2.1.8 Release

[RISCV] Better run-time detection of RVV vector instruction support

Original version posted by @ncopa in #1705.

Fixed false positive HAVE_ARMV6_INTRIN value on old ARM platforms.

Don't use chunkunroll for inflateBack

If the output buffer and the window buffer are the same
memory allocation, we cannot make the assumptions that chunkunroll
does, that it is okay to overwrite the output buffer.

Address CR feedback

Fix new Windows SDK build break

Co-authored-by: Jan Kotas <jkotas@microsoft.com>

Enable warning C4242 and treat warnings as errors for Visual C++.

 Fixes build issues C4242, C4244 and C4334 caused by loss of data bugs due to data type mismatch in various files.

IBM zSystems: Hardcode HWCAP_S390_VXRS

Compiling zlib-ng with glibc 2.17 (minimum version still supported by
crosstool-ng) fails due to the lack of HWCAP_S390_VX - it was
introduced in glibc 2.23.

Strictly speaking, this is a problem with the feature detection logic
in cmake. However, it's not worth disabling the s390x vectorized CRC32
if the hwcap constant is missing and the compiler intrinsics are
available.

So fix by hardcoding the constant. It's a part of the kernel ABI,
which does not change.

Silence possible loss of data warning in crc32_braid ZSWAPWORD call

Warning C4244
'=': conversion from 'z_word_t' to 'uint32_t', possible loss of data
zlib
crc32_braid_c.c 195

[CI] Upgrade MacOS ARM64 UBSAN to use gcc-13.

don't use zlib-ng's -Wl,--version-script in tests (#1750)

lld 18 errors when a version script assigns a version to a symbol that
is not defined in the object files. Therefore configure scripts should
not use zlib-ng's version script -- all tests will fail.

Also test whether the linker supports the flag instead of assuming.

2.1.7 Release

Fix a couple typos found by codespell

free_aligned: validate passed in pointer

Especially when replacing zlib with zlib-ng in old binaries, it is possible that the binary is mixing allocation and
deallocation functions from different libraries. As not all old binaries can be rebuild, we should validate the passed
in pointer and if the pointer doesn't seem to be allocated with alloc_aligned of zlib-ng, we should not try to adjust
the pointer.

Enabled orphaned unit tests for compare256_rle family of functions.

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

Improve source compatibility with zlib

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

Add AArch64 feature detection support for OpenBSD

- Make codecov uploads happen even if some jobs have failed.
- Fix error in generating list of reports to upload.
- Clean up the implementation a bit.

Add back-and-forth inflateCopy() test

Check that calling inflateCopy() twice does not result in memory
corruption.

Use actual compressed length

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

Fix illegal instruction usage in Xeon Phi x200 processors

The Xeon Phi x200 family of processors (Knights Landing) supports
AVX512 (F, CD, ER, PF) but does not support AVX512 (VL, DQ, BW).

Because of processors like this, the Intel Software Developer's Manual
suggests the bits AVX512 (DQ,BW,VL) are also tested in EBX together with
AVX512F before deciding to run AVX512 (DQ,BW,VL) instructions.

This also adds a new x86 feature called avx512_common that indicates
that AVX512 (F,DQ,BW,VL) are all available and start using this for both
adler32_avx512 and crc32_vpclmulqdq implementations because they are
both built with -mavx512dq -mavx512bw -mavx512vl.

This has been reported downstream as
https://bugzilla.redhat.com/show_bug.cgi?id=2280347 .

Use windows-2019 for build with toolset v141

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

Make lint check ignore *.patch files, as those can legally have trailing
whitespace or end the file with blank lines.

Update llvm version used for MSAN.
Disable unneccessary compilation of tests, benchmarks, docs.

Update s390x CI setup.
- New dockerfile
  - Using native actions-runner instead of relying on qemu.
  - To support s390x, we include patches to actions-runner.
  - Using Almalinux 9 instead of Ubuntu, with functional .Net.
- Update CI workflow.
- Update readme guide.

Fix a name conflict when compiled with zlib-compat

The benchmarks fail to compile properly when built with ZLIB_COMPAT.
The name of the class cannot have the same name as the function.

Add small compress() benchmark

IBM zSystems DFLTCC: Extend sanitizer checks

Currently the DFLTCC sanitizer instrumentation is limited to
MSAN-unpoisoning the parameter block. Add ASAN and MSAN checks;
also MSAN-unpoison the window.

Introduce the generic instrument_read(), instrument_write() and
instrument_read_write() macros, that are modeled after the repsective
functions in the Linux kernel.

Switch to python venv

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

Fix -std option in configure script

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

Update macOS Github Actions

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

.gitattributes: Enforce LF line-endings on all non-binary files

Although Git is able to automatically modify line-endings during checkin
and checkout, this brings a lot of trouble, especially when trying to
use a repository from different platforms (as Windows and Linux). This
is due to the fact that Git consults different local gitconfig settings
and uses problematic defaults if not set.

Therefore, Git should enfoce one type of line-ending (LF) and not
consult the local config, which is what the change from this commit
does.

Signed-off-by: Deniz Bahadir <deniz@code.bahadir.email>

make darwin cross compilation possible

https://github.com/madler/zlib/pull/789

Bump max CMake policy version to 3.29.0

Addresses:

  3.29.0/share/cmake/Modules/CMakeDependentOption.cmake:89 (message):
    Policy CMP0127 is not set: cmake_dependent_option() supports full Condition
    Syntax.  Run "cmake --help-policy CMP0127" for policy details.  Use the
    cmake_policy command to set the policy and suppress this warning.
  Call Stack (most recent call first):
    CMakeLists.txt:107 (cmake_dependent_option)
This warning is for project developers.  Use -Wno-dev to suppress it.

arch/riscv/riscv_features.c: fix uclibc build

Fix the following uclibc build failure raised since version 2.1.4 and
https://github.com/zlib-ng/zlib-ng/commit/6ff8b52cefe56a824fae1d53fdd687bcde2e53c9:

/home/autobuild/autobuild/instance-2/output-1/build/zlib-ng-2.1.6/arch/riscv/riscv_features.c:4:10: fatal error: sys/auxv.h: No such file or directory
    4 | #include <sys/auxv.h>
      |          ^~~~~~~~~~~~

Fixes:
 - http://autobuild.buildroot.org/results/06a7d8e59ec4de7c711d3f4a4624f67b97d78afe

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Update dependencies for 32-bit MinGW CI run.

CMake: Replace ';' by '$<SEMICOLON>' in generator-expression

Note: CMake generator-expressions should not contain semicolons,
especially if they might end up in a CMake list, because a semicolon
would be interpreted as list-item separator and therefore render the
generator-expression invalid. The generator-expression `$<SEMICOLON>`
should be used instead.

Signed-off-by: Deniz Bahadir <deniz@code.bahadir.email>

Remove obsolete TARGET_OS_MAC check

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

Fix building with NVHPC

[ARM] Override Clang x4 NEON intrinsics for Android
* Clang for Android requires 256-bit alignment for x4 loads and stores, which can't be guaranteed and is unnecessary

[CI] Fix creating list of artifacts for CodeCov.

Add more result variables to the cmake package configuration

Add test for checking if -march=native needs -mfpu=neon for 32-bit ARM.

Bump actions/upload-artifact from 3 to 4

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump github/codeql-action from 2 to 3

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Add dependabot for github actions

Fix PCLMULQDQ support for IntelLLVM.

Disable CodeCov for RISC-V as the toolchain doesn't support generating code coverage.

Update mymindstorm/setup-emsdk to v14.

Upgrade codecov/codecov-action to v4.

Add missing checks for 64bit arm/intel with msvc compiler

Disable Intel Compiler diagnostic message 10441.

Upgrade ilammy/msvc-dev-cmd to v1.13.0

Fix xsave intrinsic test for clang, and gcc 8.2 or later, and icc.

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)`.

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

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

madler/zlib#431a9b65eacab7efabf2230ba97ff426c0e07f9d

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

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

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 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.

Fixed S390X configure builds not running in forks.

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