[PATCH] D74538: [doc] Clarify responsibility for fixing experimental target problems
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 13 02:32:41 PST 2020
jhenderson created this revision.
jhenderson added reviewers: arsenm, rupprecht, asb, reames, thakis, lattner, chandlerc, jfb.
Herald added subscribers: dexonsmith, wdng.
Herald added a project: LLVM.
Experimental targets are meant to be maintained by the community behind the target. They are not monitored by the primary build bots. This change clarifies that it is this communities responsibility for things like test fixes related to the target caused by changes unrelated to that target.
See http://lists.llvm.org/pipermail/llvm-dev/2020-February/139115.html for a full discussion.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D74538
Files:
llvm/docs/DeveloperPolicy.rst
Index: llvm/docs/DeveloperPolicy.rst
===================================================================
--- llvm/docs/DeveloperPolicy.rst
+++ llvm/docs/DeveloperPolicy.rst
@@ -570,9 +570,16 @@
of reasons.
For these reasons, new targets are *always* added as *experimental* until
-they can be proven stable, and later moved to non-experimental. The difference
-between both classes is that experimental targets are not built by default
-(need to be added to -DLLVM_TARGETS_TO_BUILD at CMake time).
+they can be proven stable, and later moved to non-experimental. The differences
+between both classes are:
+
+* Experimental targets are not built by default (need to be added to
+ -DLLVM_TARGETS_TO_BUILD at CMake time).
+
+* It is solely the responsibility of the community behind the target to maintain
+ that target. This means that it is not appropriate to revert a change
+ unrelated to the target or ask for something in it to be fixed, if that change
+ only causes a problem when the experimental target is enabled.
The basic rules for a back-end to be upstreamed in **experimental** mode are:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74538.244368.patch
Type: text/x-patch
Size: 1113 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200213/ee9b8e51/attachment.bin>
More information about the llvm-commits
mailing list