-
Notifications
You must be signed in to change notification settings - Fork 120
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(stdlib): optimized emptyCell()
and emptySlice()
#1696
base: main
Are you sure you want to change the base?
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.
Need a benchmark!
emptyCell()
and emptySlice()
Optimization of emptyCell() Expected :1449n Optimization of emptySlice() Snapshot: 1667n |
# Conflicts: # src/stdlib/stdlib.ts # src/test/benchmarks/benchmarks.spec.ts
await testContract.send( | ||
treasure.getSender(), | ||
{ value: toNano(1) }, | ||
beginCell().asSlice(), |
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.
Usually we just define receive() {}
and pass null
here
@@ -0,0 +1,11 @@ | |||
contract cellsCreation { |
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.
CellsCreation
Co-authored-by: Anton Trunov <anton@ton.org>
Issue
Closes #1693 .
Benchmarks:
Optimization of emptyCell()
Expected :1449n
Actual :931n
Optimization of emptySlice()
Snapshot: 1667n
Received: 1035n
Checklist