Merging debug strings
Ben Young
ben.young@transversal.com
Thu Oct 2 09:32:00 GMT 2003
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
>
>
More information about the Gcc-patches
mailing list