[PATCH] D95965: [DeveloperPolicy] Changes should not caused major compile-time regressions

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 3 12:00:48 PST 2021


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

Document that changes should not cause major correctness, performance or compile-time regressions. In line with the previous wording, this is kept vague and what constitutes a "major" regression is left up to case-by-case evaluation.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D95965

Files:
  llvm/docs/DeveloperPolicy.rst


Index: llvm/docs/DeveloperPolicy.rst
===================================================================
--- llvm/docs/DeveloperPolicy.rst
+++ llvm/docs/DeveloperPolicy.rst
@@ -211,7 +211,7 @@
 * The code should compile cleanly on all supported platforms.
 
 * The changes should not cause any correctness regressions in the ``llvm-test``
-  suite and must not cause any major performance regressions.
+  suite and must not cause any major performance or compile-time regressions.
 
 * The change set should not cause performance or correctness regressions for the
   LLVM tools.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95965.321180.patch
Type: text/x-patch
Size: 580 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210203/7c854dc4/attachment.bin>


More information about the llvm-commits mailing list