[PATCH] D89995: Make the post-commit review expectations more explicit with respect to revert

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini updated this revision to Diff 300126.
mehdi_amini added a comment.

Remove non-ascii char


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89995

Files:
  llvm/docs/CodeReview.rst


Index: llvm/docs/CodeReview.rst
===================================================================
--- llvm/docs/CodeReview.rst
+++ llvm/docs/CodeReview.rst
@@ -44,12 +44,21 @@
 promptly to post-commit feedback and address it. Failure to do so is cause for
 the patch to be reverted.
 
-In addition, if substantial problems are identified, it is expected that the
-patch is reverted and fixed offline. Before being recommitted, the patch
-generally undergoes further review, including by the community member who
-identified the problem and, in cases where the patch triggered a
-hardware-specific buildbot failure, a community member with access to hardware
-similar to that on the buildbot that the patch previously caused to fail.
+If a developer thinks that a recently committed patch would benefit from some
+more review, the patch should be reverted promptly (preferably by the original
+author, unless they are unresponsive). Developers often disagree, and erring on
+the side of the developer asking for more review prevents any lingering
+disagreement over code in the tree. This does not indicate any fault from the
+patch author, this is inherent also to our post-commit review practices.
+Reverting a patch ensures that design discussions can happen without blocking
+other development; it's entirely possible the patch will end up being reapplied
+essentially as is once concerns have been resolved.
+
+Before being recommitted, the patch generally undergoes further review,
+including by the community member who identified the problem and, in cases
+where the patch triggered a hardware-specific buildbot failure, a community
+member with access to hardware similar to that on the buildbot that the patch
+previously caused to fail.
 
 Please note: The bar for post-commit feedback is not higher than for pre-commit
 feedback. Don't delay unnecessarily in providing feedback. However, if you see


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D89995.300126.patch
Type: text/x-patch
Size: 1912 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201023/9c4ae7f2/attachment.bin>