[PATCH] D100228: [flang][OpenMP] Add semantic check for close nesting of `master` regions

[Arnamoy B via Phabricator via llvm-commits]

arnamoy10 added inline comments.


================
Comment at: flang/lib/Semantics/check-omp-structure.h:80
+static OmpDirectiveSet nestedMasterErrSet{
+    OmpDirectiveSet{Directive::OMPD_atomic} | taskGeneratingSet | parallelSet |
+    doSet};
----------------
kiranchandramohan wrote:
> arnamoy10 wrote:
> > kiranchandramohan wrote:
> > > Why is parallelSet here? Master is allowed inside parallel.
> > Good catch, updated the code and removed it.  Now the set is inspired from [[ https://github.com/llvm/llvm-project/blob/c058a7122787c8e503cf6306b8da03c0e56a41a5/flang/lib/Semantics/check-omp-structure.cpp#L215| here ]]
> Can you add all those combined constructs also. Those combined constructs which will lead to a nested master violating this constraint, like parallel do.
Can you please review [[ https://reviews.llvm.org/D100575 | this ]] patch?  Then I should be able to use the combined constructs for worksharing from the patch here.


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

https://reviews.llvm.org/D100228