Enable Link-Time Optimization (LTO) and codegen-units = 1 for Release builds #281
Closed
zamazan4ik
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
|
Thanks. The 2 options shaved off exactly 2MB of the binary on my NixOS system, nice! |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Although the build time is brutal now /m\ |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi. Congratulations with the 3.0 release!
I noticed that after the Rust rewrite, in the
Cargo.tomlfile Link-Time Optimization (LTO) for the project is not enabled. I suggest switching it on since it will reduce the binary size (always a good thing to have) and will likely improve the application's CPU performance a bit. If you want to read more about LTO and its possible modes, I recommend starting from this Rustc documentation.I think you can enable LTO only for the Release builds so as not to sacrifice the developers' experience while working on the project, since LTO consumes an additional amount of time to finish the compilation routine. In this case, we can create a dedicated
[profile.optimized-dev]profile where LTO will be disabled (so developers experience will not be affected). If we enable it on the Cargo profile level for the Release profile, users, who install the application withcargo install, will get the LTO-optimized version of the app "automatically". Actually, I can propose to use flags directly from theripgrepprofile.Basically, it can be enabled with the following lines to the root Cargo.toml file:
I have made quick tests (AMD Ryzen 9 5900x, Fedora 42, Rust 1.89, the latest version of the project at the moment,
cargo build --releasecommand) - here are the results:codegen-units = 1+ Fat LTO: 6.8 Mib, clean build time: 57sThe same tests for MacBook M1 Pro, macOS, Rust 1.89:
codegen-units = 1+ Fat LTO: 5.6 Mib, clean build time: 1m 17sThank you.
Beta Was this translation helpful? Give feedback.
All reactions