-
Notifications
You must be signed in to change notification settings - Fork 264
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
default to neon for new enough API levels #490
Comments
While NEON became the default for new devices, there are non-NEON devices that would have been upgraded to L as well. I am not sure how to determine when all the non-NEON devices stopped taking upgrades. |
Are there really any to worry about? Nearly all ARMv7 devices (with the notable exception of some early Tegras) support NEON. Even Nexus One had NEON support, and that's several major Android versions back... |
I second this. We switched to using NEON in our recent embedded v8 build and had to disable the app for a few select Tegra 2 devices manually, but these would all have been running 4.2, tops. The only possible way for a non-NEON device to run L would be to use a custom rom and a lot of black magic. Activating it for L+ seems very fine to me. |
More than a fifth of all active Android devices are running API <= 19: https://developer.android.com/about/dashboards/index.html |
@kneth true, but they are not affected as no device running a Tegra 2 has ever been updated beyond 4.1 as far as I know (LG P990 and Acer A1 ended at 4.0.4). And even the custom roms would have a very hard time pushing a 512MB Tegra 2 system above Kitkat, I'd wager it would be technically impossible or at least nobody would wait even for the first boot ;) |
Probably safe to do this with some documentation showing how to blacklist bad CPUs. |
Slight touch up because this was actually done in M, not L: https://android-review.googlesource.com/c/platform/ndk/+/714584 (thanks @rprichard for catching that). |
In L (I think it was L, I need to check the CDD) NEON became a requirement for ARM32 devices. We should default to enabling NEON whenever the target API is at least that version.
Would be best to do this in the Clang driver so we don't need to replicate this in a thousand build systems.
The text was updated successfully, but these errors were encountered: