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

op CALLDATALOAD#346

Merged
ChihChengLiang merged 11 commits into
privacy-ethereum:mainfrom
scroll-tech:feat/calldataload
Mar 14, 2022
Merged

op CALLDATALOAD#346
ChihChengLiang merged 11 commits into
privacy-ethereum:mainfrom
scroll-tech:feat/calldataload

Conversation

@roynalnaruto
Copy link
Copy Markdown
Collaborator

@roynalnaruto roynalnaruto commented Feb 24, 2022

Specs PR here

@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 24, 2022
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs
Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs Outdated
@roynalnaruto roynalnaruto requested a review from icemelon March 3, 2022 01:48
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.

LGTM

@ed255 @ChihChengLiang @han0110 Could you help review this PR?

@CPerezz CPerezz mentioned this pull request Mar 4, 2022
19 tasks
Copy link
Copy Markdown
Contributor

@CPerezz CPerezz left a comment

Choose a reason for hiding this comment

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

LGTM!

Would appreciate a review from any of my colleges before merging JIC.

Maybe @ed255 you have a bit of time and can spot anything that I might have missed?

Comment thread zkevm-circuits/src/evm_circuit/execution/calldataload.rs
@ChihChengLiang
Copy link
Copy Markdown
Collaborator

Let's merge this as it has 2 effective reviews. Post merge review is welcomed.

@ChihChengLiang ChihChengLiang merged commit 80d20e7 into privacy-ethereum:main Mar 14, 2022
zemse pushed a commit to zemse/zkevm-circuits that referenced this pull request Mar 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

5 participants