[Lldb-commits] [PATCH] D12556: Use eAddressClassCode for address lookup for opcodes
Greg Clayton via lldb-commits
lldb-commits at lists.llvm.org
Fri Sep 4 10:05:50 PDT 2015
clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.
We really do need to restrict this for single stepping purposes. If the thread plans that single step and set breakpoints for stepping think they should place a breakpoint on 0x1004 if the example below:
0x1000: bx <addr> Non-tail call in a no return function
0x1004: [data-pool] Marked with $d mapping symbol
You will change the data with the software breakpoint instruction and change the flow of your program incorrectly. I do agree we should have a "force" option to allow this to be done by the user, but we need to do due diligence to make sure we don't do this in LLDB code.
Your updated changes look good though.
More information about the lldb-commits