ABataev added a comment.

In D65835#1624619 <https://reviews.llvm.org/D65835#1624619>, @jdenny wrote:

> In D65835#1624617 <https://reviews.llvm.org/D65835#1624617>, @ABataev wrote:
>
> > Target teams private map will produce extra private in target context, 
> > other constructs either.
>
>
> Here's what I tried:
>
>   int a;
>   #pragma omp target teams private(a) map(a)
>     ;
>
>
> The same code is generated as for:
>
>   int a;
>   #pragma omp target map(a)
>   #pragma omp teams private(a)
>     ;
>
>
> I haven't debugged it yet, but it seems orthogonal to whether you have a 
> combined directive, which is what this patch is about.


Did you check the mapping flags, generated during host codegen? They must be 
the same. With private clause it may generate just map(alloc) instead of 
map(tofrom).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65835



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

Reply via email to