[Lldb-commits] [lldb] r170975 - /lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp

Jim Ingham jingham at apple.com
Fri Dec 21 17:52:33 PST 2012


Author: jingham
Date: Fri Dec 21 19:52:32 2012
New Revision: 170975

URL: http://llvm.org/viewvc/llvm-project?rev=170975&view=rev
Log:
Remember to tell the target about dyld when we resolve the address for it.

Modified:
    lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp

Modified: lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp?rev=170975&r1=170974&r2=170975&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp (original)
+++ lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp Fri Dec 21 19:52:32 2012
@@ -415,9 +415,16 @@
             // it again (since Target::SetExecutableModule() will clear the
             // images). So append the dyld module back to the list if it is
             /// unique!
-            if (dyld_module_sp && m_process->GetTarget().GetImages().AppendIfNeeded (dyld_module_sp))
-                UpdateImageLoadAddress(dyld_module_sp.get(), m_dyld);
+            if (dyld_module_sp)
+            {
+                if (m_process->GetTarget().GetImages().AppendIfNeeded (dyld_module_sp))
+                    UpdateImageLoadAddress(dyld_module_sp.get(), m_dyld);
 
+                // At this point we should have read in dyld's module, and so we should set breakpoints in it:
+                ModuleList modules;
+                modules.Append(dyld_module_sp);
+                m_process->GetTarget().ModulesDidLoad(modules);
+            }
             return true;
         }
     }





More information about the lldb-commits mailing list