[Lldb-commits] [lldb] [LLDB][PDB] Relax check for resolving breakpoint (PR #169932)

via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 28 08:13:04 PST 2025


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 <nerixdev at outlook.de>
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)



More information about the lldb-commits mailing list