Skip to content

Conversation

@llvmbot
Copy link
Member

@llvmbot llvmbot commented Oct 21, 2025

Backport 288ef04

Requested by: @boomanaiden154

@llvmbot
Copy link
Member Author

llvmbot commented Oct 21, 2025

@tstellar What do you think about merging this PR to the release branch?

@llvmbot
Copy link
Member Author

llvmbot commented Oct 21, 2025

@llvm/pr-subscribers-github-workflow

Author: None (llvmbot)

Changes

Backport 288ef04

Requested by: @boomanaiden154


Full diff: https://github.com/llvm/llvm-project/pull/164497.diff

1 Files Affected:

  • (modified) .github/workflows/premerge.yaml (+1-1)
diff --git a/.github/workflows/premerge.yaml b/.github/workflows/premerge.yaml
index d0518fa6879e2..8594e1e91ee99 100644
--- a/.github/workflows/premerge.yaml
+++ b/.github/workflows/premerge.yaml
@@ -149,7 +149,7 @@ jobs:
         uses: llvm/actions/install-ninja@main
       - name: Build and Test
         run: |
-          source <(git diff --name-only HEAD~2..HEAD | python3 .ci/compute_projects.py)
+          source <(git diff --name-only HEAD~1...HEAD | python3 .ci/compute_projects.py)
 
           if [[ "${projects_to_build}" == "" ]]; then
             echo "No projects to build"

@c-rhodes c-rhodes moved this from Needs Triage to Needs Merge in LLVM Release Status Oct 22, 2025
Otherwise we're looking for a commit that does not exist given the fetch
depth.

Fixes llvm#164430.

(cherry picked from commit 288ef04)
@c-rhodes c-rhodes merged commit 1600cf0 into llvm:release/21.x Oct 27, 2025
1 check was pending
@github-project-automation github-project-automation bot moved this from Needs Merge to Done in LLVM Release Status Oct 27, 2025
@github-actions
Copy link

@boomanaiden154 (or anyone else). If you would like to add a note about this fix in the release notes (completely optional). Please reply to this comment with a one or two sentence description of the fix. When you are done, please add the release:note label to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

4 participants