[Openmp-commits] [openmp] 2bc4d48 - [libomptarget][amdgpu] Follow on to D110513, empty kernarg pools are not fatal

[Jon Chesterfield via Openmp-commits]

Author: Jon Chesterfield
Date: 2021-09-27T22:44:35+01:00
New Revision: 2bc4d48a78bab5ddecbcfb46a4c64a87065bbd83

URL: https://github.com/llvm/llvm-project/commit/2bc4d48a78bab5ddecbcfb46a4c64a87065bbd83
DIFF: https://github.com/llvm/llvm-project/commit/2bc4d48a78bab5ddecbcfb46a4c64a87065bbd83.diff

LOG: [libomptarget][amdgpu] Follow on to D110513, empty kernarg pools are not fatal

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 c0edc9c6478f..767a0eb32b26 100644
--- a/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
+++ b/openmp/libomptarget/plugins/amdgpu/src/rtl.cpp
@@ -383,9 +383,7 @@ FindKernargPool(const std::vector<hsa_agent_t> &HSAAgents) {
     err = hsa_amd_agent_iterate_memory_pools(
         Agent, addKernArgPool, static_cast<void *>(&KernArgPools));
     if (err != HSA_STATUS_SUCCESS) {
-      DP("[%s:%d] %s failed: %s\n", __FILE__, __LINE__,
-         "Iterate all memory pools", get_error_string(err));
-      return {err, hsa_amd_memory_pool_t{}};
+      DP("addKernArgPool returned %s, continuing\n", get_error_string(err));
     }
   }