[Lldb-commits] [lldb] c697a1f - Fix the order of modules-loaded event and the resultant breakpoint-changed event.

[Jim Ingham via lldb-commits]

Author: Jim Ingham
Date: 2022-03-03T12:10:54-08:00
New Revision: c697a1f06b6240d62eec8890ac1314728906d707

URL: https://github.com/llvm/llvm-project/commit/c697a1f06b6240d62eec8890ac1314728906d707
DIFF: https://github.com/llvm/llvm-project/commit/c697a1f06b6240d62eec8890ac1314728906d707.diff

LOG: Fix the order of modules-loaded event and the resultant breakpoint-changed event.

The order used to be breakpoint-changed first, which didn't make much sense.

Differential Revision: https://reviews.llvm.org/D120919

Added: 
    

Modified: 
    lldb/source/Target/Target.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Target/Target.cpp b/lldb/source/Target/Target.cpp
index 298db3bca6803..14b5ecfacf384 100644
--- a/lldb/source/Target/Target.cpp
+++ b/lldb/source/Target/Target.cpp
@@ -1642,11 +1642,11 @@ void Target::SymbolsDidLoad(ModuleList &module_list) {
 void Target::ModulesDidUnload(ModuleList &module_list, bool delete_locations) {
   if (m_valid && module_list.GetSize()) {
     UnloadModuleSections(module_list);
+    BroadcastEvent(eBroadcastBitModulesUnloaded,
+                   new TargetEventData(this->shared_from_this(), module_list));
     m_breakpoint_list.UpdateBreakpoints(module_list, false, delete_locations);
     m_internal_breakpoint_list.UpdateBreakpoints(module_list, false,
                                                  delete_locations);
-    BroadcastEvent(eBroadcastBitModulesUnloaded,
-                   new TargetEventData(this->shared_from_this(), module_list));
   }
 }