[Lldb-commits] [lldb] [lldb] Adds additional fields to ProcessInfo (PR #91544)
Jonas Devlieghere via lldb-commits
lldb-commits at lists.llvm.org
Mon May 20 10:59:59 PDT 2024
JDevlieghere wrote:
> Hello all. I am wondering what I can do to advance this patch? I think it is required to support process save-core in linux for lldb. I'd love to move this before adding static methods in ThreadEfCore.h to produce populated prpsinfo and prstatus structs for inclusion in a generated corefile. Is there someone else I should include on review here? @JDevlieghere? I am happy to amend this in whatever way the community thinks works best for lldb.
A message like this that the PR is ready for re-review should be all you need. Several folks left comments and they might be waiting for all of them to be addressed before they take another look. If you don't hear back within a week, you can ping the patch to get reviewer's attention.
There's some more info here: https://llvm.org/docs/CodeReview.html#lgtm-how-a-patch-is-accepted
https://github.com/llvm/llvm-project/pull/91544
More information about the lldb-commits
mailing list