[Lldb-commits] [lldb] de7f7ea - [lldb][test] Fix TestStdCXXDisassembly test (#112357)
via lldb-commits
lldb-commits at lists.llvm.org
Tue Oct 15 13:09:01 PDT 2024
Author: Dmitry Vasilyev
Date: 2024-10-16T00:08:58+04:00
New Revision: de7f7ea884525cca24e8797319452bd8bc150752
URL: https://github.com/llvm/llvm-project/commit/de7f7ea884525cca24e8797319452bd8bc150752
DIFF: https://github.com/llvm/llvm-project/commit/de7f7ea884525cca24e8797319452bd8bc150752.diff
LOG: [lldb][test] Fix TestStdCXXDisassembly test (#112357)
The patch #98694 was not enough. This test is still failed on the
buildbot https://lab.llvm.org/staging/#/builders/195/builds/4438
Use `USE_LIBSTDCPP := 1` instead for non Darwin OS and skip the test if
libstdc++.so is missing.
Added:
Modified:
lldb/test/API/lang/cpp/stl/Makefile
lldb/test/API/lang/cpp/stl/TestStdCXXDisassembly.py
Removed:
################################################################################
diff --git a/lldb/test/API/lang/cpp/stl/Makefile b/lldb/test/API/lang/cpp/stl/Makefile
index 8534fa9b00209e..4408691f01b701 100644
--- a/lldb/test/API/lang/cpp/stl/Makefile
+++ b/lldb/test/API/lang/cpp/stl/Makefile
@@ -1,5 +1,9 @@
CXX_SOURCES := main.cpp
-USE_SYSTEM_STDLIB := 1
+ifneq ($(OS),Darwin)
+ USE_LIBSTDCPP := 1
+else
+ USE_SYSTEM_STDLIB := 1
+endif
include Makefile.rules
diff --git a/lldb/test/API/lang/cpp/stl/TestStdCXXDisassembly.py b/lldb/test/API/lang/cpp/stl/TestStdCXXDisassembly.py
index 8676ee16d83c04..06f338b3ed1ded 100644
--- a/lldb/test/API/lang/cpp/stl/TestStdCXXDisassembly.py
+++ b/lldb/test/API/lang/cpp/stl/TestStdCXXDisassembly.py
@@ -43,7 +43,10 @@ def test_stdcxx_disasm(self):
# At this point, lib_stdcxx is the full path to the stdc++ library and
# module is the corresponding SBModule.
- self.expect(lib_stdcxx, "Library StdC++ is located", exe=False, substrs=["lib"])
+ if "lib" not in lib_stdcxx:
+ self.skipTest(
+ "This test requires libstdc++.so or libc++.dylib in the target's module list."
+ )
self.runCmd("image dump symtab '%s'" % lib_stdcxx)
raw_output = self.res.GetOutput()
More information about the lldb-commits
mailing list