[cfe-commits] [patch] Make it so that some warnings in DiagGroup<"all"> aren't on by default

Nico Weber thakis at chromium.org
Wed Feb 15 07:53:04 PST 2012


On Wed, Feb 15, 2012 at 3:14 AM, Chris Lattner <clattner at apple.com> wrote:
> What is the motivation for this patch?  Clang defaulting to more warnings on than gcc is a feature, not a bug.

I had the impression that these warnings are on by default without
-Wall mostly by accident. When I added -Wdangling-else, I wasn't aware
that it would fire without -Wall, and dblaikie unsuccessfully tried to
move -Wswitch to -Wall here:
http://www.mail-archive.com/cfe-commits@cs.uiuc.edu/msg45730.html

I'm just as happy with tweaking the comment above -Wall in the .td
file instead and not changing any behavior (patch for that attached)
if you think all these warnings should stay on by default.

Nico
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-wall-different.patch
Type: application/octet-stream
Size: 663 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120215/15a90fea/attachment.obj>


More information about the cfe-commits mailing list