https://github.com/mgcarrasco created https://github.com/llvm/llvm-project/pull/171815
Prior to this changes, the toolchain was expecting device libs to be present. This is not required because they are not linked. >From 90960878a5661ab860ba2e65d366c195a05433d3 Mon Sep 17 00:00:00 2001 From: Manuel Carrasco <[email protected]> Date: Thu, 11 Dec 2025 06:09:39 -0600 Subject: [PATCH] [clang][Driver] SPIRVAMDToolChain must not require device libs. Prior to this changes, the toolchain was expecting device libs to be present. This is not required because they are not linked. --- clang/lib/Driver/ToolChains/HIPAMD.cpp | 9 +++++++++ clang/lib/Driver/ToolChains/HIPAMD.h | 5 +++++ clang/test/Driver/spirv-amd-toolchain.c | 2 +- 3 files changed, 15 insertions(+), 1 deletion(-) diff --git a/clang/lib/Driver/ToolChains/HIPAMD.cpp b/clang/lib/Driver/ToolChains/HIPAMD.cpp index f2f64922cb404..130f7f2a78938 100644 --- a/clang/lib/Driver/ToolChains/HIPAMD.cpp +++ b/clang/lib/Driver/ToolChains/HIPAMD.cpp @@ -466,3 +466,12 @@ Tool *SPIRVAMDToolChain::buildLinker() const { assert(getTriple().getArch() == llvm::Triple::spirv64); return new tools::AMDGCN::Linker(*this); } + +void SPIRVAMDToolChain::addClangTargetOptions( + const llvm::opt::ArgList &DriverArgs, llvm::opt::ArgStringList &CC1Args, + Action::OffloadKind DeviceOffloadingKind) const { + // For SPIR-V we must not link any device libraries, so we call the + // AMDGPUToolChain impl instead of the ROCMToolChain one. + AMDGPUToolChain::addClangTargetOptions(DriverArgs, CC1Args, + DeviceOffloadingKind); +} \ No newline at end of file diff --git a/clang/lib/Driver/ToolChains/HIPAMD.h b/clang/lib/Driver/ToolChains/HIPAMD.h index 30fc01a2f8e40..d04e805f7e30c 100644 --- a/clang/lib/Driver/ToolChains/HIPAMD.h +++ b/clang/lib/Driver/ToolChains/HIPAMD.h @@ -104,6 +104,11 @@ class LLVM_LIBRARY_VISIBILITY SPIRVAMDToolChain final : public ROCMToolChain { SPIRVAMDToolChain(const Driver &D, const llvm::Triple &Triple, const llvm::opt::ArgList &Args); + void + addClangTargetOptions(const llvm::opt::ArgList &DriverArgs, + llvm::opt::ArgStringList &CC1Args, + Action::OffloadKind DeviceOffloadKind) const override; + protected: Tool *buildLinker() const override; }; diff --git a/clang/test/Driver/spirv-amd-toolchain.c b/clang/test/Driver/spirv-amd-toolchain.c index 8f1f0f33e53f9..91cd43622518d 100644 --- a/clang/test/Driver/spirv-amd-toolchain.c +++ b/clang/test/Driver/spirv-amd-toolchain.c @@ -12,7 +12,7 @@ // BINDINGS: # "spirv64-amd-amdhsa" - "clang", inputs: ["[[INPUT:.+]]"], output: "[[OUTPUT:.+]]" // BINDINGS: # "spirv64-amd-amdhsa" - "AMDGCN::Linker", inputs: ["[[OUTPUT]]"], output: "a.out" -// RUN: %clang -### --target=spirv64-amd-amdhsa %s -nogpulib -nogpuinc 2>&1 \ +// RUN: %clang -### --target=spirv64-amd-amdhsa %s 2>&1 \ // RUN: | FileCheck %s --check-prefix=INVOCATION // INVOCATION: "-cc1" "-triple" "spirv64-amd-amdhsa" {{.*}}"-disable-llvm-optzns" {{.*}} "-o" "[[OUTPUT:.+]]" "-x" "c" // INVOCATION: "{{.*}}llvm-link" "-o" "[[LINKED_OUTPUT:.+]]" "[[OUTPUT]]" _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
