https://github.com/Nerixyz created 
https://github.com/llvm/llvm-project/pull/169932

The test was flaky, because it assumed that the breakpoint was always resolved 
before `r` was executed 
(https://github.com/llvm/llvm-project/pull/169728#issuecomment-3589799783). 
This removes the check for this order. It still checks that the breakpoint is 
resolved before it is hit.

>From dae0c1e671eab505f867f2239344458d0399b93a Mon Sep 17 00:00:00 2001
From: Nerixyz <[email protected]>
Date: Fri, 28 Nov 2025 17:10:01 +0100
Subject: [PATCH] [LLDB][PDB] Relax check for resolving breakpoint

---
 lldb/test/Shell/SymbolFile/PDB/add-symbols.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lldb/test/Shell/SymbolFile/PDB/add-symbols.cpp 
b/lldb/test/Shell/SymbolFile/PDB/add-symbols.cpp
index ef7690b1720a6..64fbc84a98f24 100644
--- a/lldb/test/Shell/SymbolFile/PDB/add-symbols.cpp
+++ b/lldb/test/Shell/SymbolFile/PDB/add-symbols.cpp
@@ -25,7 +25,7 @@
 // CHECK-NEXT: Breakpoint 1: no locations (pending).
 // CHECK: (lldb) target symbols add
 // CHECK: 1 location added to breakpoint 1
-// CHECK: (lldb) r
+
 // CHECK: * thread #1, stop reason = breakpoint 1.1
 // CHECK: (lldb) target variable a
 // CHECK-NEXT: (A) a = (x = 47)

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

Reply via email to