<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 14, 2019 at 10:46 PM Finkel, Hal J. via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  3. All comments by reviewers should be addressed by the patch author. <br>
It is generally expected that suggested changes will be incorporated <br>
into the next revision of the patch unless the author and/or other <br>
reviewers can articulate a good reason to do otherwise (and then the <br>
reviewers must agree).<br></blockquote><div>I disagree on the high bar here. The author should acknowledge the comments; however, addressing all of the comments in one shot has similar problems as having commits that are too large (diffs between revisions become more difficult to review). This also leads to significant timing issues, where the comments made overnight in some time zone are addressed by the author locally, but someone added comments in the afternoon the next day before the author has a chance to post the new revision.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If you suggest changes in a code review, but <br>
don't wish the suggestion to be interpreted this strongly, please state <br>
so explicitly.<br>
</blockquote></div></div>