From: Christian Sigg Date: Wed, 14 Sep 2022 10:58:08 +0000 (+0200) Subject: [MLIR] Fix toy lit substitutions X-Git-Tag: upstream/17.0.6~33553 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5cff32b9f0bc7179e7cccb4233968b4a04057e0a;p=platform%2Fupstream%2Fllvm.git [MLIR] Fix toy lit substitutions The tools are called e.g. `toyc-ch1`, not `toy-ch1`. Add missing toyc-ch6/7. It turns out that the other substitutions are not needed more by specific circumstances rather than by design: The lit test exec root is set to build/mlir/test, which is where all the test tools are placed by CMake and we wouldn't need to substitute them at all. We shouldn't rely on this assumption though, because it will make things harder for standalone tests and other build systems. Reviewed By: mehdi_amini Differential Revision: https://reviews.llvm.org/D133842 --- diff --git a/mlir/test/lit.cfg.py b/mlir/test/lit.cfg.py index 73405c3..151e333 100644 --- a/mlir/test/lit.cfg.py +++ b/mlir/test/lit.cfg.py @@ -75,11 +75,13 @@ if config.enable_spirv_cpu_runner: # The following tools are optional tools.extend([ - ToolSubst('toy-ch1', unresolved='ignore'), - ToolSubst('toy-ch2', unresolved='ignore'), - ToolSubst('toy-ch3', unresolved='ignore'), - ToolSubst('toy-ch4', unresolved='ignore'), - ToolSubst('toy-ch5', unresolved='ignore'), + ToolSubst('toyc-ch1', unresolved='ignore'), + ToolSubst('toyc-ch2', unresolved='ignore'), + ToolSubst('toyc-ch3', unresolved='ignore'), + ToolSubst('toyc-ch4', unresolved='ignore'), + ToolSubst('toyc-ch5', unresolved='ignore'), + ToolSubst('toyc-ch6', unresolved='ignore'), + ToolSubst('toyc-ch7', unresolved='ignore'), ToolSubst('%mlir_lib_dir', config.mlir_lib_dir, unresolved='ignore'), ToolSubst('%mlir_src_dir', config.mlir_src_root, unresolved='ignore'), ])