[Lldb-commits] [PATCH] D45554: Make sure deleting all breakpoints clears their sites first

[Eugene Zemtsov via Phabricator via lldb-commits]

eugene marked 2 inline comments as done.
eugene added a comment.

There is an ownership cycle between BreakpointSite::m_owners and BreakpointLocation::m_bp_site_sp.
We should probably make m_owners a collection of weak references. 
But currently most of the code just works it around by calling Breakpoint::ClearAllBreakpointSites() before deleting a breakpoint.


https://reviews.llvm.org/D45554