[llvm] [lit] Use dbg instead of note for tool detection (PR #193662)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 22 21:56:56 PDT 2026
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-testing-tools
Author: Vitaly Buka (vitalybuka)
<details>
<summary>Changes</summary>
Also remove unused parameter `quiet`.
Don't output lines like these by default:
```
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using split-file: out/z/bin/split-file
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using yaml2obj: out/z/bin/yaml2obj
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using llvm-objcopy: out/z/bin/llvm-objcopy
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using ld.lld: out/z/bin/ld.lld
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using lld-link: out/z/bin/lld-link
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using ld64.lld: out/z/bin/ld64.lld
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using wasm-ld: out/z/bin/wasm-ld
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using clang: out/z/bin/clang
llvm-lit: llvm-project/llvm/utils/lit/lit/llvm/config.py:569: note: using clang: out/z/bin/clang
```
I don't believe this information is usefull for a typical test run.
Now they can be retrived with 'LIT_OPTS=--debug ninja check-llvm'
---
Full diff: https://github.com/llvm/llvm-project/pull/193662.diff
1 Files Affected:
- (modified) llvm/utils/lit/lit/llvm/config.py (+1-3)
``````````diff
diff --git a/llvm/utils/lit/lit/llvm/config.py b/llvm/utils/lit/lit/llvm/config.py
index 9713f4333d99a..2a6c60fab6125 100644
--- a/llvm/utils/lit/lit/llvm/config.py
+++ b/llvm/utils/lit/lit/llvm/config.py
@@ -527,7 +527,6 @@ def use_llvm_tool(
name,
search_env=None,
required=False,
- quiet=False,
search_paths=None,
use_installed=False,
):
@@ -565,8 +564,7 @@ def use_llvm_tool(
self.lit_config.params.get("use_normalized_slashes")
):
tool = tool.replace("\\", "/")
- if not quiet:
- self.lit_config.note("using {}: {}".format(name, tool))
+ self.lit_config.dbg("using {}: {}".format(name, tool))
return tool
def _get_clang_paths(self, additional_tool_dirs):
``````````
</details>
https://github.com/llvm/llvm-project/pull/193662
More information about the llvm-commits
mailing list