debug symbol hiding

[H.J. Lu]

On Sun, Nov 1, 2009 at 10:27 PM, Alan Modra <amodra@bigpond.net.au> wrote:
> On Sun, Sep 06, 2009 at 09:27:46PM -0400, Hans-Peter Nilsson wrote:
>> To fix that, I think I need to know if a symbol in a
>> debug-section, which is global in the input, should always be
>> forced local (or just non-dynamic).
>>
>> That's what elf_link_add_object_symbols /
>> _bfd_elf_link_hash_hide_symbol does.  Either that's wrong (i.e.
>> it should only be forced non-dynamic, not also local), or the
>> generic linker is wrong for not doing that too, in the name of
>> consistency (to ld-mmix/sec-6m.d).
>
> Let's make them consistent, and correct IMO.
>
> 2009-11-02  Alan Modra  <amodra@bigpond.net.au>
>
>        * elflink.c (elf_link_add_object_symbols): Don't force debug
>        symbols local.
>
> Index: bfd/elflink.c
> ===================================================================
> RCS file: /cvs/src/src/bfd/elflink.c,v
> retrieving revision 1.357
> diff -u -p -r1.357 elflink.c
> --- bfd/elflink.c       13 Oct 2009 04:06:20 -0000      1.357
> +++ bfd/elflink.c       2 Nov 2009 00:49:08 -0000
> @@ -4373,7 +4373,6 @@ error_free_dyn:
>          if (definition && (sec->flags & SEC_DEBUGGING) && !info->relocatable)
>            {
>              /* We don't want to make debug symbol dynamic.  */
> -             (*bed->elf_backend_hide_symbol) (info, h, TRUE);
>              dynsym = FALSE;
>            }
>

I didn't check it.Will -rdynamic make debug symbols global dynamic?


-- 
H.J.