Skip to content

Conversation

@GunaDD
Copy link
Contributor

@GunaDD GunaDD commented Nov 21, 2025

No description provided.

@github-actions

This comment has been minimized.

@GunaDD GunaDD requested a review from shuklaayush November 21, 2025 21:43
@github-actions
Copy link

group app.proof_time_ms app.cycles app.cells_used leaf.proof_time_ms leaf.cycles leaf.cells_used
verify_fibair (-6 [-2.6%]) 228 322,610 2,058,654 - - -
fibonacci (-1 [-0.1%]) 972 1,500,209 2,100,402 - - -
regex (-14 [-0.6%]) 2,378 4,137,502 17,695,216 - - -
ecrecover (+18 [+2.6%]) 720 122,859 2,265,100 - - -
pairing (-10 [-0.7%]) 1,410 1,745,742 25,468,210 - - -

Commit: 222b086

Benchmark Workflow

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 21, 2025

CodSpeed Performance Report

Merging #2276 will degrade performances by 14.46%

Comparing fix/aot-e1-instret-left (222b086) with main (b2e33af)1

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

❌ 1 regression
✅ 23 untouched
⏩ 36 skipped2

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Mode Benchmark BASE HEAD Change
WallTime benchmark_execute[revm_snailtracer] 6.9 ms 8 ms -14.46%

Footnotes

  1. No successful run was found on main (6ff454e) during the generation of this report, so b2e33af was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

  2. 36 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@Maillew Maillew self-requested a review November 21, 2025 21:54
@jonathanpwang jonathanpwang merged commit 91e34d5 into main Nov 21, 2025
68 of 70 checks passed
@jonathanpwang jonathanpwang deleted the fix/aot-e1-instret-left branch November 21, 2025 22:27
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.

5 participants