[PATCH] D87906: [Flang][OpenACC] Fix for branching out issue in OpenACC parallel construct.

Tim Keith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 07:29:40 PDT 2020


tskeith accepted this revision.
tskeith added inline comments.


================
Comment at: flang/lib/Semantics/check-do-forall.cpp:894
+  } else if (constructName &&
+      constructName.value().source == stmtName->source) {
     return stmtType == StmtType::EXIT || inDoConstruct;
----------------
The original is better. constructName is checked for null so `.value()` is unnecessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87906



More information about the llvm-commits mailing list