[clang] f2541ce - [clang] Remove usage of llvm-spirv in clang LIT tests (#131158)

via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 14 08:18:16 PDT 2025


Author: Nick Sarnie
Date: 2025-03-14T15:18:13Z
New Revision: f2541cee44edca653326895f51bea4561394d6c7

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

LOG: [clang] Remove usage of llvm-spirv in clang LIT tests (#131158)

We use the backend now, so remove the requirement from the only test
that actually executes the translator and remove the LIT requirement
variable.

Signed-off-by: Sarnie, Nick <nick.sarnie at intel.com>

Added: 
    

Modified: 
    clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp
    clang/test/Tooling/lit.local.cfg

Removed: 
    


################################################################################
diff  --git a/clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp b/clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp
index 9b16727d74192..af98904677283 100644
--- a/clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp
+++ b/clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp
@@ -1,7 +1,7 @@
 // Verify the ELF packaging of OpenMP SPIR-V device images.
 // REQUIRES: system-linux
 // REQUIRES: spirv-tools
-// REQUIRES: llvm-spirv
+// REQUIRES: spirv-registered-target
 // RUN: mkdir -p %t_tmp
 // RUN: cd %t_tmp
 // RUN: %clangxx -fopenmp -fopenmp-targets=spirv64-intel -nogpulib -c -o %t_clang-linker-wrapper-spirv-elf.o %s

diff  --git a/clang/test/Tooling/lit.local.cfg b/clang/test/Tooling/lit.local.cfg
index 61f328c91e4d3..46d32e6ebf454 100644
--- a/clang/test/Tooling/lit.local.cfg
+++ b/clang/test/Tooling/lit.local.cfg
@@ -9,6 +9,3 @@ if config.spirv_tools_tests:
     config.substitutions.append(("spirv-val", os.path.join(config.llvm_tools_dir, "spirv-val")))
     config.substitutions.append(("spirv-as", os.path.join(config.llvm_tools_dir, "spirv-as")))
     config.substitutions.append(("spirv-link", os.path.join(config.llvm_tools_dir, "spirv-link")))
-
-if lit.util.which("llvm-spirv"):
-    config.available_features.add("llvm-spirv")


        


More information about the cfe-commits mailing list