-
Notifications
You must be signed in to change notification settings - Fork 105
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
feat(wasm-builder): Set optimisation params #2973
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
It'd be good compare upload program cost for some of them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about going with O4
plus shrink-level-2
?
for -Oz
vs -O4
, the difference is optimization level 2 ( + shrink-level-2) vs optimization level 4, see
https://docs.rs/wasm-opt/0.113.0/wasm_opt/struct.OptimizationOptions.html#method.new_optimize_for_size_aggressively for structured docs
btw, super curious why tests::locking_gas_for_waitlist
works in this PR but failed in all of my PRs xd
in some cases it is faster, but the size is similar |
Added to description |
Resolves # .
fungible-token
changes from refactor: avoid unnecessary syscalls gear-foundation/dapps-fungible-token#52.opt-level = z
in tomlwasm-opt
level to 4 and shrink 2codegen-units = 1
lto = "fat"
, which is the same astrue
These changes reduce the size of wasm artifacts and speed up programs.
stress_transfer
benchmark (updated):Artifact size:
@gear-tech/dev