Skip to content

Commit

Permalink
Do not build LLVM in CI by default (#1497)
Browse files Browse the repository at this point in the history
The access to llvm tarball is now restored, no need to build LLVM in the
workflow.

Note that this is not a complete undo of #1489, because we need to keep
an option to build LLVM if something happens to the tarball again.
Relates to #1490.
  • Loading branch information
pbchekin authored Jun 26, 2024
1 parent b85b536 commit 72dab69
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/build-test-reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ on:
build_llvm:
description: Build LLVM
type: boolean
default: true
default: false

permissions: read-all

Expand Down Expand Up @@ -96,6 +96,7 @@ jobs:
echo "LLVM_COMMIT_ID=$LLVM_COMMIT_ID" | tee -a $GITHUB_ENV
- name: Load LLVM cache
if: inputs.build_llvm
id: llvm-cache
uses: ./.github/actions/load
env:
Expand Down Expand Up @@ -127,7 +128,7 @@ jobs:
echo "LLVM_SYSPATH=$HOME/packages/llvm" | tee -a $GITHUB_ENV
- name: Save LLVM cache
if: steps.llvm-cache.outputs.status == 'miss'
if: inputs.build_llvm && steps.llvm-cache.outputs.status == 'miss'
uses: ./.github/actions/save
with:
path: ${{ steps.llvm-cache.outputs.path }}
Expand Down

0 comments on commit 72dab69

Please sign in to comment.