diff options
author | Johann <johannkoenig@google.com> | 2017-02-16 17:57:44 -0800 |
---|---|---|
committer | Johann Koenig <johannkoenig@google.com> | 2017-02-24 05:25:26 +0000 |
commit | 904b957ae965bd3d67f15a75cd9db7954f810d33 (patch) | |
tree | 3d3db25d9600a19e32d105bf9a149429478f3782 /vp9/vp9cx.mk | |
parent | aa911e8b41733a950dfe96866dbf8118940bc996 (diff) | |
download | libvpx-904b957ae965bd3d67f15a75cd9db7954f810d33.tar libvpx-904b957ae965bd3d67f15a75cd9db7954f810d33.tar.gz libvpx-904b957ae965bd3d67f15a75cd9db7954f810d33.tar.bz2 libvpx-904b957ae965bd3d67f15a75cd9db7954f810d33.zip |
consolidate block_error functions
vp9_highbd_block_error_8bit_c was a very simple wrapper around
vp9_block_error_c. The SSE2 implemention was practically identical to
the non-HBD one. It was missing some minor improvements which only
went into the original version.
In quick speed tests, the AVX implementation showed minimal
improvement over SSE2 when it does not detect overflow. However, when
overflow is detected the function is run a second time. The
OperationCheck test seems to trigger this case and reverses any
speed benefits by running ~60% slower. AVX2 on the other hand is
always 30-40% faster.
Change-Id: I9fcb9afbcb560f234c7ae1b13ddb69eca3988ba1
Diffstat (limited to 'vp9/vp9cx.mk')
-rw-r--r-- | vp9/vp9cx.mk | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/vp9/vp9cx.mk b/vp9/vp9cx.mk index a54e99e2c..e73535543 100644 --- a/vp9/vp9cx.mk +++ b/vp9/vp9cx.mk @@ -108,10 +108,6 @@ endif VP9_CX_SRCS-$(HAVE_SSE2) += encoder/x86/vp9_dct_sse2.asm VP9_CX_SRCS-$(HAVE_SSE2) += encoder/x86/vp9_error_sse2.asm -ifeq ($(CONFIG_VP9_HIGHBITDEPTH),yes) -VP9_CX_SRCS-$(HAVE_SSE2) += encoder/x86/vp9_highbd_error_sse2.asm -VP9_CX_SRCS-$(HAVE_AVX) += encoder/x86/vp9_highbd_error_avx.asm -endif ifeq ($(ARCH_X86_64),yes) VP9_CX_SRCS-$(HAVE_SSSE3) += encoder/x86/vp9_quantize_ssse3_x86_64.asm |