https://github.com/charles-zablit updated https://github.com/llvm/llvm-project/pull/201605
>From 0821514e7df7d045d4861c61242604c45865d077 Mon Sep 17 00:00:00 2001 From: Charles Zablit <[email protected]> Date: Thu, 4 Jun 2026 15:33:16 +0100 Subject: [PATCH 1/2] [lldb][gdb-remote] Mark thread as stopped in RefreshStateAfterStop --- lldb/source/Plugins/Process/gdb-remote/ThreadGDBRemote.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lldb/source/Plugins/Process/gdb-remote/ThreadGDBRemote.cpp b/lldb/source/Plugins/Process/gdb-remote/ThreadGDBRemote.cpp index c0171734343a6..86c9843eeedd8 100644 --- a/lldb/source/Plugins/Process/gdb-remote/ThreadGDBRemote.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/ThreadGDBRemote.cpp @@ -271,6 +271,9 @@ void ThreadGDBRemote::WillResume(StateType resume_state) { } void ThreadGDBRemote::RefreshStateAfterStop() { + // Mark this thread as stopped. ThreadList::DidStop only transitions + // threads from a running state to stopped. + SetState(eStateStopped); // Invalidate all registers in our register context. We don't set "force" to // true because the stop reply packet might have had some register values // that were expedited and these will already be copied into the register >From c88bd7f377975063a0a909f8ff212119fea21e27 Mon Sep 17 00:00:00 2001 From: Charles Zablit <[email protected]> Date: Thu, 4 Jun 2026 16:09:52 +0100 Subject: [PATCH 2/2] fixup! [lldb][gdb-remote] Mark thread as stopped in RefreshStateAfterStop --- .../thread/break_after_join/TestBreakAfterJoin.py | 13 ------------- .../thread/multi_break/TestMultipleBreakpoints.py | 13 ------------- .../thread/state/TestThreadStates.py | 14 -------------- 3 files changed, 40 deletions(-) diff --git a/lldb/test/API/functionalities/thread/break_after_join/TestBreakAfterJoin.py b/lldb/test/API/functionalities/thread/break_after_join/TestBreakAfterJoin.py index 0e80d8b7a11fb..c246808f1bb30 100644 --- a/lldb/test/API/functionalities/thread/break_after_join/TestBreakAfterJoin.py +++ b/lldb/test/API/functionalities/thread/break_after_join/TestBreakAfterJoin.py @@ -17,19 +17,6 @@ def setUp(self): # Find the line number for our breakpoint. self.breakpoint = line_number("main.cpp", "// Set breakpoint here") - @expectedFailureAll( - oslist=["linux"], - bugnumber="llvm.org/pr15824 thread states not properly maintained", - ) - @expectedFailureAll( - oslist=lldbplatformutil.getDarwinOSTriples(), - bugnumber="llvm.org/pr15824 thread states not properly maintained and <rdar://problem/28557237>", - ) - @expectedFailureAll( - oslist=["freebsd"], - bugnumber="llvm.org/pr18190 thread states not properly maintained", - ) - @expectedFailureNetBSD def test(self): """Test breakpoint handling after a thread join.""" self.build() diff --git a/lldb/test/API/functionalities/thread/multi_break/TestMultipleBreakpoints.py b/lldb/test/API/functionalities/thread/multi_break/TestMultipleBreakpoints.py index 9c9f7f7acdb04..fe30138f78157 100644 --- a/lldb/test/API/functionalities/thread/multi_break/TestMultipleBreakpoints.py +++ b/lldb/test/API/functionalities/thread/multi_break/TestMultipleBreakpoints.py @@ -17,20 +17,7 @@ def setUp(self): # Find the line number for our breakpoint. self.breakpoint = line_number("main.cpp", "// Set breakpoint here") - @expectedFailureAll( - oslist=["linux"], - bugnumber="llvm.org/pr15824 thread states not properly maintained", - ) - @expectedFailureAll( - oslist=lldbplatformutil.getDarwinOSTriples(), - bugnumber="llvm.org/pr15824 thread states not properly maintained and <rdar://problem/28557237>", - ) - @expectedFailureAll( - oslist=["freebsd"], - bugnumber="llvm.org/pr18190 thread states not properly maintained", - ) @skipIfWindows # This is flakey on Windows: llvm.org/pr24668, llvm.org/pr38373 - @expectedFailureNetBSD def test(self): """Test simultaneous breakpoints in multiple threads.""" self.build() diff --git a/lldb/test/API/functionalities/thread/state/TestThreadStates.py b/lldb/test/API/functionalities/thread/state/TestThreadStates.py index 6eeff07955733..f2b19959ea6e4 100644 --- a/lldb/test/API/functionalities/thread/state/TestThreadStates.py +++ b/lldb/test/API/functionalities/thread/state/TestThreadStates.py @@ -11,16 +11,6 @@ class ThreadStateTestCase(TestBase): - @expectedFailureAll( - oslist=["linux"], - bugnumber="llvm.org/pr15824 thread states not properly maintained", - ) - @skipIfDarwin # llvm.org/pr15824 thread states not properly maintained and <rdar://problem/28557237> - @expectedFailureAll( - oslist=["freebsd"], - bugnumber="llvm.org/pr18190 thread states not properly maintained", - ) - @expectedFailureNetBSD def test_state_after_breakpoint(self): """Test thread state after breakpoint.""" self.build() @@ -48,13 +38,10 @@ def test_state_after_expression(self): self.build() self.thread_state_after_expression_test() - # thread states not properly maintained - @unittest.expectedFailure # llvm.org/pr15824 and <rdar://problem/28557237> @expectedFailureAll( oslist=["windows"], bugnumber="llvm.org/pr24668: Breakpoints not resolved correctly", ) - @skipIfDarwin # llvm.org/pr15824 thread states not properly maintained and <rdar://problem/28557237> @expectedFailureNetBSD def test_process_state(self): """Test thread states (comprehensive).""" @@ -190,7 +177,6 @@ def thread_state_after_expression_test(self): oslist=["windows"], bugnumber="llvm.org/pr24668: Breakpoints not resolved correctly", ) - @skipIfDarwin # llvm.org/pr15824 thread states not properly maintained and <rdar://problem/28557237> @no_debug_info_test def test_process_interrupt(self): """Test process interrupt and continue.""" _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
