From 363fd52b41808e4d5ab1afbd248ffd79e6c95990 Mon Sep 17 00:00:00 2001 From: "Ryan C. Gordon" Date: Fri, 26 Jun 2020 23:30:59 -0400 Subject: [PATCH] configure/cmake: Disable ARM SIMD and NEON by default. It's buggy at the moment. --- CMakeLists.txt | 4 ++-- configure | 18 +++++++++--------- configure.ac | 2 +- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 98cd98326..b703c34d2 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -328,8 +328,8 @@ set_option(SSE "Use SSE assembly routines" ${OPT_DEF_ASM}) set_option(SSE2 "Use SSE2 assembly routines" ${OPT_DEF_SSEMATH}) set_option(SSE3 "Use SSE3 assembly routines" ${OPT_DEF_SSEMATH}) set_option(ALTIVEC "Use Altivec assembly routines" ${OPT_DEF_ASM}) -set_option(ARMSIMD "use SIMD assembly blitters on ARM" ON) -set_option(ARMNEON "use NEON assembly blitters on ARM" ON) +set_option(ARMSIMD "use SIMD assembly blitters on ARM" OFF) +set_option(ARMNEON "use NEON assembly blitters on ARM" OFF) set_option(DISKAUDIO "Support the disk writer audio driver" ON) set_option(DUMMYAUDIO "Support the dummy audio driver" ON) set_option(VIDEO_DIRECTFB "Use DirectFB video driver" OFF) diff --git a/configure b/configure index 509c40311..6d6ab4018 100755 --- a/configure +++ b/configure @@ -852,8 +852,8 @@ enable_diskaudio enable_dummyaudio enable_libsamplerate enable_libsamplerate_shared -enable_arm_simd enable_arm_neon +enable_arm_simd enable_video_wayland enable_video_wayland_qt_touch enable_wayland_shared @@ -1635,7 +1635,6 @@ Optional Features: --enable-libsamplerate-shared dynamically load libsamplerate [[default=yes]] --enable-arm-simd use SIMD assembly blitters on ARM [[default=yes]] - --enable-arm-neon use NEON assembly blitters on ARM [[default=no]] --enable-video-wayland use Wayland video driver [[default=yes]] --enable-video-wayland-qt-touch QtWayland server support for Wayland video driver @@ -19473,9 +19472,7 @@ CheckARM() if test "${enable_arm_simd+set}" = set; then : enableval=$enable_arm_simd; enable_arm_simd=$enableval else - enable_arm_simd=yes -fi - + enable_arm_simd=no if test x$enable_video = xyes -a x$enable_assembly = xyes -a x$enable_arm_simd = xyes; then save_CFLAGS="$CFLAGS" have_arm_simd=no @@ -19494,7 +19491,7 @@ $as_echo_n "checking for ARM SIMD... " >&6; } #ifndef __ARM_EABI__ #error EABI is required (to be sure that calling conventions are compatible) #endif - pld [r0] + pld r0 uqadd8 r0, r0, r0 _ACEOF @@ -19519,7 +19516,8 @@ $as_echo "#define SDL_ARM_SIMD_BLITTERS 1" >>confdefs.h CheckNEON() { - # Check whether --enable-arm-neon was given. + --enable-arm-neon use NEON assembly blitters on ARM [[default=no]] +# Check whether --enable-arm-neon was given. if test "${enable_arm_neon+set}" = set; then : enableval=$enable_arm_neon; enable_arm_neon=$enableval else @@ -19546,7 +19544,7 @@ $as_echo_n "checking for ARM NEON... " >&6; } #ifndef __ARM_EABI__ #error EABI is required (to be sure that calling conventions are compatible) #endif - pld [r0] + pld r0 vmovn.u16 d0, q0 _ACEOF @@ -19759,7 +19757,9 @@ $as_echo "$have_gcc_Wall" >&6; } $as_echo_n "checking for necessary GCC -Wno-multichar option... " >&6; } need_gcc_Wno_multichar=no case "$host" in - *-*-haiku*) + *-*-haiku* +fi + need_gcc_Wno_multichar=yes ;; esac diff --git a/configure.ac b/configure.ac index 474ed03d1..c957d3fc7 100644 --- a/configure.ac +++ b/configure.ac @@ -1320,7 +1320,7 @@ CheckARM() { AC_ARG_ENABLE(arm-simd, AC_HELP_STRING([--enable-arm-simd], [use SIMD assembly blitters on ARM [[default=yes]]]), - enable_arm_simd=$enableval, enable_arm_simd=yes) + enable_arm_simd=$enableval, enable_arm_simd=no if test x$enable_video = xyes -a x$enable_assembly = xyes -a x$enable_arm_simd = xyes; then save_CFLAGS="$CFLAGS" have_arm_simd=no