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

Add optimized crc32 for Power 8+ processors #857

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mmatti-sw
Copy link

This is a pull request to include all Power8 optimisations rebased to v1.3
Issue #847 and the reference PR's are #750 ( Power8 optimisations rebased to v1.2.13) and #478.

Manjunath S Matti added 3 commits September 15, 2023 11:38
Optimized functions for Power will make use of GNU indirect functions,
an extension to support different implementations of the same function,
which can be selected during runtime. This will be used to provide
optimized functions for different processor versions.

Since this is a GNU extension, we placed the definition of the Z_IFUNC
macro under `contrib/gcc`. This can be reused by other archs as well.

Author: Matheus Castanho <[email protected]>
Author: Rogerio Alves <[email protected]>
Signed-off-by: Manjunath Matti <[email protected]>
This commit adds an optimized version for the crc32 function based
on crc32-vpmsum from https://github.com/antonblanchard/crc32-vpmsum/

This is the C implementation created by Rogerio Alves
<[email protected]>

It makes use of vector instructions to speed up CRC32 algorithm.

Author: Rogerio Alves <[email protected]>
Signed-off-by: Manjunath Matti <[email protected]>
Clang 7 changed the behavior of vec_xxpermdi in order to match GCC's
behavior.  After this change, code that used to work on Clang 6 stopped
to work on Clang >= 7.

Tested on Clang 6, 7, 8 and 9.

Reference: https://bugs.llvm.org/show_bug.cgi?id=38192

Signed-off-by: Tulio Magno Quites Machado Filho <[email protected]>
Signed-off-by: Manjunath Matti <[email protected]>
@Neustradamus
Copy link

@madler: Can you look this PR?

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