diff --git a/llvm/docs/CodeReview.rst b/llvm/docs/CodeReview.rst --- a/llvm/docs/CodeReview.rst +++ b/llvm/docs/CodeReview.rst @@ -44,12 +44,23 @@ 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 shortly after landing a commit, a developer expresses concerns that would +have prevented from landing the patch in a pre-commit review (including around +the need for more design discussions) they may ask for a revert to the original +author who is responsible to revert the patch promptly. 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 should undergo further review. +The community member who identified the problem is expected to engage +actively in the review. 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