[Openmp-commits] [openmp] ca17b26 - [libomptarget] [amdgpu] Fix copy-paste error setting NumThreads for a corner case.
    Dhruva Chakrabarti via Openmp-commits 
    openmp-commits at lists.llvm.org
       
    Mon May 24 15:23:58 PDT 2021
    
    
  
Author: Dhruva Chakrabarti
Date: 2021-05-24T15:23:15-07:00
New Revision: ca17b26d4d7a1bd95346184d3f3ccdf006c33781
URL: https://github.com/llvm/llvm-project/commit/ca17b26d4d7a1bd95346184d3f3ccdf006c33781
DIFF: https://github.com/llvm/llvm-project/commit/ca17b26d4d7a1bd95346184d3f3ccdf006c33781.diff
LOG: [libomptarget] [amdgpu] Fix copy-paste error setting NumThreads for a corner case.
Fix the case where NumTeams was set incorrectly instead of NumThreads
Reviewed By: JonChesterfield
Differential Revision: https://reviews.llvm.org/D103037
Added: 
    
Modified: 
    openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
Removed: 
    
################################################################################
diff  --git a/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp b/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
index de76b80be931..51fc0a188ced 100644
--- a/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
+++ b/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
@@ -836,7 +836,7 @@ int32_t __tgt_rtl_init_device(int device_id) {
      RTLDeviceInfoTy::Default_WG_Size);
   if (DeviceInfo.NumThreads[device_id] >
       DeviceInfo.ThreadsPerGroup[device_id]) {
-    DeviceInfo.NumTeams[device_id] = DeviceInfo.ThreadsPerGroup[device_id];
+    DeviceInfo.NumThreads[device_id] = DeviceInfo.ThreadsPerGroup[device_id];
     DP("Default number of threads exceeds device limit, capping at %d\n",
        DeviceInfo.ThreadsPerGroup[device_id]);
   }
        
    
    
More information about the Openmp-commits
mailing list