[cfe-commits] r128371 - in /cfe/trunk: include/clang/Driver/CC1Options.td include/clang/Driver/Options.td include/clang/Frontend/DiagnosticOptions.h include/clang/Frontend/TextDiagnosticPrinter.h lib/Driver/Tools.cpp lib/Frontend/CompilerInvocati

Chris Lattner clattner at apple.com
Mon Mar 28 16:07:17 PDT 2011


On Mar 27, 2011, at 12:43 PM, Chandler Carruth wrote:

> > Still, for now, the default behavior isn't changed. We can evaluate user
> > feedback with the flag.
> >
> > Patch by Richard Trieu, a couple of style tweaks from me.
> 
> When is printing the include stack for a note ever a good idea?  I think we should just remove it unconditionally.
> 
> That'll teach me to assume that these things were intentional... ;] I'll flip the default, and if no one complains, remove it. Sound good?

Sounds good to me, please do!  Thanks!

-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110328/06cf6a2c/attachment.html>


More information about the cfe-commits mailing list