diff --git a/Android.mk b/Android.mk index 6ae69f19..ff01894e 100644 --- a/Android.mk +++ b/Android.mk @@ -25,6 +25,7 @@ ifneq ($(findstring armeabi-v7a, $(TARGET_ARCH_ABI)),) # specifically. NEON := c.neon USE_CPUFEATURES := yes + WEBP_CFLAGS += -DHAVE_CPU_FEATURES_H else NEON := c endif diff --git a/CMakeLists.txt b/CMakeLists.txt index 07ccb0dc..d08f0b8b 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -54,6 +54,7 @@ if(ANDROID) set(WEBP_DEP_INCLUDE_DIRS ${WEBP_DEP_INCLUDE_DIRS} ${ANDROID_NDK}/sources/android/cpufeatures ) + add_definitions(-DHAVE_CPU_FEATURES_H) endif() ################################################################################ diff --git a/build.gradle b/build.gradle index 21c5c440..485d637d 100644 --- a/build.gradle +++ b/build.gradle @@ -84,6 +84,7 @@ model { // Check for NEON usage. if (getTargetPlatform() == "arm" || getTargetPlatform() == "arm64") { NEON = "c.neon" + cCompiler.define "HAVE_CPU_FEATURES_H" } else { NEON = "c" } diff --git a/configure.ac b/configure.ac index c70c9f6f..2949b19f 100644 --- a/configure.ac +++ b/configure.ac @@ -243,9 +243,13 @@ AS_IF([test "x$enable_neon" != "xno"], [ NEON_FLAGS=""], [AC_DEFINE(WEBP_HAVE_NEON_RTCD, [1], [Set to 1 if runtime detection of NEON is enabled])])]) - ;; - esac - AC_SUBST([NEON_FLAGS])]) + + case "$host_os" in + *android*) AC_CHECK_HEADERS([cpu-features.h]) ;; + esac + ;; + esac + AC_SUBST([NEON_FLAGS])]) dnl === CLEAR_LIBVARS([var_pfx]) dnl === Clears _{INCLUDES,LIBS}. diff --git a/src/dsp/dsp.h b/src/dsp/dsp.h index e2bebf97..65140e3e 100644 --- a/src/dsp/dsp.h +++ b/src/dsp/dsp.h @@ -72,18 +72,20 @@ extern "C" { #define WEBP_USE_AVX2 #endif -#if defined(__ANDROID__) && defined(__ARM_ARCH_7A__) -#define WEBP_ANDROID_NEON // Android targets that might support NEON -#endif - // The intrinsics currently cause compiler errors with arm-nacl-gcc and the // inline assembly would need to be modified for use with Native Client. -#if (defined(__ARM_NEON__) || defined(WEBP_ANDROID_NEON) || \ +#if (defined(__ARM_NEON__) || \ defined(__aarch64__) || defined(WEBP_HAVE_NEON)) && \ !defined(__native_client__) #define WEBP_USE_NEON #endif +#if !defined(WEBP_USE_NEON) && defined(__ANDROID__) && \ + defined(__ARM_ARCH_7A__) && defined(HAVE_CPU_FEATURES_H) +#define WEBP_ANDROID_NEON // Android targets that may have NEON +#define WEBP_USE_NEON +#endif + #if defined(_MSC_VER) && _MSC_VER >= 1700 && defined(_M_ARM) #define WEBP_USE_NEON #define WEBP_USE_INTRINSICS