labath added a subscriber: labath.
labath added a comment.

I believe this defeats the purpose of the whole test, which (I think) is to 
check that crashes during the actual "step" operation are handled correctly. We 
already have other tests which test crashing after a "continue" so I don't 
think this adds any value. A better course of action in my opinion would be to 
xfail or skip this test.

It will be tricky, but I think the proper way to make these kinds of things 
work with software single stepping is to detect (during the instruction 
emulation phase) that the instruction will crash (because it jumps to unmapped 
memory, for instance). Then you can omit putting the breakpoint and just issue 
the "continue" command normally, and the process will stop anyway because of 
the signal. This way the higher levels of the debugger (and the user) can be 
oblivious to the fact that software single stepping is in use, and everything 
will work normally.

But if having this working is not important for you, then I suggest you just go 
with XFAIL.


Repository:
  rL LLVM

http://reviews.llvm.org/D15182



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

Reply via email to