[llvm] [llvm][docs] Improve usage note on stacked PRs (PR #138949)

Aiden Grossman via llvm-commits llvm-commits at lists.llvm.org
Wed May 7 12:48:30 PDT 2025


================
@@ -288,16 +288,15 @@ Your options are as follows:
    in ``llvm/llvm-project``.
 
 .. note::
-   When not using user branches, GitHub will not display proper diffs for
+   When using the second approach, GitHub will not display proper diffs for
    subsequent PRs in a stack. Instead, it will show a combined diff that
    includes all commits from earlier PRs.
 
-   As described in the first option above, in such cases it is the PR author’s
-   responsibility to clearly indicate which commits are relevant to the
-   current PR. For example: “The first N commits are from the base PR.”
+   As described above, it is the PR author’s responsibility to clearly indicate
+   which commits are relevant to the current PR.
+   For example: “The first N commits are from the base PR.”
 
-   You can avoid this issue by using user branches directly in the
-   ``llvm/llvm-project`` repository.
+   You can avoid this issue by using the first or the third approach.
----------------
boomanaiden154 wrote:

Same here regarding numbering versus describing.

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


More information about the llvm-commits mailing list