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

Evm refactor (rebased on master 8a65e19, 2022-06-08) #1950

Closed
wants to merge 40 commits into from

Conversation

holgerd77
Copy link
Member

This is a cherry-picked version of #1912 rebased on the latest master (up to 8a65e19).

This can be seen as a test rebase or taken as a new work state. #1912 has been integrated up to 00e29e4, a total of 36 commits.

Rebase went relatively well, roughly 7-9 conflict resolution needs.

vm: move eei/vmstate into new EEI dir
vm/evm: fix gasRefund
vm/evm: fixup TransientStorage/EEI creation
@holgerd77 holgerd77 added PR state: do-not-merge PR/Issue state: blocked package: vm type: test all hardforks This special label enables VM state and blockchain tests for all hardforks on the respective PR. target: master Work to be done towards master branch labels Jun 10, 2022
@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #1950 (a5ba4dd) into master (8a65e19) will increase coverage by 3.23%.
The diff coverage is n/a.

Impacted file tree graph

Flag Coverage Δ
block 86.15% <ø> (ø)
blockchain 81.72% <ø> (ø)
client ?
common 95.01% <ø> (ø)
devp2p 82.64% <ø> (?)
ethash 90.81% <ø> (ø)
rlp ?
statemanager 84.55% <ø> (ø)
trie 80.19% <ø> (ø)
tx 92.18% <ø> (ø)
util 87.32% <ø> (ø)
vm ?

Flags with carried forward coverage won't be shown. Click here to find out more.

@jochem-brouwer
Copy link
Member

Ethereum/tests pass on 2c9fc7a

@holgerd77 holgerd77 changed the title Evm refactor (rebased on master 8a65e19) Evm refactor (rebased on master 8a65e19, 2022-06-08) Jun 13, 2022
@holgerd77 holgerd77 deleted the evm-refactor-rebased-on-master branch March 2, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: vm PR state: WIP target: master Work to be done towards master branch type: test all hardforks This special label enables VM state and blockchain tests for all hardforks on the respective PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants