jyu2 added inline comments.
================ Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:8923-8937 + // Extract map information. + for (const auto *C : Dir.getClausesOfKind<OMPMapClause>()) { + if (C->getMapType() != OMPC_MAP_to) + continue; + for (auto L : C->component_lists()) { + const ValueDecl *VD = std::get<0>(L); + const auto *RD = VD ? VD->getType() ---------------- ABataev wrote: > What if we have `to(lambda)` in data motion directive? What shall we do if > there mapping modifiers? Not sure we shall ignore them. to(lambda) is not isOpenMPTargetExecutionDirective. I don't see generateInfoForLambdaCaptures and adjustMemberOfForLambdaCaptures functions get called in that part. Those two functions only called in emitTargetCall. So I don't think we need that. What do you think? Thanks. Jennifer Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111115/new/ https://reviews.llvm.org/D111115 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits