Revert "pass dictionary loading method as parameter"

This reverts commit 821fc567f93a415e9fbe856271ccd452ee7acf07.

better MT fluidity

--patch-from no longer blocked on first job dictionary loading

ensure cdict is properly reset to NULL

skips the need to create a temporary cdict for --patch-from

thus saving a bit of memory and a little bit of cpu time

reduced memory usage

by avoiding to duplicate in memory
a dictionary that was passed by reference.

always free .cdictLocal

update type naming convention

minor boundary change

improves compression ratio at low levels

fix speed of --patch-from at high compression mode

Merge pull request #4287 from facebook/warn_error

visual studio compilation tests: error out on warnings

fix minor conversion warnings

more %zu warnings fixes

fix more %zu warnings

fix minor printf argument limitation on older mingw

which do not support `%zu` fields

fix minor alignment warning

fix minor alignment warning

this is a prototype definition error:
`_mm_storeu_si128()` should accept a `void*` pointer,
since it explicitly states that it accepts unaligned addresses
yet requiring a `__m128i*` tells otherwise, and requires the compiler the enforce this alignment.

fix one minor alignment warning

seems like a prototype interface error:
input parameter should have been `const void*`,
since the documentation is explicit that input doesn't have to be aligned,
but `const __m256i*` makes the compiler enforce it.

make visual studio tests error out on warnings

Merge pull request #4286 from facebook/visual_clang_avx2

Fix Visual + ClangCL + AVX2 compilation

added __clang__ to compiler-specific alignment attribute

when clang is used within msvc, `__GNUC__` isn't defined,
so testing `__clang__` explicitly is required.

changed C11 keyword to _Alignas

so that it doesn't depend on #include

adding a Visual + ClangCL + AVX2 compilation test

Merge pull request #4265 from pps83/static-bmi2-check

Check `STATIC_BMI2` instead of `STATIC_BMI2 == 1`

Merge pull request #4271 from tsdgeos/removedupe

cmake: Remove duplicated line

Merge pull request #4278 from facebook/bench_x100

bench: better result alignment

bench: better result alignment

when displaying ratios > x100

Merge pull request #4264 from pps83/dev-static-bmi2

Move STATIC_BMI2 define to portability_macros.h

cmake: Remove duplicated line

Merge pull request #4269 from luau-project/fix-rc-include

fix: quote include directory for resource compiler

fix: quote include directory for resource compiler

Merge pull request #4260 from gcabiddu/compress_sequences_kernel

[linux] Expose ZSTD_compressSequences*() in the kernel

[linux] Expose ZSTD_compressSequencesAndLiterals() in the kernel

Make the function ZSTD_compressSequencesAndLiterals() available in kernel
space. This will be used by Intel QAT driver.

Additionally, (1) expose the function ZSTD_CCtx_setParameter(), which is
required to set parameters before calling ZSTD_compressSequencesAndLiterals(),
(2) update the build process to include `compress/zstd_preSplit.o` and
(3) replace `asm/unaligned.h` with `linux/unaligned.h`.

Signed-off-by: Giovanni Cabiddu <giovanni.cabiddu@intel.com>

CI: build with CMake on source directory with spaces to reproduce #4268

Merge pull request #4266 from pps83/rm-ZSTD_decompressSequences_t

Remove unused ZSTD_decompressSequences_t typedef

fix formatting

Check `STATIC_BMI2` instead of `STATIC_BMI2 == 1`

Reorder __BMI2__ check

 + if `__BMI2__` defined, then set STATIC_BMI2 for all compilers
 + use `defined(_MSC_VER) && defined(__AVX2__)` as fallback for ms compiler

Update comment for STATIC_BMI2 macro

Move STATIC_BMI2 block as-is to portability_macros.h

Remove unused ZSTD_decompressSequences_t typedef

Merge pull request #4263 from pps83/dev-dyn-bmi2

Check `DYNAMIC_BMI2` instead of `DYNAMIC_BMI2 != 0`

Check `DYNAMIC_BMI2` instead of `DYNAMIC_BMI2 != 0`

`#if DYNAMIC_BMI2` is consistent with the rest of the code.

 + use spaces instead of tabs

Merge pull request #4257 from pps83/dev-x64test

Use _M_X64 only without mixing with _M_AMD64

Merge pull request #4258 from pps83/dev-ZSTD_ALIGNED

Implement ZSTD_ALIGNED for ms compiler

Merge pull request #4259 from Treata11/dev

Add support for Apple framework builds

Instructions for Apple Framework builds

Build: Revert min version to 3.10

Ignore generated framework artifacts

Signed-off-by: Treata11 <treata11@yahoo.com>

Build: Add support for Apple frameworks

Signed-off-by: Treata11 <treata11@yahoo.com>

Implement ZSTD_ALIGNED for ms compiler

Use _M_X64 only without mixing with _M_AMD64

Merge pull request #4255 from facebook/dependabot/github_actions/github/codeql-action-3.28.1

Bump github/codeql-action from 3.27.1 to 3.28.1

Merge pull request #4254 from facebook/dependabot/github_actions/cygwin/cygwin-install-action-5

Bump cygwin/cygwin-install-action from 4 to 5

Update dev-short-tests.yml

specify cygwin action version

Bump github/codeql-action from 3.27.1 to 3.28.1

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.27.1 to 3.28.1.
- [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/4f3212b61783c3c68e8309a0f18a699764811cda...b6a472f63d85b9c78a3ac5e89422239fc15e9b3c)

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

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

Bump cygwin/cygwin-install-action from 4 to 5

Bumps [cygwin/cygwin-install-action](https://github.com/cygwin/cygwin-install-action) from 4 to 5.
- [Release notes](https://github.com/cygwin/cygwin-install-action/releases)
- [Commits](https://github.com/cygwin/cygwin-install-action/compare/006ad0b0946ca6d0a3ea2d4437677fa767392401...f61179d72284ceddc397ed07ddb444d82bf9e559)

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

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

Merge pull request #4253 from facebook/BitContainerType

minor: use BitContainerType when appropriate

Merge pull request #4252 from facebook/x32_dynamicBmi2

enable DYNAMIC_BMI2 by default on x86 (32-bit mode)

minor: generalize BitContainerType

technically equivalent to `size_t`,
but it's the proper type for underlying register representation.

This makes it possible to control register type, and therefore size, independently from `size_t`,
which can be useful on systems where `size_t` is 32-bit, while the architecture supports 64-bit registers.

enable DYNAMIC_BMI2 by default on x86 (32-bit mode)

so far was only enabled for x64 (64-bit mode)

Merge pull request #4251 from facebook/x32_bmi2

BMI2 detection in 32-bit mode

no longer limit automated BMI2 detection to x64

this was previously no triggered in x86 32-bit mode,
due to a limitation in `bitstream.h`, that was fixed in #4248.

Now, `bmi2` will be automatically detected and triggered
at compilation time, if the corresponding instruction set is enabled,
even in 32-bit mode.

Also: updated library documentation, to feature STATIC_BMI2 build variable

Merge pull request #4250 from facebook/x86_32_avx2

added a CI test for x86 32-bit + avx2 combination

also add `-mbmi2` to the compilation test

minor: cosmetic, indentation

added -DSTATIC_BMI2=1 for the -mavx2 test

fixed minor static function declaration issue

in AVX2 mode only

added avx2 (x64) compilation test

added a CI test for x86 32-bit + avx2 combination

which is expected to be quite rare, but nonetheless possible.

This test is initially expected to fail, before integration of #4248 fix

Merge pull request #4248 from pps83/dev-bzhi32

Use _bzhi_u32 for 32-bit builds when building with STATIC_BMI2

Merge pull request #4246 from pps83/dev-asmx64-win

[asm] Enable x86_64 asm for windows builds

Merge pull request #4249 from facebook/zds_nonnull

minor: assert that state is not null

update sizeof check

remove unrelated changes

change to BitContainerType

handle 32bit size_t when building for x64

handle 32bit size_t when building for x64

minor: assert that state is not null

replaces #4016

Merge pull request #4247 from facebook/test_endMsg

Added a human-readable final success message at end of tests

CI: replaced shortest target by check

Use _bzhi_u32 for 32-bit builds when building with STATIC_BMI2

`_bzhi_u64` is available only for 64-bit builds, while `BIT_getLowerBits` expects `nbBits` to be less than `BIT_MASK_SIZE` (`BIT_MASK_SIZE` is 32)

[asm] Enable x86_64 asm for windows builds

Added a final success message at end of tests

so that a human reading the test log can determine everything was fine without consulting the shell error code.

Also: made `make check` slightly shorter by moving one longer test to `make test`

Merge pull request #4242 from lalakii/dev

add prerelease option

Merge pull request #4235 from facebook/bench_nbThreads

benchmark mode specifies nb of threads with -v

Merge pull request #4238 from szsam/patch-1

fix out-of-bounds array index access

Merge pull request #4232 from facebook/convertSequences_SSE

Improve speed of ZSTD_compressSequencesAndLiterals() using AVX2

restore full equation

do not solve the equation, even though some members cancel each other,
this is done for clarity,
we'll let the compiler do the resolution at compile time.

Alignment instruction ZSTD_ALIGNED() in common/compiler.h

changed code compilation test to employ ZSTD_ARCH_X86_AVX2

removed unused branch

create new compilation macro ZSTD_ARCH_X86_AVX2

minor code doc update

added compilation-time checks to ensure AVX2 code is valid

since it depends on a specific definition of ZSTD_Sequence structure.

removed unused variable

fix minor conversion warning

no need for specialized variant

the branch is not in the hot loop

removed erroneous #includes

that were automatically added by the editor without notification

AVX2 version of ZSTD_get1BlockSummary()

minor +10% speed improvement for scalar ZSTD_get1BlockSummary()