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