Skip to content

Enable PGO for Linux ARM64 uv releases - #21004

Merged
charliermarsh merged 4 commits into
charlie/uv-pgo-windows-x64from
charlie/uv-pgo-linux-arm64
Aug 17, 2026
Merged

charliermarsh merged 4 commits into
charlie/uv-pgo-windows-x64from
charlie/uv-pgo-linux-arm64

Conversation

@charliermarsh

@charliermarsh charliermarsh commented Aug 7, 2026 •

Copy link
Copy Markdown
Member

Summary

This PR enables PGO for uv's Linux ARM64 releases, following the approach outlined in #21001. (To enable PGO, we also move to a native ARM64 runner.)

@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from ad66463 to 4b120ad Compare August 7, 2026 21:06
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from b339894 to e67b063 Compare August 7, 2026 21:43
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from e67b063 to 8e5743a Compare August 10, 2026 20:23
@charliermarsh
charliermarsh marked this pull request as ready for review August 10, 2026 20:24
@charliermarsh charliermarsh added the internal:releases Related to building and distributing release artifacts of uv label Aug 10, 2026
@charliermarsh
charliermarsh marked this pull request as draft August 10, 2026 20:30
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 8e5743a to 63aa439 Compare August 10, 2026 20:46
@codspeed

codspeed Bot commented Aug 10, 2026 •

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 31 untouched benchmarks
⏩ 6 skipped benchmarks1


Comparing charlie/uv-pgo-linux-arm64 (7640c50) with charlie/uv-pgo-windows-x64 (61e8fc2)2

Open in CodSpeed

Footnotes

  1. 6 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

  2. No successful run was found on charlie/uv-pgo-windows-x64 (4e6ae4d) during the generation of this report, so b5383df was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 63aa439 to 72b22b0 Compare August 10, 2026 20:48
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 72b22b0 to 5cd04aa Compare August 10, 2026 20:57
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 5cd04aa to 2940b28 Compare August 10, 2026 21:13
@charliermarsh
charliermarsh marked this pull request as ready for review August 10, 2026 21:13
@charliermarsh
charliermarsh marked this pull request as draft August 11, 2026 11:10
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 03777a1 to b06222f Compare August 12, 2026 16:17
@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from b06222f to 7640c50 Compare August 12, 2026 16:25
@charliermarsh
charliermarsh marked this pull request as ready for review August 12, 2026 16:28
name: ${{ matrix.platform.target }}
runs-on: depot-ubuntu-24.04-8
timeout-minutes: 30
runs-on: ${{ matrix.platform.target == 'aarch64-unknown-linux-gnu' && 'depot-ubuntu-24.04-arm-8' || 'depot-ubuntu-24.04-8' }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd probably just add matrix.platform.runner

before-script-linux: |
scripts/install-cargo-extensions.sh

if [[ "${{ matrix.platform.target }}" == "aarch64-unknown-linux-gnu" ]]; then

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this worth a note about why we only do PGO for arm64 linux and not the rest of the matrix?

Should we just put matrix.platform.pgo in the matrix?

@charliermarsh
charliermarsh force-pushed the charlie/uv-pgo-linux-arm64 branch from 4d4e896 to fb402a4 Compare August 17, 2026 19:26

This branch was previously deployed

1 inactive deployment
automations — 2992c043 Deployed Aug 17, 2026 by charliermarsh via review / security review #44870
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal:releases Related to building and distributing release artifacts of uv

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants