Nathan Moinvaziri comments

Results 433 comments of


                                            Nathan Moinvaziri

Inflate: Unconditional refill and fastpath for literals

I wonder if can use `__nop()` https://docs.microsoft.com/en-us/cpp/intrinsics/nop?view=msvc-170

Inflate: Unconditional refill and fastpath for literals

Looks like heap-buffer-overflow from OSS-Fuzz..

Inflate: Unconditional refill and fastpath for literals

I ran two small benchmarks, one with `nop` and one without. # Silesia-small 15MB MSVC noop ## HEAD https://github.com/zlib-ng/zlib-ng/commit/89763032d57e3da5301f4cd6e1f363e7a1f85f02 ``` OS: Windows 10 10.0.22000 AMD64 CPU: Intel64 Family 6 Model...

Inflate: Unconditional refill and fastpath for literals

For the second benchmark I just removed the `asm` lines from the #define. I'm not sure if it they would have made much of a difference on x86..

Inflate: Unconditional refill and fastpath for literals

@Dead2 oddly this needs workflow approval too. It looks like it ran some but is now asking again for approval.

Optimized longest_match function (code submission)

For anybody still interested, I have incorporated these changes in https://github.com/zlib-ng/zlib-ng/pull/828.

Fix compilation with clang-cl on windows

I have submitted the issue to Microsoft. https://developercommunity.visualstudio.com/t/ClangCl-include_next-error-when-includi/1622642?space=62&entry=myfeedback

Using AVX2

[zlib-ng](https://github.com/zlib-ng/zlib-ng) has support for AVX2.

Initialize 'check' member in inflateResetKeep()

We are also currently doing this in zlib-ng and has been working well for some time. @madler

acceleration of gzip on imb z. Worth for zlib ?

[zlib-ng](https://github.com/zlib-ng/zlib-ng) has support for IBM Z acceleration.