diff options
author | 2015-07-30 05:37:06 -0700 | |
---|---|---|
committer | 2015-07-30 05:37:06 -0700 | |
commit | 6c72d5740231f47c664a8e765a8df05cd124c88c (patch) | |
tree | e4b9b65db3fff2df31f7e3b7d1ddae4fc698572d /src | |
parent | 640898f5881df1fd56d306190608dd79b1f32044 (diff) |
Optimize RGB16 blitH functions with NEON for ARM platform.
Here are some performance resultsi on Nexus 9:
SkRGB16BlitterBlitH_neon:
+--------+-----------+
|height | C/NEON |
+--------+-----------+
|1 | 0.888531 |
+--------+-----------+
|8 | 1.231800 |
+--------+-----------+
|18 | 1.073327 |
+--------+-----------+
|32 | 1.136991 |
+--------+-----------+
|76 | 1.174638 |
+--------+-----------+
|85 | 1.188551 |
+--------+-----------+
|120 | 1.180261 |
+--------+-----------+
|128 | 1.183726 |
+--------+-----------+
|512 | 1.220806 |
+--------+-----------+
BUG=skia:
Review URL: https://codereview.chromium.org/1229673008
Diffstat (limited to 'src')
-rw-r--r-- | src/core/SkBlitter_RGB16.cpp | 12 | ||||
-rw-r--r-- | src/opts/SkBlitMask_opts_arm_neon.cpp | 45 |
2 files changed, 57 insertions, 0 deletions
diff --git a/src/core/SkBlitter_RGB16.cpp b/src/core/SkBlitter_RGB16.cpp index 8caf908312..b481b27b57 100644 --- a/src/core/SkBlitter_RGB16.cpp +++ b/src/core/SkBlitter_RGB16.cpp @@ -27,6 +27,10 @@ extern void SkRGB16BlitterBlitV_neon(uint16_t* device, size_t deviceRB, unsigned scale, uint32_t src32); +extern void SkRGB16BlitterBlitH_neon(uint16_t* device, + int width, + unsigned scale, + uint32_t src32); #else // if we don't have neon, then our black blitter is worth the extra code #define USE_BLACK_BLITTER @@ -338,10 +342,14 @@ void SkRGB16_Opaque_Blitter::blitAntiH(int x, int y, uint32_t src32 = srcExpanded * scale5; scale5 = 32 - scale5; // now we can use it on the device int n = count; +#if SK_ARM_NEON_IS_ALWAYS && defined(SK_CPU_LENDIAN) + SkRGB16BlitterBlitH_neon(device, n, scale5, src32); +#else do { uint32_t dst32 = SkExpand_rgb_16(*device) * scale5; *device++ = SkCompact_rgb_16((src32 + dst32) >> 5); } while (--n != 0); +#endif goto DONE; } } @@ -601,10 +609,14 @@ void SkRGB16_Blitter::blitAntiH(int x, int y, unsigned scale5 = SkAlpha255To256(aa) * scale >> (8 + 3); uint32_t src32 = srcExpanded * scale5; scale5 = 32 - scale5; +#if SK_ARM_NEON_IS_ALWAYS && defined(SK_CPU_LENDIAN) + SkRGB16BlitterBlitH_neon(device, count, scale5, src32); +#else do { uint32_t dst32 = SkExpand_rgb_16(*device) * scale5; *device++ = SkCompact_rgb_16((src32 + dst32) >> 5); } while (--count != 0); +#endif continue; } device += count; diff --git a/src/opts/SkBlitMask_opts_arm_neon.cpp b/src/opts/SkBlitMask_opts_arm_neon.cpp index 3361a5d19d..370780c210 100644 --- a/src/opts/SkBlitMask_opts_arm_neon.cpp +++ b/src/opts/SkBlitMask_opts_arm_neon.cpp @@ -332,3 +332,48 @@ void SkRGB16BlitterBlitV_neon(uint16_t* device, #undef LOAD_LANE_16 #undef STORE_LANE_16 + +void SkRGB16BlitterBlitH_neon(uint16_t* device, + int width, + unsigned scale, + uint32_t src32) { + if (width >= 8) { + // prepare constants + uint16x8_t vdev = vdupq_n_u16(0); + uint16x8_t vmaskq_g16 = vdupq_n_u16(SK_G16_MASK_IN_PLACE); + uint16x8_t vmaskq_ng16 = vdupq_n_u16(~SK_G16_MASK_IN_PLACE); + uint32x4_t vsrc32 = vdupq_n_u32(src32); + uint32x4_t vscale5 = vdupq_n_u32((uint32_t)scale); + + while (width >= 8) { + vdev = vld1q_u16(device); + + // Expand_rgb_16 + uint16x8x2_t vdst = vzipq_u16((vdev & vmaskq_ng16), (vdev & vmaskq_g16)); + uint32x4_t vdst32_lo = vmulq_u32(vreinterpretq_u32_u16(vdst.val[0]), vscale5); + uint32x4_t vdst32_hi = vmulq_u32(vreinterpretq_u32_u16(vdst.val[1]), vscale5); + + // Compact_rgb_16 + vdst32_lo = vaddq_u32(vdst32_lo, vsrc32); + vdst32_hi = vaddq_u32(vdst32_hi, vsrc32); + vdst32_lo = vshrq_n_u32(vdst32_lo, 5); + vdst32_hi = vshrq_n_u32(vdst32_hi, 5); + + uint16x4_t vtmp_lo = vmovn_u32(vdst32_lo) & vget_low_u16(vmaskq_ng16); + uint16x4_t vtmp_hi = vshrn_n_u32(vdst32_lo, 16) & vget_low_u16(vmaskq_g16); + uint16x4_t vdst16_lo = vorr_u16(vtmp_lo, vtmp_hi); + vtmp_lo = vmovn_u32(vdst32_hi) & vget_low_u16(vmaskq_ng16); + vtmp_hi = vshrn_n_u32(vdst32_hi, 16) & vget_low_u16(vmaskq_g16); + uint16x4_t vdst16_hi = vorr_u16(vtmp_lo, vtmp_hi); + + vst1q_u16(device, vcombine_u16(vdst16_lo, vdst16_hi)); + device += 8; + width -= 8; + } + } + while (width != 0) { + uint32_t dst32 = SkExpand_rgb_16(*device) * scale; + *device++ = SkCompact_rgb_16((src32 + dst32) >> 5); + width--; + } +} |