Skip to content
This repository has been archived by the owner on Apr 2, 2021. It is now read-only.

Use getauxval to check for NEON on Linux. #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

QuLogic
Copy link
Contributor

@QuLogic QuLogic commented Jul 21, 2018

Based on this SO answer and this ARM blog post. Seems to work pretty well on an armv7hl machine.

@Auburn
Copy link
Owner

Auburn commented Jul 24, 2018

Have you tested that this still compiles for Android

@QuLogic
Copy link
Contributor Author

QuLogic commented Jul 24, 2018

TBH, I'm not sure how to set that up. Does it define one of the linux constants also? I guess checking for __ANDROID__ might help if needed then.

@QuLogic
Copy link
Contributor Author

QuLogic commented Jul 24, 2018

Based on this SO answer, it does appear that __linux__ is defined on Android, so I modified this to check that __ANDROID__ is also not defined.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants