-
Notifications
You must be signed in to change notification settings - Fork 154
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
refactor(codegen): inline contract router #1968
Conversation
This requires inlining receivers as well. Reduces gas usage of the Jetton benchmarks. But increases gas usage for Escrow.
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.
General logic is solid, good thing we have #1949 since we will catch all possible router bugs there, if there are any, after this massive codegen changes
Few notes in review
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.
Good to go!
This requires inlining receivers as well.
Reduces gas usage of the Jetton benchmarks.
But increases gas usage for Escrow (I think we can live with it for now, until #1866 is resolved).
Issue
Closes #1587.
Gas Benchmarks
Checklist