[Lldb-commits] [lldb] [lldb][test] Provide proper path to LLVM utils in API tests (PR #110837)

Vladislav Dzhidzhoev via lldb-commits lldb-commits at lists.llvm.org
Wed Oct 2 06:09:44 PDT 2024


https://github.com/dzhidzhoev updated https://github.com/llvm/llvm-project/pull/110837

>From 8deb61b462964d08b4cc5d7f296b44ea4197f65b Mon Sep 17 00:00:00 2001
From: Vladislav Dzhidzhoev <vdzhidzhoev at accesssoftek.com>
Date: Tue, 1 Oct 2024 13:14:25 +0200
Subject: [PATCH] [lldb][test] Provide proper path to LLVM utils in API tests

In aea066849, API tests were supposed to be switched to LLVM tools
usage.
However, a path to an utility is made up incorrectly there: util name
should be prefixed with `llvm-`.

Hence it is fixed here.
---
 lldb/packages/Python/lldbsuite/test/builders/builder.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lldb/packages/Python/lldbsuite/test/builders/builder.py b/lldb/packages/Python/lldbsuite/test/builders/builder.py
index e3099219e437e3..736f0ddc390736 100644
--- a/lldb/packages/Python/lldbsuite/test/builders/builder.py
+++ b/lldb/packages/Python/lldbsuite/test/builders/builder.py
@@ -153,7 +153,9 @@ def getToolchainSpec(self, compiler):
         cc_dir = cc_path.parent
 
         def getToolchainUtil(util_name):
-            return os.path.join(configuration.llvm_tools_dir, util_name + exe_ext)
+            return os.path.join(
+                configuration.llvm_tools_dir, "llvm-" + util_name + exe_ext
+            )
 
         cxx = cc_dir / (cc_prefix + cxx_type + cc_ext)
 



More information about the lldb-commits mailing list