Wasm gas: Stop charging for "empty" instructions #1245
Milestone
Comments
Stebalien
added a commit
that referenced
this issue
Dec 11, 2022
We now explicitly charge for individual instructions, and reject unsupported ones. Additionally, this patch: 1. Makes consts and basic casts free (fixes #1245). 2. Makes register operations free (locals/globals). 3. Charges 100gas per memory access (10ns). This is based on normal DDR4 memory latency. 4. Charges 0.5gas per byte copied (fills and bulk copy). This is based on our previous memcpy benchmarks.
4 tasks
Stebalien
added a commit
that referenced
this issue
Dec 19, 2022
We now explicitly charge for individual instructions, and reject unsupported ones. Additionally, this patch: 1. Makes consts and basic casts free (fixes #1245). 2. Makes register operations free (locals/globals). 3. Charges 100gas per memory access (10ns). This is based on normal DDR4 memory latency. 4. Charges 0.5gas per byte copied (fills and bulk copy). This is based on our previous memcpy benchmarks.
|
@Stebalien can you set milestone and priority? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Specifically, stop charging for:
The text was updated successfully, but these errors were encountered: