diff options
| author |  mtklein <mtklein@chromium.org> | 2016-08-08 09:06:27 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-08-08 09:06:28 -0700 |
| commit | 4e97607d9a1cef66fac16f347c5ca813ec4f9515 (patch) | |
| tree | e523cac97fae89a2abb1ec0d4c1fc87f86e3169f /src/core/SkResourceCache.cpp | |
| parent | 9a5a201472c3fb9f02b954ba09ae1ec13ebd50f4 (diff) | |
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
Diffstat (limited to 'src/core/SkResourceCache.cpp')
| -rw-r--r-- | src/core/SkResourceCache.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/core/SkResourceCache.cpp b/src/core/SkResourceCache.cpp index e465132288..4bdc8dd9e0 100644 --- a/src/core/SkResourceCache.cpp +++ b/src/core/SkResourceCache.cpp @@ -5,10 +5,10 @@ * found in the LICENSE file. */ -#include "SkChecksum.h" #include "SkMessageBus.h" #include "SkMipMap.h" #include "SkMutex.h" +#include "SkOpts.h" #include "SkPixelRef.h" #include "SkResourceCache.h" #include "SkTraceMemoryDump.h" @@ -46,9 +46,9 @@ void SkResourceCache::Key::init(void* nameSpace, uint64_t sharedID, size_t dataS fSharedID_lo = (uint32_t)sharedID; fSharedID_hi = (uint32_t)(sharedID >> 32); fNamespace = nameSpace; - // skip unhashed fields when computing the murmur - fHash = SkChecksum::Murmur3(this->as32() + kUnhashedLocal32s, - (fCount32 - kUnhashedLocal32s) << 2); + // skip unhashed fields when computing the hash + fHash = SkOpts::hash(this->as32() + kUnhashedLocal32s, + (fCount32 - kUnhashedLocal32s) << 2); } #include "SkTDynamicHash.h" |