diff options
| author | David Blaikie <dblaikie@gmail.com> | 2012-02-10 05:15:16 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2012-02-10 05:15:16 +0000 |
| commit | 0337241e054a8d580ade13edf039e6561878f8dc (patch) | |
| tree | bc176d58e577134c7b0687c317bede6f8ee8ae5a | |
| parent | d6cf912e1e33167452f328d15b98a790c58c03c0 (diff) | |
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.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@150230 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | include/clang/Basic/DiagnosticGroups.td | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/include/clang/Basic/DiagnosticGroups.td b/include/clang/Basic/DiagnosticGroups.td index 1a733d28e0..4fa19826b9 100644 --- a/include/clang/Basic/DiagnosticGroups.td +++ b/include/clang/Basic/DiagnosticGroups.td @@ -317,7 +317,6 @@ def Most : DiagGroup<"most", [ Reorder, ReturnType, SelfAssignment, - Switch, SizeofArrayArgument, Trigraphs, Uninitialized, @@ -331,8 +330,8 @@ def Most : DiagGroup<"most", [ // Thread Safety warnings def ThreadSafety : DiagGroup<"thread-safety">; -// -Wall is -Wmost -Wparentheses -Wdangling-else -def : DiagGroup<"all", [DanglingElse, Most, Parentheses]>; +// -Wall is -Wmost -Wparentheses -Wdangling-else -Wswitch +def : DiagGroup<"all", [DanglingElse, Most, Parentheses, Switch]>; // Aliases. def : DiagGroup<"", [Extra]>; // -W = -Wextra |