aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Target/CppBackend/CPPBackend.cpp
diff options
context:
space:
mode:
authorDavid Blaikie <dblaikie@gmail.com>2012-01-21 18:12:07 +0000
committerDavid Blaikie <dblaikie@gmail.com>2012-01-21 18:12:07 +0000
commit645ae0ce10cbbe44383eab948597d3432e23a441 (patch)
treee9aaa33acf1acf6639ac23ff28dfa3337e2b8095 /llvm/lib/Target/CppBackend/CPPBackend.cpp
parentfb6ecdf27c40654746780cb92f922655c59db19e (diff)
downloadllvm-645ae0ce10cbbe44383eab948597d3432e23a441.zip
llvm-645ae0ce10cbbe44383eab948597d3432e23a441.tar.gz
llvm-645ae0ce10cbbe44383eab948597d3432e23a441.tar.bz2
Add -Wswitch-enum-redundant-default.
This warning acts as the complement to the main -Wswitch-enum warning (which warns whenever a switch over enum without a default doesn't cover all values of the enum) & has been an an-doc coding convention in LLVM and Clang in my experience. The purpose is to ensure there's never a "dead" default in a switch-over-enum because this would hide future -Wswitch-enum errors. The name warning has a separate flag name so it can be disabled but it's grouped under -Wswitch-enum & is on-by-default because of this. The existing violations of this rule in test cases have had the warning disabled & I've added a specific test for the new behavior (many negative cases already exist in the same test file - and none regressed - so I didn't add more). Reviewed by Ted Kremenek ( http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120116/051690.html ) llvm-svn: 148640
Diffstat (limited to 'llvm/lib/Target/CppBackend/CPPBackend.cpp')
0 files changed, 0 insertions, 0 deletions