[llvm] [llvm-symbolizer] Recognize AIX big archive (PR #150401)
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 8 01:16:20 PDT 2025
https://github.com/jh7370 commented:
Hi @midhuncodes7,
It's generally expected that ALL reviewer comments are responded to, either via code changes or comments in return, when updating a patch (see https://llvm.org/docs/CodeReview.html#acknowledge-all-reviewer-feedback). In general, there's no need to push updates until you've addressed everything and answered any comments. However, if you do want to push, you should make it clear that you have more changes incoming, either by putting the PR in draft state, or by simply saying as much in the comments. This is important, because otherwise it wastes reviewer time as they review a piece of work that hasn't been finished.
https://github.com/llvm/llvm-project/pull/150401
More information about the llvm-commits
mailing list