[llvm] 28ad801 - [SPIR-V] Enable `spirv-sim` lit substitution unconditionally (#109451)

via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 20 13:00:08 PDT 2024


Author: Justin Bogner
Date: 2024-09-20T13:00:04-07:00
New Revision: 28ad801a7593cdb768bcd8e940ea53507653432d

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

LOG: [SPIR-V] Enable `spirv-sim` lit substitution unconditionally (#109451)

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

Added: 
    

Modified: 
    llvm/test/CodeGen/SPIRV/lit.local.cfg

Removed: 
    


################################################################################
diff  --git a/llvm/test/CodeGen/SPIRV/lit.local.cfg b/llvm/test/CodeGen/SPIRV/lit.local.cfg
index 7a20bcc82ad0ff..4655633a256827 100644
--- a/llvm/test/CodeGen/SPIRV/lit.local.cfg
+++ b/llvm/test/CodeGen/SPIRV/lit.local.cfg
@@ -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")),
-      )
-    )


        


More information about the llvm-commits mailing list