ABataev added inline comments.
================
Comment at: clang/include/clang/AST/OpenMPClause.h:5779-5780
+ /// Has iterator modifier
+ bool HasIteratorModifier = false;
+
----------------
It can be removed
================
Comment at: clang/test/OpenMP/target_map_messages.cpp:970-979
+ // ompx-error@+8 {{use of undeclared identifier 'itt'; did you mean 'it'?}}
+ // ompx-note@+7 {{'it' declared here}}
+ // omp-error@+6 {{use of undeclared identifier 'itt'; did you mean 'it'?}}
+ // omp-note@+5 {{'it' declared here}}
+ // ge51-ompx-error@+4 {{incorrect map type modifier, expected one of:
'always', 'close', 'mapper', 'present', 'ompx_hold'}}
+ // lt51-ompx-error@+3 {{incorrect map type modifier, expected one of:
'always', 'close', 'mapper', 'ompx_hold'}}
+ // ge51-omp-error@+2 {{incorrect map type modifier, expected one of:
'always', 'close', 'mapper', 'present'}}
----------------
Test cases for wrong variables in mappers?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D141871/new/
https://reviews.llvm.org/D141871
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
