[llvm] [Docs] Update contribution guide to remove the single-commit requirement (PR #212031)

via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 7 13:47:55 PDT 2026


================
@@ -51,7 +57,18 @@ Once you have a patch ready, it is time to submit it. The patch should:
 * conform to the {doc}`CodingStandards`. You can use the [clang-format-diff.py] or [git-clang-format] tools to automatically format your patch properly.
 * not contain any unrelated changes
 * be an isolated change. Independent changes should be submitted as separate patches as this makes reviewing easier.
-* have a single commit, up-to-date with the upstream `origin/main` branch, and don't have merges.
+
+```{note}
+When a PR is accepted and submitted into mainline, all changes from the PR are combined into a single commit using GitHub's "Squash and Merge" button. (We don't use merge commits on LLVM's mainline.) Because of that policy, the series of commits inside a PR branch does not make it into the "permanent record", and is thus not of critical importance.
----------------
Sirraide wrote:

Ok, I believe that page also needs updating because it says

> If your PR contains multiple commits, you need to consolidate those commits into one commit. There are three different ways to do this, [...] Interactive rebase with fixups. This is the recommended method

and I at least would argue that the 'recommended method' should be to just press Squash+Merge (which is also listed as an option there). Are we fine w/ me updating that document too as part of this?

https://github.com/llvm/llvm-project/pull/212031


More information about the llvm-commits mailing list