Consider enabling more aggressive optimizations for the Dist profile: Fat LTO and codegen-units = 1

[zamazan4ik]

Hi!

I see that the project already uses ThinLTO in the Dist profile in the root Cargo.toml file. However, ThinLTO is usually less efficient from the perspective of performed optimizations than Fat (aka Full) LTO, and cargo-dist defaults are not the most optimal by default. Additionally, I suggest enabling codegen-units = 1 (CG1) too. Enabling more advanced optimizations allows us to reduce the binary size further (always a good thing) and improve the application performance more.

Basically, it can be enabled with the following change:

[profile.dist]
inherits = "release"
codegen-units = 1
lto = true

I have made quick local tests (AMD Ryzen 5900x, Fedora 41, Rust 1.84, the latest version of this project at the moment, cargo build --profile dist command) - the results are below. I show only binary size changes since it's easier to measure them compared to performance differences in this app.

• Current Dist profile: 4.2 Mib, clean build time: 6s
• Dist + Fat LTO + codegen-units = 1: 2.9 Mib, clean build time: 16s

Thank you.

[solidiquis]

Thanks for the tip @zamazan4ik .. I'm in agreement that the more aggressive optimizations are warranted here. Would you like to submit a PR? If not I'll do it sometime this weekend which is also when I plan to do another release.

[zamazan4ik]

Sure, here is the PR: #16