[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