[PATCH] D141871: [Clang][OpenMP] Add parse and sema for iterator map modifier

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

doru1004 added inline comments.


================
Comment at: clang/test/OpenMP/declare_mapper_messages.c:33-36
+#pragma omp declare mapper(id2: struct vec vvec) map(iterator(it=0:vvec.len:2), tofrom:vvec.data[it])
+int var; // expected-note {{'var' declared here}}
+// expected-error at +1 {{only variable 'vvec' is allowed in map clauses of this 'omp declare mapper' directive}}
+#pragma omp declare mapper(id3: struct vec vvec) map(iterator(it=0:vvec.len:2), tofrom:vvec.data[var])
----------------
ABataev wrote:
> doru1004 wrote:
> > Here we have both the positive and the negative declare mapper cases. Please let me know if you meant something different.
> Would be good to have ast print case and codegen
I can add ast print! Code gen for this map modifier is going to be done in a separate patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141871/new/

https://reviews.llvm.org/D141871