- 13 Aug, 2018 1 commit
-
-
Mike Frysinger authored
Change-Id: Iffbb35cdda9de495b0612a545893d1b7d5e3cb13 Reviewed-on: https://chromium-review.googlesource.com/1168452Reviewed-by: Ryan Tseng <hinoka@chromium.org> Reviewed-by: Patrik Höglund <phoglund@chromium.org> Commit-Queue: Mike Frysinger <vapier@chromium.org>
-
- 11 Aug, 2018 1 commit
-
-
lixia zhang authored
When loading or storing the data, the unaligned address will greatly degrade the optimization performance, so non-aligned access instructions are required on the loongson platform. Also delete the optimization function:ScaleARGBFilterCols_MMI, because it degraded the performance. BUG=libyuv:804 R=fbarchard@chromium.org Change-Id: If4c15886a21cdcbac7ae8b336292e4549acf1e47 Reviewed-on: https://chromium-review.googlesource.com/1164627Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 10 Aug, 2018 6 commits
-
-
Frank Barchard authored
Bug: libyuv:812 Test: LibYUVConvertTest.H420ToRGB565_Opt Change-Id: Ie85ece74e0bc2b5f789cfcde76703fff6474c0e0 Reviewed-on: https://chromium-review.googlesource.com/1171380Reviewed-by: Mirko Bonadei <mbonadei@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
Chong Zhang authored
bug: 109762970 Change-Id: Iccfdc5dded2dc7695f8a7795b2f32b6401efea0d Reviewed-on: https://chromium-review.googlesource.com/1169687 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
Patrik Höglund authored
Change log: https://chromium.googlesource.com/chromium/src/+log/80f0c55706..35b72bf255 Full diff: https://chromium.googlesource.com/chromium/src/+/80f0c55706..35b72bf255 Changed dependencies: * src/base: https://chromium.googlesource.com/ chromium/src/base/+log/733a32608c..6c0497f398 * src/build: https://chromium.googlesource.com/ chromium/src/build/+log/8cb5352322..f79db013c7 * src/buildtools: https://chromium.googlesource.com/ chromium/buildtools.git/+log/5941c1b3df..9a90d9aaad * src/ios: https://chromium.googlesource.com/ chromium/src/ios/+log/299ef76e84..8e45eb00df * src/testing: https://chromium.googlesource.com/ chromium/src/testing/+log/60b2c69b17..d2fde4ae5b * src/third_party: https://chromium.googlesource.com/ chromium/src/third_party/+log/e755204b7a..f931bb4f2b * src/third_party/android_tools: https://chromium.googlesource.com/ android_tools.git/+log/c22a664c39..130499e252 * src/third_party/catapult: https://chromium.googlesource.com/ catapult.git/+log/f3ce003c2b..0d25dda9b1 * src/third_party/freetype/src: https://chromium.googlesource.com/ chromium/src/third_party/freetype2.git/+log/a44e20879c..578bcf103a * src/third_party/googletest/src: https://chromium.googlesource.com/ external/github.com/google/googletest.git/+log/ba96d0b116..d526632675 * src/third_party/harfbuzz-ng/src: https://chromium.googlesource.com/ external/github.com/harfbuzz/harfbuzz.git/+log/957e775663..2b76767bf5 * src/third_party/icu: https://chromium.googlesource.com/ chromium/deps/icu.git/+log/d888fd2a1b..297a4dd02b * src/third_party/mockito/src: https://chromium.googlesource.com/ external/mockito/mockito.git/+log/de83ad4598..04a2a289a4 * src/third_party/yasm/source/patched-yasm: https://chromium.googlesource.com/ chromium/deps/yasm/patched-yasm.git/+log/b98114e18d..720b70524a * src/tools: https://chromium.googlesource.com/ chromium/src/tools/+log/55c65d8fec..f2c6ed916b * src/tools/swarming_client: https://chromium.googlesource.com/ infra/luci/client-py.git/+log/88229872dd..486c9b53c4 DEPS diff: https://chromium.googlesource.com/ chromium/src/+/80f0c55706..35b72bf255/DEPS Clang version changed 334100:337439 Details: https://chromium.googlesource.com/chromium/src/+/80f0c55706..35b72bf255/tools/clang/scripts/update.py TBR= BUG=None Change-Id: Ia18ef8e7dddaf2cea4364027dfe7026bc6e4733c Reviewed-on: https://chromium-review.googlesource.com/1169812 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Patrik Höglund <phoglund@chromium.org>
-
Patrik Höglund authored
With several buckets, it appears the gerrit tryjobs plugin only displays the first bucket. In that case we want to display the main UCI bucket rather than the win stragglers in buildbot. No-try: True Tbr: fbarchard@chromium.org Bug: chromium:869249 Change-Id: I50dcbf668d29becbe493601be8b9db089de78b85 Reviewed-on: https://chromium-review.googlesource.com/1170688Reviewed-by: Patrik Höglund <phoglund@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org>
-
Patrik Höglund authored
try_job can't be repeated, but buckets can. I think this does what Ryan intended. Also add a presubmit so this doesn't happen again. No-try because CQ is broken, and even if it weren't nothing would catch errors in cq.cfq directly. No-try: True Tbr: fbarchard@chromium.org Bug: 869249 Change-Id: If84685f93275f5af691fc3fef77881156930b726 Reviewed-on: https://chromium-review.googlesource.com/1169820 Commit-Queue: Patrik Höglund <phoglund@chromium.org> Reviewed-by: Patrik Höglund <phoglund@chromium.org>
-
Ryan Tseng authored
All builders look WAI now, so flipping. In particular, the android bots have been migrated in place to LUCI, so the buildbot varients should no longer work. No-Try: True Bug:869249 Change-Id: Iab96cab5d92a106df67fc2458150c0dc93b77fc4 Reviewed-on: https://chromium-review.googlesource.com/1169593Reviewed-by: Patrik Höglund <phoglund@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org>
-
- 09 Aug, 2018 1 commit
-
-
Patrik Höglund authored
This should solve the bug where LUCI win bots try to download the SDK and fails. Bug: chromium:869249 Change-Id: I374ecf6508d6e3978559ec3251b89faaec99edd6 Reviewed-on: https://chromium-review.googlesource.com/1168500Reviewed-by: Ryan Tseng <hinoka@chromium.org>
-
- 08 Aug, 2018 1 commit
-
-
Ryan Tseng authored
This reverts commit 6703fe28. Reason for revert: Builders flipped, triggering first LUCI job Original change's description: > Add a whitespace.txt file > > Bug: 869249 > Change-Id: Iab9eeade2427255ce167fb524b11e0e3f691a685 > Reviewed-on: https://chromium-review.googlesource.com/1161382 > Reviewed-by: Frank Barchard <fbarchard@chromium.org> > Commit-Queue: Ryan Tseng <hinoka@chromium.org> > Commit-Queue: Frank Barchard <fbarchard@chromium.org> TBR=hinoka@chromium.org,fbarchard@chromium.org No-Try: True # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 869249 Change-Id: I00a29051b8a1be048afe444cd7811b19c949fd85 Reviewed-on: https://chromium-review.googlesource.com/1167906Reviewed-by: Ryan Tseng <hinoka@chromium.org>
-
- 06 Aug, 2018 1 commit
-
-
Martin Storsjö authored
This was changed in 21be9122. Change-Id: I6c04dc92f673557e10c231bd090ec8aa88b6bee4 Reviewed-on: https://chromium-review.googlesource.com/1146183Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 03 Aug, 2018 1 commit
-
-
Frank Barchard authored
Bug: libyuv:810,libyuv:811 Test: cmake . && make Change-Id: I521b45ccb6e49ff70823e415efa99fc5b9daad99 Reviewed-on: https://chromium-review.googlesource.com/1162503Reviewed-by: Johann Koenig <johannkoenig@google.com>
-
- 02 Aug, 2018 1 commit
-
-
Ryan Tseng authored
Bug: 869249 Change-Id: Iab9eeade2427255ce167fb524b11e0e3f691a685 Reviewed-on: https://chromium-review.googlesource.com/1161382Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 31 Jul, 2018 2 commits
-
-
Ryan Tseng authored
Otherwise it will inherit the empty python venv. Bug: 869249 TBR: fbarchard Change-Id: Ib77d116863f44f482b2cd7cbfaf71966ce2b7629 Reviewed-on: https://chromium-review.googlesource.com/1157535Reviewed-by: Ryan Tseng <hinoka@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
Ryan Tseng authored
Catapult uses this for win32con Bug: 869249 Change-Id: I6775ff7834100d2b4a55eee3330369fc15b2f14f Reviewed-on: https://chromium-review.googlesource.com/1157139Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org>
-
- 20 Jul, 2018 1 commit
-
-
lixia zhang authored
Currently, libyuv supports MIPS SIMD Arch(MSA), but libyuv does not supports MultiMedia Instruction(MMI)(such as loongson3a platform). In order to improve performance of libyuv on loongson3a platform, this provides optimize 98 functions with mmi. BUG=libyuv:804 Change-Id: I8947626009efad769b3103a867363ece25d79629 Reviewed-on: https://chromium-review.googlesource.com/1122064 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 09 Jul, 2018 1 commit
-
-
Frank Barchard authored
Legacy types can cause build errors with code that defines them differently. Disable them by default. Allow the types to be enabled with #define LIBYUV_LEGACY_TYPES BUG=libyuv:808 TESTED=libyuv try bots still build Change-Id: I48928329393f44a377cec781e645570b14569668 Reviewed-on: https://chromium-review.googlesource.com/1129558 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 04 Jul, 2018 1 commit
-
-
Frank Barchard authored
the built in __msa_ld_b() expects a void * without const. Cast pointers to void * to avoid build warning. TBR=johannkoenig@google.com Bug: libyuv:805 Change-Id: Iabc4820ecf4a3a7dcb0063e67ce276ae2a4f0501 Tested: gn gen out/Release "--args=is_debug=false target_os=\"android\" target_cpu=\"mips64el\" mips_arch_variant=\"r6\" mips_use_msa=true is_component_build=true is_clang=true" ninja -v -C out/Release libyuv_unittest Reviewed-on: https://chromium-review.googlesource.com/1125400 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 29 Jun, 2018 1 commit
-
-
Johann authored
xcode reports clang version 9. It does not appear to support inline assembly using avx512 instructions. Tested with cmake libyuv on Mac. Fails without this patch. BUG=libyuv:789
-
- 26 Jun, 2018 1 commit
-
-
Frank Barchard authored
H420/H422 are bt.720 variants TBR=braveyao@chromium.org BUG=libyuv:799 TESTED=try bots tested build on all platforms Change-Id: I007d8981d91ca0748c59403759109bbcd88f286c Reviewed-on: https://chromium-review.googlesource.com/1115719 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 20 Jun, 2018 2 commits
-
-
Johann authored
Consistently use one style of line endings for the repository Change-Id: Idd70e3d7f3a7a6641b268a81e51eebf9c705b67d Reviewed-on: https://chromium-review.googlesource.com/1107877Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Johann authored
Avoid warnings regarding loss of qualifiers: warning: cast from type ‘const uint8_t* {aka const unsigned char*}’ to type ‘v16i8* {aka __vector(16) signed char*}’ casts away qualifiers BUG=libyuv:793 Change-Id: Ie0d215bc07b49285b5d06ee91ccc2c9a7979799e Reviewed-on: https://chromium-review.googlesource.com/1107879Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 16 Jun, 2018 1 commit
-
-
Tom Anderson authored
BUG=libyuv:795 Change-Id: Ic1eb6eb841e763c0d58e6fb36322addb3956d8f2 Reviewed-on: https://chromium-review.googlesource.com/1103311Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 15 Jun, 2018 1 commit
-
-
Tom Anderson authored
After [1], a manual dependency on exe_and_shlib_deps is no longer necessary since it's automatically added. This CL removes all remaining manual references to exe_and_shlib_deps. [1] https://chromium.googlesource.com/chromium/src.git/+/d7ed1f0a9c28c932fddc834ca5de44f28266c7f5 BUG=chromium:845700 R=fbarchard Change-Id: I01ade690f3f098599ec271a1e82ec34c573e21d7 Reviewed-on: https://chromium-review.googlesource.com/1102100Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 12 Jun, 2018 1 commit
-
-
Frank Barchard authored
BUG=libyuv:799 TESTED=try bots build Change-Id: Ib4ce8d928069445a710c1e30ea85d9dccc820b6c Reviewed-on: https://chromium-review.googlesource.com/1097561Reviewed-by: Miguel Casas <mcasas@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 01 Jun, 2018 1 commit
-
-
Frank Barchard authored
This reverts commit a8aa921c. Reason for revert: breaks a webrtc unittest on Windows. https://bugs.chromium.org/p/webrtc/issues/detail?id=9263&can=2&start=0&num=100&q=&colspec=ID%20Pri%20M%20ReleaseBlock%20Component%20Status%20Owner%20Summary&groupby=&sort= Original change's description: > Allow negative height when ConvertToI420/ARGB is called with NV12/NV21 > > ConvertToI420 and ConvertToARGB support the use of a negative height > parameter to flip the image vertically. When converting from NV12 or > NV21 this parameter was misinterpreted, resulting in invalid output. > This CL introduces the use of abs_src_height to correctly calculate > the location of the source UV plane. > > The sign of crop_height is not used, to reduce confusion ConvertToI420 > and ConvertToARGB no longer accept negative crop height. > > Unit tests for Android420ToI420 are updated to fix miscalculation of > src_stride_uv, fix incorrect pixel strides, and to test inversion. > New unit tests are included to test inversion for ConvertToARGB, > ConvertToI420, Android420ToARGB, and Android420ToABGR. > For consistency the test NV12Crop is renamed ConvertToI420_NV12_Crop. > > Bug: libyuv:446 > Test: out/Release/libyuv_unittest --gtest_filter=*.ConvertTo*:*.Android420To* > Change-Id: Idc98e62671cb30272cfa7e24fafbc8b73712f7c6 > Reviewed-on: https://chromium-review.googlesource.com/994074 > Commit-Queue: Frank Barchard <fbarchard@chromium.org> > Reviewed-by: Frank Barchard <fbarchard@chromium.org> TBR=fbarchard@chromium.org,robert@bares.me # Not skipping CQ checks because original CL landed > 1 day ago. Bug: libyuv:446, chromium:9263, libyuv:801 Change-Id: I7c55b3fcb477f9754c249b9c2c54b24da2c29283 Reviewed-on: https://chromium-review.googlesource.com/1081267Reviewed-by: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Weiyong Yao <braveyao@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 11 May, 2018 1 commit
-
-
Frank Barchard authored
Mask was set to 32, but should have been 31. BUG=libyuv:798 TESTED=try bots tested Change-Id: I6120928873a4a2f1efef907d8e8296ca8c20bb03 Reviewed-on: https://chromium-review.googlesource.com/1054830Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 19 Apr, 2018 1 commit
-
-
Robert Bares authored
ConvertToI420 and ConvertToARGB support the use of a negative height parameter to flip the image vertically. When converting from NV12 or NV21 this parameter was misinterpreted, resulting in invalid output. This CL introduces the use of abs_src_height to correctly calculate the location of the source UV plane. The sign of crop_height is not used, to reduce confusion ConvertToI420 and ConvertToARGB no longer accept negative crop height. Unit tests for Android420ToI420 are updated to fix miscalculation of src_stride_uv, fix incorrect pixel strides, and to test inversion. New unit tests are included to test inversion for ConvertToARGB, ConvertToI420, Android420ToARGB, and Android420ToABGR. For consistency the test NV12Crop is renamed ConvertToI420_NV12_Crop. Bug: libyuv:446 Test: out/Release/libyuv_unittest --gtest_filter=*.ConvertTo*:*.Android420To* Change-Id: Idc98e62671cb30272cfa7e24fafbc8b73712f7c6 Reviewed-on: https://chromium-review.googlesource.com/994074 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 18 Apr, 2018 1 commit
-
-
Patrik Höglund authored
This is a stop-gap; it will simply cause cipd deps to not be updated, which will probably keep things working for now, but it's not what we want for the long term. Bug: chromium:659808 Change-Id: I292b96f174c8d910c0b5f0196eefd0e5a5f907c2 Reviewed-on: https://chromium-review.googlesource.com/1016380Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 13 Apr, 2018 5 commits
-
-
Johann authored
When casting a const value, ensure the cast is const as well. BUG=webm:1509 Change-Id: I5b597fdcc148d111e9824bc7cf918fc5f24e970f Reviewed-on: https://chromium-review.googlesource.com/996553Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Frank Barchard authored
Bug: libyuv:791 Test: g++ -Iinclude -I../libvpx/third_party/libwebm -I../libvpx/vp8 -I../libvpx/vp8 -I../libvpx/vp9 -I../libvpx/vp9 -Iinclude -m64 -DNDEBUG -O3 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -Wall -Wdisabled-optimization -Wfloat-conversion -Wpointer-arith -Wtype-limits -Wcast-qual -Wvla -Wuninitialized -Wunused -Wextra -I. -I"../libvpx" -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -Wno-unused-parameter -c -o third_party/libyuv/source/row_common.cc.o source/row_common.cc Change-Id: Ia006cb9212b671ae668cab5ec0b29759024a2c8a Reviewed-on: https://chromium-review.googlesource.com/1012462Reviewed-by: Johann Koenig <johannkoenig@google.com> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Johann authored
BUG=webm:1509 Change-Id: I74db6e287952b343021c0a17a80c973517080517 Reviewed-on: https://chromium-review.googlesource.com/996311Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Johann authored
When casting input for loads, include modifiers such as 'const' Clears build warnings: warning: cast from type 'const uint8_t* {aka const unsigned char*}' to type 'uint32_t* {aka unsigned int*}' casts away qualifiers [-Wcast-qual] Bug: webm:1509, libyuv:791 Test: g++ -Iinclude -I../libvpx/third_party/libwebm -I../libvpx/vp8 -I../libvpx/vp8 -I../libvpx/vp9 -I../libvpx/vp9 -Iinclude -m64 -DNDEBUG -O3 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -Wall -Wdisabled-optimization -Wfloat-conversion -Wpointer-arith -Wtype-limits -Wcast-qual -Wvla -Wuninitialized -Wunused -Wextra -I. -I"../libvpx" -D__STDC_CONSTANT_MACROS -D__STDC_LIMIT_MACROS -Wno-unused-parameter -c -o third_party/libyuv/source/row_common.cc.o source/row_common.cc Change-Id: I1e3b2fe2a4ae9dd466c3db9cde0560aceb9d1398 Reviewed-on: https://chromium-review.googlesource.com/996393Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Martin Storsjö authored
This is the same fix as in d4cfc7bc, but applied at the second block that also defines HAS_HASHDJB2_AVX2. Remove the duplicated, redundant block. Change-Id: I53e0fc5f46e2ed5c14a4531c2b571c72d969a471 Reviewed-on: https://chromium-review.googlesource.com/989432Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 12 Apr, 2018 1 commit
-
-
Johann authored
The file this referes to (x86inc.asm) was removed in commit c9986313 BUG=libyuv:790 Change-Id: I4aae28475ef11e01304daf2aaf60a0404752cfa7 Reviewed-on: https://chromium-review.googlesource.com/996373Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
- 11 Apr, 2018 1 commit
-
-
Oleh Prypin authored
Bug: webrtc:9104 No-Try: True Change-Id: I3860b0f7b07fa5110a0bc552965c49801c7a3acd Reviewed-on: https://chromium-review.googlesource.com/1005002 Commit-Queue: Oleh Prypin <oprypin@chromium.org> Reviewed-by: Patrik Höglund <phoglund@chromium.org>
-
- 03 Apr, 2018 1 commit
-
-
Frank Barchard authored
iOS simulator has the option to build with xcode instead of clang. GN use_xcode_clang=true enables the xcode build. As of version Xcode 9.2, the clang version used does not support AVX512. The version reported is version 9, but for normal clang, version 7 is sufficient to AVX512. When a version of XCode does support AVX512, the version check can be updated to allow AVX512 for newer versions of XCode. with XCode 9.2 the following macro is set. __APPLE_CC__ 6000 Bug: libyuv:789 Test: gn gen out/Release "--args=is_debug=false target_os=\"ios\" ios_enable_code_signing=false target_cpu=\"x86\" use_xcode_clang=true" Change-Id: I5a9a0b4a2760c7d09a4bcb464b3668979113b07e Reviewed-on: https://chromium-review.googlesource.com/991595Reviewed-by: richard winterton <rrwinterton@gmail.com>
-
- 31 Mar, 2018 1 commit
-
-
Martin Storsjö authored
When detected, add the necessary defines to actually use it. Change-Id: I540c3e11e480be8aaab154ad91ee08cdc52319de Reviewed-on: https://chromium-review.googlesource.com/988432 Commit-Queue: Frank Barchard <fbarchard@chromium.org> Reviewed-by: Frank Barchard <fbarchard@chromium.org>
-
- 30 Mar, 2018 2 commits
-
-
Martin Storsjö authored
In these cases, CLANG_HAS_AVX2 is defined, but _MSC_VER isn't. The corresponding codepaths are in source/compare_win.cc, and the whole file is guarded with a defined(_MSC_VER). This is the same fix as in d8680893, but this issue wasn't noticed since compare.o wasn't linked in in earlier tested configurations. Change-Id: Ifff6d4bd7b1071256df1169881490a52026c183b Reviewed-on: https://chromium-review.googlesource.com/988573Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-
Martin Storsjö authored
Change-Id: Ib6367fa94c60f7239fe02ace81c06054b1b4a837 Reviewed-on: https://chromium-review.googlesource.com/988532Reviewed-by: Frank Barchard <fbarchard@chromium.org> Commit-Queue: Frank Barchard <fbarchard@chromium.org>
-