ABataev added a comment. In D64356#1576813 <https://reviews.llvm.org/D64356#1576813>, @NoQ wrote:
> Something like this: > > F9509417: photo_2019-07-09_12-21-46.jpg <https://reviews.llvm.org/F9509417> > > Like, you can construct a CFG for an arbitrary statement. CFG₁ is the CFG for > xxx() and CFG₂ is the CFG for the CapturedStmt (and its children). I'm trying > to say that even if used expressions are duplicated in the AST, they should > not be duplicated in our CFGs. > > But that's more, like, for the future patches. I'll be happy to accept this > patch once you add CFG tests :) I don't expect such duplication here. Those expressions associated with the directive have no relation with the CapturedStmt. They are associated only with the directive itself through the OpenMP clauses. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64356/new/ https://reviews.llvm.org/D64356 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
