implement borsh serialize and deserialize artifact - #1
Merged
Conversation
olonho
approved these changes
Dec 22, 2020
olonho
left a comment
Contributor
There was a problem hiding this comment.
LGTM, maybe rename map -> index_map.
Contributor
Author
|
There is almost no change in cost, if consider volatility in each run. second == borsh_wasmer, first == master |
ailisp
pushed a commit
that referenced
this pull request
Jan 15, 2021
…lized-results-memory test(c-api) Enable the memory test example.
willemneal
approved these changes
Jan 22, 2021
Fixed cargo test pass again after change. There's a benchmark
fn create_instance_from_cache_benchmark(c: &mut Criterion) {covers the change of this PR, and it gives a 20%-30% performance increase in this bench (a nightly rust is required):I also run param estimator with serde wasmer nearcore vs borsh wasmer nearcore, it's still running in progress