From 4026e34e3f2edaf4b61824e06994527e45d2b6d8 Mon Sep 17 00:00:00 2001 From: James Zern Date: Mon, 26 Jun 2017 11:32:20 -0700 Subject: [PATCH] dsp/cpu: correct wasmCPUInfo placement WASM needs to precede platform specific architecture checks as the defines will still be present when building this target. Change-Id: If25467ea286e582b928e26e716e41aff72898c50 --- src/dsp/cpu.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/src/dsp/cpu.c b/src/dsp/cpu.c index 11b780d0..e5b5d776 100644 --- a/src/dsp/cpu.c +++ b/src/dsp/cpu.c @@ -22,12 +22,17 @@ #include #endif -//------------------------------------------------------------------------------ -// SSE2 detection. -// +// 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; // apple/darwin gcc-4.0.1 defines __PIC__, but not __pic__ with -fPIC. -#if (defined(__pic__) || defined(__PIC__)) && defined(__i386__) +#elif (defined(__pic__) || defined(__PIC__)) && defined(__i386__) static WEBP_INLINE void GetCPUInfo(int cpu_info[4], int info_type) { __asm__ volatile ( "mov %%ebx, %%edi\n" @@ -217,12 +222,6 @@ 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