[Lldb-commits] [lldb] r185170 - Don't check for "are there any loaded sections" before trying to resolve a breakpoint site. We've already got a process,

Jim Ingham jingham at apple.com
Fri Jun 28 10:51:33 PDT 2013


Author: jingham
Date: Fri Jun 28 12:51:33 2013
New Revision: 185170

URL: http://llvm.org/viewvc/llvm-project?rev=185170&view=rev
Log:
Don't check for "are there any loaded sections" before trying to resolve a breakpoint site.  We've already got a process,
and any breakpoints with section relative addresses won't resolve their load addresses so they will error out at that point.

<rdar://problem/13900130>

Modified:
    lldb/trunk/source/Breakpoint/BreakpointLocation.cpp

Modified: lldb/trunk/source/Breakpoint/BreakpointLocation.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Breakpoint/BreakpointLocation.cpp?rev=185170&r1=185169&r2=185170&view=diff
==============================================================================
--- lldb/trunk/source/Breakpoint/BreakpointLocation.cpp (original)
+++ lldb/trunk/source/Breakpoint/BreakpointLocation.cpp Fri Jun 28 12:51:33 2013
@@ -484,9 +484,6 @@ BreakpointLocation::ResolveBreakpointSit
     if (process == NULL)
         return false;
 
-    if (m_owner.GetTarget().GetSectionLoadList().IsEmpty())
-        return false;
-
     lldb::break_id_t new_id = process->CreateBreakpointSite (shared_from_this(), false);
 
     if (new_id == LLDB_INVALID_BREAK_ID)





More information about the lldb-commits mailing list