Fix a cornercase in breakpoint reporting
authorPavel Labath <labath@google.com>
Wed, 6 Apr 2016 09:10:29 +0000 (09:10 +0000)
committerPavel Labath <labath@google.com>
Wed, 6 Apr 2016 09:10:29 +0000 (09:10 +0000)
commit3ce324af6b8df4be38c09e4fb571f7c99a9dc776
tree43d7d8fb817e93d2cdd374e1ee7591a418126683
parenta95e0effc01d0eb124c489058ae30922c8a5508d
Fix a cornercase in breakpoint reporting

Summary:
This resolves a similar problem as D16720 (which handled the case when we single-step onto a
breakpoint), but this one deals with involutary stops: when we stop a thread (e.g. because
another thread has hit a breakpont and we are doing a full stop), we can end up stopping it right
before it executes a breakpoint instruction. In this case, the stop reason will be empty, but we
will still step over the breakpoint when do the next resume, thereby missing a breakpoint hit.

I have observed this happening in TestConcurrentEvents, but I have no idea how to reproduce this
behavior more reliably.

Reviewers: clayborg

Subscribers: lldb-commits

Differential Revision: http://reviews.llvm.org/D18692

llvm-svn: 265525
lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp