[vlc-commits] configure: fix check for ARM NEON support

Rémi Denis-Courmont git at videolan.org
Thu Aug 2 23:41:17 CEST 2012


vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Fri Aug  3 00:01:47 2012 +0300| [be3f36669187cd61489b8e4e187c1c5eb611c9cf] | committer: Rémi Denis-Courmont

configure: fix check for ARM NEON support

Hardware floating point must be enabled explicitly.

This also removes the NEON_CFLAGS substition variable and the check for
ARMv6, neither of which are anymore not used.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=be3f36669187cd61489b8e4e187c1c5eb611c9cf
---

 configure.ac |   17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/configure.ac b/configure.ac
index 7ac07e4..d951751 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1314,28 +1314,21 @@ AC_ARG_ENABLE(neon,
 ])
 AS_IF([test "${enable_neon}" != "no"], [
   VLC_SAVE_FLAGS
-  CFLAGS="${CFLAGS} -mfpu=neon"
-  AC_CACHE_CHECK([if $CC groks NEON inline assembly], [ac_cv_neon_inline], [
+  CFLAGS="${CFLAGS} -mfpu=neon -mhard-float"
+  AC_CACHE_CHECK([if $CCAS groks ARM NEON assembly], [ac_cv_arm_neon], [
     AC_COMPILE_IFELSE([
       AC_LANG_PROGRAM(,[[
 asm volatile("vqmovun.s64 d0, q1":::"d0");
-asm volatile("ssat r0, #1, r0":::"r0"); /* assume ARMv6 */
 ]])
     ], [
-      ac_cv_neon_inline="-mfpu=neon"
+      ac_cv_arm_neon="yes"
     ], [
-      ac_cv_neon_inline="no"
+      ac_cv_arm_neon="no"
     ])
   ])
   VLC_RESTORE_FLAGS
-  AS_IF([test "$ac_cv_neon_inline" != "no"], [
-    NEON_CFLAGS="$ac_cv_neon_inline"
-  ])
-], [
-  ac_cv_neon_inline="no"
 ])
-AC_SUBST(NEON_CFLAGS)
-AM_CONDITIONAL(HAVE_NEON, [test "${ac_cv_neon_inline}" != "no"])
+AM_CONDITIONAL(HAVE_NEON, [test "${ac_cv_arm_neon}" = "yes"])
 
 
 AC_ARG_ENABLE(altivec,



More information about the vlc-commits mailing list