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

[arm64] implement with native NEON instructions #57

Closed
wants to merge 1 commit into from

Conversation

sebpop
Copy link
Contributor

@sebpop sebpop commented Jun 28, 2023

This patch removes the dependence on sse2neon by rewriting SSE2 and AVX code with native NEON instructions.

This patch removes the dependence on sse2neon by rewriting SSE2 and AVX code
with native NEON instructions.
@sebpop
Copy link
Contributor Author

sebpop commented Aug 14, 2023

Ping patch.

@alexeyserbin
Copy link
Contributor

Hi @sebpop,

Thank you very much for the patch!

The Kudu project doesn't merge new code through GitHub PRs; we use the gerrit tool to do code review and submission. Could you please follow the directions here to adapt this PR into a gerrit change?

BTW, both the Apache Impala and Apache Kudu projects strive to share the common code, so please be aware of the following activity in the related area (it seems you've commented on [IMPALA-11694|https://issues.apache.org/jira/browse/IMPALA-11694], but just in case :) )

@sebpop
Copy link
Contributor Author

sebpop commented Aug 17, 2023

Thanks Alexey, I pushed the patch to https://gerrit.cloudera.org/#/c/20374/

@sebpop sebpop closed this Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants