This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH][GOLD] Avoid generating unused dynamic relocation for __exidx_{start,end}
- From: Doug Kwan (éæå) <dougkwan at google dot com>
- To: Ian Lance Taylor <iant at google dot com>, binutils <binutils at sources dot redhat dot com>
- Date: Thu, 15 Mar 2012 12:50:29 -0700
- Subject: Re: [PATCH][GOLD] Avoid generating unused dynamic relocation for __exidx_{start,end}
- References: <CAH9SEo56WaFUw+CGCMVnE-EmG9pqjEr8yPQ36J0U+xrC8-uneg@mail.gmail.com>
Ping?
On Tue, Mar 13, 2012 at 4:18 PM, Doug Kwan (éæå) <dougkwan@google.com> wrote:
> [resend a previously sent e-mail to include binutils]
>
> Hi Ian,
>
> ÂThis patch fixes a problem in which unnecessary dynamic relocations
> are generated for __exidx_{start,end}. ÂThese symbols are special and
> references to them should use the local definition. ÂCurrently we
> define these symbols in the ARM backend too late. ÂThe patch adds a
> new target hook to generate target-specific symbols.
>
>
> 2012-03-13 ÂDoug Kwan Â<dougkwan@google.com>
>
> Â Â Â * arm.cc (Target_arm::do_define_standard_symbols): New method.
> Â Â Â (Target_arm::do_finalize_sections): Remove code which defines
> Â Â Â __exidx_start and __exidx_end. ÂMake symbol table parameter
> Â Â Â anonymous as it is not used.
> Â Â Â * gold.cc (queue_middle_tasks): Call target hook to define any
> Â Â Â target-specific symbols.
> Â Â Â * target.h (Target::define_standard_symbols): New method.
> Â Â Â (Target::do_define_standard_symbols): Same.
> Â Â Â * testsuite/Makefile.am (arm_exidx_test): Dump relocations also.
> Â Â Â * testsuite/Makefile.in: Regenerate.
> Â Â Â * testsuite/arm_exidx.s: Generate data relocations for __exidx_start
> Â Â Â and __exidx_end.
> Â Â Â * testsuite/arm_exidx_test.sh: Check that no unused dynamic
> Â Â Â relocations are generated for __exidx_start and __exidx_end.
>
>
>
> -Doug