Skip to content

Replace DynVarBinBuilder with a generic VarBinBuilder - #9096

Open
robert3005 wants to merge 4 commits into
developfrom
rk/varbin-builder-trait
Open

Replace DynVarBinBuilder with a generic VarBinBuilder#9096
robert3005 wants to merge 4 commits into
developfrom
rk/varbin-builder-trait

Address review: cover every offset width and push values into the bui…

e072374
Select commit
Loading
Failed to load commit list.
CodSpeed HQ / CodSpeed Performance Analysis failed Jul 31, 2026 in 0s

Performance Regression: -10.9%

❌ 2 regressed benchmarks
✅ 1840 untouched benchmarks
⏩ 55 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
❌ Simulation execute_scalar_struct_simple 426.6 µs 483 µs -11.67%
❌ Simulation execute_scalar_struct_wide 2 ms 2.3 ms -10.13%

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.

Comparing rk/varbin-builder-trait (e072374) with develop (08c336f)

Open in CodSpeed

Footnotes

  1. 55 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. ↩

View more details on CodSpeed HQ