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

[Valentin Clement via Phabricator via llvm-commits]

clementval added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:150
+  // dirContext_ size `1` means the current construct is nested
+  if (dirContext_.size() == 1) {
+    CheckSIMDNest(x);
----------------
It is also considered nested is it's bigger than 1. So it should be >= 1? 


================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:270
+    std::visit(
+        Fortran::common::visitors{
+            [&](const parser::OpenMPBlockConstruct &c) {
----------------
Apply clang-format please. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99757