-
Notifications
You must be signed in to change notification settings - Fork 812
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
[pallet-revive] pack exceeding syscall arguments into registers #7319
Merged
Merged
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
15220bb
pack call arguments into registers instead of the stack
xermicus 4095129
Merge branch 'master' into cl/call-arguments
xermicus 73b9f3d
".git/.scripts/commands/fmt/fmt.sh"
b5f69bf
Merge branch 'master' into cl/call-arguments
xermicus 866e0b8
Update from xermicus running command 'prdoc --audience runtime_dev --…
github-actions[bot] 3fd9de7
update benchmarks
xermicus 18931a7
fmt
xermicus 30ca3f1
Update from xermicus running command 'bench --runtime dev --pallet pa…
github-actions[bot] ee45486
Merge branch 'master' into cl/call-arguments
xermicus 2ff0be8
the pvm fixtures
xermicus 9056664
Merge branch 'master' into cl/call-arguments
xermicus 6926c32
Merge branch 'master' into cl/call-arguments
xermicus cf43e34
Merge branch 'master' into cl/call-arguments
xermicus cf583a5
Update substrate/frame/revive/uapi/src/host/riscv64.rs
xermicus 1d17236
improve comment
xermicus File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
the pvm fixtures
Signed-off-by: xermicus <cyrill@parity.io>
- Loading branch information
commit 2ff0be81ff78a28a3ef331a91886ff8931d55bd1
There are no files selected for viewing
Binary file modified
BIN
+31 Bytes
(100%)
substrate/frame/revive/rpc/examples/js/pvm/FlipperCaller.polkavm
Binary file not shown.
Binary file modified
BIN
+2 Bytes
(100%)
substrate/frame/revive/rpc/examples/js/pvm/PiggyBank.polkavm
Binary file not shown.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I was hoping it reduces code size not increase it :(
But we mainly do that to reduce complexity of having two different ways of passing arguments.