search

Narendrabrsoft / libyuv

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

ARGBToUVJRow_SSSE3 used but expected ARGBToUVJRow_AVX2 #553

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
ARGBToUVJRow_SSSE3  used but expected ARGBToUVJRow_AVX2
port RGBToUVRow_AVX2 to J version RGBToUVJRow_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:43

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/+/22e062a4485d4d69736cf900ae1de32f4a7058a1

commit 22e062a4485d4d69736cf900ae1de32f4a7058a1
Author: Frank Barchard <fbarchard@google.com>
Date: Thu Feb 18 07:16:39 2016

Port ARGBToJ420 to AVX2

ARGBToJ420 had an SSSE3 version, but not AVX2.
ARGBToI420 had an AVX2, so adapt that code to J420.

TBR=harryjin@google.com
BUG=libyuv:553

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

[modify] 
http://crrev.com/22e062a4485d4d69736cf900ae1de32f4a7058a1/source/row_gcc.cc
[modify] 
http://crrev.com/22e062a4485d4d69736cf900ae1de32f4a7058a1/source/row_win.cc

Original comment by bugdroid1@chromium.org on 18 Feb 2016 at 7:17

GoogleCodeExporter commented 8 years ago

Original comment by fbarch...@google.com on 18 Feb 2016 at 8:25