[Openmp-commits] [openmp] f184147 - [amdgpu] Default to 1.0, instead of unspecified, for dynamic hsa (#74098)

[via Openmp-commits]

Author: Jon Chesterfield
Date: 2023-12-01T16:37:49Z
New Revision: f184147706f5430387fee99d2e94c7a3361c642b

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

LOG: [amdgpu] Default to 1.0, instead of unspecified, for dynamic hsa (#74098)

The plugin checks the values of HSA_AMD_INTERFACE_VERSION_* so we now
set them to something safe in the header.

Added: 
    

Modified: 
    openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h

Removed: 
    


################################################################################
diff  --git a/openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h b/openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h
index 188dd2600a610c6..9c59d3bf824de3c 100644
--- a/openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h
+++ b/openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h
@@ -14,6 +14,15 @@
 
 #include "hsa.h"
 
+/* Using this header means we do not know what version library will be linked.
+   Until such point as a CMake level override is requested, default to the
+   minimum. */
+/*
+ * - 1.0 - initial version
+ */
+#define HSA_AMD_INTERFACE_VERSION_MAJOR 1
+#define HSA_AMD_INTERFACE_VERSION_MINOR 0
+
 #ifdef __cplusplus
 extern "C" {
 #endif