-
Notifications
You must be signed in to change notification settings - Fork 233
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
Significant iOS binary size increase with Rust 1.50 #907
Comments
➤ Ryan Kelly commented: (Which, +5M still seems like a lot for Nimbus to be honest, but that's a separate question...) |
➤ Dan Mosedale commented: Random thoughts:
|
➤ Dan Mosedale commented: Other things to try:
|
➤ Dan Mosedale commented: rfk since you've got both builds around, can you try running cargo bloat to see if it yields any interesting info? |
➤ Dan Mosedale commented: If the problem is LLVM-upgrade-related, interestingly, LLVM12 has landed in nightly rustc ( rust-lang/rust#81451 ), so maybe that'll change things yet again... |
➤ Dan Mosedale commented: LLVM links, including release notes ( https://releases.llvm.org/ ) |
➤ Dan Mosedale commented: cargo-bisect-rustc ( https://blog.rust-lang.org/inside-rust/2019/12/18/bisecting-rust-compiler.html ) might help us sort this out too... |
As noted in mozilla-mobile/firefox-ios#8044 (comment), the
v73.0.0
release of application-services has significantly increased compiled binary size (by nearly 30%, from 35M to 45M).Some of that increase is due to the inclusion of Nimbus, but actually the largest contributor seems to be the switch from Rust 1.43 to Rust 1.50. Here's the size of a local build of
v73.0.0
done using Rust 1.50, which matches what we see in the release artifact:Here's the same when built with Rust 1.43, the toolchain we used for previous releases:
I confirmed that if I manually remove Nimbus from the build, we get back down to around 35M, which was approximately the size of the previous release.
The text was updated successfully, but these errors were encountered: