This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Merging debug strings
- From: Ben Young <ben dot young at transversal dot com>
- To: Andi Kleen <ak at suse dot de>
- Cc: Ben Young <ben dot young at transversal dot com>, "" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 2 Oct 2003 09:31:59 +0000 (GMT)
- Subject: Re: Merging debug strings
- References: <Pine.LNX.4.50.0310020809040.14254-100000@fileserver.cluster.local.suse.lists.egcs-patches><p733cecyrk2.fsf@oldwotan.suse.de>
On Thu, 2 Oct 2003, Andi Kleen wrote:
> Ben Young <ben.young@transversal.com> writes:
> >
> > if (node->form == DW_FORM_strp)
> > {
> > - named_section_flags (DEBUG_STR_SECTION, DEBUG_STR_SECTION_FLAGS);
> > + int flags = DEBUG_STR_SECTION_FLAGS;
> > + if ( !flag_merge_debug_strs )
> > + flags &=~ ( SECTION_MERGE | SECTION_STRINGS | SECTION_ENTSIZE );
>
> Hmm. This looks like a hack. Wouldn't it be better to just add
> a nomerge option to ld ?
Possibly, but we understand gcc much better than ld. Also, we added it to
be in parallel with -fno-merge-constants which is a gcc option. You're
right that the patch is a bit of a hack though. We thought that someone
more qualified would be able to take this and alter it to something more
suitable.
Cheers
Ben
---
>
> -Andi
>
>