Increase alignment from 8 to 16 to avoid warnings with ms compiler

Fixing align attribute, makes ms compiler warn: 'internal_state': Alignment specifier is less than actual alignment (16), and will be ignored.
Increasing alignemnt fixes the warning

Fix `deflate_state` alignment with MS or clang-cl compilers

When building with clang-cl, compiler produces the following warning:

zlib-ng/deflate.h(287,3): warning : attribute 'align' is ignored, place it after "struct" to apply attribute to type declaration [-Wignored-attributes]
zlib-ng/zbuild.h(196,34): note: expanded from macro 'ALIGNED_'

Repositioning align attribute after "struct" fixes the warning and aligns `deflate_state` correctly.

Avoid empty function by renaming bi_flush to zng_tr_flush_bits.

Remove branch that is almost impossible to hit in bi_flush().
Add case that handles sizes >=48.

Relocate CMake target export definitions

When exporting a CMake target the `install(TARGETS ${ZLIB_INSTALL_LIBRARIES})` is repeated with an `EXPORT` parameter. On Windows this would install the `.dll` files into the `lib` directory.

Set the `EXPORT_NAME` earlier and unconditionally on the `TARGET`. Then move the `EXPORT` to a single `install(TARGETS)`.

Use zng_alloc_aligned in unit tests to prevent having to use C++17.

alloc_aligned when using in C++ requires C++17 standard. zutil_p.h
include removed from test_crc32 since it was causing the same issue and was
not really needed.

Memory alloc size must be a multiple of alignment for aligned_alloc.

Prefer HAVE_ALIGNED_ALLOC when available in zng_alloc

Added some more helpful comments for people who come across this code.

zlib 1.3.1

madler/zlib#51b7f2abdade71cd9bb0e7a373ef2610ec6f9daf

Remove -w compile option in configure test.

Not all C compilers have a -w option.

madler/zlib#88ec24670e9c484219392def5f2f4b5e3f839db3

Fix cmake build on AIX.

The --version-script linker option is not supported by the linker on AIX systems

madler/zlib#ade6825c4996f73d776b0ffa038360762bd318ae

Remove mentions of an official zlib DLL distribution.

There used to be one, but no more. It is up to the user or vendor
to compile zlib.

madler/zlib#25263462377fa5c569947a40315815bd59e37f4b

Add back gz_intmax for zlib compatibility when linking statically

Make the existence of gz_intmax() unconditional.

gz_intmax() is noted in zlib.map. This assures it's always there.

madler/zlib#01253ecd7e0a01d311670f2d03c61b82fc12d338

Note that the len2 argument of crc_combine*() must be non-negative.

If it is negative, then the code will enter an infinite loop.

madler/zlib#36e369e1a54b35a978dc584496af69a07ec2d71a

Enable LIT_MEM by default expect when WITH_REDUCED_MEM is ON.

Fix the copy of pending_buf in deflateCopy() for the LIT_MEM case.

madler/zlib#60c31985ecdc2b40873564867e1ad2aef0b88697

Fix pending buffer overflow assert with LIT_MEM allocation.

Since each element in s->d_buf is 2 bytes, the sx index should be
multiplied by 2 in the assert.

Fixes madler/zlib#897

madler/zlib#ee474ff2d11715485a87b123edbdd615ba218b88

Add bounds checking to ERR_MSG() macro, used by zError().

madler/zlib#431a9b65eacab7efabf2230ba97ff426c0e07f9d

Add LIT_MEM define to use more memory for a small deflate speedup.

A bug fix in zlib 1.2.12 resulted in a slight slowdown (1-2%) of
deflate. This commit provides the option to #define LIT_MEM, which
uses more memory to reverse most of that slowdown. The memory for
the pending buffer and symbol buffers is increased by 25%, which
increases the total memory usage with the default parameters by
about 6%.

madler/zlib#ac8f12c97d1afd9bafa9c710f827d40a407d3266

Make internal functions static in the test code.

To avoid warnings when building with -Wmissing-prototypes.

madler/zlib#bd9c329c1055a9265812352655ed2eec93f36e92

Fix bug in inflateSync() for data held in bit buffer.

madler/zlib#5af7cef45eeef86ddf6ab00b4e363c1eecaf47b6

Update copyright years in other source files.

Update copyright year in win32 resource files.

madler/zlib#8988e03256e9c80766ac6899e86c3bc57c347efc

Remove unused code in makecrct.c

Add missing clean rule for arch/generic

Rename crc32_braid.c to crc32.c

Add support for handling alignment correction for input buffers down to 16 bytes in crc32_fold_[v]pclmulqdq

Speed up crc32_[v]pclmulqdq on small strings.

Add more fine-grained small-length benchmarking of adler32 and crc32.

Generate CMake package configuration files
* For COMPAT mode, zlib-config.cmake and zlib-config-version.cmake
* For non-COMPAT mode, zlib-ng-config.cmake and zlib-ng-config-version.cmake

Remove extern keyword from cpu_feature function declarations.

Remove type declarations for z_stream/zng_stream from cpu_features.

Split cpu_features.h by architecture.

Fix regression caused by 2fa631e029084b75acd81db5d33fd4aa802fd082
* POWER8/9 feature checks were enabled even if the toolchain didn't support AT_HWCAP2
* Add detection if we need to include <linux/auxvec.h>

Removing some outdated comments

These were left on my part, the inline copy + checksum is the very
thing the function is doing.

Set registers to zero to prevent GCC __get_cpuid bug.

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77756

Simplify architecture #ifdef in test_crc32.cc.

Add VPCLMULQDQ crc32 tests to Google benchmarks

Improve x86 intrinsics dependencies.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Remove unused macro X86_MASK_INTRIN.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Reorder in which intrinsics support is checked.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Fix x86 WITH_* options dependencies.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Remove always true arch conditions.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Move compare256 and longest_match C fallbacks to arch/generic

Move include files out of match_tpl.h, to simplify dependency handling and
avoid attempting to parse the includes more times than needed.

Move slide_hash C fallback to arch/generic

Move insert_string and update_hash C fallbacks to arch/generic.
Also add missing insert_string dependencies to windows makefiles.

Move chunkset and inffast C fallbacks to arch/generic

Move crc32 C fallbacks to arch/generic

Move adler32 C fallbacks to arch/generic

Simplify includes

Improve code coverage for SSE2.

We need to turn off support all higher instruction sets in order to hit code
for a lower instruction set.

Fixed S390X CMake builds not running in forks.

Upload all coverage reports at once after tests.

Remove unneeded pointer for functable.longest_match in deflate_slow

2.1.6 Release

Fix building tests with cmake 3.10 and older.

Add FAR macro to zlib-compat headers to improve compatibility.

Added CI instance for WITH_NATIVE_INSTRUCTIONS.

Remove tab-in-indent rule from lint workflow

Makefiles require using tabs as indents. So we need to remove this rule.

Cancel previously running CI jobs for PR or branch.

Fix memory corruption introduced in 61e181c8ae93dbf56040336179c9954078bd1399

Fixed tab character in ARM ACLE uqsub16 assembly.

Fix trailing whitespace in minideflate.c.

Add linter workflow for whitespace errors. zlib-ng/zlib-ng#1190

Don't attempt ARMv6 detection on AARCH64.

Don't incorrectly disable GoogleTest because of old CMake version.

Add status output when running git checkout of GoogleTest

Update to GoogleTest 1.12.1.
This requires minimum CMake 3.5 and C++11, this matches nicely with zlib-ng 2.1.x requirements.

Remove unused Z_TLS and HAVE_THREAD_LOCAL

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Atomic functable

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Make sure uqsub16 mnemonic doesn't get optimized away.

Add support for __attribute__((__target__(...))) to overcome limitations of -march=native.

Always run CMake tests without LTO.

Check if compiler supports -march=native or -mcpu=native.

Improve performance of crc32_acle on 32-bit ARM

Initialize functable earlier, during inflateInit

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Add force initialization functable, because deflate captures function pointers from functable

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Fix buffer overflow in compare256_rle benchmark

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Only read version information from zlib.h.in

This is a patch for https://github.com/zlib-ng/zlib-ng/commit/cd458c79c7923e76ffdd0e2638bf2a0eed4f3996
As the commit message says: "Add code to extract version information
for both ZLIB and ZLIBNG variants from the zlib.h file."

The logic of ${SUFFIX} is that zlib-ng reads version information after
parse ZLIB_COMPAT before the commit mentioned above. ${SUFFIX} will only
be set by ZLIB_COMPAT and is meaningless before ZLIB_COMPAT.

Update copyright to sync with zlib 1.3

https://github.com/zlib-ng/zlib-ng/pull/1563 this patch forgets to
update copyright string.

Remove duplicate enable tests option

There are two identical ZLIB_ENABLE_TESTS option in CMakeLists.txt which
means the latter one is meaningless. This issue is introduced in
https://github.com/zlib-ng/zlib-ng/commit/ce1a64b5147b874a2dd32920765be333f54e7095
which adds the option to top but forgets to remove the original one.

Fix pkgcheck mismatch with configure due to cmake target files.

Export cmake target (#1601)

* Export cmake target
* Mimic zlib target name

---------

Signed-off-by: Uilian Ries <uilianries@gmail.com>

Detemplatize adler32_fold_copy for avx512

Measurements show deltas basically in the noise floor for the
benchmarks.

The AVX512 VNNI implementation is a bit special in that it seems to be
be hurt by the clock penalty enough versus a normal copy that a 256 bit
wide variant was warranted.

Simplify AVX2 adler32_fold_copy by removing templates

Prevent tests writing into the source directory.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Run OSB tests with readonly source directory.

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>

Add adler32_fold_copy_rvv implementation.

Fix clang-cl warnings

2.1.5 Release

Fix an issue with regard to finishing out the window

if inflate is invoked with Z_FINISH, and it deems a window was not
necessary, there's a corner case where we never checksum the bytes.
Detect this by checking the window size against zero and the value
of the flush parameter.

This should fix issue #1600, and possibly #1565 as well.

Added unit test for inflate with Z_FINISH and no window.

This test decompresses a small git pack object with the text:
"alone in the dark". By providing inflate with Z_FINISH, it should not use a
sliding window according to zlib docs. We purposefully limit the read buffer
size to trigger CHECK mode to bail early.

[configure] Propagate -DWITH_GZFILEOP to pkgconfig.

pkgconfig file: propagate WITH_GZFILEOP

fix cmake header path in install with custom CMAKE_INSTALL_INCLUDEDIR

[README] configure no longer supports --native

Fix status badges

Update to actions/checkout@v4, and disable checkout progress logging.