diff options
author | Jim Ingham <jingham@apple.com> | 2015-07-23 19:55:02 +0000 |
---|---|---|
committer | Jim Ingham <jingham@apple.com> | 2015-07-23 19:55:02 +0000 |
commit | 9b03fa0c692acb19c5e9cfbb89932fef1c1ad1f9 (patch) | |
tree | 0b3b4b96bc9dc0da781321d8b1d372209a84000c /lldb/source/Target/ThreadPlanStepOut.cpp | |
parent | d53132214980e5540033569ee8220542a030f4cc (diff) | |
download | bcm5719-llvm-9b03fa0c692acb19c5e9cfbb89932fef1c1ad1f9.tar.gz bcm5719-llvm-9b03fa0c692acb19c5e9cfbb89932fef1c1ad1f9.zip |
Most thread plans don't handle eStopReasonInstrumentation stop reasons,
but that wasn't added to the list of reasons they don't explain. That
would mean we keep stepping after hitting the AsanDie breakpoint rather
than stopping when the Asan event occurred.
<rdar://problem/21925479>
llvm-svn: 243035
Diffstat (limited to 'lldb/source/Target/ThreadPlanStepOut.cpp')
-rw-r--r-- | lldb/source/Target/ThreadPlanStepOut.cpp | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/lldb/source/Target/ThreadPlanStepOut.cpp b/lldb/source/Target/ThreadPlanStepOut.cpp index 8b702a20ccd..9b39d1ceaf2 100644 --- a/lldb/source/Target/ThreadPlanStepOut.cpp +++ b/lldb/source/Target/ThreadPlanStepOut.cpp @@ -262,9 +262,7 @@ ThreadPlanStepOut::DoPlanExplainsStop (Event *event_ptr) if (stop_info_sp) { StopReason reason = stop_info_sp->GetStopReason(); - switch (reason) - { - case eStopReasonBreakpoint: + if (reason == eStopReasonBreakpoint) { // If this is OUR breakpoint, we're fine, otherwise we don't know why this happened... BreakpointSiteSP site_sp (m_thread.GetProcess()->GetBreakpointSiteList().FindByID (stop_info_sp->GetValue())); @@ -310,16 +308,10 @@ ThreadPlanStepOut::DoPlanExplainsStop (Event *event_ptr) } return false; } - case eStopReasonWatchpoint: - case eStopReasonSignal: - case eStopReasonException: - case eStopReasonExec: - case eStopReasonThreadExiting: + else if (IsUsuallyUnexplainedStopReason(reason)) return false; - - default: + else return true; - } } return true; } |