avx and avx2 detection

avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.  Those messages should match the -SSE4-, -AVX-, or -AVX2- in
the Test-...-Debug-Trybots below.  The Release ones are just running by accident.

So far they look right to me.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/844a0b425741f07cb233332405143931586bbb7d

[mtklein, 2015-11-06 20:51:53]

Description was changed from

==========
avx and avx2 detection

I think I've got AVX and AVX2 detection correct here, though
AVX2 detection does not seem to be working on my laptop.

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

to

==========
avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.

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

[mtklein, 2015-11-06 20:53:32]

Description was changed from

==========
avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.

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

to

==========
avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.  Those messages should match the -SSE4-, -AVX-, or -AVX2- in
the Test-...-Debug-Trybots below.  The Release ones are just running by
accident.

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

[mtklein, 2015-11-06 20:54:12]

mtklein@google.com changed reviewers:
+ msarett@google.com, mtklein@google.com

[mtklein, 2015-11-06 21:02:59]

Description was changed from

==========
avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.  Those messages should match the -SSE4-, -AVX-, or -AVX2- in
the Test-...-Debug-Trybots below.  The Release ones are just running by
accident.

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

to

==========
avx and avx2 detection

This doesn't do anything yet beyond print out a message in Debug mode,
but it's a start.  Those messages should match the -SSE4-, -AVX-, or -AVX2- in
the Test-...-Debug-Trybots below.  The Release ones are just running by
accident.

So far they look right to me.

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

[msarett, 2015-11-06 21:57:42]

I trust this if the test bots seem to be doing what we expect.  I can't make
much sense of the code.

Do we also try to check at compile time when possible?

lgtm

[mtklein, 2015-11-06 22:26:18]

On 2015/11/06 at 21:57:42, msarett wrote:
> I trust this if the test bots seem to be doing what we expect.  I can't make
much sense of the code.
Pretty much same here.  It's a lot of "if this is set, and this is set, check
this, then check this...".

> Do we also try to check at compile time when possible?
Yep.  Whatever we've got at compile time ends up as the sk_default::foo methods.
 Then they'll be overwritten with perhaps-better runtime detected methods at
startup.  But we'll never rewrite a newer method with an older one.

> lgtm

[mtklein, 2015-11-06 22:26:22]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-11-06 22:26:27]

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

[commit-bot: I haz the power, 2015-11-06 22:27:38]

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

[commit-bot: I haz the power, 2015-11-06 22:27:39]

Try jobs failed on following builders:
  Build-Mac10.8-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac10.8-Clang-...)

[mtklein, 2015-11-07 03:36:54]

The CQ bit was checked by mtklein@google.com

[mtklein, 2015-11-07 03:36:54]

The patchset sent to the CQ was uploaded after l-g-t-m from msarett@google.com
Link to the patchset: https://codereview.chromium.org/1428153003/#ps60001
(title: "__cpuid_count")

[commit-bot: I haz the power, 2015-11-07 03:37:05]

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

[commit-bot: I haz the power, 2015-11-07 03:48:24]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/844a0b425741f07cb233332405143931586bbb7d