intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-02-06 23:31:50 +08:00
Code Issues Packages Projects Releases Wiki Activity

[lldb] Fix thread step until to not set breakpoint(s) on incorrect line numbers

Browse Source 
The requirements for "thread until <line number>" are:

a) If any code contributed by <line number> or the nearest subsequent of <line number> is executed before leaving the function, stop
b) If you end up leaving the function w/o triggering (a), then stop

In case of (a), since the <line number> may have multiple entries in the line table and the compiler might have scheduled/moved the relevant code across, and the lldb does not know the control flow, set breakpoints on all the line table entries of best match of <line number> i.e. exact or the nearest subsequent line.

Along with the above, currently, CommandObjectThreadUntil is also setting the breakpoints on all the subsequent line numbers after the best match and this latter part is wrong.

This issue is discussed at http://lists.llvm.org/pipermail/lldb-dev/2018-August/013979.html.

In fact, currently `TestStepUntil.py` is not actually testing step until scenarios and `test_missing_one` test fails without this patch if tests are made to run. Fixed the test as well.

Reviewed By: jingham

Differential Revision: https://reviews.llvm.org/D50304
This commit is contained in:
Venkata Ramanaiah Nalamothu
2022-06-02 23:19:30 +05:30
parent 4817b7729a
commit a57b62deef
2 changed files with 16 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
lldb/test/API/functionalities/thread/step_until/TestStepUntil.py
View File

@@ -19,7 +19,7 @@ class StepUntilTestCase(TestBase):
self.greater_than_two = line_number('main.c', 'Greater than or equal to 2.')
self.back_out_in_main = line_number('main.c', 'Back out in main')
def do_until (self, args, until_lines, expected_linenum):
def common_setup (self, args):
self.build()
exe = self.getBuildArtifact("a.out")
@@ -48,7 +48,8 @@ class StepUntilTestCase(TestBase):
thread = threads[0]
return thread
thread = self.common_setup(None)
def do_until (self, args, until_lines, expected_linenum):
thread = self.common_setup(args)
cmd_interp = self.dbg.GetCommandInterpreter()
ret_obj = lldb.SBCommandReturnObject()
@@ -77,7 +78,7 @@ class StepUntilTestCase(TestBase):
self.do_until(None, [self.less_than_two, self.greater_than_two], self.less_than_two)
def test_missing_one (self):
"""Test thread step until - targeting one line and missing it."""
"""Test thread step until - targeting one line and missing it by stepping out to call site"""
self.do_until(["foo", "bar", "baz"], [self.less_than_two], self.back_out_in_main)