[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