Skip to content

stop popping callframe unnecessarily#2584

Merged
JereSalo merged 1 commit into
levm/cache_rollbackfrom
levm/cache_rollback_improvements
Apr 24, 2025
Merged

stop popping callframe unnecessarily#2584
JereSalo merged 1 commit into
levm/cache_rollbackfrom
levm/cache_rollback_improvements

Conversation

@JereSalo

@JereSalo JereSalo commented Apr 24, 2025

Copy link
Copy Markdown
Contributor

Motivation

  • stop popping callframe in finalize execution

Description

Closes #issue_number

@JereSalo JereSalo marked this pull request as ready for review April 24, 2025 19:36
@JereSalo JereSalo requested a review from a team as a code owner April 24, 2025 19:36
@JereSalo JereSalo merged commit 2524914 into levm/cache_rollback Apr 24, 2025
@JereSalo JereSalo deleted the levm/cache_rollback_improvements branch April 24, 2025 19:36
@github-actions

github-actions Bot commented Apr 24, 2025

Copy link
Copy Markdown

Lines of code report

Total lines added: 1
Total lines removed: 8
Total lines changed: 9

Detailed view 
+-------------------------------------------------+-------+------+
| File                                            | Lines | Diff |
+-------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/hooks/default_hook.rs | 262   | +1   |
+-------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/hooks/hook.rs         | 12    | -2   |
+-------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/hooks/l2_hook.rs      | 189   | -1   |
+-------------------------------------------------+-------+------+
| ethrex/crates/vm/levm/src/vm.rs                 | 469   | -5   |
+-------------------------------------------------+-------+------+

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@JereSalo