llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Justin Stitt (JustinStitt)

<details>
<summary>Changes</summary>

Remove the `liblldb` link target for these tests to fix CI failures.

Commit 02572c6e9bbb ("[lldb] Enforce that only the LLDB API unit tests can link 
liblldb") disallows linking against liblldb for most tests. Then, Commit 
f3e2c20a23b1 ("Make
SBBreakpoint/SBBreakpointLocation.SetCondition(nullptr) work again") got merged 
just after this commit (but before CI could report the regression).

The tests still pass for me with ` $ 
./tools/lldb/unittests/Breakpoint/LLDBBreakpointTests`.

---
Full diff: https://github.com/llvm/llvm-project/pull/162571.diff


1 Files Affected:

- (modified) lldb/unittests/Breakpoint/CMakeLists.txt (-1) 


``````````diff
diff --git a/lldb/unittests/Breakpoint/CMakeLists.txt 
b/lldb/unittests/Breakpoint/CMakeLists.txt
index 3e4161313cd9d..c38b38bc5e90e 100644
--- a/lldb/unittests/Breakpoint/CMakeLists.txt
+++ b/lldb/unittests/Breakpoint/CMakeLists.txt
@@ -5,7 +5,6 @@ add_lldb_unittest(LLDBBreakpointTests
   LINK_COMPONENTS
     Support
   LINK_LIBS
-    liblldb
     lldbBreakpoint
     lldbCore
     LLVMTestingSupport

``````````

</details>


https://github.com/llvm/llvm-project/pull/162571
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to