[PATCH] D65835: [OpenMP] Permit map with DSA on combined directive
Joel E. Denny via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 11:05:14 PDT 2019
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
More information about the cfe-commits
mailing list