maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Move -Wswitch from -Wmost to -Wall, consitent with GCC. 

This is a relatively noisy warning for a codebase not explicitly designed for
it (effectively enforcing a stylistic constraint about the use of defaults
in switches over enums) & there's nothing Clang does to clean up the noise
when compared to GCC's implementation so the same decision seems suitable.

llvm-svn: 150230
This commit is contained in:
David Blaikie 2012-02-10 05:15:16 +00:00
parent 5b6562f58c
commit 54f5d859ad
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
clang/include/clang/Basic/DiagnosticGroups.td
View File

@ -317,7 +317,6 @@ def Most : DiagGroup<"most", [
Reorder, Reorder,
ReturnType, ReturnType,
SelfAssignment, SelfAssignment,
Switch,
SizeofArrayArgument, SizeofArrayArgument,
Trigraphs, Trigraphs,
Uninitialized, Uninitialized,
@ -331,8 +330,8 @@ def Most : DiagGroup<"most", [
// Thread Safety warnings // Thread Safety warnings
def ThreadSafety : DiagGroup<"thread-safety">; def ThreadSafety : DiagGroup<"thread-safety">;
// -Wall is -Wmost -Wparentheses -Wdangling-else // -Wall is -Wmost -Wparentheses -Wdangling-else -Wswitch
def : DiagGroup<"all", [DanglingElse, Most, Parentheses]>; def : DiagGroup<"all", [DanglingElse, Most, Parentheses, Switch]>;
// Aliases. // Aliases.
def : DiagGroup<"", [Extra]>; // -W = -Wextra def : DiagGroup<"", [Extra]>; // -W = -Wextra