From 586eda373d6a35f82ec185a9097b5cf3e08a10a9 Mon Sep 17 00:00:00 2001 From: James Zern Date: Mon, 26 Jun 2017 15:36:57 -0700 Subject: [PATCH] Revert "dsp/cpu: correct wasmCPUInfo placement" This reverts commit 4026e34e3f2edaf4b61824e06994527e45d2b6d8. fails to build; not all x86 paths were protected Change-Id: I27bcc83e74440205bfd99c31c6da319c205ef145 --- src/dsp/cpu.c | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/dsp/cpu.c b/src/dsp/cpu.c index e5b5d776..11b780d0 100644 --- a/src/dsp/cpu.c +++ b/src/dsp/cpu.c @@ -22,17 +22,12 @@ #include #endif -// WASM needs to precede platform specific architecture checks as the defines -// will still be present when building this target. -#if defined(WEBP_USE_WASM) -static int wasmCPUInfo(CPUFeature feature) { - if (feature != kWASM) return 0; - return 1; -} -VP8CPUInfo VP8GetCPUInfo = wasmCPUInfo; +//------------------------------------------------------------------------------ +// SSE2 detection. +// // apple/darwin gcc-4.0.1 defines __PIC__, but not __pic__ with -fPIC. -#elif (defined(__pic__) || defined(__PIC__)) && defined(__i386__) +#if (defined(__pic__) || defined(__PIC__)) && defined(__i386__) static WEBP_INLINE void GetCPUInfo(int cpu_info[4], int info_type) { __asm__ volatile ( "mov %%ebx, %%edi\n" @@ -222,6 +217,12 @@ static int mipsCPUInfo(CPUFeature feature) { } VP8CPUInfo VP8GetCPUInfo = mipsCPUInfo; +#elif defined(WEBP_USE_WASM) +static int wasmCPUInfo(CPUFeature feature) { + if (feature != kWASM) return 0; + return 1; +} +VP8CPUInfo VP8GetCPUInfo = wasmCPUInfo; #else VP8CPUInfo VP8GetCPUInfo = NULL; #endif