Skip to content

Commit

Permalink
[SPIR-V] Enable spirv-sim lit substitution unconditionally (#109451)
Browse files Browse the repository at this point in the history
The uses of spirv-sim aren't gated by LLVM_INCLUDE_SPIRV_TOOLS_TESTS, so
the substitution shouldn't be gated either. Fixes tests after #107408
  • Loading branch information
bogner committed Sep 20, 2024
1 parent a065295 commit 28ad801
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions llvm/test/CodeGen/SPIRV/lit.local.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,16 @@ if not "SPIRV" in config.root.targets:

spirv_sim_root = os.path.join(config.llvm_src_root, "utils", "spirv-sim")

config.substitutions.append(
(
"spirv-sim",
"'%s' %s" % (config.python_executable,
os.path.join(spirv_sim_root, "spirv-sim.py")),
)
)

if config.spirv_tools_tests:
config.available_features.add("spirv-tools")
config.substitutions.append(("spirv-dis", os.path.join(config.llvm_tools_dir, "spirv-dis")))
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-sim",
"'%s' %s"
% (config.python_executable, os.path.join(spirv_sim_root, "spirv-sim.py")),
)
)

0 comments on commit 28ad801

Please sign in to comment.