[Openmp-commits] [PATCH] D87269: [OpenMP] Introduce GOMP taskwait depend in the runtime

Andrey Churbanov via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Sep 14 01:11:54 PDT 2020


AndreyChurbanov added inline comments.


================
Comment at: openmp/runtime/test/tasking/omp50_taskwait_depend.c:17
+int main(int argc, char** argv) {
+  #pragma omp parallel shared(a)
+  {
----------------
I think the test is going to hang if executed by a single thread. So omp_get_max_threads() should be checked before parallel, and if it is 1 then  omp_set_num_threads() could be used to increase number of threads explicitly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87269



More information about the Openmp-commits mailing list