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]

Re: Add more tree dump options, remove inconsistencies



  4) rather than `preoptimize' & `postoptimize' I've named them `raw' &
  `cooked'. If you have a better suggestion, please make it.

Stick with your original terminology.  It's bad enough that
colloquialisms like `grok' appear in our source code; they certainly
shouldn't appear in our external interface.

  2) I added the functionality of the first version of this patch to help
  track down a tree-based optimizer bug in 3.0. I've now added the additional
  class information for bug 3006 (an ABI problem). Without this patch in the
  branch, I'd have to apply and unapply the patch locally, which I suspect
  would be more error prone in the long run.

That's compelling enough.  I don't think the issues about
command-lines and such really matter -- these switches aren't, in
practice, used for much other than debugging.  (In fact, I can imagine
packagers disabling them entirely for "production" versions of the
compiler.)  But, making it easier for you to find and fix bugs is
definitely important.

Thanks,

--
Mark Mitchell                   mark@codesourcery.com
CodeSourcery, LLC               http://www.codesourcery.com


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