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

Side by Side Diff: src/core/SkBlitRow_D32.cpp

Issue 1145283003: Re-proc SkBlitRow::Color32 for ARM. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: ARM builds may be using _opts_none.cpp (iOS) Created 5 years, 7 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 unified diff | Download patch
« no previous file with comments | « include/core/SkBlitRow.h ('k') | src/opts/SkBlitRow_opts_arm.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2011 Google Inc. 2 * Copyright 2011 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkBlitRow.h" 8 #include "SkBlitRow.h"
9 #include "SkBlitMask.h" 9 #include "SkBlitMask.h"
10 #include "SkColorPriv.h" 10 #include "SkColorPriv.h"
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 // It's not quite perfect, but it's never wrong in the interesting edge cases, 137 // It's not quite perfect, but it's never wrong in the interesting edge cases,
138 // and it's quite a bit faster than blend_perfect. 138 // and it's quite a bit faster than blend_perfect.
139 // 139 //
140 // blend_256_round_alt is our currently blessed algorithm. Please use it or an analogous one. 140 // blend_256_round_alt is our currently blessed algorithm. Please use it or an analogous one.
141 void SkBlitRow::Color32(SkPMColor dst[], const SkPMColor src[], int count, SkPMC olor color) { 141 void SkBlitRow::Color32(SkPMColor dst[], const SkPMColor src[], int count, SkPMC olor color) {
142 switch (SkGetPackedA32(color)) { 142 switch (SkGetPackedA32(color)) {
143 case 0: memmove(dst, src, count * sizeof(SkPMColor)); return; 143 case 0: memmove(dst, src, count * sizeof(SkPMColor)); return;
144 case 255: sk_memset32(dst, color, count); return; 144 case 255: sk_memset32(dst, color, count); return;
145 } 145 }
146 146
147 // This Sk4px impl works great on other platforms or when we have NEON.
148 #if defined(SK_CPU_ARM32) && !defined(SK_ARM_HAS_NEON)
149 if (auto proc = PlatformColor32Proc()) { return proc(dst, src, count, color) ; }
150 #endif
151
147 unsigned invA = 255 - SkGetPackedA32(color); 152 unsigned invA = 255 - SkGetPackedA32(color);
148 invA += invA >> 7; 153 invA += invA >> 7;
149 SkASSERT(invA < 256); // We've already handled alpha == 0 above. 154 SkASSERT(invA < 256); // We've already handled alpha == 0 above.
150 155
151 Sk16h colorHighAndRound = Sk4px(color).widenHi() + Sk16h(128); 156 Sk16h colorHighAndRound = Sk4px(color).widenHi() + Sk16h(128);
152 Sk16b invA_16x(invA); 157 Sk16b invA_16x(invA);
153 158
154 Sk4px::MapSrc(count, dst, src, [&](const Sk4px& src4) -> Sk4px { 159 Sk4px::MapSrc(count, dst, src, [&](const Sk4px& src4) -> Sk4px {
155 return src4.mulWiden(invA_16x).addNarrowHi(colorHighAndRound); 160 return src4.mulWiden(invA_16x).addNarrowHi(colorHighAndRound);
156 }); 161 });
157 } 162 }
OLDNEW
« no previous file with comments | « include/core/SkBlitRow.h ('k') | src/opts/SkBlitRow_opts_arm.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698