Commits · 2cc1a2b233488297d2590f744fdee8ff604f5c11 · submodule / libyuv

30 Sep, 2015 1 commit

Remove sse2 functions that also have ssse3 · 2cc1a2b2

Frank Barchard authored Sep 30, 2015

ARGBBlendRow_SSE2, ARGBAttenuateRow_SSE2, and MirrorRow_SSE2
Since vast majority of CPUs have SSSE3 now, removing the SSE2
improves the performance of CPU dispatching.

R=harryjin@google.com
BUG=none

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

2cc1a2b2

25 Sep, 2015 5 commits

Width use memory instead of register for 32 bit fpic. · d039ad6e

Frank Barchard authored Sep 25, 2015

Code runs out of registers on 32 bit fpic builts.

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

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

d039ad6e

win64 version of I422AlphaToARGB. · febc26a2

Frank Barchard authored Sep 25, 2015

Was
I420AlphaToARGB_Premult (8861 ms)
I420AlphaToARGB_Opt (7119 ms)
Now
I420AlphaToABGR_Premult (2840 ms)
I420AlphaToARGB_Opt (484 ms)

C function switched to 1 step.
Was
I420AlphaToARGB_Premult (8862 ms)
I420AlphaToABGR_Opt (6718 ms)

Now
I420AlphaToARGB_Premult (8706 ms)
I420AlphaToARGB_Opt (6541 ms)

R=harryjin@google.com
BUG=libyuv:496, libyuv:473

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

febc26a2

AVX2 1 step I422AlphaToARGB for gcc and win. · 9a0e12f5

Frank Barchard authored Sep 25, 2015

C     I420AlphaToARGB_Opt (5169 ms)
SSSE3 I420AlphaToARGB_Opt (432 ms)
AVX2  I420AlphaToARGB_Opt (358 ms)

and with premultiplication as 2 step process:
I420AlphaToARGB_Premult (7029 ms)
I420AlphaToARGB_Premult (757 ms)
I420AlphaToARGB_Premult (508 ms)

R=harryjin@google.com
BUG=libyuv:496,libyuv:473

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

9a0e12f5

I420Alpha row function in 1 pass. · e365cdde

Frank Barchard authored Sep 25, 2015

API change - I420AlphaToARGB takes flag indicating if RGB should be
premultiplied by alpha.

This version implements an efficient SSSE3 version for Windows.
C version done in 2 steps.

Was
libyuvTest.I420AlphaToARGB_Any (1136 ms)
libyuvTest.I420AlphaToARGB_Unaligned (1210 ms)
libyuvTest.I420AlphaToARGB_Invert (966 ms)
libyuvTest.I420AlphaToARGB_Opt (1031 ms)
libyuvTest.I420AlphaToABGR_Any (1020 ms)
libyuvTest.I420AlphaToABGR_Unaligned (1359 ms)
libyuvTest.I420AlphaToABGR_Invert (1082 ms)
libyuvTest.I420AlphaToABGR_Opt (986 ms)

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

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

e365cdde

switch from ebp to ebx. · d4594bee

Frank Barchard authored Sep 25, 2015

ebx encodes more efficiently (1 byte less) for most address modes, than ebp.
previously it was used for 411 format, but the reader uses pinsrw now avoiding
gpr register.

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

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

d4594bee

24 Sep, 2015 1 commit

Fix nv12 64 bit gcc increment. · 8fb2048e

Frank Barchard authored Sep 24, 2015

Should be 16 bytes, but was 0x16 causing memory corruption.

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

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

8fb2048e

23 Sep, 2015 2 commits

NV12ToARGB_AVX2 ported to gcc · accc04e6

Frank Barchard authored Sep 23, 2015

TBR=harryjin@google.com
BUG=none

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

accc04e6

YUY2ToARGB avx2 in 1 step conversion. · 000cf89c

Frank Barchard authored Sep 23, 2015

Includes UYVYToARGB ssse3 fix.

Was
YUY2ToARGB_Opt (433 ms)
69.79%  libyuv_unittest  libyuv_unittest      [.] I422ToARGBRow_AVX2
20.73%  libyuv_unittest  libyuv_unittest      [.] YUY2ToUV422Row_AVX2
 6.04%  libyuv_unittest  libyuv_unittest      [.] YUY2ToYRow_AVX2
 0.77%  libyuv_unittest  libyuv_unittest      [.] YUY2ToARGBRow_AVX2

Now
YUY2ToARGB_Opt (280 ms)
95.66%  libyuv_unittest  libyuv_unittest      [.] YUY2ToARGBRow_AVX2

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

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

000cf89c

22 Sep, 2015 5 commits

Replace random with fastrand · 16f12b58

Frank Barchard authored Sep 22, 2015

random / rand is slow and impacts performance testing.
Although its only called to clear a frame once, a typical profile shows
it high in the overall profile, when doing 1000 frames for a benchmark.

95.10%  libyuv_unittest  libyuv_unittest      [.] YUY2ToARGBRow_SSSE3
 2.01%  libyuv_unittest  libc-2.19.so         [.] __random_r
 1.13%  libyuv_unittest  libc-2.19.so         [.] __random

Replace random is a faster version for unittests.

set LIBYUV_WIDTH=1280
set LIBYUV_HEIGHT=720
set LIBYUV_REPEAT=999
set LIBYUV_FLAGS=-1
out\release\libyuv_unittest --gtest_filter=*YUY2ToARGB*  | findms

Was
libyuvTest.YUY2ToARGB_Opt (497 ms)

Now
libyuvTest.YUY2ToARGB_Opt (454 ms)

R=harryjin@google.com
BUG=none

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

16f12b58

Fix git markers introduced on landing previous CL · 2b92ec8d
Frank Barchard authored Sep 22, 2015
```
BUG=none

Review URL: https://codereview.chromium.org/1359023003 .
```
2b92ec8d

yuy2 to rgb gcc versions · 5f3d4270

Frank Barchard authored Sep 22, 2015

read in read function for yuv conversion

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

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

5f3d4270

Read Y channel in read function for yuv conversion. · 03cd8584

Frank Barchard authored Sep 22, 2015

Allows reader to support YUY2 format.
Also contains fix for win64 build for yuv conversion.

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

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

03cd8584

yuvconstants for all YUV to RGB conversion functions. · f96890a0
Frank Barchard authored Sep 22, 2015
```
R=harryjin@google.com
BUG=libyuv:488

Review URL: https://codereview.chromium.org/1363503002 .
```
f96890a0

18 Sep, 2015 3 commits

move constants into common · 62c49dc8

Frank Barchard authored Sep 18, 2015

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

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

62c49dc8

port I444 to ARGB to matrix. Add I444 to ABGR. · 0381673d

Frank Barchard authored Sep 18, 2015

R=harryjin@google.com
BUG=libyuv:488,libyuv:490

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

0381673d

I444ToABGR for android · 28427a53

Frank Barchard authored Sep 18, 2015

Reimplements I444ToARGB as a matrix function.
new I444ToABGR as matrix functions with wrappers and any functions.
Allows for future J444 and H444 versions.
I444ToABGR user level function added.

BUG=libyuv:490, libyuv:449
R=harryjin@google.com

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

28427a53

17 Sep, 2015 6 commits

NEON J422ToABGR and H422ToABGR missing prototypes · 158d4079
Frank Barchard authored Sep 17, 2015
```
TBR=harryjin@google.com
BUG=none

Review URL: https://codereview.chromium.org/1351993003 .
```
158d4079

NEON constants · bdfd59a7

Frank Barchard authored Sep 17, 2015

TBR=harryjin@google.com
BUG=none

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

bdfd59a7

j422toabgr neon port using i422toabgr matrix function. · 28ce7d94
Frank Barchard authored Sep 17, 2015
```
R=harryjin@google.com
BUG=libyuv:488

Review URL: https://codereview.chromium.org/1353923003 .
```
28ce7d94

J422ToARGB Neon but not aarch64 · 6fcbae14

Frank Barchard authored Sep 17, 2015

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

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

6fcbae14

Add H422ToARGB armv7 neon version. · 6a6b67e7

Frank Barchard authored Sep 17, 2015

Patch provided by zhongwei.yao@linaro.org

R=fbarchard@chromium.org, fbarchard@google.com
BUG=libyuv:488

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

6a6b67e7

Remove redundent rounding test from scale. · ef09d53f

Frank Barchard authored Sep 17, 2015

There is still a rounding test in color test.

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

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

ef09d53f

15 Sep, 2015 2 commits

j422 not available on aarch64 · bb0a521c

Frank Barchard authored Sep 15, 2015

The aarch64 version does not have I422ToARGBMatrix yet,
so adding this to the ifdef section of row.h

R=harryjin@google.com
TBR=harryjin@google.com, zhongwei.yao@linaro.org
BUG=libyuv:488

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

bb0a521c

Add J422ToARGB armv7 neon version. · 509c6442

Frank Barchard authored Sep 15, 2015

R=fbarchard@chromium.org, fbarchard@google.com
BUG=libyuv:488

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

509c6442

14 Sep, 2015 4 commits

neon64 use yuvconstants like 32 bit code. · 73c32d92

Frank Barchard authored Sep 14, 2015

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

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

73c32d92

use struct instead of vectors · a67927c9

Frank Barchard authored Sep 14, 2015

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

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

a67927c9

use same macros as row_gcc.cc · 909160b3

Frank Barchard authored Sep 14, 2015

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

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

909160b3

validate scan EOI from end for better coverage · fcacbfb2
Frank Barchard authored Sep 14, 2015
```
R=tpsiaki@google.com
BUG=libyuv:478

Review URL: https://codereview.chromium.org/1344623003 .
```
fcacbfb2

11 Sep, 2015 1 commit

neon yuv matrix function · 67a9e302

Frank Barchard authored Sep 11, 2015

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

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

67a9e302

09 Sep, 2015 2 commits

Add a sanitizer.gni import to the GN build. · 602fdbef

Frank Barchard authored Sep 09, 2015

The is_*san build flags are moving to this .gni file. This patch will allow
libyuv to continue building when this move happens.

R=brettw@chromium.org, harryjin@google.com
BUG=none

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

602fdbef

avx2 width parameter bug fix · 316e1ab9

Frank Barchard authored Sep 09, 2015

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

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

316e1ab9

08 Sep, 2015 1 commit

disable avx2 · 8467f14e

Frank Barchard authored Sep 08, 2015

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

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

8467f14e

06 Sep, 2015 2 commits

H420 functionality · ed55d24d

Frank Barchard authored Sep 06, 2015

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

Review URL: https://webrtc-codereview.appspot.com/54869004 .

ed55d24d

Change code review host to Chromium's Rietveld. · 3f893126

Henrik Kjellander authored Sep 06, 2015

Since WebRTC's codereview instance at webrtc-codereview.appspot.com
is going to be read-only, libyuv needs to switch to Chromium's instance.

Also update the ViewVC URL to match the Git repo that is now the
master repo.

After this CL has landed, existing Git branches will continue to remember which Rietveld URL they uploaded to.
To reset a branch and re-upload a new CL to the new instance, do like this:
git cl issue 0
git pull --rebase origin master
git cl upload

BUG=libyuv:450
R=fbarchard@chromium.org

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

3f893126

03 Sep, 2015 2 commits

I422ToABGR for win64. Moves any functions to accomidate win64 subset of formats. · 67b06e66
Frank Barchard authored Sep 03, 2015
```
TBR=harryjin@google.com
BUG=libyuv:488

Review URL: https://webrtc-codereview.appspot.com/57679004 .
```
67b06e66

I420ToABGRMatrix functions with J420ToABGR wrapper. · 7060e0d8

Frank Barchard authored Sep 03, 2015

Allows direct conversion from JPeg to ABGR for android.

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

Review URL: https://webrtc-codereview.appspot.com/55719004 .

7060e0d8

02 Sep, 2015 2 commits

define yuvconstants structure all the time, so its can be referred to on all builds. · fbc3d595

Frank Barchard authored Sep 02, 2015

currently only intel code uses this structure, but the prototypes are there for neon and lack of a structure cases a compile error on arm.

R=tpsiaki@google.com
BUG=none

Review URL: https://webrtc-codereview.appspot.com/52799004 .

fbc3d595

I420ToARGB conversion with matrix. · 925c3d9e

Frank Barchard authored Sep 02, 2015

Take color conversion constants as a parameter to row function for I420ToARGBMatrixRow_SSSE3.
Allows future variations of color space using a single low level.

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

Review URL: https://webrtc-codereview.appspot.com/56669004 .

925c3d9e

31 Aug, 2015 1 commit

nolint removed · 0bc626a5

Frank Barchard authored Aug 31, 2015

R=harryjin@google.com
BUG=none

Review URL: https://webrtc-codereview.appspot.com/59389004.

0bc626a5