[clang] [clang] Add break/continue stmts with missing lable to the AST (PR #168332)

[via cfe-commits]

================
@@ -2313,10 +2313,8 @@ StmtResult Parser::ParseBreakOrContinueStatement(bool IsContinue) {
       // TODO: Make this a compatibility/extension warning instead once the
       // syntax of this feature is finalised.
       Diag(LabelLoc, diag::err_c2y_labeled_break_continue) << IsContinue;
-    if (!Target) {
+    if (!Target)
       Diag(LabelLoc, diag::err_break_continue_label_not_found) << IsContinue;
-      return StmtError();
----------------
Sirraide wrote:

> Yes, so how do we communicate that there was an error at all?

In the static analyser, we just check if there was an error anywhere in the TU (iirc it calls `Sema::hasUnrecoverableErrorOccurred()` or whatever it it’s called); I’m not sure we have a way of tracking this on a per-function basis at the moment but I feel like it should be possible.

https://github.com/llvm/llvm-project/pull/168332