Conversation
|
GNU testsuite comparison: |
Merging this PR will degrade performance by 13.4%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Simulation | shuf_lines[100000] |
27.4 ms | 31.7 ms | -13.4% |
| ⚡ | Memory | shuf_lines[100000] |
10.3 MB | 9.8 MB | +5.31% |
| ⚡ | Memory | shuf_repeat_sampling[50000] |
1.2 MB | 1.1 MB | +4.72% |
Comparing xtqqczze:memchr/split_seps (651812e) with main (2d5e77d)2
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. ↩
-
No successful run was found on
main(02104f3) during the generation of this report, so 2d5e77d was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
|
It touches data twice. I think it is why perf -13.49% . |
|
You might try to scan first few byte and predict capacity. |
|
first we should improve benchmarks to include short average line length: #11364 |
Can we get length of one SIMD scan? Averadge is enough (no need to get from system)? |
40cc40e to
651812e
Compare
|
GNU testsuite comparison: |
No description provided.