[PATCH] D92732: [Flang][OpenMP 4.5] Add semantic check for OpenMP Do Loop Constructs

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:113
+      context_.Say(*cyclesource_,
+          "CYCLE statement to non-innermost collapsed !$OMP DO loop"_err_en_US);
+    }
----------------
yhegde wrote:
> kiranchandramohan wrote:
> > We usually do not add the "!$" in the error message.
>  That is gfortran error message. Kindly suggest a better one. Will look into the rest of the review comments. Thank you. 
Would the following work?

"CYCLE statement to non-innermost associated loop of an OpenMP DO construct"


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92732