This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: [PATCH] Don't put .debug_* symbols into .dynsym
- To: Jakub Jelinek <jakub at redhat dot com>
- Subject: Re: [PATCH] Don't put .debug_* symbols into .dynsym
- From: Alan Modra <amodra at bigpond dot net dot au>
- Date: Mon, 12 Nov 2001 10:11:02 +1030
- Cc: binutils at sources dot redhat dot com, dan at cgsoftware dot com
- References: <20011106193748.Q739@sunsite.ms.mff.cuni.cz>
On Tue, Nov 06, 2001 at 07:37:48PM +0100, Jakub Jelinek wrote:
>
> * elflink.h (elf_fix_symbol_flags): Avoid putting symbols
> defined in debugging sections into .dynsym.
Since Nick hasn't yet reviewed this patch, I'm going to stick my neck out
and say it's OK, but as rth noted it is probably better to not limit
the patch to SEC_DEBUGGING. Instead, test for !SEC_ALLOC (I think rth
probably meant non-alloc sections rather than non-load). Oh, and please
run through a qt build or something similar before committing with the
changed test, just in case I'm completely off-beam.
Alan