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.

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

Superblock metering #8

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Superblock metering #8

wants to merge 5 commits into from

Conversation

cdetrio
Copy link
Contributor

@cdetrio cdetrio commented Mar 15, 2019

[WIP dont merge]

When executing metered wasm (where the gas cost table is 1 for every instruction), the gas used should be equal to the number of wasm instructions executed. This achieves that for bn128_mul, except for an off-by-one error.

## ecmul_minified_unmetered.wasm
total instructions executed: 8039712

## ecmul_minified_metered.wasm
total instructions executed: 8217634
total useGas calls: 88961
total gas used: 8039713

8039713 - 8039712 = 1

@axic axic force-pushed the superblock-metering branch from d324344 to 5c8c8cb Compare June 13, 2019 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant