- 12 Apr, 2016 1 commit
-
-
Henrik Kjellander authored
This requires you don't have target_os=["ios"] set in your libyuv root .gclient file, since that will make native_client not being downloaded due to https://code.google.com/p/chromium/codesearch#chromium/src/DEPS&l=357 BUG=libyuv:573 TESTED= rm chromium/.last_sync_chromium rm chromium/.gclient.tmp_entries gclient sync native_client/build/gyp_nacl all.gyp -Dgtest_target_type=executable -Dmsan=0 -Duse_system_yasm=0 ninja -C out/Debug Review URL: https://codereview.chromium.org/1845003004 .
-
- 11 Apr, 2016 3 commits
-
-
Frank Barchard authored
BUG=libyuv:578 TESTED=local unittests pass R=harryjin@google.com Review URL: https://codereview.chromium.org/1879783002 .
-
Henrik Kjellander authored
BUG=601140 TBR=fbarchard@chromium.org Review URL: https://codereview.chromium.org/1878763002 .
-
Frank Barchard authored
normally this warning is disabled but for nacl builds that use clang its not. this CL makes the option universally disabled for all clang based builds. BUG=libyuv:581 TESTED=native_client/build/gyp_nacl all.gyp -Dgtest_target_type=executable -Dmsan=0 & ninja -C out/Debug libyuv R=kjellander@chromium.org Review URL: https://codereview.chromium.org/1865183002 .
-
- 08 Apr, 2016 2 commits
-
-
Frank Barchard authored
When using C version of I420Interpolate for msan, a 50% interpolation would cause stride to be cast to int, which could cause erroneous memory reads on 64 bit build. This CL makes the stride use ptrdiff_t for HalfRow_C BUG=libyuv:582 TESTED=try bots tests R=dhrosa@google.com Review URL: https://codereview.chromium.org/1872953002 .
-
Henrik Kjellander authored
This is how GYP is skipped on the GN bots. If GYP runs, it will currently produce files in out/{Debug,Releaese} that makes GN fail. BUG=601140 TBR=fbarchard@chromium.org Review URL: https://codereview.chromium.org/1874653003 .
-
- 07 Apr, 2016 1 commit
-
-
Frank Barchard authored
R=kjellander@chromium.org BUG=libyuv:583 TESTED=gn gen out/Debug Review URL: https://codereview.chromium.org/1870473002 .
-
- 06 Apr, 2016 1 commit
-
-
Frank Barchard authored
Affected Linux GN build, not Windows. R=kjellander@chromium.org BUG=libyuv:583 TESTED=gn gen out/Debug --args=is_debug=true Review URL: https://codereview.chromium.org/1866743002 .
-
- 02 Apr, 2016 1 commit
-
-
Frank Barchard authored
TBR=kjellander@chromium.org BUG=libyuv:573 TESTED=try bots Review URL: https://codereview.chromium.org/1850053002 .
-
- 01 Apr, 2016 1 commit
-
-
Frank Barchard authored
BUG=libyuv:579 TESTED=mkdir build && cd build && cmake .. && cmake --build . --config Release R=kjellander@chromium.org Review URL: https://codereview.chromium.org/1847233002 .
-
- 31 Mar, 2016 1 commit
-
-
Magnus Jedvert authored
R=aiolos@chromium.org, torbjorng@chromium.org Review URL: https://codereview.chromium.org/1809653002 .
-
- 16 Mar, 2016 2 commits
-
-
Frank Barchard authored
GYP_DEFINES="target_arch=x64 msan=1" ./gyp_libyuv ninja -j7 -C out/Release R=impjdi@google.com BUG=libyuv:575 Review URL: https://codereview.chromium.org/1805683003 .
-
Henrik Kjellander authored
Thanks to Alexander Kolesnik (alexkol75@gmail.com) for noticing this. BUG= R=fbarchard@google.com Review URL: https://codereview.chromium.org/1805523002 .
-
- 10 Mar, 2016 1 commit
-
-
Frank Barchard authored
They are not needed, and due to them there was a call to _xgetbv() without a declaration of the function. This used to work because we implicitly included intrin.h in all translation units with clang-cl, but we want to stop doing that. BUG=chromium:592745 R=fbarchard@google.com Review URL: https://codereview.chromium.org/1780473003 .
-
- 04 Mar, 2016 1 commit
-
-
Henrik Kjellander authored
This brings in the renamed libvpx that happened in https://codereview.chromium.org/1734613003/ TBR=fbarchard@chromium.org Review URL: https://codereview.chromium.org/1764083002 .
-
- 29 Feb, 2016 1 commit
-
-
Frank Barchard authored
The 64 bit version of ARGBToRGB565 to 32 bit. 64 bit is using sri which shifts and inserts, saving some masking. The instruction is available for neon 32 bit as well. R=magjed@chromium.org, harryjin@google.com BUG=libyuv:571 Review URL: https://codereview.chromium.org/1724393002 .
-
- 24 Feb, 2016 1 commit
-
-
Frank Barchard authored
Name of objdump tool updated. TBR=kjellander@chromium.org BUG=none Review URL: https://codereview.chromium.org/1715743003 . Review URL: https://codereview.chromium.org/1727993002 .
-
- 19 Feb, 2016 1 commit
-
-
Frank Barchard authored
Name of objdump tool updated. R=kjellander@chromium.org BUG=none Review URL: https://codereview.chromium.org/1715743003 .
-
- 18 Feb, 2016 4 commits
-
-
Frank Barchard authored
Based on webrtc roll https://codereview.webrtc.org/1503883002 https://chromium.googlesource.com/chromium/src/+/0c2a94ab87e6351b924a19932442e85a8f786fa0 needed the addition of tools/telemetry and third_party/catapult symlinks for Android. GN changes required modification of .gn. Change log: https://chromium.googlesource.com/chromium/src/+log/626eecf..72c3265 Full diff: https://chromium.googlesource.com/chromium/src/+/626eecf..72c3265 Changed dependencies: * src/buildtools: https://chromium.googlesource.com/chromium/buildtools.git/+log/818123d..b747a9e * src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/fde89b4..6d9e5a7 DEPS diff: https://chromium.googlesource.com/chromium/src/+/626eecf..72c3265/DEPS TBR=jkellander@chromium.org BUG=libyuv:570 Review URL: https://codereview.chromium.org/1709193002 .
-
Frank Barchard authored
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 .
-
Frank Barchard authored
document play services update R=jkellander@chromium.org BUG=none Review URL: https://codereview.chromium.org/1712463002 .
-
Frank Barchard authored
R=jkellander@chromium.org BUG=none Review URL: https://codereview.chromium.org/1703243003 .
-
- 17 Feb, 2016 2 commits
-
-
Frank Barchard authored
Make environment variables match ifdefs with one SIMD instruction set word as suffix for functions, ifdefs and environment variables. R=kjellander@chromium.org BUG=libyuv:486 Review URL: https://codereview.chromium.org/1704053002 .
-
Henrik Kjellander authored
This is based on today's content at https://code.google.com/p/libyuv/w/list BUG=chromium:587193 Review URL: https://codereview.chromium.org/1703653002 .
-
- 16 Feb, 2016 1 commit
-
-
Henrik Kjellander authored
The drover.properties file is no longer useful after migrating the repo to Git. BUG=libyuv:568 R=mflodman@chromium.org Review URL: https://codereview.chromium.org/1696173002 .
-
- 12 Feb, 2016 2 commits
-
-
Frank Barchard authored
The gflags library was missing from the CMake build and the gtest includes were not being set correctly. R=fbarchard@google.com Review URL: https://codereview.chromium.org/1692083002 .
-
Frank Barchard authored
An SSSE3 version already exists, and an AVX2 version is available for Visual C. This ports the function to AVX2 completing the AVX2 ports of all YUV to RGB functions for AVX2 on gcc. TBR=harryjin@google.com BUG=libyuv:555 Review URL: https://codereview.chromium.org/1687253002 .
-
- 10 Feb, 2016 2 commits
-
-
Frank Barchard authored
NV12ToRGB565Row for Intel is implemented as a 2 step conversion: NV12ToARGBRow_SSSE3 and ARGBToRGB565Row_SSE2 NV12ToARGBRow has an AVX2 version, so this CL implements NV12ToRGB565Row_AVX2 with call to NV12ToARGBRow_AVX2 and ARGBToRGB565Row_SSE2. R=harryjin@google.com BUG=libyuv:554 Review URL: https://codereview.chromium.org/1687953002 .
-
Frank Barchard authored
R=harryjin@google.com BUG=libyuv:557 Review URL: https://codereview.chromium.org/1687713002 .
-
- 09 Feb, 2016 3 commits
-
-
Henrik Kjellander authored
TBR=fbarchard@chromium.org Review URL: https://codereview.chromium.org/1679123006 .
-
Frank Barchard authored
R=harryjin@google.com BUG=libyuv:564 Review URL: https://codereview.chromium.org/1685723002 .
-
Frank Barchard authored
TBR=harryjin@google.com BUG=libyuv:566 Review URL: https://codereview.chromium.org/1673313003 .
-
- 08 Feb, 2016 1 commit
-
-
Frank Barchard authored
R=harryjin@google.com BUG=none Review URL: https://codereview.chromium.org/1676383002 .
-
- 05 Feb, 2016 1 commit
-
-
Frank Barchard authored
When attempting to normalize function names to end in Row_SIMD it was made harder with MIPS_DSPR2 naming convention. Other CPUs do not include the vendor. This should be named consistently. Removed the DISABLE_MIPS in favour of DISABLE_ASM for consistency with other processors. TBR=harryjin@google.com BUG=libyuv:562 Review URL: https://codereview.chromium.org/1677633002 .
-
- 03 Feb, 2016 2 commits
-
-
Henrik Kjellander authored
TBR=fbarchard@chromium.org BUG=libyuv:564 Review URL: https://codereview.chromium.org/1665573003 .
-
Henrik Kjellander authored
TBR=fbarchard@chromium.org Review URL: https://codereview.chromium.org/1656423002 .
-
- 02 Feb, 2016 3 commits
-
-
Frank Barchard authored
internal math of the fastrand function uses a multiply and add that overflows a signed int. This triggers a ubsan failure: ../../unit_test/../unit_test/unit_test.h:60:33: runtime error: signed integer overflow: 56248274 * 214013 cannot be represented in type 'int' This change casts the intermediate math to unsigned int to avoid the overflow. For more info on ubsan, see http://dev.chromium.org/developers/testing/undefinedbehaviorsanitizer TESTED=Passing compilation using: GYP_DEFINES="ubsan=1" GYP_DEFINES="ubsan_vptr=1" R=harryjin@google.com, pbos@webrtc.org BUG=libyuv:563 Review URL: https://codereview.chromium.org/1662453003 .
-
Frank Barchard authored
For more info on ubsan, see http://dev.chromium.org/developers/testing/undefinedbehaviorsanitizer TESTED=Passing compilation using: GYP_DEFINES="ubsan=1" GYP_DEFINES="ubsan_vptr=1" R=harryjin@google.com, pbos@webrtc.org BUG=libyuv:563 Review URL: https://codereview.chromium.org/1654253004 .
-
Frank Barchard authored
For more info, see http://dev.chromium.org/developers/testing/undefinedbehaviorsanitizer TESTED=Passing compilation using: GYP_DEFINES="ubsan=1" GYP_DEFINES="ubsan_vptr=1" R=harryjin@google.com, pbos@webrtc.org BUG=libyuv:563,webrtc:5124 Review URL: https://codereview.chromium.org/1659713002 .
-
- 01 Feb, 2016 1 commit
-
-
Frank Barchard authored
This is an UBSan error reported by libjingle [ RUN ] WebRtcVideoFrameTest.ConvertToYUY2BufferStride [000:000] (videoframe.cc:375): Validate frame passed. format: I420 bpp: 12 size: 1280x720 bytes: 1382400 expected: 1382400 sample[0..3]: 73, 73, 73, 73 ../../chromium/src/third_party/libyuv/source/row_gcc.cc:2903:25: runtime error: signed integer overflow: 128 * 16843009 cannot be represented in type 'int' [8/614] WebRtcVideoFrameTest.ConvertToYUY2BufferStride returned/aborted with exit code 1 (32 ms) [9/614] WebRtcVideoFrameTest.ConvertToYUY2BufferInverted (29 ms) Note: Google Test filter = WebRtcVideoFrameTest.ConvertToYUY2BufferInverted The source is uint8 and the multiply is by 0x01010101 to replicate the byte to 4 bytes. Changing the constant to 0x01010101u should avoid overflow. R=harryjin@google.com TBR=harryjin@google.com BUG=libyuv:563 Review URL: https://codereview.chromium.org/1657533005 .
-