-
Notifications
You must be signed in to change notification settings - Fork 6
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 support for 32bit arm #26
Comments
@igor-microblink Does this change fix this? If so, I will merge this into |
Thanks. Proposed change is fix for compilation error on armebi v7a. __ARM_FEATURE_SVE is not defined for armeabi-v7a. Is it possible to add impl which utilizes arm neon for 32bit ARMs? AFAIK ARM NEON is supported from armeabi-v7. |
Yes, cause Arm SVE is an extension for armv8 and one of basic instructions in armv9.
Hmm, currently qoixx doesn't support armv7 as SIMD-available environment, because I don't have any development environment of armv7. It is possible technically cause NEON is an extension for armv7, but qoixx basically assumes 64-bit architecture (e.g. fast hash calculation using 64bit integer arithmetic; this might be not fast on 32bit environment), and currently I don't think I'd support it aggressively. Of course PR for supporting armv7 with NEON would be welcome, I will review it. |
At the moment 32bit android arm is not supported, so the fallback without SIMD is required.
The text was updated successfully, but these errors were encountered: