Skip to content
This repository was archived by the owner on Jul 5, 2024. It is now read-only.

Update EXTCODEHASH to handle reverts#163

Merged
icemelon merged 3 commits into
privacy-ethereum:masterfrom
scroll-tech:fix/extcodehash
Mar 28, 2022
Merged

Update EXTCODEHASH to handle reverts#163
icemelon merged 3 commits into
privacy-ethereum:masterfrom
scroll-tech:fix/extcodehash

Conversation

@z2trillion
Copy link
Copy Markdown
Contributor

Update specs for EXTCODEHASH to match its implementation in privacy-ethereum/zkevm-circuits#353

@z2trillion
Copy link
Copy Markdown
Contributor Author

@ed255

Copy link
Copy Markdown
Contributor

@ed255 ed255 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update! LGTM!

Copy link
Copy Markdown
Collaborator

@icemelon icemelon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise LGTM.

Comment thread tests/evm/test_extcodehash.py Outdated
@icemelon icemelon merged commit 473ff9d into privacy-ethereum:master Mar 28, 2022
@icemelon icemelon deleted the fix/extcodehash branch March 28, 2022 20:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants