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

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 19 09:54:40 PST 2023


ABataev 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])
----------------
doru1004 wrote:
> 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.
Ok, a st print should be enough


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

https://reviews.llvm.org/D141871



More information about the cfe-commits mailing list