GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

Remember https://codereview.chromium.org/1290423007 ?
This is part 2, and parts 1 and 3 have already landed.

GYP config in Skia:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/skia/gyp/opts.gyp&l=140

GYP config for libvpx:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libvpx_new/libvpx_srcs_x86_64_intrinsics.gypi&l=97

GN config for libvpx:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libvpx_new/BUILD.gn&l=159

BUG=skia:4117
Committed: https://crrev.com/6c88d596d0ab59817b358250d6bf51a2264b6461
Cr-Commit-Position: refs/heads/master@{#359108}

[mtklein_C, 2015-11-09 16:27:29]

Description was changed from

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

BUG=
==========

to

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

BUG=skia:4117
==========

[mtklein_C, 2015-11-09 16:28:30]

The CQ bit was checked by mtklein@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-11-09 16:28:42]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1431143002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1431143002/1

[mtklein_C, 2015-11-09 16:32:26]

Description was changed from

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

BUG=skia:4117
==========

to

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

Remember https://codereview.chromium.org/1290423007 ?
This is part 2, and parts 1 and 3 have already landed.

BUG=skia:4117
==========

[mtklein_C, 2015-11-09 16:33:57]

mtklein@chromium.org changed reviewers:
+ djsollen@google.com

[djsollen, 2015-11-09 16:59:13]

I follow the logic for the Skia stuff, but am at a total loss when it comes to
the MSVC changes for both GYP and GN.  Is there someone in chromium who knows
MSVC that could comment on that?

[mtklein, 2015-11-09 17:01:37]

On 2015/11/09 at 16:59:13, djsollen wrote:
> I follow the logic for the Skia stuff, but am at a total loss when it comes to
the MSVC changes for both GYP and GN.  Is there someone in chromium who knows
MSVC that could comment on that?

Luckily, we've already worked this out in Skia:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/skia/g...

[commit-bot: I haz the power, 2015-11-09 17:51:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-09 17:51:57]

Dry run: This issue passed the CQ dry run.

[mtklein_C, 2015-11-09 19:55:53]

Description was changed from

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

Remember https://codereview.chromium.org/1290423007 ?
This is part 2, and parts 1 and 3 have already landed.

BUG=skia:4117
==========

to

==========
GYP+GN plumbing for SSE4.2, AVX, and AVX2 in Skia.

This doesn't change any code yet: each of the sse42, avx, and avx2_sources
is a list with a single empty .cpp file.

Remember https://codereview.chromium.org/1290423007 ?
This is part 2, and parts 1 and 3 have already landed.


GYP config in Skia:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/skia/g...

GYP config for libvpx: 
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libvpx...

GN config for libvpx:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libvpx...

BUG=skia:4117
==========

[mtklein_C, 2015-11-09 19:56:54]

Updated the CL description with some GYP/GN references: our local GYPs for
{SSE42, AVX, AVX2} , and how libvpx turns on AVX2 in GYP and GN.

[djsollen, 2015-11-10 03:31:32]

lgtm

[mtklein_C, 2015-11-10 04:09:57]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2015-11-10 04:10:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1431143002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1431143002/1

[mtklein_C, 2015-11-10 04:10:42]

The CQ bit was unchecked by mtklein@chromium.org

[mtklein_C, 2015-11-10 04:11:05]

On 2015/11/10 at 04:10:19, commit-bot wrote:
> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1431143002/1
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1431143002/1

On second thought, going to save this for Wednesday morning.

[mtklein_C, 2015-11-11 16:18:49]

The CQ bit was checked by mtklein@chromium.org

[commit-bot: I haz the power, 2015-11-11 16:19:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1431143002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1431143002/1

[commit-bot: I haz the power, 2015-11-11 17:37:27]

Committed patchset #1 (id:1)

[dcheng, 2015-11-12 03:43:36]

dcheng@chromium.org changed reviewers:
+ dcheng@chromium.org

[dcheng, 2015-11-12 03:43:36]

FYI, this breaks the win clang x64 debug build:
http://build.chromium.org/p/chromium.fyi/builders/CrWinClang64%28dbg%29/build...

[commit-bot: I haz the power, 2015-11-12 19:58:19]

Patchset 1 (id:??) landed as
https://crrev.com/6c88d596d0ab59817b358250d6bf51a2264b6461
Cr-Commit-Position: refs/heads/master@{#359108}