Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[x/programs] [poc] validate wasmtime-go #464

Closed
wants to merge 4 commits into from
Closed

[x/programs] [poc] validate wasmtime-go #464

wants to merge 4 commits into from

Conversation

hexfusion
Copy link
Contributor

@hexfusion hexfusion commented Sep 12, 2023 •
edited
Loading

POC hack PR please do not review. The goal is to evaluate performance of wasmtime-go with fuel metering.

wazero in the slower interpreter mode appears faster than wasmtime-go with only a partial token example.

goos: linux
goarch: amd64
pkg: github.com/ava-labs/hypersdk/x/programs/examples
cpu: Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
BenchmarkTokenWazeroProgram/benchmark_token_program_wazero-12 78 15678089 ns/op 4653056 B/op 42360 allocs/op

cpu: Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
BenchmarkTokenWasmtimeProgram/benchmark_token_program-12 55 18746760 ns/op 104953 B/op 394 allocs/op

@hexfusion
[x/programs] [poc] validate wasmtime-go
3ca7dc5 
Signed-off-by: Sam Batschelet <sam.batschelet@avalabs.org>
@hexfusion hexfusion added the DO NOT MERGE This PR is not meant to be merged in its current state label Sep 12, 2023
@hexfusion
basic token works
26d2965 
Signed-off-by: Sam Batschelet <sam.batschelet@avalabs.org>
@hexfusion
set cranelift as default strategy
8c8935f 
Signed-off-by: Sam Batschelet <sam.batschelet@avalabs.org>
@hexfusion
token bench works
c6004df 
Signed-off-by: Sam Batschelet <sam.batschelet@avalabs.org>
@hexfusion
Copy link
Contributor Author

as per #466 we are making the change closing this and will open a new PR

@hexfusion hexfusion closed this Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aaronbuchwald aaronbuchwald Awaiting requested review from aaronbuchwald aaronbuchwald will be requested when the pull request is marked ready for review aaronbuchwald is a code owner

@joshua-kim joshua-kim Awaiting requested review from joshua-kim joshua-kim will be requested when the pull request is marked ready for review joshua-kim is a code owner

Assignees
No one assigned
Labels
DO NOT MERGE This PR is not meant to be merged in its current state
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hexfusion