[amdgpu] Default to 1.0, instead of unspecified, for dynamic hsa

[JonChesterfield]

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

[llvmbot]

@llvm/pr-subscribers-backend-amdgpu

Author: Jon Chesterfield (JonChesterfield)

Changes

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

---

Full diff: https://github.com/llvm/llvm-project/pull/74098.diff

1 Files Affected:

• (modified) openmp/libomptarget/plugins-nextgen/amdgpu/dynamic_hsa/hsa_ext_amd.h (+9)

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

[jhuber6]

Thanks

[JonChesterfield]

Discovered this while debugging openmp's runtime refusing to build but that turned out to be a bug in Debian's packaging, reported on their mailing list.