[cfe-commits] r158796 - in /cfe/trunk: include/clang/Driver/Options.td lib/Driver/Tools.cpp test/Driver/warning-options.cpp

[Chris Lattner]

On Jun 20, 2012, at 11:53 AM, Chandler Carruth wrote:
> > Thanks John.  That's sums it up well.  Right now the workflow people know is to pass -Wno- to silence a warning, and seeing the warning flag in the diagnostic.  This flag is completely different from that simple workflow.
> >
> > My understanding was that ever clang warning should be controllable under a -W flag.  That's not the case with all -pedantic warnings.
> 
> I'd also like it if we never produced [-pedantic] in a diagnostic as the warning flag.  [-Wpedantic] would be much more consistent.
> 
> I completely agree with presenting the user *only* with '-Wpedantic' and variants.
> 
> That said, for compatibility, I think we should support '-pedantic' and '-no-pedantic' as aliases for '-Wpedantic' and '-Wno-pedantic' respectively. I don't really like the flags either, but I'm not thrilled about explaining that the solution to negate '-pedantic' is to pass '-Wno-pedantic'. =/

Yes, makes sense to me.

-Chris

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120620/37db93fa/attachment.html>