commit 13e3faebc619cd2a448187d48de763b0b5c8e794
author Herb Derby <herb@google.com> Thu Jul 23 13:24:53 2020 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Fri Jul 24 13:26:06 2020 +0000
tree 7b3ab128db9442bda4b0944091ad7850b764e3f3
parent b6d4ad92adcac186c3bc52feef85de38752a1f1d

reduce mutex use and switch to spinlock

* Combine find and makeMRU.
* Switch from SkMutex to SkSpinLock

In the gm 'paragraph_$' this reduces acquiring the lock from 1.2% to 0.7%
as measured by instruments and nanobench.

Change-Id: I33e3af31825f175c9de42f001acf68ffe3623a8a
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/305564
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Herb Derby <herb@google.com>

src/gpu/GrRenderTargetContext.cpp

diff --git a/src/gpu/GrRenderTargetContext.cpp b/src/gpu/GrRenderTargetContext.cpp
index fd1f2a1..e68bbb1 100644
--- a/src/gpu/GrRenderTargetContext.cpp
+++ b/src/gpu/GrRenderTargetContext.cpp
@@ -482,11 +482,7 @@
     }
 
     const SkMatrix& drawMatrix(viewMatrix.localToDevice());
-    if (blob != nullptr && blob->canReuse(blobPaint, blurRec, drawMatrix, drawOrigin)) {
-        // Reusing the blob. Move it to the front of LRU cache.
-        textBlobCache->makeMRU(blob.get());
-    } else {
-        // Build or Rebuild the GrTextBlob
+    if (blob == nullptr || !blob->canReuse(blobPaint, blurRec, drawMatrix, drawOrigin)) {
         if (blob != nullptr) {
             // We have to remake the blob because changes may invalidate our masks.
             // TODO we could probably get away with reuse most of the time if the pointer is unique,