OHNope wrote: > Since your test also covers invalid OpenMP combinations, you could actually > just reuse the test file I added in #200663 (Renaming it is better). > > I'm not sure if adding these assertions is the right approach, as I haven't > consulted the rest of the source code.
Thanks! I deleted those asserts, reused the regression test added in #200663 and renamed it to "invalid-clause-on-directive.c". It now covers the original transparent case! https://github.com/llvm/llvm-project/pull/212822 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
