refactor(array): use buffer allocator in execution context - #9669
Conversation
Merging this PR will degrade performance by 3.01%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | cast_i32_to_u32[8192] |
340.4 µs | 416.4 µs | -18.23% |
| ❌ | WallTime | deferred_i64_avx2[PerRowPerRow] |
10 µs | 11.1 µs | -10.32% |
| ⚡ | Simulation | new_raw_prim_test_between[i32, 2048] |
77.3 µs | 62.1 µs | +24.43% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing ngates/buffer-allocator-context (884d9d1) with ngates/buffer-allocator-core (4911c36)3
Footnotes
-
106 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. ↩
-
4 benchmarks were run, but are now archived. If they were deleted in another branch, consider rebasing to remove them from the report. Instead if they were added back, click here to restore them. ↩
-
No successful run was found on
ngates/buffer-allocator-core(94c47cb) during the generation of this report, so 28b9baa was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
318a807 to
ea37874
Compare
16b6ae2 to
8b88427
Compare
Signed-off-by: Nicholas Gates <nick@nickgates.com>
Signed-off-by: Nicholas Gates <nick@nickgates.com>
8b88427 to
884d9d1
Compare
Summary
Add a buffer allocator to execution and I/O contexts.
Changes
BufferAllocatorRefinExecutionContext.