[Offload][AMDGPU] Enable SPIRV target in build conf

[jplehr]

Enable the SPIRV backend on the CMake-cache file buildbots.

[llvmbot]

@llvm/pr-subscribers-offload

@llvm/pr-subscribers-backend-amdgpu

Author: Jan Patrick Lehr (jplehr)

Changes

Enable the SPIRV backend on the CMake-cache file buildbots.

---

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

1 Files Affected:

• (modified) offload/cmake/caches/AMDGPUBot.cmake (+1-1)

diff --git a/offload/cmake/caches/AMDGPUBot.cmake b/offload/cmake/caches/AMDGPUBot.cmake
index f04f3de2cb145..521841166be82 100644
--- a/offload/cmake/caches/AMDGPUBot.cmake
+++ b/offload/cmake/caches/AMDGPUBot.cmake
@@ -14,7 +14,7 @@ set(LLVM_ENABLE_RUNTIMES "compiler-rt;openmp;offload" CACHE STRING "")
 
 set(LLVM_ENABLE_PER_TARGET_RUNTIME_DIR ON CACHE BOOL "")
 set(LLVM_ENABLE_ASSERTIONS ON CACHE BOOL "")
-set(LLVM_TARGETS_TO_BUILD "host;AMDGPU" CACHE STRING "")
+set(LLVM_TARGETS_TO_BUILD "host;AMDGPU;SPIRV" CACHE STRING "")
 set(LLVM_LIT_ARGS "-v --show-unsupported --timeout 100 --show-xfail -j 32" CACHE STRING "")
 
 set(CLANG_DEFAULT_LINKER "lld" CACHE STRING "")

[jplehr]

In case this is OK, I'll land this on the weekend to see how much damage (if any) it does to our buildbots while not disrupting too many people.

[jhuber6]

It's reasonable to turn this on because we are targeting SPIR-V for AMD machines and want coverage, but it's not reasonable to make this the solution for failing tests.

[AlexVlx]

LGTM, thanks.

[jplehr]

It's reasonable to turn this on because we are targeting SPIR-V for AMD machines and want coverage, but it's not reasonable to make this the solution for failing tests.

Agreed. We also still have bots that do not turn on this target.