[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:98
+    OmpDirectiveSet{
+        llvm::omp::Directive::OMPD_atomic, llvm::omp::Directive::OMPD_master} |
+    taskGeneratingSet | workShareSet};
----------------
kiranchandramohan wrote:
> Is a master construct not allowed in a master construct?
Thanks for the catch, just updated the code.


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

https://reviews.llvm.org/D100228