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: Enable tree-browser with --enable-checking


On Thu, 2004-05-27 at 09:45, Bryce McKinlay wrote:

> - tree-browser.o adds only a very small size overhead (approx 20k 
> stripped or 0.5% of cc1's total size on i686)
> - its a useful debugging feature
> - it sucks to have to remember yet another configure option
> - it sucks to have to reconfigure gcc when you just want to 
> browse_tree() during a debugging session
> - conceptually it isn't much different from debug_tree and other 
> debugging routines that are always enabled
> - without enabling it by default, most developers would not build the 
> tree-browser code, thus it can be subject to bit-rot
> 
> OK to commit?
> 
I agree with the rationale and the patch does look OK to me, but I can't
approve patches in this area.


Diego.


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