[Lldb-commits] [PATCH] D134882: [lldb] Move breakpoint hit reset code to Target::CleanupProcess

Tue Oct 4 07:52:02 PDT 2022

labath added a comment.

In D134882#3831137 <https://reviews.llvm.org/D134882#3831137>, @jingham wrote:

> That seems fine.  I think it's useful to be able to see breakpoint hit counts up to the point where you start a new process.  From looking at the code, it looks like putting the clear in CleanupProcess will do that.  If you agree this is useful, can you add to the test that the hit count is still 1 between process.Kill and connecting to the new process?

That makes sense to me. I've added the extra check.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134882/new/

https://reviews.llvm.org/D134882