[Openmp-commits] [PATCH] D47903: [OpenMP] Use C++11 Atomics - barrier, tasking, and lock code

Joachim Protze via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Jun 7 14:19:27 PDT 2018

protze.joachim added inline comments.

Comment at: runtime/src/kmp_runtime.cpp:3393
         __kmp_printf("    Nested?:      %2d\n", root->r.r_nested);
-        __kmp_printf("    In Parallel:  %2d\n", root->r.r_in_parallel);
+        __kmp_printf("    In Parallel:  %2d\n", root->r.r_in_parallel.load());
Is there a reason, why this does not use a macro?

Comment at: runtime/src/kmp_tasking.cpp:2988
                 (thread ? __kmp_gtid_from_thread(thread) : -1), task_team,
-                task_team->tt.tt_unfinished_threads));
+                task_team->tt.tt_unfinished_threads.load()));
   return task_team;
Same here

  rOMP OpenMP


More information about the Openmp-commits mailing list