-
Notifications
You must be signed in to change notification settings - Fork 856
Feat/CREATE Part B - Error cases that will be handled within opcode gadgets #1425
Conversation
88db436
to
eb9514d
Compare
eb9514d
to
0b66d2a
Compare
@@ -1295,9 +1292,6 @@ impl<F: Field> ExecutionConfig<F> { | |||
ExecutionState::ErrorDepth => { | |||
assign_exec_step!(self.error_depth) | |||
} | |||
ExecutionState::ErrorContractAddressCollision => { | |||
assign_exec_step!(self.error_contract_address_collision) | |||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since ErrDepth will be processed inside create gadget and call gadget, we can remove this code too?
ExecutionState::ErrorDepth => {
assign_exec_step!(self.error_depth)
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed in 4147e93
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Hi @lispc and @han0110 , in order to remove |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
### Description NOTE: This is an updated version of #1358 This PR is actually based on top of #1425 ### Issue Link #1130 ### Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] This change requires a documentation update ### Contents Bus mapping implementation for CREATE/CREATE2 EVM circuit's gadget for CREATE/CREATE2 ### How Has This Been Tested? Tests can be found here: https://github.com/scroll-tech/zkevm-circuits/blob/2d2bfc6ccf179ade1a8d063f9586b93e5283a557/zkevm-circuits/src/evm_circuit/execution/create.rs#L678 --------- Co-authored-by: Rohit Narurkar <[email protected]>
Description
NOTE: This is an updated version of #1357
This PR is actually based on top of #1419
Issue Link
#1130
Type of change
Contents
Error types for insufficient balance and nonce overflow. These are supporting changes required for the CREATE/CREATE2 opcodes' gadget.
Rationale
The above errors will be handled within the CREATE/CREATE2 opcodes' gadget. In case of insufficient balance, it is also handled in the CallOp gadget for call related opcodes.