This is an archive of the discontinued LLVM Phabricator instance.

Differential D110992

[mlir] [test] Include mlir_tools_dir in PATH to fix mlir-reduce
ClosedPublic

Authored by mgorny on Oct 2 2021, 3:01 AM.

Details

Reviewers
mehdi_amini
msifontes
Commits
rG93769e81ed2e: [mlir] [test] Include mlir_tools_dir in PATH to fix mlir-reduce
Summary

Include mlir_tools_dir in the PATH used in test environment,
as otherwise mlir-reduce is unable to find mlir-opt when building
standalone (and hence mlir_tools_dir != llvm_tools_dir).

Diff Detail

Revision Contents

PathSize
mlir/
test/
1 line

Diff 376686

mlir/test/lit.cfg.py

Show First 20 Lines • Show All 45 Lines • ▼ Show 20 Lines
# test_source_root: The root path where tests are located. # test_source_root: The root path where tests are located.
config.test_source_root = os.path.dirname(__file__) config.test_source_root = os.path.dirname(__file__)
# test_exec_root: The root path where tests should be run. # test_exec_root: The root path where tests should be run.
config.test_exec_root = os.path.join(config.mlir_obj_root, 'test') config.test_exec_root = os.path.join(config.mlir_obj_root, 'test')
# Tweak the PATH to include the tools dir. # Tweak the PATH to include the tools dir.
llvm_config.with_environment('PATH', config.mlir_tools_dir, append_path=True)
llvm_config.with_environment('PATH', config.llvm_tools_dir, append_path=True) llvm_config.with_environment('PATH', config.llvm_tools_dir, append_path=True)
tool_dirs = [config.mlir_tools_dir, config.llvm_tools_dir] tool_dirs = [config.mlir_tools_dir, config.llvm_tools_dir]
tools = [ tools = [
'mlir-opt', 'mlir-opt',
'mlir-tblgen', 'mlir-tblgen',
'mlir-translate', 'mlir-translate',
'mlir-lsp-server', 'mlir-lsp-server',
▲ Show 20 Lines • Show All 54 Lines • Show Last 20 Lines