[Lldb-commits] [PATCH] D32168: [LLDB][MIPS] Fix TestStepOverBreakpoint.py failure

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Apr 18 09:06:18 PDT 2017


clayborg resigned from this revision.
clayborg added a reviewer: jingham.
clayborg added a comment.

Jim, can you take a look at this and see if this could be fixed in a nicer way? I would prefer to not see anything related to delay slots in a test. Can we abstract this better? Maybe ask the SBInstruction() if you can set a breakpoint on it instead of speaking in delay slot terms? Or maybe we ask the SBInstructionList to count the number of instructions between two addresses? Maybe something like:

  uint32_t SBInstructionList::GetInstructionCount(const SBAddress &start, const SBAddress &end); 

Or improve the test so it doesn't rely on instruction count??


https://reviews.llvm.org/D32168





More information about the lldb-commits mailing list