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

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini created this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
mehdi_amini requested review of this revision.

See  http://lists.llvm.org/pipermail/llvm-dev/2016-March/096529.html for
context.


Repository:
  rG LLVM Github Monorepo

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.300123.patch
Type: text/x-patch
Size: 1914 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201022/afa0ab4c/attachment.bin>