================ @@ -15,18 +15,19 @@ void f(int n) { return; } case 2: + // FIXME: Should we emit an error {{fallthrough annotation does not directly precede switch label}}? ---------------- yronglin wrote:
Hmm, this because I originally set the new diagnostic's severity to `Error`, which will interrupt the warning static analysis based warnings. https://github.com/llvm/llvm-project/blob/da01257c3a16b728718f9bde8bbe435678d96fd8/clang/lib/Sema/AnalysisBasedWarnings.cpp#L2632-L2633 I have changed the severity to `Warning` now. https://github.com/llvm/llvm-project/pull/146224 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits