From: mayeut Date: Mon, 9 Apr 2018 22:02:57 +0000 (+0200) Subject: jsimd_can_encode_mcu_AC_*(): Remove useless checks X-Git-Tag: 2.0.0~25 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=269e84c971067f6228be9c720b405efa0fdf3d6c;p=libjpeg-turbo jsimd_can_encode_mcu_AC_*(): Remove useless checks These were necessary for the first iteration of the feature (see #46), which provided a different C front end for the SIMD version of the function. The final version of the feature uses a common C front end for both SIMD and non-SIMD implementations, so these checks are no longer necessary. Closes #231 --- diff --git a/simd/i386/jsimd.c b/simd/i386/jsimd.c index 0b5a410..563949a 100644 --- a/simd/i386/jsimd.c +++ b/simd/i386/jsimd.c @@ -1210,15 +1210,10 @@ jsimd_can_encode_mcu_AC_first_prepare(void) return 0; if (SIZEOF_SIZE_T != 4) return 0; - if (!(simd_support & JSIMD_SSE2)) - return 0; -#if defined(HAVE_BUILTIN_CTZL) - return 1; -#elif defined(HAVE_BITSCANFORWARD) - return 1; -#else + if (simd_support & JSIMD_SSE2) + return 1; + return 0; -#endif } GLOBAL(void) @@ -1241,15 +1236,10 @@ jsimd_can_encode_mcu_AC_refine_prepare(void) return 0; if (SIZEOF_SIZE_T != 4) return 0; - if (!(simd_support & JSIMD_SSE2)) - return 0; -#if defined(HAVE_BUILTIN_CTZL) - return 1; -#elif defined(HAVE_BITSCANFORWARD) - return 1; -#else + if (simd_support & JSIMD_SSE2) + return 1; + return 0; -#endif } GLOBAL(int) diff --git a/simd/x86_64/jsimd.c b/simd/x86_64/jsimd.c index 7927875..1e5698b 100644 --- a/simd/x86_64/jsimd.c +++ b/simd/x86_64/jsimd.c @@ -1033,15 +1033,10 @@ jsimd_can_encode_mcu_AC_first_prepare(void) return 0; if (SIZEOF_SIZE_T != 8) return 0; - if (!(simd_support & JSIMD_SSE2)) - return 0; -#if defined(HAVE_BUILTIN_CTZL) - return 1; -#elif defined(HAVE_BITSCANFORWARD64) - return 1; -#else + if (simd_support & JSIMD_SSE2) + return 1; + return 0; -#endif } GLOBAL(void) @@ -1064,15 +1059,10 @@ jsimd_can_encode_mcu_AC_refine_prepare(void) return 0; if (SIZEOF_SIZE_T != 8) return 0; - if (!(simd_support & JSIMD_SSE2)) - return 0; -#if defined(HAVE_BUILTIN_CTZL) - return 1; -#elif defined(HAVE_BITSCANFORWARD64) - return 1; -#else + if (simd_support & JSIMD_SSE2) + return 1; + return 0; -#endif } GLOBAL(int)