[PATCH] D99757: [flang][OpenMP] Add semantic check for occurrence of constructs nested inside a SIMD region

[Arnamoy B via Phabricator via llvm-commits]

arnamoy10 added inline comments.


================
Comment at: flang/test/Semantics/omp-nested-simd.f90:75
+    IF (I <= 10) THEN
+      !ERROR: OpenMP constructs other than `!$OMP ORDERED SIMD` or '!$OMP ATOMIC' may not be nested inside 'SIMD' region.
+      !$OMP SIMD
----------------
kiranchandramohan wrote:
> Which revision of the standard are you following here?
> 
> 4.5: 
> An ordered construct with the simd clause is the only OpenMP construct that can be encountered during execution of a simd region.
> 
> 5.0, 5.1:
> The only OpenMP constructs that can be encountered during execution of a simd region are the atomic construct, the loop construct, the simd construct and the ordered construct with the simd clause.
Thanks for the comment.  Initially the check I implemented was based on `gfortran` (latest) error messages.  I updated the checks to reflect the `OpenMP 5.0/5.1` standard.


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

https://reviews.llvm.org/D99757