[clang] [OpenACC] Implement compound construct parsing (PR #72692)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 20 07:35:43 PST 2023


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


More information about the cfe-commits mailing list