kpdev42 updated this revision to Diff 484745.
kpdev42 added a comment.
Renamed the test, added more tests for unconditional (enabled/disabled)
breakpoints and breakpoints with callbacks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140368/new/
https://reviews.llvm.org/D140368
Files:
lldb/source/Target/StopInfo.cpp
lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/Makefile
lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/TestThreadPlanUserBreakpoint.py
lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/main.cpp
Index: lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/main.cpp
===================================================================
--- /dev/null
+++ lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/main.cpp
@@ -0,0 +1,11 @@
+int func_1() { return 1; }
+
+int func_2() {
+ func_1(); // breakpoint_1
+ return 1 + func_1(); // breakpoint_2
+}
+
+int main(int argc, char const *argv[]) {
+ func_2(); // breakpoint_0
+ return 0; // breakpoint_3
+}
Index: lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/TestThreadPlanUserBreakpoint.py
===================================================================
--- /dev/null
+++ lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/TestThreadPlanUserBreakpoint.py
@@ -0,0 +1,124 @@
+"""
+Test that breakpoints (reason = breakpoint) have more priority than
+plan completion (reason = step in/out/over) when reporting stop reason after step,
+in particular 'step out' and 'step over', and in addition 'step in'.
+Check for correct StopReason when stepping to the line with breakpoint,
+which should be eStopReasonBreakpoint in general,
+and eStopReasonPlanComplete when breakpoint's condition fails or it is disabled.
+"""
+
+
+import unittest2
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+from lldbsuite.test import lldbutil
+
+class ThreadPlanUserBreakpointsTestCase(TestBase):
+
+ def setUp(self):
+ TestBase.setUp(self)
+
+ self.build()
+ exe = self.getBuildArtifact("a.out")
+ src = lldb.SBFileSpec("main.cpp")
+
+ # Create a target by the debugger.
+ self.target = self.dbg.CreateTarget(exe)
+ self.assertTrue(self.target, VALID_TARGET)
+
+ # Setup four breakpoints
+ self.lines = [line_number('main.cpp', "breakpoint_%i" % i) for i in range(4)]
+
+ self.breakpoints = [self.target.BreakpointCreateByLocation(src, line) for line in self.lines]
+ self.assertTrue(
+ self.breakpoints[0] and self.breakpoints[0].GetNumLocations() == 1,
+ VALID_BREAKPOINT)
+
+ # Start debugging
+ self.process = self.target.LaunchSimple(
+ None, None, self.get_process_working_directory())
+ self.assertIsNotNone(self.process, PROCESS_IS_VALID)
+ self.thread = lldbutil.get_one_thread_stopped_at_breakpoint(self.process, self.breakpoints[0])
+ self.assertIsNotNone(self.thread, "Didn't stop at breakpoint 0.")
+
+ def check_correct_stop_reason(self, breakpoint_idx, condition):
+ self.assertState(self.process.GetState(), lldb.eStateStopped)
+ if condition:
+ # All breakpoints active, stop reason is breakpoint
+ thread1 = lldbutil.get_one_thread_stopped_at_breakpoint(self.process, self.breakpoints[breakpoint_idx])
+ self.assertEquals(self.thread, thread1, "Didn't stop at breakpoint %i." % breakpoint_idx)
+ else:
+ # Breakpoints are inactive, stop reason is plan complete
+ self.assertEquals(self.thread.GetStopReason(), lldb.eStopReasonPlanComplete,
+ "Expected stop reason to be step into/over/out for inactive breakpoint %i line." % breakpoint_idx)
+
+ def check_thread_plan_user_breakpoint(self, condition, set_up_breakpoints_func):
+ # Make breakpoints active/inactive in different ways
+ set_up_breakpoints_func(condition)
+
+ self.thread.StepInto()
+ # We should be stopped at the breakpoint_1 line with the correct stop reason
+ self.check_correct_stop_reason(1, condition)
+
+ # This step-over creates a step-out from `func_1` plan
+ self.thread.StepOver()
+ # We should be stopped at the breakpoint_2 line with the correct stop reason
+ self.check_correct_stop_reason(2, condition)
+
+ # Check explicit step-out
+ self.thread.StepOut()
+ # We should be stopped at the breakpoint_3 line with the correct stop reason
+ self.check_correct_stop_reason(3, condition)
+
+ # Run the process until termination
+ self.process.Continue()
+ self.assertState(self.process.GetState(), lldb.eStateExited)
+
+ def change_breakpoints(self, action):
+ action(self.breakpoints[1])
+ action(self.breakpoints[2])
+ action(self.breakpoints[3])
+
+ def set_up_breakpoints_condition(self, condition):
+ # Set breakpoint condition to true/false
+ conditionStr = 'true' if condition else 'false'
+ self.change_breakpoints(lambda b: b.SetCondition(conditionStr))
+
+ def set_up_breakpoints_enable(self, condition):
+ # Enable/disable breakpoint
+ self.change_breakpoints(lambda b: b.SetEnabled(condition))
+
+ def set_up_breakpoints_callback(self, condition):
+ # Set breakpoint callback to return True/False
+ self.change_breakpoints(lambda b: b.SetScriptCallbackBody('return %s' % condition))
+
+ def test_thread_plan_user_breakpoint_conditional_active(self):
+ # Test with breakpoints 1, 2, 3 having true condition
+ self.check_thread_plan_user_breakpoint(condition=True,
+ set_up_breakpoints_func=self.set_up_breakpoints_condition)
+
+ def test_thread_plan_user_breakpoint_conditional_inactive(self):
+ # Test with breakpoints 1, 2, 3 having false condition
+ self.check_thread_plan_user_breakpoint(condition=False,
+ set_up_breakpoints_func=self.set_up_breakpoints_condition)
+
+ def test_thread_plan_user_breakpoint_unconditional_active(self):
+ # Test with breakpoints 1, 2, 3 enabled unconditionally
+ self.check_thread_plan_user_breakpoint(condition=True,
+ set_up_breakpoints_func=self.set_up_breakpoints_enable)
+
+ def test_thread_plan_user_breakpoint_unconditional_inactive(self):
+ # Test with breakpoints 1, 2, 3 disabled unconditionally
+ self.check_thread_plan_user_breakpoint(condition=False,
+ set_up_breakpoints_func=self.set_up_breakpoints_enable)
+
+ def test_thread_plan_user_breakpoint_callback_active(self):
+ # Test with breakpoints 1, 2, 3 with callback that returns 'True'
+ self.check_thread_plan_user_breakpoint(condition=True,
+ set_up_breakpoints_func=self.set_up_breakpoints_callback)
+
+ def test_thread_plan_user_breakpoint_callback_inactive(self):
+ # Test with breakpoints 1, 2, 3 with callback that returns 'False'
+ self.check_thread_plan_user_breakpoint(condition=False,
+ set_up_breakpoints_func=self.set_up_breakpoints_callback)
Index: lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/Makefile
===================================================================
--- /dev/null
+++ lldb/test/API/functionalities/breakpoint/thread_plan_user_breakpoint/Makefile
@@ -0,0 +1,8 @@
+CXX_SOURCES := main.cpp
+
+ifneq (,$(findstring icc,$(CC)))
+ CXXFLAGS_EXTRAS := -debug inline-debug-info
+endif
+
+
+include Makefile.rules
Index: lldb/source/Target/StopInfo.cpp
===================================================================
--- lldb/source/Target/StopInfo.cpp
+++ lldb/source/Target/StopInfo.cpp
@@ -421,8 +421,6 @@
continue;
}
- internal_breakpoint = bp_loc_sp->GetBreakpoint().IsInternal();
-
// First run the precondition, but since the precondition is per
// breakpoint, only run it once per breakpoint.
std::pair<std::unordered_set<break_id_t>::iterator, bool> result =
@@ -509,7 +507,7 @@
loc_desc.GetData());
// We want this stop reported, so you will know we auto-continued
// but only for external breakpoints:
- if (!internal_breakpoint)
+ if (!bp_loc_sp->GetBreakpoint().IsInternal())
thread_sp->SetShouldReportStop(eVoteYes);
auto_continue_says_stop = false;
}
@@ -539,6 +537,9 @@
actually_said_continue = true;
}
+ if (m_should_stop && !bp_loc_sp->GetBreakpoint().IsInternal())
+ internal_breakpoint = false;
+
// If we are going to stop for this breakpoint, then remove the
// breakpoint.
if (callback_says_stop && bp_loc_sp &&
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
