Skip to content

perf: optimize release build dependencies #634

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

Draft
wan9chi wants to merge 1 commit into
base: main
Choose a base branch
from
+6 −24
Draft

perf: optimize release build dependencies #634

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
30 changes: 6 additions & 24 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -209,31 +209,13 @@ strip = "symbols" # set to `false` for debug information
debug = false # set to `true` for debug information
panic = "abort" # Let it crash and force ourselves to write safe Rust.

# Cargo compiles artifact dependencies declared under `[build-dependencies]`
# with the build-override profile — opt-level 0, no codegen-units setting —
# even in release builds, and even with `target = "target"`. Their output
# still lands under `target/<triple>/release/`, so the missing optimization
# is easy to miss. (Artifact deps under `[dependencies]`/`[dev-dependencies]`
# get the normal profile and need nothing here.) See
# Cargo normally compiles build scripts, proc macros, artifact dependencies,
# and their dependencies with opt-level 0 and 256 codegen units, even in
# release builds. Artifact dependencies are runtime code, so optimize the
# entire build graph instead of maintaining a list of individual artifacts.
# Profile overrides do not inherit `strip`, so restate it here. See
# https://github.com/rust-lang/cargo/issues/16719.
#
# These are the workspace's build-dependency artifacts: the preload libraries
# are injected into every traced process and run inside every intercepted
# libc call, and the napi library backs the Node.js client.
#
# `strip` must be restated: a package override resets an inherited `strip`
# to "debuginfo" when the field is omitted (Cargo quirk in profile merging).
[profile.release.package.fspy_preload_unix]
opt-level = 3
codegen-units = 1
strip = "symbols"

[profile.release.package.fspy_preload_windows]
opt-level = 3
codegen-units = 1
strip = "symbols"

[profile.release.package.vt_client_napi]
[profile.release.build-override]
opt-level = 3
codegen-units = 1
strip = "symbols"
Loading