[clang] [OpenMP] Allow OMP6.0 features. (PR #122108)
Zahira Ammarguellat via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 21 07:46:02 PST 2025
zahiraam wrote:
> I don't think it is necessary to test each value of `-fopenmp-version=` separately, but don't mind either as long it doesn't affect maintanability.
>
> * Instead of repeating each clang diagnositic, consider passing multiple values to `-verify`, e.g. `-verify=omp51,omp60`, `-verify=omp51-exact,omp60-or-lafter`
> * Write expected diagnostics on separate lines using `@<relative line>`-syntax
I think this mostly applies to `declare_target_messages.cpp` and `distribute_simd_misc_messages.c` right? Should I change all the lines to the form `@<relative line>`-syntax?
https://github.com/llvm/llvm-project/pull/122108
More information about the cfe-commits
mailing list