-
Notifications
You must be signed in to change notification settings - Fork 437
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
Change 'ANDROID_ABI' default value to "armeabi-v7a with NEON" #21
Comments
+1 |
Will change as soon as Google enables Neon by default in NDK build scripts. |
+1 |
Also I'd change the default SDK to 22 since it really just is the compiler and not the target api version. |
Is it really just the compiler? It uses a different set of headers, and also (for instance) turns on PIE/PIC for 16 and up, etc. The changed headers in 21 or 22 broke an app that built earlier - Google removed a function from the headers that was there before, so had to make source changes. |
Andrey,
I would recommend you change the default
ANDROID_ABI
value to"armeabi-v7a with NEON"
instead of"armeabi-v7a"
since the 1st one is much more popular in real world.The text was updated successfully, but these errors were encountered: