[PATCH] D51222: [OPENMP][NVPTX] Lightweight runtime support for SPMD mode.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 3 10:43:54 PDT 2018


ABataev added inline comments.


================
Comment at: openmp/trunk/libomptarget/deviceRTLs/nvptx/src/loop.cu:146-167
+    case kmp_sched_static_balanced_chunk: {
+      if (chunk > 0) {
+        entityId =
+            GetOmpThreadId(tid, IsSPMDExecutionMode, IsRuntimeUninitialized);
+        numberOfEntities = GetNumberOfOmpThreads(tid, IsSPMDExecutionMode,
+                                                 IsRuntimeUninitialized);
+
----------------
Hahnfeld wrote:
> Has this change related to `kmp_sched_static_balanced_chunk` slipped into this patch on commit? I can't find it in the original review.
> 
> (No need to revert, just curious...)
Yes, during testing I found some inconsistency between clang and NVPTX libomp abd fixed all of them in a single patch. 


Repository:
  rL LLVM

https://reviews.llvm.org/D51222





More information about the llvm-commits mailing list