erichkeane wrote: > Need to add tests
I mentioned in the commit message, that unfortunately this doesn't change behavior in a way that we can discover in our testing (unless you know how to check which character a diagnostic was issued on with `VerifyDiagnosticsConsumer`?). I'd left the tests for these in place already, but the `loop` was being interpreted as the beginning of a `clause`, so it ends up being diagnosed as `unknown clause`. However, now that we're consuming 'loop' as well, we still get the same diagnostic for the NEXT token. https://github.com/llvm/llvm-project/pull/72692 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
