[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