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

Markdown doc for ErrorOutOfGasEXP state#388

Merged
han0110 merged 1 commit into
privacy-ethereum:masterfrom
scroll-tech:doc/error-oog-exp
Mar 1, 2023
Merged

Markdown doc for ErrorOutOfGasEXP state#388
han0110 merged 1 commit into
privacy-ethereum:masterfrom
scroll-tech:doc/error-oog-exp

Conversation

@silathdiir
Copy link
Copy Markdown
Contributor

@silathdiir silathdiir commented Feb 22, 2023

Add Markdown doc for EXP OOG eror.

Gas calculation could reference this go-ethereum function.

Circuit PR privacy-ethereum/zkevm-circuits#1232

lispc pushed a commit to privacy-ethereum/zkevm-circuits that referenced this pull request Feb 23, 2023
### Description

Add `ErrorOOGExpGadget` to handle OOG error of `EXP` opcode.

Gas calculation could reference [this go-ethereum
function](https://github.com/ethereum/go-ethereum/blob/master/core/vm/gas_table.go#L351).

Spec Markdown PR
privacy-ethereum/zkevm-specs#388

### Issue Link

Close
#1231

### 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

### How Has This Been Tested?

Tested with unit-test cases.
@andyguzmaneth andyguzmaneth requested a review from adria0 February 23, 2023 12:26
@han0110 han0110 self-requested a review February 24, 2023 07:43
Copy link
Copy Markdown
Contributor

@han0110 han0110 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@adria0 adria0 left a comment

Choose a reason for hiding this comment

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

LGTM

@han0110 han0110 merged commit df56809 into privacy-ethereum:master Mar 1, 2023
@silathdiir silathdiir deleted the doc/error-oog-exp branch March 1, 2023 07:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants