-
Notifications
You must be signed in to change notification settings - Fork 314
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
Add support for EIP-1087: Net storage gas metering for the EVM #50
Comments
cc @Arachnid |
The EIP suggests different costs when the slot is modified for the first time in transaction vs subsequent modification in the same transaction, so I think this proposal doesn't cover this difference. |
You are right. I didn't wanted to make it so much detailed. I meant "include any distinctive storage status need to implement Constantinople". I added |
Sufficient statuses I think would be
(Refunds for deleting are handled outside of VM in transaction finalization) |
I will work on this. |
I'm closing it as partly done in #52. The list would have to extended for EIP-1087, but nothing complicated is expected. |
I propose to extend the function
evmc_set_storage()
by returning additional information toThe
enum evmc_storage_status
would contain the information what happened with the storage, likeThe gas cost is going to be applied after the
evmc_set_storage()
is executed (is this a problem?).This also avoids calling
evmc_get_storage()
first.The text was updated successfully, but these errors were encountered: