1===================== 2LLVM Developer Policy 3===================== 4 5.. contents:: 6 :local: 7 8Introduction 9============ 10 11This document contains the LLVM Developer Policy which defines the project's 12policy towards developers and their contributions. The intent of this policy is 13to eliminate miscommunication, rework, and confusion that might arise from the 14distributed nature of LLVM's development. By stating the policy in clear terms, 15we hope each developer can know ahead of time what to expect when making LLVM 16contributions. This policy covers all llvm.org subprojects, including Clang, 17LLDB, libc++, etc. 18 19This policy is also designed to accomplish the following objectives: 20 21#. Attract both users and developers to the LLVM project. 22 23#. Make life as simple and easy for contributors as possible. 24 25#. Keep the top of tree as stable as possible. 26 27#. Establish awareness of the project's :ref:`copyright, license, and patent 28 policies <copyright-license-patents>` with contributors to the project. 29 30This policy is aimed at frequent contributors to LLVM. People interested in 31contributing one-off patches can do so in an informal way by sending them to the 32`llvm-commits mailing list 33<http://lists.llvm.org/mailman/listinfo/llvm-commits>`_ and engaging another 34developer to see it through the process. 35 36Developer Policies 37================== 38 39This section contains policies that pertain to frequent LLVM developers. We 40always welcome `one-off patches`_ from people who do not routinely contribute to 41LLVM, but we expect more from frequent contributors to keep the system as 42efficient as possible for everyone. Frequent LLVM contributors are expected to 43meet the following requirements in order for LLVM to maintain a high standard of 44quality. 45 46Stay Informed 47------------- 48 49Developers should stay informed by reading at least the "dev" mailing list for 50the projects you are interested in, such as `llvm-dev 51<http://lists.llvm.org/mailman/listinfo/llvm-dev>`_ for LLVM, `cfe-dev 52<http://lists.llvm.org/mailman/listinfo/cfe-dev>`_ for Clang, or `lldb-dev 53<http://lists.llvm.org/mailman/listinfo/lldb-dev>`_ for LLDB. If you are 54doing anything more than just casual work on LLVM, it is suggested that you also 55subscribe to the "commits" mailing list for the subproject you're interested in, 56such as `llvm-commits 57<http://lists.llvm.org/mailman/listinfo/llvm-commits>`_, `cfe-commits 58<http://lists.llvm.org/mailman/listinfo/cfe-commits>`_, or `lldb-commits 59<http://lists.llvm.org/mailman/listinfo/lldb-commits>`_. Reading the 60"commits" list and paying attention to changes being made by others is a good 61way to see what other people are interested in and watching the flow of the 62project as a whole. 63 64We recommend that active developers register an email account with `LLVM 65Bugzilla <https://bugs.llvm.org/>`_ and preferably subscribe to the `llvm-bugs 66<http://lists.llvm.org/mailman/listinfo/llvm-bugs>`_ email list to keep track 67of bugs and enhancements occurring in LLVM. We really appreciate people who are 68proactive at catching incoming bugs in their components and dealing with them 69promptly. 70 71Please be aware that all public LLVM mailing lists are public and archived, and 72that notices of confidentiality or non-disclosure cannot be respected. 73 74.. _patch: 75.. _one-off patches: 76 77Making and Submitting a Patch 78----------------------------- 79 80When making a patch for review, the goal is to make it as easy for the reviewer 81to read it as possible. As such, we recommend that you: 82 83#. Make your patch against git main, not a branch, and not an old version 84 of LLVM. This makes it easy to apply the patch. For information on how to 85 clone from git, please see the :ref:`Getting Started Guide 86 <checkout>`. 87 88#. Similarly, patches should be submitted soon after they are generated. Old 89 patches may not apply correctly if the underlying code changes between the 90 time the patch was created and the time it is applied. 91 92#. Patches should be made with ``git format-patch``, or similar (see special 93 commands for `Requesting Phabricator review via the web interface 94 <Phabricator.html#phabricator-request-review-web>`_ ). If you use a 95 different tool, make sure it uses the ``diff -u`` format and that it 96 doesn't contain clutter which makes it hard to read. 97 98Once your patch is ready, submit it by emailing it to the appropriate project's 99commit mailing list (or commit it directly if applicable). Alternatively, some 100patches get sent to the project's development list or component of the LLVM bug 101tracker, but the commit list is the primary place for reviews and should 102generally be preferred. 103 104When sending a patch to a mailing list, it is a good idea to send it as an 105*attachment* to the message, not embedded into the text of the message. This 106ensures that your mailer will not mangle the patch when it sends it (e.g. by 107making whitespace changes or by wrapping lines). 108 109*For Thunderbird users:* Before submitting a patch, please open *Preferences > 110Advanced > General > Config Editor*, find the key 111``mail.content_disposition_type``, and set its value to ``1``. Without this 112setting, Thunderbird sends your attachment using ``Content-Disposition: inline`` 113rather than ``Content-Disposition: attachment``. Apple Mail gamely displays such 114a file inline, making it difficult to work with for reviewers using that 115program. 116 117When submitting patches, please do not add confidentiality or non-disclosure 118notices to the patches themselves. These notices conflict with the LLVM 119licensing terms and may result in your contribution being excluded. 120 121.. _code review: 122 123Code Reviews 124------------ 125 126LLVM has a code-review policy. Code review is one way to increase the quality of 127software. Please see :doc:`CodeReview` for more information on LLVM's code-review 128process. 129 130.. _code owners: 131 132Code Owners 133----------- 134 135The LLVM Project relies on two features of its process to maintain rapid 136development in addition to the high quality of its source base: the combination 137of code review plus post-commit review for trusted maintainers. Having both is 138a great way for the project to take advantage of the fact that most people do 139the right thing most of the time, and only commit patches without pre-commit 140review when they are confident they are right. 141 142The trick to this is that the project has to guarantee that all patches that are 143committed are reviewed after they go in: you don't want everyone to assume 144someone else will review it, allowing the patch to go unreviewed. To solve this 145problem, we have a notion of an 'owner' for a piece of the code. The sole 146responsibility of a code owner is to ensure that a commit to their area of the 147code is appropriately reviewed, either by themself or by someone else. The list 148of current code owners can be found in the file `CODE_OWNERS.TXT 149<https://github.com/llvm/llvm-project/blob/main/llvm/CODE_OWNERS.TXT>`_ in the 150root of the LLVM source tree. 151 152Note that code ownership is completely different than reviewers: anyone can 153review a piece of code, and we welcome code review from anyone who is 154interested. Code owners are the "last line of defense" to guarantee that all 155patches that are committed are actually reviewed. 156 157Being a code owner is a somewhat unglamorous position, but it is incredibly 158important for the ongoing success of the project. Because people get busy, 159interests change, and unexpected things happen, code ownership is purely opt-in, 160and anyone can choose to resign their "title" at any time. For now, we do not 161have an official policy on how one gets elected to be a code owner. 162 163.. _include a testcase: 164 165Test Cases 166---------- 167 168Developers are required to create test cases for any bugs fixed and any new 169features added. Some tips for getting your testcase approved: 170 171* All feature and regression test cases are added to the ``llvm/test`` 172 directory. The appropriate sub-directory should be selected (see the 173 :doc:`Testing Guide <TestingGuide>` for details). 174 175* Test cases should be written in :doc:`LLVM assembly language <LangRef>`. 176 177* Test cases, especially for regressions, should be reduced as much as possible, 178 by :doc:`bugpoint <Bugpoint>` or manually. It is unacceptable to place an 179 entire failing program into ``llvm/test`` as this creates a *time-to-test* 180 burden on all developers. Please keep them short. 181 182Note that llvm/test and clang/test are designed for regression and small feature 183tests only. More extensive test cases (e.g., entire applications, benchmarks, 184etc) should be added to the ``llvm-test`` test suite. The llvm-test suite is 185for coverage (correctness, performance, etc) testing, not feature or regression 186testing. 187 188Quality 189------- 190 191The minimum quality standards that any change must satisfy before being 192committed to the main development branch are: 193 194#. Code must adhere to the `LLVM Coding Standards <CodingStandards.html>`_. 195 196#. Code must compile cleanly (no errors, no warnings) on at least one platform. 197 198#. Bug fixes and new features should `include a testcase`_ so we know if the 199 fix/feature ever regresses in the future. 200 201#. Code must pass the ``llvm/test`` test suite. 202 203#. The code must not cause regressions on a reasonable subset of llvm-test, 204 where "reasonable" depends on the contributor's judgement and the scope of 205 the change (more invasive changes require more testing). A reasonable subset 206 might be something like "``llvm-test/MultiSource/Benchmarks``". 207 208Additionally, the committer is responsible for addressing any problems found in 209the future that the change is responsible for. For example: 210 211* The code should compile cleanly on all supported platforms. 212 213* The changes should not cause any correctness regressions in the ``llvm-test`` 214 suite and must not cause any major performance regressions. 215 216* The change set should not cause performance or correctness regressions for the 217 LLVM tools. 218 219* The changes should not cause performance or correctness regressions in code 220 compiled by LLVM on all applicable targets. 221 222* You are expected to address any `Bugzilla bugs <https://bugs.llvm.org/>`_ that 223 result from your change. 224 225We prefer for this to be handled before submission but understand that it isn't 226possible to test all of this for every submission. Our build bots and nightly 227testing infrastructure normally finds these problems. A good rule of thumb is 228to check the nightly testers for regressions the day after your change. Build 229bots will directly email you if a group of commits that included yours caused a 230failure. You are expected to check the build bot messages to see if they are 231your fault and, if so, fix the breakage. 232 233Commits that violate these quality standards (e.g. are very broken) may be 234reverted. This is necessary when the change blocks other developers from making 235progress. The developer is welcome to re-commit the change after the problem has 236been fixed. 237 238.. _commit messages: 239 240Commit messages 241--------------- 242 243Although we don't enforce the format of commit messages, we prefer that 244you follow these guidelines to help review, search in logs, email formatting 245and so on. These guidelines are very similar to rules used by other open source 246projects. 247 248Most importantly, the contents of the message should be carefully written to 249convey the rationale of the change (without delving too much in detail). It 250also should avoid being vague or overly specific. For example, "bits were not 251set right" will leave the reviewer wondering about which bits, and why they 252weren't right, while "Correctly set overflow bits in TargetInfo" conveys almost 253all there is to the change. 254 255Below are some guidelines about the format of the message itself: 256 257* Separate the commit message into title and body separated by a blank line. 258 259* If you're not the original author, ensure the 'Author' property of the commit is 260 set to the original author and the 'Committer' property is set to yourself. 261 You can use a command similar to 262 ``git commit --amend --author="John Doe <[email protected]>"`` to correct the 263 author property if it is incorrect. See `Attribution of Changes`_ for more 264 information including the method we used for attribution before the project 265 migrated to git. 266 267* The title should be concise. Because all commits are emailed to the list with 268 the first line as the subject, long titles are frowned upon. Short titles 269 also look better in `git log`. 270 271* When the changes are restricted to a specific part of the code (e.g. a 272 back-end or optimization pass), it is customary to add a tag to the 273 beginning of the line in square brackets. For example, "[SCEV] ..." 274 or "[OpenMP] ...". This helps email filters and searches for post-commit 275 reviews. 276 277* The body, if it exists, should be separated from the title by an empty line. 278 279* The body should be concise, but explanatory, including a complete 280 reasoning. Unless it is required to understand the change, examples, 281 code snippets and gory details should be left to bug comments, web 282 review or the mailing list. 283 284* If the patch fixes a bug in bugzilla, please include the PR# in the message. 285 286* Text formatting and spelling should follow the same rules as documentation 287 and in-code comments, ex. capitalization, full stop, etc. 288 289* If the commit is a bug fix on top of another recently committed patch, or a 290 revert or reapply of a patch, include the git commit hash of the prior 291 related commit. This could be as simple as "Revert commit NNNN because it 292 caused PR#". 293 294For minor violations of these recommendations, the community normally favors 295reminding the contributor of this policy over reverting. Minor corrections and 296omissions can be handled by sending a reply to the commits mailing list. 297 298Obtaining Commit Access 299----------------------- 300 301We grant commit access to contributors with a track record of submitting high 302quality patches. If you would like commit access, please send an email to 303`Chris <mailto:[email protected]>`_ with your GitHub username. This is true 304for former contributors with SVN access as well as new contributors. 305 306Prior to obtaining commit access, it is common practice to request that 307someone with commit access commits on your behalf. When doing so, please 308provide the name and email address you would like to use in the Author 309property of the commit. 310 311Your first commit to a repository may require the autogenerated email to be 312approved by a moderator of the mailing list. 313This is normal and will be done when the mailing list owner has time. 314 315If you have recently been granted commit access, these policies apply: 316 317#. You are granted *commit-after-approval* to all parts of LLVM. For 318 information on how to get approval for a patch, please see :doc:`CodeReview`. 319 When approved, you may commit it yourself. 320 321#. You are allowed to commit patches without approval which you think are 322 obvious. This is clearly a subjective decision --- we simply expect you to 323 use good judgement. Examples include: fixing build breakage, reverting 324 obviously broken patches, documentation/comment changes, any other minor 325 changes. Avoid committing formatting- or whitespace-only changes outside of 326 code you plan to make subsequent changes to. Also, try to separate 327 formatting or whitespace changes from functional changes, either by 328 correcting the format first (ideally) or afterward. Such changes should be 329 highly localized and the commit message should clearly state that the commit 330 is not intended to change functionality, usually by stating it is 331 :ref:`NFC <nfc>`. 332 333#. You are allowed to commit patches without approval to those portions of LLVM 334 that you have contributed or maintain (i.e., have been assigned 335 responsibility for), with the proviso that such commits must not break the 336 build. This is a "trust but verify" policy, and commits of this nature are 337 reviewed after they are committed. 338 339#. Multiple violations of these policies or a single egregious violation may 340 cause commit access to be revoked. 341 342In any case, your changes are still subject to `code review`_ (either before or 343after they are committed, depending on the nature of the change). You are 344encouraged to review other peoples' patches as well, but you aren't required 345to do so. 346 347.. _discuss the change/gather consensus: 348 349Making a Major Change 350--------------------- 351 352When a developer begins a major new project with the aim of contributing it back 353to LLVM, they should inform the community with an email to the `llvm-dev 354<http://lists.llvm.org/mailman/listinfo/llvm-dev>`_ email list, to the extent 355possible. The reason for this is to: 356 357#. keep the community informed about future changes to LLVM, 358 359#. avoid duplication of effort by preventing multiple parties working on the 360 same thing and not knowing about it, and 361 362#. ensure that any technical issues around the proposed work are discussed and 363 resolved before any significant work is done. 364 365The design of LLVM is carefully controlled to ensure that all the pieces fit 366together well and are as consistent as possible. If you plan to make a major 367change to the way LLVM works or want to add a major new extension, it is a good 368idea to get consensus with the development community before you start working on 369it. 370 371Once the design of the new feature is finalized, the work itself should be done 372as a series of `incremental changes`_, not as a long-term development branch. 373 374.. _incremental changes: 375 376Incremental Development 377----------------------- 378 379In the LLVM project, we do all significant changes as a series of incremental 380patches. We have a strong dislike for huge changes or long-term development 381branches. Long-term development branches have a number of drawbacks: 382 383#. Branches must have mainline merged into them periodically. If the branch 384 development and mainline development occur in the same pieces of code, 385 resolving merge conflicts can take a lot of time. 386 387#. Other people in the community tend to ignore work on branches. 388 389#. Huge changes (produced when a branch is merged back onto mainline) are 390 extremely difficult to `code review`_. 391 392#. Branches are not routinely tested by our nightly tester infrastructure. 393 394#. Changes developed as monolithic large changes often don't work until the 395 entire set of changes is done. Breaking it down into a set of smaller 396 changes increases the odds that any of the work will be committed to the main 397 repository. 398 399To address these problems, LLVM uses an incremental development style and we 400require contributors to follow this practice when making a large/invasive 401change. Some tips: 402 403* Large/invasive changes usually have a number of secondary changes that are 404 required before the big change can be made (e.g. API cleanup, etc). These 405 sorts of changes can often be done before the major change is done, 406 independently of that work. 407 408* The remaining inter-related work should be decomposed into unrelated sets of 409 changes if possible. Once this is done, define the first increment and get 410 consensus on what the end goal of the change is. 411 412* Each change in the set can be stand alone (e.g. to fix a bug), or part of a 413 planned series of changes that works towards the development goal. 414 415* Each change should be kept as small as possible. This simplifies your work 416 (into a logical progression), simplifies code review and reduces the chance 417 that you will get negative feedback on the change. Small increments also 418 facilitate the maintenance of a high quality code base. 419 420* Often, an independent precursor to a big change is to add a new API and slowly 421 migrate clients to use the new API. Each change to use the new API is often 422 "obvious" and can be committed without review. Once the new API is in place 423 and used, it is much easier to replace the underlying implementation of the 424 API. This implementation change is logically separate from the API 425 change. 426 427If you are interested in making a large change, and this scares you, please make 428sure to first `discuss the change/gather consensus`_ then ask about the best way 429to go about making the change. 430 431Attribution of Changes 432---------------------- 433 434When contributors submit a patch to an LLVM project, other developers with 435commit access may commit it for the author once appropriate (based on the 436progression of code review, etc.). When doing so, it is important to retain 437correct attribution of contributions to their contributors. However, we do not 438want the source code to be littered with random attributions "this code written 439by J. Random Hacker" (this is noisy and distracting). In practice, the revision 440control system keeps a perfect history of who changed what, and the CREDITS.txt 441file describes higher-level contributions. If you commit a patch for someone 442else, please follow the attribution of changes in the simple manner as outlined 443by the `commit messages`_ section. Overall, please do not add contributor names 444to the source code. 445 446Also, don't commit patches authored by others unless they have submitted the 447patch to the project or you have been authorized to submit them on their behalf 448(you work together and your company authorized you to contribute the patches, 449etc.). The author should first submit them to the relevant project's commit 450list, development list, or LLVM bug tracker component. If someone sends you 451a patch privately, encourage them to submit it to the appropriate list first. 452 453Our previous version control system (subversion) did not distinguish between the 454author and the committer like git does. As such, older commits used a different 455attribution mechanism. The previous method was to include "Patch by John Doe." 456in a separate line of the commit message and there are automated processes that 457rely on this format. 458 459.. _IR backwards compatibility: 460 461IR Backwards Compatibility 462-------------------------- 463 464When the IR format has to be changed, keep in mind that we try to maintain some 465backwards compatibility. The rules are intended as a balance between convenience 466for llvm users and not imposing a big burden on llvm developers: 467 468* The textual format is not backwards compatible. We don't change it too often, 469 but there are no specific promises. 470 471* Additions and changes to the IR should be reflected in 472 ``test/Bitcode/compatibility.ll``. 473 474* The current LLVM version supports loading any bitcode since version 3.0. 475 476* After each X.Y release, ``compatibility.ll`` must be copied to 477 ``compatibility-X.Y.ll``. The corresponding bitcode file should be assembled 478 using the X.Y build and committed as ``compatibility-X.Y.ll.bc``. 479 480* Newer releases can ignore features from older releases, but they cannot 481 miscompile them. For example, if nsw is ever replaced with something else, 482 dropping it would be a valid way to upgrade the IR. 483 484* Debug metadata is special in that it is currently dropped during upgrades. 485 486* Non-debug metadata is defined to be safe to drop, so a valid way to upgrade 487 it is to drop it. That is not very user friendly and a bit more effort is 488 expected, but no promises are made. 489 490C API Changes 491---------------- 492 493* Stability Guarantees: The C API is, in general, a "best effort" for stability. 494 This means that we make every attempt to keep the C API stable, but that 495 stability will be limited by the abstractness of the interface and the 496 stability of the C++ API that it wraps. In practice, this means that things 497 like "create debug info" or "create this type of instruction" are likely to be 498 less stable than "take this IR file and JIT it for my current machine". 499 500* Release stability: We won't break the C API on the release branch with patches 501 that go on that branch, with the exception that we will fix an unintentional 502 C API break that will keep the release consistent with both the previous and 503 next release. 504 505* Testing: Patches to the C API are expected to come with tests just like any 506 other patch. 507 508* Including new things into the API: If an LLVM subcomponent has a C API already 509 included, then expanding that C API is acceptable. Adding C API for 510 subcomponents that don't currently have one needs to be discussed on the 511 mailing list for design and maintainability feedback prior to implementation. 512 513* Documentation: Any changes to the C API are required to be documented in the 514 release notes so that it's clear to external users who do not follow the 515 project how the C API is changing and evolving. 516 517.. _toolchain: 518 519Updating Toolchain Requirements 520------------------------------- 521 522We intend to require newer toolchains as time goes by. This means LLVM's 523codebase can use newer versions of C++ as they get standardized. Requiring newer 524toolchains to build LLVM can be painful for those building LLVM; therefore, it 525will only be done through the following process: 526 527 * It is a general goal to support LLVM and GCC versions from the last 3 years 528 at a minimum. This time-based guideline is not strict: we may support much 529 older compilers, or decide to support fewer versions. 530 531 * An RFC is sent to the `llvm-dev mailing list`_ 532 533 - Detail upsides of the version increase (e.g. which newer C++ language or 534 library features LLVM should use; avoid miscompiles in particular compiler 535 versions, etc). 536 - Detail downsides on important platforms (e.g. Ubuntu LTS status). 537 538 * Once the RFC reaches consensus, update the CMake toolchain version checks as 539 well as the :doc:`getting started<GettingStarted>` guide. This provides a 540 softer transition path for developers compiling LLVM, because the 541 error can be turned into a warning using a CMake flag. This is an important 542 step: LLVM still doesn't have code which requires the new toolchains, but it 543 soon will. If you compile LLVM but don't read the mailing list, we should 544 tell you! 545 546 * Ensure that at least one LLVM release has had this soft-error. Not all 547 developers compile LLVM top-of-tree. These release-bound developers should 548 also be told about upcoming changes. 549 550 * Turn the soft-error into a hard-error after said LLVM release has branched. 551 552 * Update the :doc:`coding standards<CodingStandards>` to allow the new 553 features we've explicitly approved in the RFC. 554 555 * Start using the new features in LLVM's codebase. 556 557Here's a `sample RFC 558<http://lists.llvm.org/pipermail/llvm-dev/2019-January/129452.html>`_ and the 559`corresponding change <https://reviews.llvm.org/D57264>`_. 560 561.. _new-llvm-components: 562 563Introducing New Components into LLVM 564==================================== 565 566The LLVM community is a vibrant and exciting place to be, and we look to be 567inclusive of new projects and foster new communities, and increase 568collaboration across industry and academia. 569 570That said, we need to strike a balance between being inclusive of new ideas and 571people and the cost of ongoing maintenance that new code requires. As such, we 572have a general :doc:`support policy<SupportPolicy>` for introducing major new 573components into the LLVM world, depending on the degree of detail and 574responsibility required. *Core* projects need a higher degree of scrutiny 575than *peripheral* projects, and the latter may have additional differences. 576 577However, this is really only intended to cover common cases 578that we have seen arise: different situations are different, and we are open 579to discussing unusual cases as well - just start an RFC thread on the 580`llvm-dev mailing list`_. 581 582Adding a New Target 583------------------- 584 585LLVM is very receptive to new targets, even experimental ones, but a number of 586problems can appear when adding new large portions of code, and back-ends are 587normally added in bulk. New targets need the same level of support as other 588*core* parts of the compiler, so they are covered in the *core tier* of our 589:doc:`support policy<SupportPolicy>`. 590 591We have found that landing large pieces of new code and then trying to fix 592emergent problems in-tree is problematic for a variety of reasons. For these 593reasons, new targets are *always* added as *experimental* until they can be 594proven stable, and later moved to non-experimental. 595 596The differences between both classes are: 597 598* Experimental targets are not built by default (they need to be explicitly 599 enabled at CMake time). 600 601* Test failures, bugs, and build breakages that only appear when the 602 experimental target is enabled, caused by changes unrelated to the target, are 603 the responsibility of the community behind the target to fix. 604 605The basic rules for a back-end to be upstreamed in **experimental** mode are: 606 607* Every target must have a :ref:`code owner<code owners>`. The `CODE_OWNERS.TXT` 608 file has to be updated as part of the first merge. The code owner makes sure 609 that changes to the target get reviewed and steers the overall effort. 610 611* There must be an active community behind the target. This community 612 will help maintain the target by providing buildbots, fixing 613 bugs, answering the LLVM community's questions and making sure the new 614 target doesn't break any of the other targets, or generic code. This 615 behavior is expected to continue throughout the lifetime of the 616 target's code. 617 618* The code must be free of contentious issues, for example, large 619 changes in how the IR behaves or should be formed by the front-ends, 620 unless agreed by the majority of the community via refactoring of the 621 (:doc:`IR standard<LangRef>`) **before** the merge of the new target changes, 622 following the :ref:`IR backwards compatibility`. 623 624* The code conforms to all of the policies laid out in this developer policy 625 document, including license, patent, and coding standards. 626 627* The target should have either reasonable documentation on how it 628 works (ISA, ABI, etc.) or a publicly available simulator/hardware 629 (either free or cheap enough) - preferably both. This allows 630 developers to validate assumptions, understand constraints and review code 631 that can affect the target. 632 633In addition, the rules for a back-end to be promoted to **official** are: 634 635* The target must have addressed every other minimum requirement and 636 have been stable in tree for at least 3 months. This cool down 637 period is to make sure that the back-end and the target community can 638 endure continuous upstream development for the foreseeable future. 639 640* The target's code must have been completely adapted to this policy 641 as well as the :doc:`coding standards<CodingStandards>`. Any exceptions that 642 were made to move into experimental mode must have been fixed **before** 643 becoming official. 644 645* The test coverage needs to be broad and well written (small tests, 646 well documented). The build target ``check-all`` must pass with the 647 new target built, and where applicable, the ``test-suite`` must also 648 pass without errors, in at least one configuration (publicly 649 demonstrated, for example, via buildbots). 650 651* Public buildbots need to be created and actively maintained, unless 652 the target requires no additional buildbots (ex. ``check-all`` covers 653 all tests). The more relevant and public the new target's CI infrastructure 654 is, the more the LLVM community will embrace it. 655 656To **continue** as a supported and official target: 657 658* The maintainer(s) must continue following these rules throughout the lifetime 659 of the target. Continuous violations of aforementioned rules and policies 660 could lead to complete removal of the target from the code base. 661 662* Degradation in support, documentation or test coverage will make the target as 663 nuisance to other targets and be considered a candidate for deprecation and 664 ultimately removed. 665 666In essences, these rules are necessary for targets to gain and retain their 667status, but also markers to define bit-rot, and will be used to clean up the 668tree from unmaintained targets. 669 670Adding an Established Project To the LLVM Monorepo 671-------------------------------------------------- 672 673The `LLVM monorepo <https://github.com/llvm/llvm-project>`_ is the centerpoint 674of development in the LLVM world, and has all of the primary LLVM components, 675including the LLVM optimizer and code generators, Clang, LLDB, etc. `Monorepos 676in general <https://en.wikipedia.org/wiki/Monorepo>`_ are great because they 677allow atomic commits to the project, simplify CI, and make it easier for 678subcommunities to collaborate. 679 680Like new targets, most projects already in the monorepo are considered to be in 681the *core tier* of our :doc:`support policy<SupportPolicy>`. The burden to add 682things to the LLVM monorepo needs to be very high - code that is added to this 683repository is checked out by everyone in the community. As such, we hold 684components to a high bar similar to "official targets", they: 685 686 * Must be generally aligned with the mission of the LLVM project to advance 687 compilers, languages, tools, runtimes, etc. 688 * Must conform to all of the policies laid out in this developer policy 689 document, including license, patent, coding standards, and code of conduct. 690 * Must have an active community that maintains the code, including established 691 code owners. 692 * Should have reasonable documentation about how it works, including a high 693 quality README file. 694 * Should have CI to catch breakage within the project itself or due to 695 underlying LLVM dependencies. 696 * Should have code free of issues the community finds contentious, or be on a 697 clear path to resolving them. 698 * Must be proposed through the LLVM RFC process, and have its addition approved 699 by the LLVM community - this ultimately mediates the resolution of the 700 "should" concerns above. 701 702If you have a project that you think would make sense to add to the LLVM 703monorepo, please start an RFC thread on the `llvm-dev mailing list`_ to kick off 704the discussion. This process can take some time and iteration - please don’t 705be discouraged or intimidated by that! 706 707If you have an earlier stage project that you think is aligned with LLVM, please 708see the "Incubating New Projects" section. 709 710Incubating New Projects 711----------------------- 712 713The burden to add a new project to the LLVM monorepo is intentionally very high, 714but that can have a chilling effect on new and innovative projects. To help 715foster these sorts of projects, LLVM supports an "incubator" process that is 716much easier to get started with. It provides space for potentially valuable, 717new top-level and sub-projects to reach a critical mass before they have enough 718code to prove their utility and grow a community. This also allows 719collaboration between teams that already have permissions to make contributions 720to projects under the LLVM umbrella. 721 722Projects which can be considered for the LLVM incubator meet the following 723criteria: 724 725 * Must be generally aligned with the mission of the LLVM project to advance 726 compilers, languages, tools, runtimes, etc. 727 * Must conform to the license, patent, and code of conduct policies laid out 728 in this developer policy document. 729 * Must have a documented charter and development plan, e.g. in the form of a 730 README file, mission statement, and/or manifesto. 731 * Should conform to coding standards, incremental development process, and 732 other expectations. 733 * Should have a sense of the community that it hopes to eventually foster, and 734 there should be interest from members with different affiliations / 735 organizations. 736 * Should have a feasible path to eventually graduate as a dedicated top-level 737 or sub-project within the `LLVM monorepo 738 <https://github.com/llvm/llvm-project>`_. 739 * Should include a notice (e.g. in the project README or web page) that the 740 project is in ‘incubation status’ and is not included in LLVM releases (see 741 suggested wording below). 742 * Must be proposed through the LLVM RFC process, and have its addition 743 approved by the LLVM community - this ultimately mediates the resolution of 744 the "should" concerns above. 745 746That said, the project need not have any code to get started, and need not have 747an established community at all! Furthermore, incubating projects may pass 748through transient states that violate the "Should" guidelines above, or would 749otherwise make them unsuitable for direct inclusion in the monorepo (e.g. 750dependencies that have not yet been factored appropriately, leveraging 751experimental components or APIs that are not yet upstream, etc). 752 753When approved, the llvm-admin group can grant the new project: 754 * A new repository in the LLVM Github Organization - but not the LLVM monorepo. 755 * New mailing list, discourse forum, and/or discord chat hosted with other LLVM 756 forums. 757 * Other infrastructure integration can be discussed on a case-by-case basis. 758 759Graduation to the mono-repo would follow existing processes and standards for 760becoming a first-class part of the monorepo. Similarly, an incubating project 761may be eventually retired, but no process has been established for that yet. If 762and when this comes up, please start an RFC discussion on llvm-dev. 763 764This process is very new - please expect the details to change, it is always 765safe to ask on the `llvm-dev mailing list`_ about this. 766 767Suggested disclaimer for the project README and the main project web page: 768 769:: 770 771 This project is participating in the LLVM Incubator process: as such, it is 772 not part of any official LLVM release. While incubation status is not 773 necessarily a reflection of the completeness or stability of the code, it 774 does indicate that the project is not yet endorsed as a component of LLVM. 775 776.. _copyright-license-patents: 777 778Copyright, License, and Patents 779=============================== 780 781.. note:: 782 783 This section deals with legal matters but does not provide legal advice. We 784 are not lawyers --- please seek legal counsel from a licensed attorney. 785 786This section addresses the issues of copyright, license and patents for the LLVM 787project. The copyright for the code is held by the contributors of 788the code. The code is licensed under permissive `open source licensing terms`_, 789namely the Apache-2.0 with LLVM-exception license, which includes a copyright 790and `patent license`_. When you contribute code to the LLVM project, you 791license it under these terms. 792 793If you have questions or comments about these topics, please contact the 794`LLVM Developer's Mailing List <mailto:[email protected]>`_. However, 795please realize that most compiler developers are not lawyers, and therefore you 796will not be getting official legal advice. 797 798Copyright 799--------- 800 801The LLVM project does not collect copyright assignments, which means that the 802copyright for the code in the project is held by the respective contributors. 803Because you (or your company) 804retain ownership of the code you contribute, you know it may only be used under 805the terms of the open source license you contributed it under: the license for 806your contributions cannot be changed in the future without your approval. 807 808Because the LLVM project does not require copyright assignments, changing the 809LLVM license requires tracking down the 810contributors to LLVM and getting them to agree that a license change is 811acceptable for their contributions. We feel that a high burden for relicensing 812is good for the project, because contributors do not have to fear that their 813code will be used in a way with which they disagree. 814 815Relicensing 816----------- 817 818The last paragraph notwithstanding, the LLVM Project is in the middle of a large 819effort to change licenses, which aims to solve several problems: 820 821* The old licenses made it difficult to move code from (e.g.) the compiler to 822 runtime libraries, because runtime libraries used a different license from the 823 rest of the compiler. 824* Some contributions were not submitted to LLVM due to concerns that 825 the patent grant required by the project was overly broad. 826* The patent grant was unique to the LLVM Project, not written by a lawyer, and 827 was difficult to determine what protection was provided (if any). 828 829The scope of relicensing is all code that is considered part of the LLVM 830project, including the main LLVM repository, runtime libraries (compiler_rt, 831OpenMP, etc), Polly, and all other subprojects. There are a few exceptions: 832 833* Code imported from other projects (e.g. Google Test, Autoconf, etc) will 834 remain as it is. This code isn't developed as part of the LLVM project, it 835 is used by LLVM. 836* Some subprojects are impractical or uninteresting to relicense (e.g. llvm-gcc 837 and dragonegg). These will be split off from the LLVM project (e.g. to 838 separate GitHub projects), allowing interested people to continue their 839 development elsewhere. 840 841To relicense LLVM, we will be seeking approval from all of the copyright holders 842of code in the repository, or potentially remove/rewrite code if we cannot. 843This is a large 844and challenging project which will take a significant amount of time to 845complete. In the interim, **all contributions to the project will be made under 846the terms of both the new license and the legacy license scheme** (each of which 847is described below). The exception to this is the legacy patent grant, which 848will not be required for new contributions. 849 850When all of the code in the project has been converted to the new license or 851removed, we will drop the requirement to contribute under the legacy license. 852This will achieve the goal of having 853a single standardized license for the entire codebase. 854 855If you are a prior contributor to LLVM and have not done so already, please do 856*TODO* to allow us to use your code. *Add a link to a separate page here, which 857is probably a click through web form or something like that. Details to be 858determined later*. 859 860 861.. _open source licensing terms: 862 863New LLVM Project License Framework 864---------------------------------- 865 866Contributions to LLVM are licensed under the `Apache License, Version 2.0 867<https://www.apache.org/licenses/LICENSE-2.0>`_, with two limited 868exceptions intended to ensure that LLVM is very permissively licensed. 869Collectively, the name of this license is "Apache 2.0 License with LLVM 870exceptions". The exceptions read: 871 872:: 873 874 ---- LLVM Exceptions to the Apache 2.0 License ---- 875 876 As an exception, if, as a result of your compiling your source code, portions 877 of this Software are embedded into an Object form of such source code, you 878 may redistribute such embedded portions in such Object form without complying 879 with the conditions of Sections 4(a), 4(b) and 4(d) of the License. 880 881 In addition, if you combine or link compiled forms of this Software with 882 software that is licensed under the GPLv2 ("Combined Software") and if a 883 court of competent jurisdiction determines that the patent provision (Section 884 3), the indemnity provision (Section 9) or other Section of the License 885 conflicts with the conditions of the GPLv2, you may retroactively and 886 prospectively choose to deem waived or otherwise exclude such Section(s) of 887 the License, but only in their entirety and only with respect to the Combined 888 Software. 889 890 891We intend to keep LLVM perpetually open source and available under a permissive 892license - this fosters the widest adoption of LLVM by 893**allowing commercial products to be derived from LLVM** with few restrictions 894and without a requirement for making any derived works also open source. In 895particular, LLVM's license is not a "copyleft" license like the GPL. 896 897The "Apache 2.0 License with LLVM exceptions" allows you to: 898 899* freely download and use LLVM (in whole or in part) for personal, internal, or 900 commercial purposes. 901* include LLVM in packages or distributions you create. 902* combine LLVM with code licensed under every other major open source 903 license (including BSD, MIT, GPLv2, GPLv3...). 904* make changes to LLVM code without being required to contribute it back 905 to the project - contributions are appreciated though! 906 907However, it imposes these limitations on you: 908 909* You must retain the copyright notice if you redistribute LLVM: You cannot 910 strip the copyright headers off or replace them with your own. 911* Binaries that include LLVM must reproduce the copyright notice (e.g. in an 912 included README file or in an "About" box), unless the LLVM code was added as 913 a by-product of compilation. For example, if an LLVM runtime library like 914 compiler_rt or libc++ was automatically included into your application by the 915 compiler, you do not need to attribute it. 916* You can't use our names to promote your products (LLVM derived or not) - 917 though you can make truthful statements about your use of the LLVM code, 918 without implying our sponsorship. 919* There's no warranty on LLVM at all. 920 921We want LLVM code to be widely used, and believe that this provides a model that 922is great for contributors and users of the project. For more information about 923the Apache 2.0 License, please see the `Apache License FAQ 924<http://www.apache.org/foundation/license-faq.html>`_, maintained by the 925Apache Project. 926 927 928.. note:: 929 930 The LLVM Project includes some really old subprojects (dragonegg, 931 llvm-gcc-4.0, and llvm-gcc-4.2), which are licensed under **GPL 932 licenses**. This code is not actively maintained - it does not even 933 build successfully. This code is cleanly separated into distinct SVN 934 repositories from the rest of LLVM, and the LICENSE.txt files specifically 935 indicate that they contain GPL code. When LLVM transitions from SVN to Git, 936 we plan to drop these code bases from the new repository structure. 937 938 939.. _patent license: 940 941Patents 942------- 943 944Section 3 of the Apache 2.0 license is a patent grant under which 945contributors of code to the project contribute the rights to use any of 946their patents that would otherwise be infringed by that code contribution 947(protecting uses of that code). Further, the patent grant is revoked 948from anyone who files a patent lawsuit about code in LLVM - this protects the 949community by providing a "patent commons" for the code base and reducing the 950odds of patent lawsuits in general. 951 952The license specifically scopes which patents are included with code 953contributions. To help explain this, the `Apache License FAQ 954<http://www.apache.org/foundation/license-faq.html>`_ explains this scope using 955some questions and answers, which we reproduce here for your convenience (for 956reference, the "ASF" is the Apache Software Foundation, the guidance still 957holds though):: 958 959 Q1: If I own a patent and contribute to a Work, and, at the time my 960 contribution is included in that Work, none of my patent's claims are subject 961 to Apache's Grant of Patent License, is there a way any of those claims would 962 later become subject to the Grant of Patent License solely due to subsequent 963 contributions by other parties who are not licensees of that patent. 964 965 A1: No. 966 967 Q2: If at any time after my contribution, I am able to license other patent 968 claims that would have been subject to Apache's Grant of Patent License if 969 they were licensable by me at the time of my contribution, do those other 970 claims become subject to the Grant of Patent License? 971 972 A2: Yes. 973 974 Q3: If I own or control a licensable patent and contribute code to a specific 975 Apache product, which of my patent claims are subject to Apache's Grant of 976 Patent License? 977 978 A3: The only patent claims that are licensed to the ASF are those you own or 979 have the right to license that read on your contribution or on the 980 combination of your contribution with the specific Apache product to which 981 you contributed as it existed at the time of your contribution. No additional 982 patent claims become licensed as a result of subsequent combinations of your 983 contribution with any other software. Note, however, that licensable patent 984 claims include those that you acquire in the future, as long as they read on 985 your original contribution as made at the original time. Once a patent claim 986 is subject to Apache's Grant of Patent License, it is licensed under the 987 terms of that Grant to the ASF and to recipients of any software distributed 988 by the ASF for any Apache software product whatsoever. 989 990.. _legacy: 991 992Legacy License Structure 993------------------------ 994 995.. note:: 996 The code base was previously licensed under the Terms described here. 997 We are in the middle of relicensing to a new approach (described above), but 998 until this effort is complete, the code is also still available under these 999 terms. Once we finish the relicensing project, new versions of the code will 1000 not be available under these terms. However, nothing takes away your right 1001 to use old versions under the licensing terms under which they were 1002 originally released. 1003 1004We intend to keep LLVM perpetually open source and to use a permissive open 1005source license. The code in 1006LLVM is available under the `University of Illinois/NCSA Open Source License 1007<http://www.opensource.org/licenses/UoI-NCSA.php>`_, which boils down to 1008this: 1009 1010* You can freely distribute LLVM. 1011* You must retain the copyright notice if you redistribute LLVM. 1012* Binaries derived from LLVM must reproduce the copyright notice (e.g. in an 1013 included README file). 1014* You can't use our names to promote your LLVM derived products. 1015* There's no warranty on LLVM at all. 1016 1017We believe this fosters the widest adoption of LLVM because it **allows 1018commercial products to be derived from LLVM** with few restrictions and without 1019a requirement for making any derived works also open source (i.e. LLVM's 1020license is not a "copyleft" license like the GPL). We suggest that you read the 1021`License <http://www.opensource.org/licenses/UoI-NCSA.php>`_ if further 1022clarification is needed. 1023 1024In addition to the UIUC license, the runtime library components of LLVM 1025(**compiler_rt, libc++, and libclc**) are also licensed under the `MIT License 1026<http://www.opensource.org/licenses/mit-license.php>`_, which does not contain 1027the binary redistribution clause. As a user of these runtime libraries, it 1028means that you can choose to use the code under either license (and thus don't 1029need the binary redistribution clause), and as a contributor to the code that 1030you agree that any contributions to these libraries be licensed under both 1031licenses. We feel that this is important for runtime libraries, because they 1032are implicitly linked into applications and therefore should not subject those 1033applications to the binary redistribution clause. This also means that it is ok 1034to move code from (e.g.) libc++ to the LLVM core without concern, but that code 1035cannot be moved from the LLVM core to libc++ without the copyright owner's 1036permission. 1037 1038.. _llvm-dev mailing list: http://lists.llvm.org/mailman/listinfo/llvm-dev 1039