search

jaqquery / libyuv

Automatically exported from code.google.com/p/libyuv
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

I422ToARGBRow_SSSE3 used; expected I422ToARGBRow_AVX2 #557

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
I422ToARGBRow_SSSE3 is not expected.  Should call I422ToARGBRow_AVX2

Run conversion tests

LIBYUV_FLAGS=-1 LIBYUV_WIDTH=1280 LIBYUV_HEIGHT=720 LIBYUV_REPEAT=1000 perf 
record out/Release/libyuv_unittest --gtest_filter=*ConvertTest*

perf report

Original issue reported on code.google.com by fbarch...@google.com on 26 Jan 2016 at 1:47

GoogleCodeExporter commented 8 years ago

Original comment by fbarch...@google.com on 26 Jan 2016 at 1:49

GoogleCodeExporter commented 8 years ago 
The following revision refers to this bug:
  https://chromium.googlesource.com/libyuv/libyuv.git/+/c39509c8e5c529fa1dd444dc116de877cf48c518

commit c39509c8e5c529fa1dd444dc116de877cf48c518
Author: Frank Barchard <fbarchard@google.com>
Date: Wed Feb 10 01:14:29 2016

add avx2 wrappers for functions that can call I422ToARGBRow_AVX2

R=harryjin@google.com
BUG=libyuv:557

Review URL: https://codereview.chromium.org/1687713002 .

[modify] 
http://crrev.com/c39509c8e5c529fa1dd444dc116de877cf48c518/README.chromium
[modify] 
http://crrev.com/c39509c8e5c529fa1dd444dc116de877cf48c518/include/libyuv/row.h
[modify] 
http://crrev.com/c39509c8e5c529fa1dd444dc116de877cf48c518/include/libyuv/version
.h
[modify] 
http://crrev.com/c39509c8e5c529fa1dd444dc116de877cf48c518/source/row_common.cc

Original comment by bugdroid1@chromium.org on 10 Feb 2016 at 1:15

GoogleCodeExporter commented 8 years ago

Original comment by fbarch...@google.com on 10 Feb 2016 at 1:15

GoogleCodeExporter commented 8 years ago

Original comment by fbarch...@google.com on 10 Feb 2016 at 1:31

GoogleCodeExporter commented 8 years ago 
Was I420ToARGB4444_Opt (626 ms)
Now I420ToARGB4444_Opt (560 ms)

Original comment by fbarch...@google.com on 10 Feb 2016 at 11:39