https://github.com/kwk updated https://github.com/llvm/llvm-project/pull/197129
>From 8001f6c67ea960cd6382114028a7fa1977e2c131 Mon Sep 17 00:00:00 2001 From: Konrad Kleine <[email protected]> Date: Tue, 12 May 2026 08:42:59 +0000 Subject: [PATCH 1/2] libclc: use clang triples when configuring in README In #196483 I've changed the triples in the README for the first time. Back then the triple `spirv32-mesa-mesa3d` was recognized and the correct triple was passed to clang: ```cmake set(clang_triple ${LIBCLC_TARGET}) if(ARCH STREQUAL spirv AND LIBCLC_USE_SPIRV_BACKEND) set(clang_triple spirv32--) ``` But when #196351 landed, the triple `spirv32-mesa-mesa3d` was passed to clang as is: ```cmake set(clang_triple ${LIBCLC_TARGET}) if(ARCH STREQUAL spirv OR ARCH STREQUAL spirv32 OR ARCH STREQUAL spirv64) if(NOT OS STREQUAL vulkan AND NOT LIBCLC_USE_SPIRV_BACKEND) if(ARCH STREQUAL spirv) set(clang_triple spir--) else() set(clang_triple spir64--) endif() endif() endif() ``` Notice, that the OS `mesa3d` is not vulkan in `spirv32-mesa-mesa3d`. We don't want to fight with libclc's `CMakeLists.txt` file so we think it is better to simply use the `spirv32--` and `spirv64--` triples as they are passed to clang anyway. The files will be installed in `spirv32-unknown-unknown` and `spirv64-unknown-unknown`. --- libclc/README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libclc/README.md b/libclc/README.md index 169d59b75c5c6..2d72a7edc9d81 100644 --- a/libclc/README.md +++ b/libclc/README.md @@ -60,9 +60,9 @@ cmake ../llvm -G Ninja -DLLVM_ENABLE_PROJECTS="clang" -DCMAKE_BUILD_TYPE=Release #### Configure for SPIR-V targets ``` cmake ../llvm -G Ninja -DLLVM_ENABLE_PROJECTS="clang" -DCMAKE_BUILD_TYPE=Release \ - -DRUNTIMES_spirv-mesa-mesa3d_LLVM_ENABLE_RUNTIMES=libclc \ - -DRUNTIMES_spirv64-mesa-mesa3d_LLVM_ENABLE_RUNTIMES=libclc \ - -DLLVM_RUNTIME_TARGETS="spirv-mesa-mesa3d;spirv64-mesa-mesa3d" + -DRUNTIMES_spirv32--_LLVM_ENABLE_RUNTIMES=libclc \ + -DRUNTIMES_spirv64--_LLVM_ENABLE_RUNTIMES=libclc \ + -DLLVM_RUNTIME_TARGETS="spirv32--;spirv64--" ``` To build multiple targets, pass them as a semicolon-separated list in >From 1b89e75aca12c0c60326780d0953ac96e6a6d9f1 Mon Sep 17 00:00:00 2001 From: Konrad Kleine <[email protected]> Date: Tue, 12 May 2026 09:10:44 +0000 Subject: [PATCH 2/2] Use spirv32-unknown-unknown instead of spirv32-- --- libclc/README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libclc/README.md b/libclc/README.md index 2d72a7edc9d81..54429d71b3e32 100644 --- a/libclc/README.md +++ b/libclc/README.md @@ -60,9 +60,9 @@ cmake ../llvm -G Ninja -DLLVM_ENABLE_PROJECTS="clang" -DCMAKE_BUILD_TYPE=Release #### Configure for SPIR-V targets ``` cmake ../llvm -G Ninja -DLLVM_ENABLE_PROJECTS="clang" -DCMAKE_BUILD_TYPE=Release \ - -DRUNTIMES_spirv32--_LLVM_ENABLE_RUNTIMES=libclc \ - -DRUNTIMES_spirv64--_LLVM_ENABLE_RUNTIMES=libclc \ - -DLLVM_RUNTIME_TARGETS="spirv32--;spirv64--" + -DRUNTIMES_spirv32-unknown-unknown_LLVM_ENABLE_RUNTIMES=libclc \ + -DRUNTIMES_spirv64-unknown-unknown_LLVM_ENABLE_RUNTIMES=libclc \ + -DLLVM_RUNTIME_TARGETS="spirv32-unknown-unknown;spirv64-unknown-unknown" ``` To build multiple targets, pass them as a semicolon-separated list in _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
