Commits · dee77a4ebeaebc781cb3acd80aa6627fd1c7c825 · submodule / libyuv

09 Dec, 2015 2 commits

Optimize yuv alpha blend AVX2 code to do 32 pixels at time. · dee77a4e

Frank Barchard authored Dec 09, 2015

out/Release/libyuv_unittest --libyuv_width=1280 --libyuv_height=720 --libyuv_repeat=9999 --libyuv_flags=-1 --gtest_filter=*I420Blend_Opt

Was LibYUVPlanarTest.I420Blend_Opt (2335 ms)
Now LibYUVPlanarTest.I420Blend_Opt (1937 ms)

vs SSSE3
LibYUVPlanarTest.I420Blend_Opt (2599 ms)

BUG=libyuv:527
R=dhrosa@google.com

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

dee77a4e

Work around bug in xgetbv for Visual Studio. · fae1a105

Frank Barchard authored Dec 09, 2015

xgetbv is generating bad code, falsely disabling AVX2 and AVX512.
disable optimization for the function affected on older versions of Visual C 32 bit.

R=brucedawson@chromium.org, dhrosa@google.com, harryjin@google.com
BUG=libyuv:529

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

fae1a105

07 Dec, 2015 3 commits

Add support for odd height YUVA alpha blending. · 2657688e

Frank Barchard authored Dec 07, 2015

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:527

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

2657688e

Unroll C version of YUV blender for improved performance. · b0b22f88

Frank Barchard authored Dec 07, 2015

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:527

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

b0b22f88

Bug fix for UYVYToNV12 odd height · 48a919d8

Frank Barchard authored Dec 07, 2015

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

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

48a919d8

06 Dec, 2015 1 commit

AVX2 YUV alpha blender and improved unittests · bea690b3

Frank Barchard authored Dec 06, 2015

AVX2 version can process 16 pixels at a time for improved memory bandwidth and fewer instructions.

unittests improved to test unaligned memory, and test exactness when alpha is 0 or 255.

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:527

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

bea690b3

04 Dec, 2015 1 commit

Port BlendPlaneRow_SSSE3 to GCC · fa2618ee

Frank Barchard authored Dec 04, 2015

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:527

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

fa2618ee

02 Dec, 2015 1 commit

planar blend use signed images · 8af0ebf8

Frank Barchard authored Dec 02, 2015

R=dhrosa@google.com, harryjin@google.com, jzern@chromium.org
BUG=libyuv:527

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

8af0ebf8

26 Nov, 2015 1 commit

Interpolate plane initial implementation. · b6f37bd8

Frank Barchard authored Nov 26, 2015

YUV version of interpolation between two images.

R=dhrosa@google.com, harryjin@google.com
BUG=libyuv:526

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

b6f37bd8

20 Nov, 2015 1 commit

disable 411 on x86 due to compile error · 88552486

Frank Barchard authored Nov 20, 2015

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

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

88552486

19 Nov, 2015 2 commits
- disable debug build of 411 to work around compiler bug · 526558b2
  Frank Barchard authored Nov 19, 2015
```
TBR=harryjin@google.com
BUG=libyuv:524

Review URL: https://codereview.chromium.org/1461013002 .
```
  526558b2
- fix for I411 build error on 32 bit x86 · b7dfb725
  Frank Barchard authored Nov 19, 2015
```
TBR=harrjin@google.com
BUG=libyuv:525

Review URL: https://codereview.chromium.org/1461693004 .
```
  b7dfb725
18 Nov, 2015 3 commits

syntax fix for gcc movzwl · 528356a1

Frank Barchard authored Nov 18, 2015

TBR=harryjin@google.com
BUG=libtyv:525

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

528356a1

port I411 movzx 2 byte reader to gcc · 50f8cb2d

Frank Barchard authored Nov 18, 2015

previously the I411 format used movd to read U, V pixels.
But this reads 4 bytes, and can cause a memory exception.
pinsrw can be used, but fails on drmemory 1.5, and is slow.
So in this change a movzxw is used to read 2 bytes into EBX,
then copy to xmm0 with movd.
Slightly slower, but no memory exception
Was LibYUVConvertTest.I411ToARGB_Opt (577 ms)
Now LibYUVConvertTest.I411ToARGB_Opt (608 ms)

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

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

50f8cb2d

Fix for drmemory failure on I411ToARGB · 5eefbe23

Frank Barchard authored Nov 18, 2015

Before
I420ToARGB_Opt (594 ms)
I422ToARGB_Opt (483 ms)
I411ToARGB_Opt (748 ms) ***
I444ToARGB_Opt (452 ms)
I400ToARGB_Opt (218 ms)

After
I420ToARGB_Opt (591 ms)
I422ToARGB_Opt (454 ms)
I411ToARGB_Opt (502 ms)  ***
I444ToARGB_Opt (441 ms)
I400ToARGB_Opt (216 ms)

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

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

5eefbe23

17 Nov, 2015 2 commits

free src_a in unittest to fix leak · ec4b258d

Frank Barchard authored Nov 17, 2015

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

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

ec4b258d

test for unaligned vs aligned for CopyRow_SSE2 · 0815568a

Frank Barchard authored Nov 17, 2015

improves performance on older CPUs where movdqa is faster.
TBR=harryjin@google.com
BUG=libyuv:492

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

0815568a

14 Nov, 2015 1 commit

port I444ToARGB avx2 code from Visual C to GCC. · 1019e453

Frank Barchard authored Nov 14, 2015

SSSE3
Note: Google Test filter = *I444ToARGB*
[==========] Running 8 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 8 tests from LibYUVConvertTest
[ RUN      ] LibYUVConvertTest.I444ToARGB_Any
[       OK ] LibYUVConvertTest.I444ToARGB_Any (435 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Unaligned
[       OK ] LibYUVConvertTest.I444ToARGB_Unaligned (418 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Invert
[       OK ] LibYUVConvertTest.I444ToARGB_Invert (417 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Opt
[       OK ] LibYUVConvertTest.I444ToARGB_Opt (411 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Any
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Any (419 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Unaligned
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Unaligned (432 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Invert
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Invert (435 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Opt
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Opt (421 ms)
[----------] 8 tests from LibYUVConvertTest (3389 ms total)

AVX2
Note: Google Test filter = *I444ToARGB*
[==========] Running 8 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 8 tests from LibYUVConvertTest
[ RUN      ] LibYUVConvertTest.I444ToARGB_Any
[       OK ] LibYUVConvertTest.I444ToARGB_Any (340 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Unaligned
[       OK ] LibYUVConvertTest.I444ToARGB_Unaligned (325 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Invert
[       OK ] LibYUVConvertTest.I444ToARGB_Invert (316 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_Opt
[       OK ] LibYUVConvertTest.I444ToARGB_Opt (316 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Any
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Any (315 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Unaligned
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Unaligned (341 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Invert
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Invert (331 ms)
[ RUN      ] LibYUVConvertTest.I444ToARGB_ARGB_Opt
[       OK ] LibYUVConvertTest.I444ToARGB_ARGB_Opt (329 ms)
[----------] 8 tests from LibYUVConvertTest (2615 ms total)

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

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

1019e453

13 Nov, 2015 1 commit

scale with conversion using 2 steps with unittest · 60adcbaf

Frank Barchard authored Nov 13, 2015

a prototype function to implement the yuv to rgb with conversion and scale.
replace with 1 step function in future version, using same API.

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

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

60adcbaf

10 Nov, 2015 2 commits

fix yvu constants for avx2 yuv to rgb · 6100f50f

Frank Barchard authored Nov 10, 2015

the yvu matrix for yuv to rgb had an incorrect entry, affecting yuv to bgra,
yuv to abgr and yuv to raw.
fix the matrix and reenable avx2 functions.

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

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

6100f50f

disable more avx2 functions that dont link in chrome · 72a9e282

Frank Barchard authored Nov 10, 2015

libyuv builds/runs, but when integrated into chromium, produces link errors. unclear why but this disables affected functions.
will followup with re-enabling them once the root cause in the runtime error is found.

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

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

72a9e282

09 Nov, 2015 1 commit

disable 4 AVX2 YUV to RGB conversions which fails tests. · fb5ed1f4

Frank Barchard authored Nov 09, 2015

disable I422ALPHATOARGBROW_AVX2 I422TOARGBROW_AVX2 I422TORGB24ROW_AVX2 I422TORGBAROW_AVX2 in row.h.
SSSE3 versions will be used instead.
Short term fix until issue can be resolved.

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

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

fb5ed1f4

06 Nov, 2015 1 commit

set d19 alpha on inner loop · 98eb102b

Frank Barchard authored Nov 06, 2015

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

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

98eb102b

05 Nov, 2015 1 commit

YUV to RGB for x64 use registers instead of memory. · 431cb366

Frank Barchard authored Nov 05, 2015

On Arm the YVU to RGB conversions move constants into registers.
This change does the same for 64 bit intel builds where additional
registers are available.
The AVX2 saves 3 instructions by because the 2nd argument needs to be a register, so a vmovdqu was avoided.

x64 builds using memory:
AVX2  I420ToARGB_Opt (3059 ms)
SSSE3 I420ToARGB_Opt (3959 ms)

Now using registers
AVX2  I420ToARGB_Opt (2906 ms)
SSSE3 I420ToARGB_Opt (3928 ms)

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

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

431cb366

04 Nov, 2015 7 commits

add .gn file for gn builds · c2bff1a1

Frank Barchard authored Nov 04, 2015

using a stripped down gn file from webrtc.

BUG=libyuv:411,libyuv:519
R=kjellander@chromium.org

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

c2bff1a1

Neon versions of I420AlphaToARGB · 860cc035

Frank Barchard authored Nov 04, 2015

Add alpha version of YUV to RGB to neon code for ARMv7 and aarch64.
For other YUV to RGB conversions, hoist alpha set to 255 out of loop.

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

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

860cc035

unittest fix for cpu flags. · 82d74a37

Frank Barchard authored Nov 04, 2015

cpu flags of 1 disables SIMD and uses C.  This used to be 0, but the change
in auto init behavior means that 0 now means uninitialized, and will cause
auto detect to reinit the cpu info.  A value of 1 disables the auto init.

TBR=harryjin@google.com
BUG=none

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

82d74a37

cpuid tool fix - remove mipsdsp and add avx3 · ab9a42d9
Frank Barchard authored Nov 04, 2015
```
TBR=harryjin@google.com
BUG=none

Review URL: https://codereview.chromium.org/1412373006 .
```
ab9a42d9
rename yuv matrix constants to be more clear about what they are · d95d2169
Frank Barchard authored Nov 04, 2015
```
R=harryjin@google.com
BUG=none

Review URL: https://codereview.chromium.org/1429693006 .
```
d95d2169

add command line cpu info to allow android neon test · c629cb3a

Frank Barchard authored Nov 04, 2015

in order to compare C and Neon code, a new command line flag is added.
historically environment variables controlled cpu features, but on
android apk it is easier to pass a command line option to disable cpu
optimizations.

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

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

c629cb3a

remove mips dsp detect · 1f1d140b

Frank Barchard authored Nov 04, 2015

DSP code is not actually used, only DSPR2.  Remove the detect.

TBR=harryjin@google.com
BUG=none

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

1f1d140b

03 Nov, 2015 1 commit

Raw 24 bit RGB to RGB24 (bgr) · ce4c2fad

Frank Barchard authored Nov 03, 2015

Add unittests that do 1 step conversion vs 2 step conversion.

Tests end swapping versions match direct conversions.

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

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

ce4c2fad

02 Nov, 2015 2 commits

remove store bgra, abgr, raw unused macros · 87926cec

Frank Barchard authored Nov 02, 2015

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

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

87926cec

remove I422ToBGRA and use I422ToRGBA internally · 2c7aa007

Frank Barchard authored Nov 02, 2015

Removes low levels for I420ToBGRA and I420ToRAW and reimplements them as I420ToRGBA and I420ToRGB24 with transposed color matrix.

Adds unittests that do 1 step conversion vs 2 steps to test end swapping versions match direct conversions.

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

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

2c7aa007

30 Oct, 2015 1 commit

refactor I420ToABGR to use I420ToARGBRow · 5d97b933

Frank Barchard authored Oct 30, 2015

Using a transposed conversion matrix, I420ToARGB can output ABGR.

R=harryjin@google.com, xhwang@chromium.org
BUG=libyuv:473

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

5d97b933

29 Oct, 2015 1 commit
- disable I420AlphaToARGB for 32 bit intel debug build · 254ef015
  Frank Barchard authored Oct 29, 2015
```
R=harryjin@google.com
BUG=libyuv:517

Review URL: https://codereview.chromium.org/1428843003 .
```
  254ef015
28 Oct, 2015 2 commits

Fix debug compilation problems for gcc and 32 bit x86. · cdbdf5b7

Frank Barchard authored Oct 28, 2015

In some methods with 7 arguments gcc fails to find enough registers
to compile the assembler code when compiling debug. Simplest solution
is to skip the assembler version in debug of those particular functions

(I422Alpha -> ARBG/ABGR)

R=harryjin@google.com,bratell@opera.com
BUG=libyuv:517

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

cdbdf5b7

pass clangcl compile options to ignore warnings in gflags.cc · 811a5ec4

Frank Barchard authored Oct 28, 2015

R=ajm@chromium.org, ajm@google.com
BUG=libyuv:513,webrtc:760

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

811a5ec4

27 Oct, 2015 2 commits

refactor I420AlphaToABGR to use I420AlphaToARGB internally · b86dbf24

Frank Barchard authored Oct 27, 2015

swap U and V and transpose conversion matrix, so I420AlphaToARGB and
I420AlphaToABGR share low level code.

Having less code with same performance allows more focused
optimization for future ARM versions.

R=harryjin@google.com
TBR=harryjin@chromium.org
BUG=libyuv:473,libyuv:516

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

b86dbf24

implement I444ToABGR by swapping uv and transpose matrix · cf160cdb

Frank Barchard authored Oct 27, 2015

U contributes to B and G. V contributes to R and G.
By swapping U and V, they contribute to the opposite channels. Adjust the matrix so the U contribution is in the matrix location such that it till contribute to the
new B channel and vice versa.
This allows ABGR versions of YUV conversion to use the same low level code as ARGB, just using a different matrix and swapping U and V pointers.

As a result the existing I444ToABGRRow functions are no longer needed and are removed.

Previously this function was only Intel AVX2 optimized for Windwos. Now it is also optimized for Arm and GCC.

ARMv7 Neon
Was LibYUVConvertTest.I444ToABGR_Opt (75971 ms)
Now LibYUVConvertTest.I444ToABGR_Opt (3672 ms)
20.6 times faster.

R=xhwang@chromium.org
BUG=libyuv:515

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

cf160cdb