[PATCH] D100575: [flang][OpenMP][FIX] Fix the worksharing nesting check with inclusion of more constructs to cover entire workshare family.

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
This revision is now accepted and ready to land.

LGTM. Have two Nit comments.



================
Comment at: flang/lib/Semantics/check-omp-structure.cpp:369
   case llvm::omp::OMPD_parallel_workshare:
     CheckWorkshareBlockStmts(block, beginDir.source);
     break;
----------------
Nit: Can a call to HasInvalidWorksharingNesting be added here for the worksharing construct with name workshare? And add one test.



================
Comment at: flang/lib/Semantics/check-omp-structure.h:58
     Directive::OMPD_teams_distribute_parallel_do_simd};
+static OmpDirectiveSet workShareSet{
+    Directive::OMPD_distribute_parallel_do,
----------------
Nit: Can you construct this set using the doSet?


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

https://reviews.llvm.org/D100575