Skip to content

Perf/cpu lde rework with default hasher#722

Closed
MauroToscano wants to merge 23 commits into
mainfrom
perf/cpu-lde-rework-default-hasher
Closed

Perf/cpu lde rework with default hasher#722
MauroToscano wants to merge 23 commits into
mainfrom
perf/cpu-lde-rework-default-hasher

Conversation

@MauroToscano

Copy link
Copy Markdown
Contributor

Based on PR #650, with the PR-added deterministic FxHasher removed so op-dedup maps use Rust default keyed HashMap. This keeps the LDE and trace parallelization changes intact for benchmarking the hasher delta. If benchmark timing is similar, this avoids the deterministic-hasher DoS concern.

@MauroToscano

Copy link
Copy Markdown
Contributor Author

/bench

@github-actions

Copy link
Copy Markdown

Benchmark — ethrex 20 transfers (median of 3)

Table parallelism: auto (cores / 3)

Metric main PR Δ
Peak heap 79872 MB 81977 MB +2105 MB (+2.6%) ⚪
Prove time 50.360s 44.776s -5.584s (-11.1%) 🟢

🎉 Improvement detected — heap or time decreased by more than 5%.

✅ Low variance (time: 1.8%, heap: 1.3%)

Commit: d1bf465 · Baseline: built from main · Runner: self-hosted bench

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.

3 participants