[PATCH] RISC-V: Fix .p2align is not at smallest instruction
Nelson Chu
nelson.chu@sifive.com
Thu Dec 5 02:55:00 GMT 2019
Dear Kuan-Lin,
On Thu, Dec 5, 2019 at 10:30 AM Kuan-Lin Chen <kuanlinchentw@gmail.com> wrote:
> About the fixup cross fragments, I think it's ok.
> Because the addend just records the possible range for alignment.
> We just have to be sure the next fragment has the enough space for alignment.
Thanks for your explanation, I'm OK for this now.
I think ideally solution may be like ARM, support the mapping symbol
and do the related works. But before we supporting these features,
your solution looks good and can cover lots of mixed cases.
However, my review may not help so much at the moment, it is better to
wait for the approvals by Jim, Palmer and Andrew. At least one of
them. I think they should be very busy now since last week is
Thanksgiving :)
Thanks and regards
Nelson
More information about the Binutils
mailing list