This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [AArch64][Backport 2.25] PR18270, fix handling of GOT entry for local symbol


2015-04-24 23:37 GMT+01:00 Jiong Wang <wong.kwongyuan.tools@gmail.com>:
> 2015-04-24 15:23 GMT+01:00 Nicholas Clifton <nickc@redhat.com>:
>> Hi Jiong,
>>
>>> bfd/
>>>    PR18270
>>>    * elfnn-aarch64.c (elfNN_aarch64_size_dynamic): Count local symbol for
>>>    GOT_NORMAL for both sgot/srelgot section.
>>>    (elfNN_aarch64_final_link_relocate): Relocate against GOT entry
>>>    address and generate necessary runtime relocation for GOT entry.
>>
>>
>> Approved - but ... I do not like having calls to abort() inside a library.
>> It rarely helps the user.  It would be better I think to replace them with
>> calls to bdf_error_handler, providing an error message that tells the user
>> that there has been an internal error.

This patch cleanly applied to 2.25 branch.

OK to backport?

Thanks.

Regards,
Jiong


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]