Author: Weining Lu Date: 2023-02-15T09:16:14+08:00 New Revision: 82c6e90d6a3d6773372b845a246adfed1682cb65
URL: https://github.com/llvm/llvm-project/commit/82c6e90d6a3d6773372b845a246adfed1682cb65 DIFF: https://github.com/llvm/llvm-project/commit/82c6e90d6a3d6773372b845a246adfed1682cb65.diff LOG: [Clang][Test] Add llvm-lto, llvm-lto2 and llvm-profdata to the tool substitutions list Similar to issue fixed in D107155, some lit tests invoke `llvm-lto`, `llvm-lto2` and `llvm-profdata` without going through the substitution system. While the test runner correctly picks up these binaries from the build directory, it doesn't print its absolute path. When copying the invocations when reproducing test failures, this can result in `command not found: llvm-lto` or other errors (caused by using wrong version of the tool). This patch adds these tools to the `tools` variable in `clang/test/lit.cfg.py` which will then call add_tool_substitutions. Reviewed By: dblaikie Differential Revision: https://reviews.llvm.org/D143806 Added: Modified: clang/test/lit.cfg.py Removed: ################################################################################ diff --git a/clang/test/lit.cfg.py b/clang/test/lit.cfg.py index cc55c3c44a413..839960e9633df 100644 --- a/clang/test/lit.cfg.py +++ b/clang/test/lit.cfg.py @@ -61,6 +61,7 @@ tools = [ 'apinotes-test', 'c-index-test', 'clang- diff ', 'clang-format', 'clang-repl', 'clang-offload-packager', 'clang-tblgen', 'clang-scan-deps', 'opt', 'llvm-ifs', 'yaml2obj', 'clang-linker-wrapper', + 'llvm-lto', 'llvm-lto2', 'llvm-profdata', ToolSubst('%clang_extdef_map', command=FindTool( 'clang-extdef-mapping'), unresolved='ignore'), ] _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits