dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2024-12-27 06:08:21 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

add LOCAL_CLANG_PREREQ and avoid WORK_AROUND_GCC w/3.8+

Browse Source 
this results in a 15-20% speedup for lossy decoding on a N5/S6/CM1

BUG=webp:339

Change-Id: Icdeb84c3e0b8908147ac276b4d8f76c3d565b735
This commit is contained in:
James Zern 2017-09-19 20:15:31 -07:00
parent 01c426f1e7
commit f78da3dea6
2 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/dsp/dsp.h
View File

@ -38,6 +38,15 @@ extern "C" {
# define LOCAL_GCC_PREREQ(maj, min) 0 # define LOCAL_GCC_PREREQ(maj, min) 0
#endif #endif
#if defined(__clang__)
# define LOCAL_CLANG_VERSION ((__clang_major__ << 8) | __clang_minor__)
# define LOCAL_CLANG_PREREQ(maj, min) \
(LOCAL_CLANG_VERSION >= (((maj) << 8) | (min)))
#else
# define LOCAL_CLANG_VERSION 0
# define LOCAL_CLANG_PREREQ(maj, min) 0
#endif
#ifndef __has_builtin #ifndef __has_builtin
# define __has_builtin(x) 0 # define __has_builtin(x) 0
#endif #endif

6
src/dsp/neon.h
View File

@ -17,9 +17,9 @@
#include "./dsp.h" #include "./dsp.h"
// Right now, some intrinsics functions seem slower, so we disable them // Right now, some intrinsics functions seem slower, so we disable them
// everywhere except newer gcc or aarch64 where the inline assembly is // everywhere except newer clang/gcc or aarch64 where the inline assembly is
// incompatible. // incompatible.
#if LOCAL_GCC_PREREQ(4,9) || defined(__aarch64__) #if LOCAL_CLANG_PREREQ(3,8) || LOCAL_GCC_PREREQ(4,9) || defined(__aarch64__)
#define WEBP_USE_INTRINSICS // use intrinsics when possible #define WEBP_USE_INTRINSICS // use intrinsics when possible
#endif #endif
@ -44,7 +44,7 @@
// if using intrinsics, this flag avoids some functions that make gcc-4.6.3 // if using intrinsics, this flag avoids some functions that make gcc-4.6.3
// crash ("internal compiler error: in immed_double_const, at emit-rtl."). // crash ("internal compiler error: in immed_double_const, at emit-rtl.").
// (probably similar to gcc.gnu.org/bugzilla/show_bug.cgi?id=48183) // (probably similar to gcc.gnu.org/bugzilla/show_bug.cgi?id=48183)
#if !(LOCAL_GCC_PREREQ(4,8) || defined(__aarch64__)) #if !(LOCAL_CLANG_PREREQ(3,8) || LOCAL_GCC_PREREQ(4,8) || defined(__aarch64__))
#define WORK_AROUND_GCC #define WORK_AROUND_GCC
#endif #endif