Skip to content

Includes EIP-7702 address in CODECOPY #2014

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.

Sign up for GitHub

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

Jump to bottom
Closed
elijahhampton wants to merge 104 commits into from
Closed

Includes EIP-7702 address in CODECOPY #2014

elijahhampton wants to merge 104 commits into from

Conversation

@elijahhampton
Copy link

@elijahhampton elijahhampton commented Jan 23, 2025

No description provided.

Wodann and others added 30 commits September 9, 2024 12:27
@Wodann @rakita
feat: introducing EvmWiring, a chain-specific configuration (blueallo…
077c2c3 
…y#1672)

* refactor: allow chain-specific configuration of Evm

* refactor: rename Transaction::transact_to and clarify docs

* refactor: remove trait bounds on Transaction super trait

* refactor: remove trait bounds from Block supertrait

* fix: clippy warnings

* fix: cargo doc

* refactor: limit trait bounds on HaltReason

* refactor: allow moving of kind

* refactor: rename Transaction::nonce to nonce_opt to signal that it's optional

* refactor: replace AccessList with alloy version

* refactor: rename gas_priority_fee to max_priority_fee_per_gas

* refactor: correct trait bound on ExecutionResult::clone

* Clone

* refactor: only allow optional nonce check via CfgEnv

* fix: revme

* refactor: derive DummyHost

* refactor: derive Clone for ExecutionResult

* refactor: remove EVMErrorForChain

* refactor: derive Clone for CfgEnvWithChainSpec

* refactor: use EVMResultGeneric

* refactor: add convenience EVMErrorForChain type alias

* feat: export OptimismBlock

* refactor: add handler constructor and Context to revm::ChainSpec

* refactor: generalise optimism implementation using traits

* fix: no-default-features

* fix: CI

* chore: Add default fn to Tx/Block traits

* Chore: rename ChainSpec to EvmWiring

* chore: clippy comments fix

* chore: rename EthEvmWiring to EthereumWiring

* chore: re add serde, restring HaltReasonTrait

* chore: move custom opcode to examples

* chore: remove op feature from test wiring

* nit use Self::EvmWiringT

* nit indents

* feat(Wiring): Add Database and EXT to EvmWiring

* some fixes

* temp

* feat: make builder compile. EnvWiring and Result Halt

* chore: cleanup rename

* nit

* fix: make string conversion complete

* fix compile

* compiles

* wip builder

* wip

* fix compile

* wip

* fix optimism test

* fix docs ci

* cleanup

* cleanup

* use core::error::Error

* cleanup

* use core error

* fix builer

* fix docs

* final doc fix

* rm alloy provider

---------

Co-authored-by: rakita <[email protected]>
@DaniPopes
chore(revme): replace structopt with clap (bluealloy#1754)
a914ddd 
`structopt` has long been discontinued in favor of clap:
<https://docs.rs/structopt/0.3.26/structopt/index.html#maintenance>
@rakita
fix: statetest json set spec_id (bluealloy#1766)
f57e3e6
@Wodann
refactor: simplify SuccessOrHalt trait bound (bluealloy#1768)
8d6c5d1
@dependabot
chore(deps): bump once_cell from 1.19.0 to 1.20.0 (bluealloy#1773)
1bdd5bd 
Bumps [once_cell](https://github.com/matklad/once_cell) from 1.19.0 to 1.20.0.
- [Changelog](https://github.com/matklad/once_cell/blob/master/CHANGELOG.md)
- [Commits](matklad/once_cell@v1.19.0...v1.20.0)

---
updated-dependencies:
- dependency-name: once_cell
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps): bump anyhow from 1.0.88 to 1.0.89 (bluealloy#1772)
acf1a70 
Bumps [anyhow](https://github.com/dtolnay/anyhow) from 1.0.88 to 1.0.89.
- [Release notes](https://github.com/dtolnay/anyhow/releases)
- [Commits](dtolnay/anyhow@1.0.88...1.0.89)

---
updated-dependencies:
- dependency-name: anyhow
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps): bump secp256k1 from 0.29.0 to 0.29.1 (bluealloy#1770)
9b43aa1 
Bumps [secp256k1](https://github.com/rust-bitcoin/rust-secp256k1) from 0.29.0 to 0.29.1.
- [Changelog](https://github.com/rust-bitcoin/rust-secp256k1/blob/secp256k1-0.29.1/CHANGELOG.md)
- [Commits](rust-bitcoin/rust-secp256k1@secp256k1-0.29.0...secp256k1-0.29.1)

---
updated-dependencies:
- dependency-name: secp256k1
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps): bump alloy-transport from 0.3.0 to 0.3.5 (bluealloy#1771)
3359b0e 
Bumps [alloy-transport](https://github.com/alloy-rs/alloy) from 0.3.0 to 0.3.5.
- [Release notes](https://github.com/alloy-rs/alloy/releases)
- [Changelog](https://github.com/alloy-rs/alloy/blob/main/CHANGELOG.md)
- [Commits](alloy-rs/alloy@v0.3.0...v0.3.5)

---
updated-dependencies:
- dependency-name: alloy-transport
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@clearloop
feat(example): deploy bytecode from scratch (bluealloy#1767)
54d2415 
* feat(example): deploy bytecode from stratch

* feat(example): passing constructor parameters via creation code

* chore(example): use opcode constants for example deploy

* chore(example): typo EOF
@emhane
chore: Test for l1 gas used and l1 fee for ecotone tx (bluealloy#1748)
a340cf9
@prestwich
feat: to_plain_state (bluealloy#1778)
9ded690 
* feat: to_plain_state

* feat: extend to reverts
@rakita
chore: release v44 bump main CHANGELOG (bluealloy#1783)
b6ebcc8
@rakita
feat: project restructuring Part1 (bluealloy#1776)
3e5a307 
* feat: project restructuring

* refactor

* restructuring

* restructure

* restructure

* move revm-test bins to revme bench

* rm ethersdb and revm-test

* move kzg to wiring

* fmt, clippy, cleanup

* fix test and doc

* fix kzg-rs build

* Use hashbrown for revme

* docs fix

* fix merged change

* fmt
@royvardhan
feat: use TestAuthorization and skip decoding of eip7702 tx (blueallo…
be1d324 
…y#1785)
@rakita
feat: restructure Part2 database crate (bluealloy#1784)
fa1e546 
* feat: restructure database crate

* fix tests

* wip examples

* split examples to standalone crates
@rakita
feat: Restructuring Part3 inspector crate (bluealloy#1788)
66adad0 
* feat: Restructuring Part3 inspector crate

* fix serde include

* fix docs
@prestwich
chore: migrate off alloy-eips (bluealloy#1789)
5b6846b
@rakita
feat: restructure part3 fix examples (bluealloy#1792)
f75998a 
* fix contract deployment example

* database component example

* fix examples

* fix uniswap usdc swap

* fmt

* docs

* rm comment
@juanimedone
Replace PrecompileError variant (bluealloy#1797)
3562d2e
@jsvisa
chore: add ReentrancySentryOOG for SSTORE (bluealloy#1795)
5b24f61 
* feat: add ReentrancySentryOOG for SSTORE

Signed-off-by: jsvisa <[email protected]>

* sstore_cost return u64 instead of Option

Signed-off-by: jsvisa <[email protected]>

* fix testcase

Signed-off-by: jsvisa <[email protected]>

---------

Signed-off-by: jsvisa <[email protected]>
@rakita
feat: Merge validation/analyzis with Bytecode (bluealloy#1793)
e7de0bb 
* feat: Merge validation/analyzis with Bytecode

* cleanup

* enable parse and std for bytecode
@rakita
chore: refactor -copy common code (bluealloy#1799)
47169b3 
* chore: refactor *copy common code

* doc link fix
@DaniPopes
chore: fix constants module typo (bluealloy#1801)
4102ecd
@DaniPopes
chore: fix some typos (bluealloy#1800)
30c9199
@Wodann
fix: make macro crate-agnostic (bluealloy#1802)
e477c7f 
* fix: make macro crate-agnostic

* fix: clippy
@royvardhan
feat: push NonceChange to Journal in deduct_caller (bluealloy#1804)
f8d8ae9
@morph-dev
feat: add support for async database (bluealloy#1809)
411d0ef 
* feat: add support for async database

* fix: update link in docs
@royvardhan
feat: extract statetest models/structs to standalone crate (bluealloy…
ca5b2fe 
…#1808)

* feat: split test models to statetest-tests

* chore: attempt documentation

* Update Cargo.toml

* Update Cargo.toml


---------
@cairoeth
chore: Bump to [email protected] tests (bluealloy#1815)
798d08c
@DaniPopes
chore(primitives): replace HashMap re-exports with alloy_primitives::…
a47d3e5 
…map (bluealloy#1805)

* chore(primitives): replace HashMap re-exports with alloy_primitives::map

* chore: update methods

* chore: update

* use default hasher
FredCoen and others added 25 commits December 31, 2024 20:55
@FredCoen
Add bytecode hash in interpreter bluealloy#1888 (bluealloy#1952)
e3c7abf 
* add bytecode hash to interpreter

* fmt

* lazy setting bytecode_hash to prevent performance degradation

* move hash to extended bytecode

* remove trait bound

* Update crates/interpreter/src/interpreter/ext_bytecode.rs

* Update crates/interpreter/src/interpreter/ext_bytecode.rs

* Update crates/interpreter/src/interpreter/ext_bytecode.rs

* fix tests

* set hash for interpreter
@varun-doshi
feat: Split inspector.rs (bluealloy#1958)
6f3d9f8 
* split inspector

* fixed errors

* fmt

* fixed failing test

* fmt

* clippy fix

* fmt
@savvar9991
fix: Documentation Corrections and Clarifications (bluealloy#1962)
223a2d6 
* Update precompile.md

* Update builder.md

* Update kzg.md
@rakita
feat: simplify Transaction trait (bluealloy#1959)
12a790f 
* chore: relax some impl and exec_with_tx

* add helper created_address

* add error to database error

* ext journal with state

* fix: handle inspector final return

* call final_return

* expose precompile addresses to Journal

* precompile addresses fn for journal

* feat: align Block trait

* fix mul

* feat: simplify transaction trait

* wip impl for TxEnv

* compiled simlified tx trait

* tweek

* add perf on access list load

* rm debug print

* op enabled. Rm Error from Tx trait

* fix tests

* rn old prague tests
@rakita
feat: bump eof validation tests (bluealloy#1963)
f6cf872
@rakita
fix(eof): dont run precompile on ext delegate call (bluealloy#1964)
894bcfb 
* feat: bump eof validation tests

* fix(eof): dont run precompile on ext delegate call

* updates eof validation paths

* some cleanup
@rakita
feat(EIP-7623): Increase calldata cost. backport from rel/v51 (blueal…
04688b7 
…loy#1965)

* feat(EIP-7623): Increase calldata cost. backport from rel/v51

* bump devnet5 tests
@FredCoen
chore: fix journal naming for inc/dec balance (bluealloy#1976)
7fcbd7d
@FredCoen
chore: Add helper functions for JournalInit bluealloy#1879 (bluealloy…
c28ea1b 
…#1961)

* add journalinit type

* replace from into convention with explicit helpers
@rakita
feat(eip7702): apply latest EIP-7702 changes, backport from v52 (blue…
6a69c71 
…alloy#1969)

* feat(eip7702): apply latest EIP-7702 changes, backport from v52

* fix eip7702 tests
@rakita
fix: test case state root revert (bluealloy#1979)
67200ca
@rakita
feat(EIP-7840): Add blob schedule to execution client cfg (bluealloy#…
cd5a659 
…1980)

* feat(EIP-7840): Add blob schedule to execution client configuration files

* fix test

* no_std include vec

* doc
@rakita
core: blst reprice, remove g1/g2 mul (bluealloy#1981)
77a981c 
* core: blst reprice, remove g1/g2 mul

* fmt

* rm old bls tests
@rakita
chore: align crates versions (bluealloy#1983)
a2451cd
@meyer9
feat: extract and export estimate_tx_compressed_size (bluealloy#1985)
9bef7d9 
* feat: extract and export `estimate_tx_compressed_size`

* Lint
@rakita
feat(EIP-7623): adjuct floor gas check order (main) (bluealloy#1991)
bc44ae1 
* feat(EIP-7623): adjuct floor gas check order

* fmt
@rakita
fix(revme): statetest remove redundant json output (bluealloy#1994)
88c5a94
@rakita
fix(revme): Statetest stop exec when print output is true (bluealloy#…
234405c 
…1995)

* fix(revme): statetest remove redundant json output

* fix(revme): Statetest stop exec when print output is true
@nhtyy
chore: bump kzg.rs (bluealloy#2002)
39eaab7
@agostbiro
feat(examples): add cheatcode inspector example (bluealloy#2003)
242f61f
@0xAlcibiades
chore: bump alloy versions to match latest (bluealloy#2007)
b4578fe 
* fix: bumps alloy version for provider compat

* fix: bump examples
@Wodann
fix: deduplicate validate_initial_tx_gas API (bluealloy#2006)
25512fd 
* fix: deduplicate validate_initial_tx_gas API

* misc: apply review suggestions
@rakita
feat: EthHandler trait (bluealloy#2001)
34bc873 
* feat: EthHandler trait

* work work

* frame contexts

* inspector compiled

* transact_main and transact_main_commit

* Generalize InterpreterTypes

* big cleanup, rm all partial handlers

* compile

* Some cleanup

* some cleanup

* Erc20 example, need to fix logic

* fix erc20 example

* cleanup and testing

* fix eip7702 order, pass all devnet tests

* fix compilation

* docs
Includes EIP-7702 address in CODECOPY
34957e5
clippy,fmt
9ae1af5
rakita
rakita requested changes Jan 23, 2025
View reviewed changes
Copy link
Member

@rakita rakita left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Hey, the change is not about CODECOPY but about EXTCODECOPY. Hane wrote wrong opcode in the issue (have edited issue rn
), but if you go to the link you can see the change ethereum/EIPs#9248

And can you make a PR on top of release/v52 branch. And the line is here:

revm/crates/revm/src/context/inner_evm_context.rs

Line 186 in 26f0cf8

EIP7702_MAGIC_BYTES.clone()

@elijahhampton elijahhampton changed the base branch from main to release/v52 January 23, 2025 17:08
@elijahhampton
Copy link
Author

elijahhampton commented Jan 23, 2025

Hey, the change is not about CODECOPY but about EXTCODECOPY. Hane wrote wrong opcode in the issue (have edited issue rn ), but if you go to the link you can see the change ethereum/EIPs#9248

And can you make a PR on top of release/v52 branch. And the line is here:

revm/crates/revm/src/context/inner_evm_context.rs

Line 186 in 26f0cf8

EIP7702_MAGIC_BYTES.clone()

I created another PR here: #2016 to avoid potential problems from resolving the conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rakita rakita rakita requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.