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@+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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to