diff options
| author | Ted Kremenek <kremenek@apple.com> | 2012-11-12 21:20:48 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2012-11-12 21:20:48 +0000 |
| commit | da5919ffa10c251091482f5aea0c30eca123aa54 (patch) | |
| tree | 006743df515ddb1c089073fe4da5ac2a56c72331 /clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp | |
| parent | 039136664db74a346f94ce4caa3e378d2d315ee8 (diff) | |
| download | bcm5719-llvm-da5919ffa10c251091482f5aea0c30eca123aa54.tar.gz bcm5719-llvm-da5919ffa10c251091482f5aea0c30eca123aa54.zip | |
Per discussion on cfe-dev, re-enable suppression of -Wimplicit-fallthrough on C, but also include dialects of C++ earlier than C++11.
There was enough consensus that we *can* get a good language solution
to have an annotation outside of C++11, and without this annotation
this warning doesn't quite mean's completeness criteria for this
kind of warning. For now, restrict this warning to C++11 (where an
annotation exists), and make this the behavior for the LLVM 3.2 release.
Afterwards, we will hammer out a language solution that we are all
happy with.
llvm-svn: 167749
Diffstat (limited to 'clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp')
| -rw-r--r-- | clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp b/clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp index 14ffcef704d..8b24c4a3b13 100644 --- a/clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp +++ b/clang/test/SemaCXX/switch-implicit-fallthrough-cxx98.cpp @@ -1,5 +1,8 @@ // RUN: %clang_cc1 -fsyntax-only -verify -std=c++98 -Wimplicit-fallthrough %s +// XFAIL: * +// NOTE: This test is marked XFAIL until we come up with a good language design +// for a worfklow to use this warning outside of C++11. int fallthrough(int n) { switch (n / 10) { |
