fdeazeve added a comment. In D133858#3790013 <https://reviews.llvm.org/D133858#3790013>, @jingham wrote:
> Resetting the hit counts on rerun is a more useful behavior, so this is all > fine. But the Target is the one that does all the breakpoint management, so > I think the resetting should go through the Target, not the Process. And we > generally delegate "do on all breakpoints" operations to the BreakpointList, > so it would be more consistent with the current structure to go > Process::WillLaunch -> Target::ResetHitCounts -> > BreakpointList::ResetHitCounts. This makes sense! I've fixed this in the latest revision Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133858/new/ https://reviews.llvm.org/D133858 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
