Repository navigation
For decompression works on owned arrays - #10256
robert3005 wants to merge 2 commits into
Conversation
Merging this PR will regress 1 benchmark
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | WallTime | bitpack_blocked_compress_avx2 |
6.7 µs | 7.6 µs | -11.59% |
| ⚡ | WallTime | decode_chunked_avx512[i64, 524288] |
19.1 µs | 13.8 µs | +38.86% |
| ⚡ | WallTime | decode_chunked_avx2[i64, 524288] |
18.9 µs | 13.7 µs | +37.34% |
| ⚡ | WallTime | decode_chunked_neon[i64, 524288] |
13.5 µs | 11.9 µs | +14.15% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/ownedfor (4bdf7fa) with develop (5314209)
Footnotes
-
518 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. ↩
Signed-off-by: Robert Kruszewski <github@robertk.io>
de18cc9 to
5d0378c
Compare
We have fast in place decompression path, pass the array by value and use them.