[Lldb-commits] [PATCH] D13296: [LLDB] Fix watchpoint ignore feature for architectures with watchpoint_exceptions_received=before

[Jim Ingham via lldb-commits]

jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.

This change alters the timing for the handling of ignore counts for watchpoints.  The original implementation (and the way ignore counts work for breakpoints) is that the breakpoint's ignore count gets checked during the "synchronous" phase of breakpoint stop evaluation.  So it happens when the private stop event is received, and if the ignore count is not satisfied then a public event is not generated.

Your version will move the decision making to the async ShouldStop, i.e. when an event is getting pulled from the public event queue.  If these were breakpoints it would mean that the synchronous callbacks would still fire even if they fail their ignore count test.  Watchpoints don't currently have synchronous callbacks - and I'm not sure we would ever need to support that.  If you moved the adjustment code you do in PerformAction to the synchronous callback (Watchpoint::ShouldStop) that would keep breakpoints & watchpoints behaving symmetrically.  It's worth trying that, I am pretty sure that will be a safe place to do your little dance.  But if it isn't, then I don't mind having it in PerformAction provided you put a note to that effect somewhere obvious in Watchpoint.h.

Note, if you end up going with the current patch, isn't quite right, however.  You need to move the check for the watchpoint ignore count up before printing out the old & new values.  An ignored watchpoint shouldn't print anything, it should be as if it never stopped.  Where you have the check, the old & new values will get printed even if the watchpoint is ignored.


Repository:
  rL LLVM

http://reviews.llvm.org/D13296