jdenny added a comment.

In D65835#1639622 <https://reviews.llvm.org/D65835#1639622>, @ABataev wrote:

> We don't need this new level counter to correctly handle this situation. Just 
> check for the combined target directive in `Sema::isOpenMPCapturedByRef` and 
> return true if it is mapped as to from or just from. The change must be very 
> simple.


I don't see a way to do that without also affecting capturing on the teams 
region in the case of a combined target teams construct.

If it's simple, it might save time for both of us if you post the change you're 
thinking of here.


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