Skip to content

Commit 2d025d9

Browse files
committed
TEMP(ci:windows-tools): baseline measurement for Option B rework — REVERT BEFORE MERGE
Re-applies the hooks removed in 7f8e255 to collect a warm-ccache baseline on the current LLVM_BUILD_TOOLS=Off + force-build-llvm-config approach. Provides a reference wall-clock for comparing the follow-up rework (install-distribution + Rust-side flag move) in isolation. 1. `if: false` on the LLVM artifact cache restore → forces Build LLVM to run every push, even on warm artifact cache. 2. `save: true` on the ccache-action → populates ccache on PR events so a second push observes warm-ccache behaviour. ccache scope for this PR (refs/pull/365/merge) is already warm from the prior `save: true` window — two 965 MB Windows entries saved 2026-04-22, inside the 7-day TTL. So the first push after this commit already measures the warm-ccache target configuration. Revert before merge. End-state behaviour (artifact restore works, ccache saves only on non-PR events) is what ships.
1 parent 7f8e255 commit 2d025d9

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

.github/actions/build-llvm/action.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,11 @@ runs:
107107
KEY="${KEY}-args${{ steps.build-config.outputs.args-hash }}-${SHA}"
108108
echo "key=${KEY}" | tee -a "${GITHUB_OUTPUT}"
109109
110+
# TEMP(ci:windows-tools): forced-miss so Build LLVM always runs and the
111+
# Windows tool-disable change is actually exercised. REVERT BEFORE MERGE.
110112
- name: Restore LLVM artifact cache
111113
id: llvm-artifact-cache
114+
if: false
112115
uses: actions/cache/restore@668228422ae6a00e4ad889ee87cd7109ec5666a7 # v5.0.4
113116
with:
114117
path: ${{ inputs.working-dir != '.' && format('{0}/target-llvm/target-final', inputs.working-dir) || 'target-llvm/target-final' }}
@@ -175,7 +178,7 @@ runs:
175178
append-timestamp: true
176179
max-size: "10G"
177180
verbose: 2
178-
save: ${{ github.event_name != 'pull_request' }}
181+
save: true # TEMP(ci:windows-tools): populate ccache on PR events so a follow-up push to this branch can measure warm-ccache + tool-disable. REVERT BEFORE MERGE to `${{ github.event_name != 'pull_request' }}`.
179182

180183
- name: Build LLVM
181184
if: steps.llvm-artifact-cache.outputs.cache-hit != 'true'

0 commit comments

Comments
 (0)