This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [3/6] Fix overcounting of ARM PLT references
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: binutils at sourceware dot org, Richard Sandiford <rdsandiford at googlemail dot com>
- Date: Thu, 31 May 2012 14:10:04 -0700
- Subject: Re: [3/6] Fix overcounting of ARM PLT references
- References: <g47hcevpgr.fsf@linaro.org>
On Fri, Mar 4, 2011 at 8:07 AM, Richard Sandiford
<richard.sandiford@linaro.org> wrote:
> If a shared object has only non-call references to a function that is
> defined in the same shared object, we will nevertheless generate an
> unused PLT entry for it. ?This is due to some overeager counting in
> elf32_arm_check_relocs.
>
> The patch below restructures check_relocs and its gc_sweep_hook
> counterpart to make things a little more obvious (I hope).
> This also makes it easier to slip in the ifunc handling.
> I've kept a fair bit of the existing commentary, but moved
> it around.
>
> Note that the R_ARM_ABS12 case was misplaced: it fell through
> into the PLT relocs instead of the absolute relocs.
>
> Tested on arm-linux-gnueabi. ?OK to install?
>
> Richard
>
>
> bfd/
> ? ? ? ?* elf32-arm.c (elf32_arm_check_relocs): Use call_reloc_p,
> ? ? ? ?may_need_local_target_p and may_become_dynamic_p to classify
> ? ? ? ?the relocation type. ?Don't check info->symbolic or h->def_regular
> ? ? ? ?when deciding whether to record a potential dynamic reloc.
> ? ? ? ?Don't treat potential dynamic relocs as PLT references.
> ? ? ? ?(elf32_arm_gc_sweep_hook): Update to match. ?Assert that we don't
> ? ? ? ?try to make the PLT reference count go negative.
>
This caused:
http://sourceware.org/bugzilla/show_bug.cgi?id=14189
--
H.J.