[PATCH] D88655: [Flang][OpenMP][OpenACC] Support branching out issue in OpenMP parallel construct.

[Valentin Clement via Phabricator via llvm-commits]

clementval requested changes to this revision.
clementval added inline comments.
This revision now requires changes to proceed.


================
Comment at: flang/lib/Semantics/check-acc-structure.cpp:76
     // Restriction - 840 (PARALLEL)
-    CheckNoBranching(block, GetContext().directive, blockDir.source);
+    DirectiveStructureChecker::CheckNoBranching(
+        block, GetContext().directive, blockDir.source);
----------------
`DirectiveStructureChecker::` is not necessary here and it's not added for other checks. 


================
Comment at: flang/lib/Semantics/check-acc-structure.cpp:84
     // Restriction - 916
-    CheckNoBranching(block, llvm::acc::Directive::ACCD_serial, blockDir.source);
+    DirectiveStructureChecker::CheckNoBranching(
+        block, llvm::acc::Directive::ACCD_serial, blockDir.source);
----------------
`DirectiveStructureChecker::` is not necessary here and it's not added for other checks.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88655/new/

https://reviews.llvm.org/D88655