[PATCH] D98733: [fix][Flang][OpenMP] Allow ordered construct within a non-worksharing loop

Arnamoy B via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 19 09:05:46 PDT 2021


arnamoy10 added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:359-360
+      context_.Say(blkDirective.source,
+          "`ordered` region may not be closely nested inside of "
+          "`critical`, `ordered`, explicit `task` or `taskloop` region."_err_en_US);
+    }
----------------
kiranchandramohan wrote:
> Nit: I think we should captialise all the regions. Space missing before critical.
Done, thanks


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

https://reviews.llvm.org/D98733



More information about the llvm-commits mailing list