Skip to content

perf(docker): enable -Zshare-generics in the image build #845

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jlucaso1 merged 7 commits into from
Jun 11, 2026
Merged

perf(docker): enable -Zshare-generics in the image build #845

Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5c2fcd6
perf(docker): enable -Zshare-generics in the image build
jlucaso1 Jun 11, 2026
f34d9d1
docker: copy the toolchain override into /app before the dependency cook
jlucaso1 Jun 11, 2026
7f7085f
docker: install cargo-chef with --locked
jlucaso1 Jun 11, 2026
12121ed
perf(docker): rebuild std with the release profile via build-std
jlucaso1 Jun 11, 2026
1489771
docker: pin cargo-chef to an exact version
jlucaso1 Jun 11, 2026
ac669a0
docker: derive the build target from the image host triple
jlucaso1 Jun 11, 2026
881b4b1
docker: fail fast if the host triple extraction comes up empty
jlucaso1 Jun 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,17 @@ RUN cargo chef prepare --recipe-path recipe.json
# --- Builder: cook deps (cached layer), then compile source ---
FROM chef AS builder

ENV RUSTFLAGS="-C target-cpu=native"
# -Zshare-generics reuses upstream monomorphizations instead of re-codegening
# them per crate, deduplicating most cross-crate generic/coroutine copies
# (measured: -666 KiB, -5.6% .text). Nightly-only, which this image already
# pins via rust-toolchain.toml; with fat LTO the historical inlining downside
# does not apply since LTO sees all bitcode anyway.
ENV RUSTFLAGS="-C target-cpu=native -Zshare-generics=y"
Comment thread
jlucaso1 marked this conversation as resolved.
Comment thread
jlucaso1 marked this conversation as resolved.

# The dependency cook runs before the source COPY, so make the nightly
# override explicit in /app instead of relying on rustup walking up to the
# chef stage's copy at /; the nightly-only RUSTFLAGS above depends on it.
COPY rust-toolchain.toml .
COPY --from=planner /app/recipe.json recipe.json
RUN cargo chef cook --release --recipe-path recipe.json
COPY . .
Expand Down
Loading