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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
