Reduce vortex-fastlanes debug symbols size by introducing debug-only path - #8502
Reduce vortex-fastlanes debug symbols size by introducing debug-only path#8502AdamGS wants to merge 2 commits into
Conversation
Merging this PR will not alter performance
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | decompress_rd[f64, (10000, 0.1)] |
109 µs | 135.5 µs | -19.55% |
| ❌ | Simulation | decompress_rd[f64, (10000, 0.01)] |
108.7 µs | 135.1 µs | -19.54% |
| ❌ | Simulation | decompress_rd[f64, (10000, 0.0)] |
108.7 µs | 135.1 µs | -19.54% |
| ❌ | Simulation | decompress_rd[f32, (10000, 0.1)] |
78.1 µs | 86.8 µs | -10.04% |
| ⚡ | Simulation | chunked_varbinview_opt_canonical_into[(1000, 10)] |
206.8 µs | 170.2 µs | +21.46% |
| ⚡ | Simulation | decompress_rd[f64, (100000, 0.0)] |
978.9 µs | 843.2 µs | +16.1% |
| ⚡ | Simulation | chunked_varbinview_into_canonical[(100, 100)] |
307.1 µs | 272.8 µs | +12.59% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing adamg/reduce-debug-symbols-fastlanes (31b9d58) with develop (3a7b9a0)
532f345 to
aefa6b8
Compare
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
aefa6b8 to
8b3b046
Compare
|
This PR has been marked as stale because it has been open for 14 days with no activity. Please comment or remove the stale label if you wish to keep it active, otherwise it will be closed in 7 days |
|
This PR was closed because it has been inactive for 7 days since being marked as stale. |
Summary
vortex-fastlanesis currently a pretty big crate, due to its underlying structure/generics.When running
cargo clean; cargo build -p vortex-fastlanesCurrently on `develop:
We currently disable debug symbols for it to improve our development experience, but underlying crates can't enjoy that without explicitly adding the same or similar configuration.
With this branch, the build takes 30s and generates a 60MB rlib, with no performance impact for release builds.