Implement a NEON version of the RGBA gaussian blur. This shows a 9-15% speedup on Nexus-10.

src/opts/SkBlurImage_opts_neon.cpp

Index: src/opts/SkBlurImage_opts_neon.cpp
diff --git a/src/opts/SkBlurImage_opts_neon.cpp b/src/opts/SkBlurImage_opts_neon.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..05607a63cb91ace43089d4bb598cd65635297db4
--- /dev/null
+++ b/src/opts/SkBlurImage_opts_neon.cpp
@@ -0,0 +1,103 @@
+/*
+ * Copyright 2013 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.
+ */
+
+
+#include "SkBitmap.h"
+#include "SkColorPriv.h"
+#include "SkBlurImage_opts.h"
+#include "SkRect.h"
+
+#include <arm_neon.h>
+
+namespace {
+
+enum BlurDirection {
+    kX, kY
+};
+
+/**
+ * Helper function to spread the components of a 32-bit integer into the
+ * lower 8 bits of each 32-bit element of a NEON register.
+ */
+
+inline uint32x4_t expand(uint32_t a) {
+    // ( A R G B A R G B )
+    uint8x8_t v8 = vreinterpret_u8_u32(vdup_n_u32(a));
+    // ( A R G B A R G B ) -> (   A   R   G   B   A   R   G   B ) -> (   A   R   G   B )

[mtklein, 2013/12/04 16:52:23]

Maybe use _ for 0's leaving spaces for spacing?

+    const uint16x4_t v16 = vget_low_u16(vmovl_u8(v8));
+    // (       A       R       G       B )
+    return vmovl_u16(v16);
+}
+
+template<BlurDirection srcDirection, BlurDirection dstDirection>
+void SkBoxBlur_NEON(const SkPMColor* src, int srcStride, SkPMColor* dst, int kernelSize,
+                    int leftOffset, int rightOffset, int width, int height)
+{
+    const int rightBorder = SkMin32(rightOffset + 1, width);
+    const int srcStrideX = srcDirection == kX ? 1 : srcStride;
+    const int dstStrideX = dstDirection == kX ? 1 : height;
+    const int srcStrideY = srcDirection == kX ? srcStride : 1;
+    const int dstStrideY = dstDirection == kX ? width : 1;
+    const uint32x4_t scale = vdupq_n_u32((1 << 24) / kernelSize);
+    const uint32x4_t half = vdupq_n_u32(1 << 23);
+    for (int y = 0; y < height; ++y) {
+        uint32x4_t sum = vdupq_n_u32(0);
+        const SkPMColor* p = src;
+        for (int i = 0; i < rightBorder; ++i) {
+            sum = vaddq_u32(sum, expand(*p));
+            p += srcStrideX;
+        }
+
+        const SkPMColor* sptr = src;
+        SkPMColor* dptr = dst;
+        for (int x = 0; x < width; ++x) {
+            // ( half+sumA*scale half+sumR*scale half+sumG*scale half+sumB*scale )
+            uint32x4_t result = vmlaq_u32(half, sum, scale);
+
+            // Shift down to lower 8 bits of each element.
+            // (       A       R       G       B )
+            result = vshrq_n_u32(result, 24);
+
+            // (   A   R   G   B )
+            uint16x4_t result16 = vqmovn_u32(result);
+
+            // ( A R G B A R G B )
+            uint8x8_t result8 = vqmovn_u16(vcombine_u16(result16, result16));
+
+            // Store low 32 bits to destination.
+            vst1_lane_u32(dptr, vreinterpret_u32_u8(result8), 0);
+            if (x >= leftOffset) {
+                const SkPMColor* l = sptr - leftOffset * srcStrideX;
+                sum = vsubq_u32(sum, expand(*l));
+            }
+            if (x + rightOffset + 1 < width) {
+                const SkPMColor* r = sptr + (rightOffset + 1) * srcStrideX;
+                sum = vaddq_u32(sum, expand(*r));
+            }
+            sptr += srcStrideX;
+            if (srcDirection == kY) {
+                SK_PREFETCH(sptr + (rightOffset + 1) * srcStrideX);
+            }
+            dptr += dstStrideX;
+        }
+        src += srcStrideY;
+        dst += dstStrideY;
+    }
+}
+
+} // namespace
+
+bool SkBoxBlurGetPlatformProcs(SkBoxBlurProc* boxBlurX,
+                               SkBoxBlurProc* boxBlurY,
+                               SkBoxBlurProc* boxBlurXY,
+                               SkBoxBlurProc* boxBlurYX) {
+    *boxBlurX = SkBoxBlur_NEON<kX, kX>;
+    *boxBlurY = SkBoxBlur_NEON<kY, kY>;
+    *boxBlurXY = SkBoxBlur_NEON<kX, kY>;
+    *boxBlurYX = SkBoxBlur_NEON<kY, kX>;
+    return true;
+}