refactor(core): further decouple LEVM and rEVM implementations. #2179

mpaulucci · 2025-03-07T18:54:06Z

Motivation
Separate code from LEVM and rEVM implementations and try not to expose rEVM specific logic outside of vm crate.

Description

Moved levm specific code to backends/levm/...
Moved revm specific coed to backens/revm/...

crates/l2/prover/bench/src/cache.rs

crates/networking/rpc/eth/transaction.rs

crates/vm/backends/mod.rs

crates/vm/backends/revm/aux.rs

…to decouple-levm-revm-2

github-actions · 2025-03-07T19:12:20Z

Lines of code report

Total lines added: 1954
Total lines removed: 453
Total lines changed: 2407

Detailed view

+----------------------------------------------------+-------+------+
| File                                               | Lines | Diff |
+----------------------------------------------------+-------+------+
| ethrex/cmd/ef_tests/state/utils.rs                 | 77    | +3   |
+----------------------------------------------------+-------+------+
| ethrex/crates/l2/prover/bench/src/rpc/db.rs        | 416   | +3   |
+----------------------------------------------------+-------+------+
| ethrex/crates/networking/rpc/eth/transaction.rs    | 578   | +2   |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/levm/db.rs               | 40    | +40  |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/levm/mod.rs              | 436   | +436 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/mod.rs                   | 296   | +61  |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/db.rs               | 320   | +320 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/execution_db.rs     | 175   | +175 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/execution_result.rs | 116   | +116 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/helpers.rs          | 93    | +93  |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/mod.rs              | 655   | +655 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/backends/revm/mods.rs             | 38    | +38  |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/constants.rs                      | 12    | +12  |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/db.rs                             | 7     | -359 |
+----------------------------------------------------+-------+------+
| ethrex/crates/vm/vm.rs                             | 35    | -94  |
+----------------------------------------------------+-------+------+

cmd/ef_tests/state/utils.rs

crates/vm/backends/mod.rs

crates/vm/backends/levm/db.rs

crates/networking/rpc/eth/transaction.rs

crates/vm/Cargo.toml

crates/vm/backends/levm/db.rs

crates/vm/backends/mod.rs

crates/vm/backends/revm/aux.rs

…to decouple-levm-revm-2

**Motivation** - When executing ethereum/rpc simulation with LEVM we have errors in a lot of tests. The error is when executing the blocks for the setup for these tests and how we created the withdrawals requests introduced in EIP-7002 **Description** - First, when we execute the transactions, we weren't handling if the account was delegated and we weren't using the correct bytecode. For this, we add the check for delegated accounts and get the bytecode. This is made when creating a new VM and using the function `eip7702_get_code()`. - Second, when creating the withdrawals requests, we were overwritting and loosing some of the new values in the account updates. We now merge both updates. - When running the system contract for the withdrawals requests introduced in EIP 7002, we were creating a new `CacheDB` instead of using the one with updated values. **Status** - All RPC Hive tests were passing, but with the changed made [here](#2179), 12 are failing due to a new missing implementation. Links to #2158 --------- Co-authored-by: Tomás Paradelo <[email protected]>

refactor(core): further decouple LEVM and rEVM implementations.

b5a6319

mpaulucci requested a review from a team as a code owner March 7, 2025 18:54