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

[Arnamoy B via Phabricator via llvm-commits]

arnamoy10 created this revision.
arnamoy10 added reviewers: kiranchandramohan, bryanpkc, SouraVX, clementval, kiranktp, AMDChirag.
arnamoy10 added a project: Flang.
Herald added subscribers: jdoerfert, jfb, guansong, yaxunl.
Herald added a reviewer: sscalpone.
arnamoy10 requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: llvm-commits, sstefan1.
Herald added a project: LLVM.

This patch adds the semantic check  for occurrence of constructs nested inside a SIMD region.

To make the test generalized for multiple constructs, to avoid repeats, the check is done while `Enter()`-ing the top level `OpenMPConstruct`

Also adds a test case.

It also modifies an existing test case for nested ordered clause check because with the current check introduced, the previous test shows //two// sema check failures instead of one.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D99757

Files:
  flang/lib/Semantics/check-omp-structure.cpp
  flang/lib/Semantics/check-omp-structure.h
  flang/test/Semantics/omp-nested-simd.f90
  flang/test/Semantics/omp-ordered-simd.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D99757.334782.patch
Type: text/x-patch
Size: 10458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210401/2cedd71d/attachment.bin>