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: [3/6] Fix overcounting of ARM PLT references


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.


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