ABataev added inline comments.
================ Comment at: clang/lib/Sema/TreeTransform.h:122 + /// the RebuildME uses to set if member expression needs to be rebuilt. + bool RebuildME = false; + ---------------- jyu2 wrote: > ABataev wrote: > > I think we don't need to add a new field here. Can instead we have a check > > for regions with default clauses, if possible? > It seems during the TransformMemberExpr, I can not get Directives info for > omp regions to check. Could you give me hand? > Thanks. > Yes, we do not rebuild the DSA stack at the time of the instantiation. Can you just check that we're inside OpenMPCapturedRegion? Something like getCurCapturedRegion()->CapRegionKind == CR_OpenMP? Or walk the stack of regions to find the outer OpenMP region, if any. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127803/new/ https://reviews.llvm.org/D127803 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
