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,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