Add sum oracle to the fuzzer and reject samples that have order dependent overflow behaviour - #9801
Add sum oracle to the fuzzer and reject samples that have order dependent overflow behaviour#9801robert3005 wants to merge 5 commits into
Conversation
Merging this PR will regress 1 benchmark
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | WallTime | mul_u32_nonnull_avx512 |
5.5 µs | 6.2 µs | -10.24% |
| ⚡ | Simulation | decompress[u64, (4000, 1024)] |
86.1 µs | 70.8 µs | +21.62% |
| ⚡ | WallTime | arrow_checked_add_u32_avx2[16384] |
21.4 µs | 17.7 µs | +21.12% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing rk/sumfuzzoracle (eff1c5b) with develop (2a5b1a7)
Footnotes
-
218 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. ↩
4fc3399 to
69ecab6
Compare
| } | ||
| } | ||
|
|
||
| fn accumulate<T>( |
There was a problem hiding this comment.
please add a comment. But I think you want to filter >0 and <0 and sum each and it will read better
Fuzzer sum oracle does chunk by chunk aggregation instead of canonicalising and
running aggregate on single chunk