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

[Joachim Protze via Phabricator via Openmp-commits]

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());
         __kmp_printf("\n");
----------------
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


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D47903