[flang-commits] [flang] [Flang][OpenMP] Add Semantics support for Nested OpenMPLoopConstructs (PR #145917)

Krzysztof Parzyszek via flang-commits flang-commits at lists.llvm.org
Thu Jun 26 11:30:18 PDT 2025


================
@@ -761,10 +761,13 @@ void OmpStructureChecker::Enter(const parser::OpenMPLoopConstruct &x) {
   }
   SetLoopInfo(x);
 
-  if (const auto &doConstruct{
-          std::get<std::optional<parser::DoConstruct>>(x.t)}) {
-    const auto &doBlock{std::get<parser::Block>(doConstruct->t)};
-    CheckNoBranching(doBlock, beginDir.v, beginDir.source);
+  auto &optLoopCons = std::get<1>(x.t);
----------------
kparzysz wrote:

Same here, please put the type name in <>.  It's longer, but it makes it a bit easier to read, especially since you have `auto` in the declaration.

https://github.com/llvm/llvm-project/pull/145917


More information about the flang-commits mailing list