[PATCH] D24728: [ARM][LLD] Add support for .ARM.exidx sections

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 15:39:51 PDT 2016


On 28 September 2016 at 13:31, Rui Ueyama <ruiu at google.com> wrote:
> On Tue, Sep 27, 2016 at 3:55 PM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>>
>> Another interesting thing I noticed: it looks like gold doesn't
>> implement the order restriction at all. The reason things work is
>> because they are concatenated in the same order as the sections they
>> point to.
>>
>> For example, given the script
>>
>> SECTIONS
>> {
>>   .text : { test1.o(.text) test3.o(.text) test2.o(.text) }
>> }
>>
>> and a command line of "test1.o test2.o test3.o", the output .ARM.exidx
>> will be in the wrong order.
>
>
> That's an interesting finding, but I'm wondering how much gold's ARM support
> mature. IIUC, Mips support in gold is immature compared to ld.bfd, so if
> that's the case for ARM, this might not just be implemented yet.

Yes, I pointed it out not to say that we should not implement it, but
as an indication that it provides a path for splitting the patch up.

Cheers,
Rafael


More information about the llvm-commits mailing list