Skip to content

Commit

Permalink
Fix the order of modules-loaded event and the resultant breakpoint-ch…
Browse files Browse the repository at this point in the history
…anged event.

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

Differential Revision: https://reviews.llvm.org/D120919
  • Loading branch information
jimingham committed Mar 3, 2022
1 parent 3f43818 commit c697a1f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lldb/source/Target/Target.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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));
}
}

Expand Down

0 comments on commit c697a1f

Please sign in to comment.