Skip to content

Commit f2541ce

Browse files
authored
[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@intel.com>
1 parent e811702 commit f2541ce

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

clang/test/Tooling/clang-linker-wrapper-spirv-elf.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Verify the ELF packaging of OpenMP SPIR-V device images.
22
// REQUIRES: system-linux
33
// REQUIRES: spirv-tools
4-
// REQUIRES: llvm-spirv
4+
// REQUIRES: spirv-registered-target
55
// RUN: mkdir -p %t_tmp
66
// RUN: cd %t_tmp
77
// RUN: %clangxx -fopenmp -fopenmp-targets=spirv64-intel -nogpulib -c -o %t_clang-linker-wrapper-spirv-elf.o %s

clang/test/Tooling/lit.local.cfg

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,3 @@ if config.spirv_tools_tests:
99
config.substitutions.append(("spirv-val", os.path.join(config.llvm_tools_dir, "spirv-val")))
1010
config.substitutions.append(("spirv-as", os.path.join(config.llvm_tools_dir, "spirv-as")))
1111
config.substitutions.append(("spirv-link", os.path.join(config.llvm_tools_dir, "spirv-link")))
12-
13-
if lit.util.which("llvm-spirv"):
14-
config.available_features.add("llvm-spirv")

0 commit comments

Comments
 (0)