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.


http://reviews.llvm.org/D12556



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to