-
Notifications
You must be signed in to change notification settings - Fork 98
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.
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
Enable LTO on the release profile #175
Comments
Wow, that's a big improvement! Seems like a good idea. What's the difference after also |
@blyxxyz I also tried when stripping, it was smaller than when LTO is disabled.
|
@sorairolake a 15% reduction in binary size is still an improvement. Would you mind creating a PR for it? Thanks On a slightly related note, there is a nightly cargo feature that can automate stripping binaries. Once that stabilizes, we can remove the |
This makes the binary smaller and may also improve performance.
.rwxr-xr-x 16,163,864 root 11 Sep 18:37 target/release/xh # disabled
(about 15 MiB).rwxr-xr-x 11,236,416 root 11 Sep 18:42 target/release/xh # enalbed
(about 11 MiB)Add to
Cargo.toml
:The text was updated successfully, but these errors were encountered: