nathanchance marked an inline comment as done. nathanchance 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}} + ; ---------------- nickdesaulniers wrote: > These 4 lines don't add anything to the test coverage. Remove them? Done. 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