Repository navigation
bench: add memchr vs scalar comparison for #718 - #1014
dhimasardinata wants to merge 2 commits into
Conversation
Provides benchmark from tafia#718 table: memchr 5.89 GB/s vs stringzilla 8.37 GB/s (Intel forward), 1.08 vs 8.21 reverse. This bench reproduces scalar iter.position vs SIMD memchr3 for escape 6-char case using sample_rss.xml:194K x20. Run: cargo bench --bench issue718 -- --sample-size 10 Related: tafia#405 escape SIMD, tafia#718 stringzilla
|
Name the file / bench |
Address dralley review on tafia#1014: name file/bench simd_vs_scalar not issue718
|
Renamed per review: |
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1014 +/- ##
==========================================
- Coverage 57.31% 55.00% -2.32%
==========================================
Files 46 52 +6
Lines 18197 18847 +650
==========================================
- Hits 10429 10366 -63
- Misses 7768 8481 +713
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
I think, that we do not need that benchmark, because it is not related to quick-xml. #1013 already shows us effect of using memchr escaping instead of built-in Rust |
Related to #718, #405
Adds
benches/issue718.rsto reproduce scalar vs SIMD gap from memchr_vs_stringzilla table in #718:This bench uses existing
sample_rss.xml:194K x20and compares:iter.position(|&b| matches!(b, <>&'"\r))(old _escape)2x memchr3(new SIMD accelerated escape routines #405 impl)Run:
cargo bench --bench issue718 -- --sample-size 10Provides baseline for #405 and future stringzilla/AVX-512 exploration without adding C++ dep. Can extend to stringzilla-rs binding later.