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 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
