[lldb-tests] Add compiler version check in TestFunctionStarts
authorFelipe de Azevedo Piovezan <fpiovezan@apple.com>
Tue, 18 Oct 2022 15:01:38 +0000 (11:01 -0400)
committerFelipe de Azevedo Piovezan <fpiovezan@apple.com>
Tue, 18 Oct 2022 17:48:26 +0000 (13:48 -0400)
This test requires compiling its input program without debug
information. To do so, it uses certain Makefile variables that are never
populated with custom libcxx paths (if present). Doing so would not
necessarily be correct: we cannot guarantee that said standard library
has no debug symbols.

As such, we keep using the system libraries but disable the tests in
clang versions that are too old to work with more modern system
libraries, as in the case of the lldb-matrix bot.

Differential Revision: https://reviews.llvm.org/D136178

lldb/test/API/macosx/function-starts/TestFunctionStarts.py

index 9b8ef06..f829c76 100644 (file)
@@ -17,6 +17,7 @@ class FunctionStartsTestCase(TestBase):
 
     @skipIfRemote
     @skipUnlessDarwin
+    @skipIf(compiler="clang", compiler_version=['<', '13.0'])
     def test_function_starts_binary(self):
         """Test that we make synthetic symbols when we have the binary."""
         self.build(dictionary={'CODESIGN': ''}) # Binary is getting stripped later.
@@ -24,6 +25,7 @@ class FunctionStartsTestCase(TestBase):
 
     @skipIfRemote
     @skipUnlessDarwin
+    @skipIf(compiler="clang", compiler_version=['<', '13.0'])
     def test_function_starts_no_binary(self):
         """Test that we make synthetic symbols when we don't have the binary"""
         self.build(dictionary={'CODESIGN': ''}) # Binary is getting stripped later.