Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated API for static allocation #65

Merged
merged 14 commits into from
Jun 2, 2016
Merged

updated API for static allocation #65

merged 14 commits into from
Jun 2, 2016

Conversation

Cyan4973
Copy link
Owner

@Cyan4973 Cyan4973 commented Jun 1, 2016

No description provided.

cota and others added 14 commits March 24, 2016 18:08
Signed-off-by: Emilio G. Cota <cota@braap.org>
This is particularly useful when inlining the library.

Signed-off-by: Emilio G. Cota <cota@braap.org>
The name change also includes inverting the logic, which makes the evaluation
logic clearer: if FORCE_ALIGN_CHECK is set, we perform the check;
if not, we don't.

While at it, allow calling code to set the flag. The use case for this
is the following: calling code might have a guarantee that the input data
is aligned. Thus, for maximum performance, it would set:

  XXH_FORCE_ALIGN_CHECK to 0, to skip the alignment check
  XXH_FORCE_MEMORY_ACCESS 2, to copy data in chunks of 32 or 64 bits

Without this change, there is no way to skip the alignment check on
certain architectures.

Signed-off-by: Emilio G. Cota <cota@braap.org>
[v2] changes to increase xxhash' configurability when inlining it
introduce XXH64_mergeRound() to simplify code.
change library version to v0.5.2
expose definition of `XXH32_state_t` and `XXH64_state_t` in the new section
@Cyan4973 Cyan4973 merged commit 9d791cf into master Jun 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants