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

Circuit for opcode SELFBALANCE #328

Merged

Conversation

scroll-dev
Copy link
Collaborator

Implement the SelfbalanceGadget for opcode SELFBALANCE.

Specs: privacy-scaling-explorations/zkevm-specs#113

Will update the bus mapping for SELFBALANCE once #292 is merged in upstream.

@github-actions github-actions bot added crate-bus-mapping Issues related to the bus-mapping workspace member T-opcode Type: opcode-related and focused PR/Issue labels Feb 10, 2022
@ChihChengLiang
Copy link
Collaborator

@scroll-dev Mason Liang Can we do a rebase for this?

bus-mapping/src/evm/opcodes.rs Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the crate-bus-mapping Issues related to the bus-mapping workspace member label Feb 14, 2022
@ed255 ed255 self-requested a review February 14, 2022 16:48
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.

LGTM! Thanks, good work!

@ed255
Copy link
Member

ed255 commented Feb 18, 2022

@z2trillion #292 has been merged! Do you plan on updating this PR to implement the SELFBALANCE in the bus mapping? Or do you prefer doing that in another PR?
Either way the PR needs a rebase with main to be merged :)

@github-actions github-actions bot added the crate-bus-mapping Issues related to the bus-mapping workspace member label Feb 23, 2022
@z2trillion
Copy link
Collaborator

@ed255, I've rebased and added the bus mapping.

@ed255
Copy link
Member

ed255 commented Feb 24, 2022

@z2trillion There seems to be a linting error coming from clippy.

@z2trillion
Copy link
Collaborator

@ed255 it should be fixed now.

@ed255 ed255 merged commit 1f4f1a8 into privacy-scaling-explorations:main Feb 24, 2022
@z2trillion z2trillion deleted the feat/opcode-selfbalance branch June 30, 2022 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crate-bus-mapping Issues related to the bus-mapping workspace member T-opcode Type: opcode-related and focused PR/Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants