[PATCH] D81951: [OPENMP]Fix PR46347: several ordered directives in a single region.

2020-06-16 Thread Alexey Bataev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3488e8c21cec: [OPENMP]Fix PR46347: several ordered directives in a single region. (authored by ABataev). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81951/

[PATCH] D81951: [OPENMP]Fix PR46347: several ordered directives in a single region.

2020-06-16 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev added a comment. In D81951#2096397 , @jdoerfert wrote: > LGTM, can you add the nested ordered + depend test case too, assuming we > don't have it. We have such tests in ordered_ast_print.cpp and ordered_codegen.cpp tests Repository: rG LLVM

[PATCH] D81951: [OPENMP]Fix PR46347: several ordered directives in a single region.

2020-06-16 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. LGTM, can you add the nested ordered + depend test case too, assuming we don't have it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D8195

[PATCH] D81951: [OPENMP]Fix PR46347: several ordered directives in a single region.

2020-06-16 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev created this revision. ABataev added a reviewer: jdoerfert. Herald added subscribers: sstefan1, guansong, yaxunl. Herald added a project: clang. According to OpenMP, During execution of an iteration of a worksharing-loop or a loop nest within a worksharing-loop, simd, or worksharing-loop