Skip to content

Conversation

@shekhirin
Copy link
Collaborator

Follow-up to #3231 that makes use of alloy-core fixed bytes hashmaps throughout the code for Address, U256, and B256 keys.

@shekhirin shekhirin marked this pull request as ready for review December 18, 2025 17:53
@codspeed-hq
Copy link

codspeed-hq bot commented Dec 18, 2025

CodSpeed Performance Report

Merging #3238 will degrade performances by 5.84%

Comparing alexey/alloy-fixed-bytes-maps (bd83723) with main (7b505e6)

Summary

❌ 2 regressions
✅ 171 untouched

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

Benchmarks breakdown

Benchmark BASE HEAD Change
SLOAD_50 34.4 µs 36.5 µs -5.84%
SSTORE_50 39.5 µs 41.7 µs -5.39%

@rakita
Copy link
Member

rakita commented Dec 22, 2025

Not sure why we are seeing 5% perf degradation

@rakita
Copy link
Member

rakita commented Dec 22, 2025

Will read it in detail, but PR makes sense

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