search

Frommi / miniz_oxide

Rust replacement for miniz
MIT License
168 stars 48 forks source link

Optimize match_len == 3 #146

Closed kornelski closed 5 months ago

kornelski commented 5 months ago

The fast path has fewer instructions, and gets inlined into decompress.

oyvindln commented 5 months ago

Thanks!

I can see a measurable impact in decompression speed with the old test app I had sitting around. The previous PR was mainly impacting the init speed so it was a bit harder to measure without writing something new or looking with a profiler or similar.

If you're planning to do some more work on this I'll hold on on doing a new release for a little, need to look into #143 in any case.