[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