Skip to content

feat(revme): add benchmarking capability to statetest command#3254

Open
woojinnn wants to merge 1 commit intobluealloy:mainfrom
woojinnn:feat/statetest-bench
Open

feat(revme): add benchmarking capability to statetest command#3254
woojinnn wants to merge 1 commit intobluealloy:mainfrom
woojinnn:feat/statetest-bench

Conversation

@woojinnn
Copy link
Contributor

Summary

Resolves #2573
Add --bench flag to run Criterion benchmarks on state test JSON files with --warmup (-w) and --time (-m) flags for benchmark configuration

Usage

# Run benchmarks on state tests
revme statetest --bench legacytests/Cancun/GeneralStateTests

# With custom warmup (500ms) and measurement time (5s)
revme statetest --bench -w 500 -m 5 legacytests/Cancun/GeneralStateTests

@woojinnn woojinnn marked this pull request as draft December 24, 2025 14:08
@woojinnn woojinnn force-pushed the feat/statetest-bench branch from c1ae4ec to 252ffbc Compare December 24, 2025 14:10
@woojinnn woojinnn marked this pull request as ready for review December 24, 2025 14:16
@codspeed-hq
Copy link

codspeed-hq bot commented Dec 24, 2025

CodSpeed Performance Report

Merging #3254 will not alter performance

Comparing woojinnn:feat/statetest-bench (252ffbc) with main (3b17ee6)

Summary

✅ 173 untouched

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

lgtm.

Didn't run it on my own but willl do later, this is local to revme binary so it is not consensus sensitive PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add benching capability to revm statetest

2 participants