[PATCH] D58153: [Driver] Default all Android ARM targets to NEON.

2019-02-15 Thread Dan Albert via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL354166: [Driver] Default all Android ARM targets to NEON. (authored by danalbert, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://

[PATCH] D58153: [Driver] Default all Android ARM targets to NEON.

2019-02-13 Thread Dan Albert via Phabricator via cfe-commits
danalbert added a comment. In D58153#1395601 , @efriedma wrote: > The official documentation still says "Your app must perform runtime > detection to confirm that NEON-capable machine code can be run on the target > device" > (https://developer.android.

[PATCH] D58153: [Driver] Default all Android ARM targets to NEON.

2019-02-12 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added a comment. The official documentation still says "Your app must perform runtime detection to confirm that NEON-capable machine code can be run on the target device" (https://developer.android.com/ndk/guides/cpu-arm-neon#runtime_detection). Is that wrong? Repository: rG LLVM

[PATCH] D58153: [Driver] Default all Android ARM targets to NEON.

2019-02-12 Thread Dan Albert via Phabricator via cfe-commits
danalbert created this revision. danalbert added reviewers: srhines, pirama, kristof.beyls. danalbert added a project: clang. Herald added a subscriber: javed.absar. There are an insignificant number of ARM Android devices that don't support NEON. Default to using NEON since that will improve perf