Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(123)

Unified Diff: src/core/SkBlitter_RGB16.cpp

Issue 143423004: ARM Skia NEON patches - 35 - First AArch64 support (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Rebase Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/core/SkBlitter_RGB16.cpp
diff --git a/src/core/SkBlitter_RGB16.cpp b/src/core/SkBlitter_RGB16.cpp
index 256cbc6936f6daad6a1c48c798936a3b184dbfc0..4edd00d768bfb30be2f82faa4c04fae06e6e2deb 100644
--- a/src/core/SkBlitter_RGB16.cpp
+++ b/src/core/SkBlitter_RGB16.cpp
@@ -16,8 +16,7 @@
#include "SkUtils.h"
#include "SkXfermode.h"
-#if defined(__ARM_HAVE_NEON) && defined(SK_CPU_LENDIAN)
- #define SK_USE_NEON
+#if SK_ARM_NEON_IS_ALWAYS && defined(SK_CPU_LENDIAN)
djsollen 2014/03/20 14:40:16 can we do this in a separate CL since it doesn't s
kevin.petit 2014/03/20 16:05:09 You're right. I've uploaded https://codereview.chr
#include <arm_neon.h>
#else
// if we don't have neon, then our black blitter is worth the extra code
@@ -385,7 +384,7 @@ void SkRGB16_Opaque_Blitter::blitMask(const SkMask& mask,
unsigned maskRB = mask.fRowBytes - width;
uint32_t expanded32 = fExpandedRaw16;
-#ifdef SK_USE_NEON
+#if SK_ARM_NEON_IS_ALWAYS && defined(SK_CPU_LENDIAN)
#define UNROLL 8
do {
int w = width;

Powered by Google App Engine
This is Rietveld 408576698