[llvm] [lit] Resolve extra_args in ToolSubst (PR #94319)

Dmitry Vasilyev via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 00:45:10 PDT 2024


https://github.com/slydiman created https://github.com/llvm/llvm-project/pull/94319

We are working on lldb shell tests running on a remote target. This patch is necessary for flexible lldb params.

>From f15fdea6257db89eb5311b58f5975fc0364c6220 Mon Sep 17 00:00:00 2001
From: Dmitry Vasilyev <dvassiliev at accesssoftek.com>
Date: Tue, 4 Jun 2024 11:06:22 +0400
Subject: [PATCH] [lit] Resolve extra_args in ToolSubst

We are working on lldb shell tests running on a remote target. This patch is necessary for flexible lldb params.
---
 llvm/utils/lit/lit/llvm/subst.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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.



More information about the llvm-commits mailing list