Skip to content

Commit 38c9811

Browse files
svenvhjsji
authored andcommitted
Clarify optionality of spirv-val in test suite (#2765)
Original commit: KhronosGroup/SPIRV-LLVM-Translator@e8fce056867bb1b
1 parent 8659507 commit 38c9811

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

llvm-spirv/test/lit.cfg.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,8 @@
7575
config.available_features.add('spirv-link')
7676
using_spirv_tools = True
7777

78+
# Unlike spirv-{as,dis,link} above, running spirv-val is optional: if spirv-val is
79+
# not available, the test must still run and just skip any spirv-val commands.
7880
if config.spirv_tools_have_spirv_val:
7981
llvm_config.add_tool_substitutions(['spirv-val'], [config.spirv_tools_bin_dir])
8082
using_spirv_tools = True

0 commit comments

Comments
 (0)