Skip to content

docs(tests): clarify gas-cap semantics and SSTORE stub layout assumption - #13

Open
spencer-tb wants to merge 1 commit into
port-benchmark-from-devnetfrom
benchmark-port-followup
Open

docs(tests): clarify gas-cap semantics and SSTORE stub layout assumption#13
spencer-tb wants to merge 1 commit into
port-benchmark-from-devnetfrom
benchmark-port-followup

Conversation

@spencer-tb

Copy link
Copy Markdown
Owner

Debugging PR for the ethereum#2977 follow-up, based on the upstream PR head (18b0fbb). Shows only the follow-up changes before the real PR against upstream forks/amsterdam.

  • tx_iterations_by_gas_limit docstring: the tx gas-limit cap constrains compute gas; under EIP-8037 the combined gas limit may exceed it by the state-gas portion.
  • test_sstore_bloated docstring: restore the linear value==key stub storage layout assumption that existing_slots=True gas metadata relies on.
  • tx_iterations_by_total_iteration_count: comment why the cap also bounds combined gas (no block budget in this context; keeps the search finite, slightly under-fills under state gas).

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