[PATCH] D107933: [clang] Expose unreachable fallthrough annotation warning

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 12 11:26:41 PDT 2021


nickdesaulniers added inline comments.


================
Comment at: clang/test/SemaCXX/switch-implicit-fallthrough.cpp:211-214
+    case 224:
+      n += 400;
+    case 225: // expected-warning{{unannotated fall-through between switch labels}} expected-note{{insert '[[clang::fallthrough]];' to silence this warning}} expected-note{{insert 'break;' to avoid fall-through}}
+        ;
----------------
These 4 lines don't add anything to the test coverage.  Remove them?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107933/new/

https://reviews.llvm.org/D107933



More information about the cfe-commits mailing list