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.
Actually, just realized there would be a difference in behavior for `pragma acc parallel loop` (with no clause). So I'll add tests for those. 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
