[vlc-devel] [PATCH] contrib: force HAVE_NEON only on ARMv7

Edward Wang edward.c.wang at compdigitec.com
Thu Nov 22 01:11:25 CET 2012


Since NO_NEON is not used anymore, leaving it undefined on ARMv6 will result in HAVE_NEON showing up, resulting in crashes.

ARMv7 Android builds are now built to tegra2 with a special dynamic NEON loader for libav.
---
 contrib/bootstrap |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/contrib/bootstrap b/contrib/bootstrap
index 0567a57..b8d88c9 100755
--- a/contrib/bootstrap
+++ b/contrib/bootstrap
@@ -191,7 +191,7 @@ check_android_sdk()
 	add_make "ANDROID_NDK := ${ANDROID_NDK}"
 	[ -z "${ANDROID_ABI}" ] && echo "You must set ANDROID_ABI environment variable" && exit 1
 	add_make "ANDROID_ABI := ${ANDROID_ABI}"
-	test -z "${NO_NEON}" && add_make_enabled "HAVE_NEON"
+	[ ${ANDROID_ABI} = "armeabi-v7a" ] && add_make_enabled "HAVE_NEON"
 	[ ${ANDROID_ABI} = "armeabi-v7a" ] && add_make_enabled "HAVE_ARMV7A"
 }
 
-- 
1.7.5.4




More information about the vlc-devel mailing list