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
merged 3 commits into from
Mar 28, 2022
Merged

Update EXTCODEHASH to handle reverts #163

merged 3 commits into from
Mar 28, 2022

Conversation

z2trillion
Copy link
Collaborator

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

@z2trillion
Copy link
Collaborator Author

@ed255

ed255
ed255 approved these changes Mar 28, 2022
View reviewed changes
Copy link
Member

@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!

icemelon
icemelon approved these changes Mar 28, 2022
View reviewed changes
Copy link
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.

@icemelon icemelon merged commit 473ff9d into privacy-scaling-explorations: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.
Reviewers
2 more reviewers

@icemelon icemelon icemelon approved these changes

@ed255 ed255 ed255 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@z2trillion @icemelon @ed255