labath added a comment.

@clayborg, @shafik, what do you make of this version?

In D124370#3481338 <https://reviews.llvm.org/D124370#3481338>, @dblaikie wrote:

> Should the cleanups you found for the previous test case be included in this 
> patch (or a separate pre/post-commit?)? The description in the updated patch 
> sounds plausible - though delves into parts of lldb I'm not especially 
> qualified to make a firm assessment on.

I already checked those in --> rG089a1d9deba5 
<https://reviews.llvm.org/rG089a1d9deba5579487d77c97241b3f9e5e3d4745>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124370

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to