Rewrite memset benches, then use results to add a small-N optimization.

include/core/SkUtils.h

 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkUtils_DEFINED
 #define SkUtils_DEFINED
 
 #include "SkTypes.h"
 
 ///////////////////////////////////////////////////////////////////////////////
 
 /** Similar to memset(), but it assigns a 16bit value into the buffer.
     @param buffer   The memory to have value copied into it
     @param value    The 16bit value to be copied into buffer
     @param count    The number of times value should be copied into the buffer.
 */
-void sk_memset16(uint16_t dst[], uint16_t value, int count);
+void sk_memset16_large(uint16_t dst[], uint16_t value, int count);

[reed1, 2015/04/09 18:55:48]

Seems like its not much harder to provide both N values...

// These values were determined running ...
#ifdef SK_BUILD_FOR_ARM && NOT_64BIT
    #define kMEMSET_SMALL_CUTOFF    10
#else
    #define kMEMSET_SMALL_CUTOFF    10000
#endif

[mtklein, 2015/04/09 19:11:10]

Good point!  Done.

+inline void sk_memset16(uint16_t dst[], uint16_t value, int count) {
+    // 10 determined empirically using bench/MemsetBench.cpp on a Nexus 7, Nexus 9, and desktop.
+    // (N7 was the limiting factor; desktops and N9 consider 10,000 small but 100,000 large).
+    if (count <= 10) {
+        for (int i = 0; i < count; i++) {
+            dst[i] = value;
+        }
+    } else {
+        sk_memset16_large(dst, value, count);
+    }
+}
 typedef void (*SkMemset16Proc)(uint16_t dst[], uint16_t value, int count);
 SkMemset16Proc SkMemset16GetPlatformProc();
 
 /** Similar to memset(), but it assigns a 32bit value into the buffer.
     @param buffer   The memory to have value copied into it
     @param value    The 32bit value to be copied into buffer
     @param count    The number of times value should be copied into the buffer.
 */
-void sk_memset32(uint32_t dst[], uint32_t value, int count);
+void sk_memset32_large(uint32_t dst[], uint32_t value, int count);
+inline void sk_memset32(uint32_t dst[], uint32_t value, int count) {
+    // 10 determined empirically using bench/MemsetBench.cpp on a Nexus 7, Nexus 9, and desktop.
+    // (N7 was the limiting factor; desktops and N9 consider 10,000 small but 100,000 large).
+    if (count <= 10) {
+        for (int i = 0; i < count; i++) {
+            dst[i] = value;
+        }
+    } else {
+        sk_memset32_large(dst, value, count);
+    }
+}
+
 typedef void (*SkMemset32Proc)(uint32_t dst[], uint32_t value, int count);
 SkMemset32Proc SkMemset32GetPlatformProc();
 
 /** Similar to memcpy(), but it copies count 32bit values from src to dst.
     @param dst      The memory to have value copied into it
     @param src      The memory to have value copied from it
     @param count    The number of values should be copied.
 */
 void sk_memcpy32(uint32_t dst[], const uint32_t src[], int count);
 typedef void (*SkMemcpy32Proc)(uint32_t dst[], const uint32_t src[], int count);
@@ skipping 69 matching lines @@
     }
     ~SkAutoTrace() {
         SkDebugf("--- trace: %s Leave\n", fLabel);
     }
 private:
     const char* fLabel;
 };
 #define SkAutoTrace(...) SK_REQUIRE_LOCAL_VAR(SkAutoTrace)
 
 #endif