From a672ecefef25bcc8cf0d7ac8d54c5139f6386df4 Mon Sep 17 00:00:00 2001 From: Jim Ingham Date: Wed, 22 Oct 2014 01:54:17 +0000 Subject: The breakpoint location hit counts were getting incremented in BreakpointLocation::ShouldStop. That worked but wasn't really right, since there's nothing to guarantee that won't get called more than once. So this change moves that responsibility to the StopInfoBreakpoint directly, and then it uses the BreakpointSite to actually do the bumping. Also fix a test case that was assuming if you had many threads running some code with a breakpoint in it, the hit count when you stopped would always be 1. Many of the threads could have hit it at the same time... llvm-svn: 220358 --- lldb/source/Breakpoint/BreakpointLocationCollection.cpp | 1 + 1 file changed, 1 insertion(+) (limited to 'lldb/source/Breakpoint/BreakpointLocationCollection.cpp') diff --git a/lldb/source/Breakpoint/BreakpointLocationCollection.cpp b/lldb/source/Breakpoint/BreakpointLocationCollection.cpp index ee3f56f928d..5756ccedfaa 100644 --- a/lldb/source/Breakpoint/BreakpointLocationCollection.cpp +++ b/lldb/source/Breakpoint/BreakpointLocationCollection.cpp @@ -196,3 +196,4 @@ BreakpointLocationCollection::GetDescription (Stream *s, lldb::DescriptionLevel (*pos)->GetDescription(s, level); } } + -- cgit v1.2.3