[llvm] [lit] Resolve extra_args in ToolSubst (PR #94319)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 4 00:45:41 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-testing-tools
Author: Dmitry Vasilyev (slydiman)
<details>
<summary>Changes</summary>
We are working on lldb shell tests running on a remote target. This patch is necessary for flexible lldb params.
---
Full diff: https://github.com/llvm/llvm-project/pull/94319.diff
1 Files Affected:
- (modified) llvm/utils/lit/lit/llvm/subst.py (+1-1)
``````````diff
diff --git a/llvm/utils/lit/lit/llvm/subst.py b/llvm/utils/lit/lit/llvm/subst.py
index 09ab3555a6b44..9e579d91a4b98 100644
--- a/llvm/utils/lit/lit/llvm/subst.py
+++ b/llvm/utils/lit/lit/llvm/subst.py
@@ -119,7 +119,7 @@ def resolve(self, config, search_dirs):
if command_str:
if self.extra_args:
- command_str = " ".join([command_str] + self.extra_args)
+ command_str = " ".join(str(a) for a in [command_str] + self.extra_args)
else:
if self.unresolved == "warn":
# Warn, but still provide a substitution.
``````````
</details>
https://github.com/llvm/llvm-project/pull/94319
More information about the llvm-commits
mailing list