[PATCH] D133887: [Clang] Support label at end of compound statement

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 17 06:37:02 PDT 2022


cor3ntin added inline comments.


================
Comment at: clang/lib/Parse/ParseStmt.cpp:738
+    } else if (getLangOpts().C99 && !getLangOpts().C2x) {
+      Diag(Tok, diag::ext_c_label_end_of_compound_statement);
+    }
----------------
I do not understand this. Why checking `getLangOpts().C99`  instead of just C? Whyt no compatibility warning in C23 mode?
I think the code should look very similar for both C++ and C


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133887



More information about the cfe-commits mailing list