This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH] RISC-V: Fix .p2align is not at smallest instruction
- From: Nelson Chu <nelson dot chu at sifive dot com>
- To: Kuan-Lin Chen <kuanlinchentw at gmail dot com>
- Cc: Binutils <binutils at sourceware dot org>, Jim Wilson <jimw at sifive dot com>, Palmer Dabbelt <palmerdabbelt at google dot com>, Andrew Waterman <andrew at sifive dot com>, Kito Cheng <kito dot cheng at sifive dot com>
- Date: Thu, 5 Dec 2019 10:55:26 +0800
- Subject: Re: [PATCH] RISC-V: Fix .p2align is not at smallest instruction
- References: <CAJYME4GHNojA5+h_xCqiNej95iNEz3ucxgYPx+rpAzprs=wApg@mail.gmail.com> <CAJr6u0iHrm-p71+c5xgxCv1Gyexg=0fQS6rd6r+wcrvq4F4z0w@mail.gmail.com>
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