This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [PATCH] add option to eliminate unused types in dwarf2 output (v3)


scott snyder writes:
> Here's a revised version of the -feliminate-unused-dwarf2-types
> patch i first sent a months ago.

I have no comment on the substance of the patch, but one question about the
interface.  Is there a reason you need to specify "dwarf2" in the option
name?  It seems like, in principle, this feature could be supported for
other debugging formats.

If the option were named '-feliminate-unused-debug-types' instead, people
who were interested in supporting other debugging formats could potentially
add support for them in the future, without having to add additional option
names or having cognitive dissonance in the name.

In the meantime, your patch can just print a warning if some debug format
other than DWARF-2 is selected with this option.  (I'm not suggesting you
actually implement this feature for other formats.)

-- 
Jonathan Lennox
lennox@cs.columbia.edu


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