commit 4e97607d9a1cef66fac16f347c5ca813ec4f9515
author mtklein <mtklein@chromium.org> Mon Aug 08 09:06:27 2016 -0700
committer Commit bot <commit-bot@chromium.org> Mon Aug 08 09:06:28 2016 -0700
tree e523cac97fae89a2abb1ec0d4c1fc87f86e3169f
parent 9a5a201472c3fb9f02b954ba09ae1ec13ebd50f4

Use sse4.2 CRC32 instructions to hash when available.

About 9x faster than Murmur3 for long inputs.

Most of this is a mechanical change from SkChecksum::Murmur3(...) to SkOpts::hash(...).

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2208903002
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot;master.client.skia.compile:Build-Ubuntu-GCC-x86_64-Release-CMake-Trybot,Build-Mac-Clang-x86_64-Release-CMake-Trybot

Review-Url: https://codereview.chromium.org/2208903002

gyp/opts.gypi

diff --git a/gyp/opts.gypi b/gyp/opts.gypi
index 6db9d0c..e89334d 100644
--- a/gyp/opts.gypi
+++ b/gyp/opts.gypi
@@ -49,13 +49,13 @@
         'sse41_sources': [
             '<(skia_src_path)/opts/SkOpts_sse41.cpp',
         ],
+        'sse42_sources': [
+            '<(skia_src_path)/opts/SkOpts_sse42.cpp',
+        ],
         'avx_sources': [
             '<(skia_src_path)/opts/SkOpts_avx.cpp',
         ],
         # These targets are empty, but XCode doesn't like that, so add an empty file to each.
-        'sse42_sources': [
-            '<(skia_src_path)/core/SkForceCPlusPlusLinking.cpp',
-        ],
         'avx2_sources': [
             '<(skia_src_path)/core/SkForceCPlusPlusLinking.cpp',
         ],