[PATCH] D64356: [OPENMP]Initial fix PR42392: Improve -Wuninitialized warnings for OpenMP programs.
Alexey Bataev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 9 12:40:20 PDT 2019
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
More information about the cfe-commits
mailing list