[llvm] [libc++] Shuffle the order of pre-commit CI jobs a bit (PR #128008)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 20 06:24:29 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-github-workflow
Author: Louis Dionne (ldionne)
<details>
<summary>Changes</summary>
I've recently noticed that our CI is bottlenecked around platforms on which we don't have a lot of capacity like macOS (mostly) and Windows. To try to alleviate that, this patch moves the macOS builds and the Windows builds further down the pipeline so that they will get triggered less often (if an earlier job fails).
---
Full diff: https://github.com/llvm/llvm-project/pull/128008.diff
1 Files Affected:
- (modified) .github/workflows/libcxx-build-and-test.yaml (+3-3)
``````````diff
diff --git a/.github/workflows/libcxx-build-and-test.yaml b/.github/workflows/libcxx-build-and-test.yaml
index d0193a4ec2faa..93e673ca513a4 100644
--- a/.github/workflows/libcxx-build-and-test.yaml
+++ b/.github/workflows/libcxx-build-and-test.yaml
@@ -124,7 +124,7 @@ jobs:
**/crash_diagnostics/*
stage3:
if: github.repository_owner == 'llvm'
- needs: [ stage1, stage2 ]
+ needs: [ stage2 ]
continue-on-error: false
strategy:
fail-fast: false
@@ -188,7 +188,7 @@ jobs:
**/crash_diagnostics/*
macos:
- needs: [ stage1 ]
+ needs: [ stage3 ]
strategy:
fail-fast: false
matrix:
@@ -232,7 +232,7 @@ jobs:
windows:
runs-on: windows-2022
- needs: [ stage1 ]
+ needs: [ stage2 ]
strategy:
fail-fast: false
matrix:
``````````
</details>
https://github.com/llvm/llvm-project/pull/128008
More information about the llvm-commits
mailing list