factor: add benchmark coverage#12479
Conversation
303750d to
32c0a7e
Compare
32c0a7e to
5951495
Compare
|
GNU testsuite comparison: |
|
please fix the jobs |
5951495 to
e3a997c
Compare
|
44 new benchmarks is too much for a program that it isn't super important |
e3a997c to
c427443
Compare
|
updated, now only 3 benchmarks total |
Merging this PR will improve performance by 91.24%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | true_consecutive_calls |
170.3 ns | 199.4 ns | -14.62% |
| 🆕 | Memory | factor_large_u64_prime |
N/A | 54.1 KB | N/A |
| 🆕 | Memory | factor_64bit_semiprime |
N/A | 6.1 KB | N/A |
| ⚡ | Memory | factor_multiple_u64s[2] |
54.1 KB | 5.6 KB | ×9.6 |
| 🆕 | Simulation | factor_64bit_semiprime |
N/A | 109 µs | N/A |
| 🆕 | Simulation | factor_large_u64_prime |
N/A | 149.4 µs | N/A |
| ❌ | Simulation | false_consecutive_calls |
170.3 ns | 199.4 ns | -14.62% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing naoNao89:factor-bench-migration (c427443) with main (55549fa)
Footnotes
-
46 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. ↩
expands
factorDivan bench with internal factorization cases and e2e CLI casesCovers: small ranges, semiprimes, large primes, u128-sized composites, and PR timing inputs
#9587