[PATCH] D70726: [OpenMP50] Add parallel master construct

Chi Chun Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 3 12:34:47 PST 2019


cchen marked an inline comment as done.
cchen added inline comments.


================
Comment at: clang/test/OpenMP/nesting_of_regions.cpp:3024
+  {
+#pragma omp master // OK, though second 'master' is redundant
+    {
----------------
ABataev wrote:
> Is this allowed by the standard?
I didn't see any restriction about master directive inside master directive. Also, nesting_of_regions also have tests for nested master construct.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70726





More information about the cfe-commits mailing list