ABataev added inline comments.
================
Comment at: clang/include/clang/Basic/DiagnosticParseKinds.td:1252-1253
+ "incorrect map type modifier, expected 'always', 'close', 'mapper', or
'present'">;
+def err_omp_map_type_modifier_wrong_version : Error<
+ "map type modifier '%0' requires OpenMP version %1 or above">;
def err_omp_map_type_missing : Error<
----------------
jdenny wrote:
> ABataev wrote:
> > Better to keep original message for <= 5.0. This is what we usually do
> What message do you mean? Which `err_` diag id?
For <= 5.0 the error message should be the same.
```
def err_omp_unknown_map_type_modifier : Error<
"incorrect map type modifier, expected 'always', 'close', %select{or
'mapper'|'mapper', or 'presence'}0">;
```
In code:
```
Diag(..., diags::err_omp_unknown_map_type_modifier) << (LangOpts.OpenMP <= 50 ?
0 : 1);
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83061/new/
https://reviews.llvm.org/D83061
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
