v0.8.2 #873
Merged
v0.8.2 #873
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- XXH_SIZE_OPT is a value from 0-2 which indicates how much xxHash
should care about code size, default 1 for -Os/-Oz, default 0
otherwise
- XXH_NO_STREAM disables the streaming API.
- These two interact: if XXH_SIZE_OPT == 2 and XXH_NO_STREAM is not
defined, XXH32 and XXH64 use the streaming API for single shot
- TODO: apply this to XXH3 as well
since some versions are buggy. Use `_Static_assert` instead, which is part of the C11 language.
do no longer depend on `<assert.h>` for XXH_STATIC_ASSERT
It seems that using the streaming API is not beneficial on GCC so it isn't worth the complexity.
Uses `#pragma push_macro`/`#pragma pop_macro` instead of assuming that `__APPLE_ALTIVEC__` can be defined safely. Safer and less hacky fix for #668.
Properly fix altivec namespace collisions
Introduce XXH_SIZE_OPT and XXH_NO_STREAM
Remove duplicated definition of XXH3_128bits
Removed windows-2016 from ci.yml
tipi.build instructions
For details, see issue #695. If filename contains special character (\n, 0x0a, LF), - Put '\' (0x5c) at the beginning of the line. - Escape special character by '\'.
Build fix for Haiku
Use inline assembler for Power/IBM XL Compiler
test filename-escape
Generate with the following commands: rm cli/xxhsum.1 make man man cli/xxhsum.1
Add --ignore-missing
Add missing doxygen style comments to XXH64 family
Add missing doxygen style comments to XXH3 family
Add --ignore-missing and --binary to --help and xxhsum.1
as suggested by @t-mat, warning about the upcoming deprecation of this feature.
added a compilation warning message for XXH_OLD_NAMES
clang-cl warns the first argument of __cpuidex() because it's int[]. void __cpuidex( int cpuInfo[4], int function_id, int subfunction_id ); https://learn.microsoft.com/en-us/cpp/intrinsics/cpuid-cpuidex
fix: __cpuidex pointer type warning
clang-cl defines both __clang__ and _MSC_VER. Also it seems clang-cl needs extra headers to compile AVX2/512F intrinsic.
- Actual platform name of x64 for MSVC/clang-cl is "AMD64" - When we compile xxhsum with dispatch, we need xxh_x86dispatch.c
Fix clang-cl dispatch
test that #763 is fixed and remains fixed
add a test combining Windows + Visual + Clang-Cl + DISPATCH=1
fix: enable clang-cl test for VC++2022
Intel compiler expects "unsigned char" as an argument of __builtin_assume().
fix: Intel compiler's warning for XXH_ASSERT()
"Doxyfile" is configured to generate public document. Differences are: - INTERNAL_DOCS - EXTRACT_STATIC - EXTRACT_PRIVATE
Hide internals to simplify public document.
fix: hide internal macros from doxygen
hide x86 dispatcher internals
and re-employ original nest order to reproduce same performance as v0.8.1.
CI: Add gcc-13 and clang-{13,14,15}
revert XXH3's 17to128 variant to single accumulator
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
v0.8.2