diff options
author | mtklein <mtklein@chromium.org> | 2015-09-10 10:38:02 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-09-10 10:38:02 -0700 |
commit | 4a37d08382a16717cde52c3d2687b021c5413464 (patch) | |
tree | 873cdb0493b2039cd3d3d478b5849c9d181163b8 /src/core/SkBlitRow_D32.cpp | |
parent | d69ece6a03d497cead8dc5c5ea2d682e49800df4 (diff) |
Port SkBlitRow::Color32 to SkOpts.
This was a pre-SkOpts attempt that we can bring under its wing now.
This should be a perf no-op, deo volente.
BUG=skia:4117
Review URL: https://codereview.chromium.org/1314863006
Diffstat (limited to 'src/core/SkBlitRow_D32.cpp')
-rw-r--r-- | src/core/SkBlitRow_D32.cpp | 25 |
1 files changed, 2 insertions, 23 deletions
diff --git a/src/core/SkBlitRow_D32.cpp b/src/core/SkBlitRow_D32.cpp index 0f8cbe8104..75330ae337 100644 --- a/src/core/SkBlitRow_D32.cpp +++ b/src/core/SkBlitRow_D32.cpp @@ -8,6 +8,7 @@ #include "SkBlitRow.h" #include "SkBlitMask.h" #include "SkColorPriv.h" +#include "SkOpts.h" #include "SkUtils.h" #define UNROLL @@ -131,32 +132,10 @@ SkBlitRow::Proc32 SkBlitRow::Factory32(unsigned flags) { return proc; } -#include "Sk4px.h" - -// Color32 uses the blend_256_round_alt algorithm from tests/BlendTest.cpp. -// It's not quite perfect, but it's never wrong in the interesting edge cases, -// and it's quite a bit faster than blend_perfect. -// -// blend_256_round_alt is our currently blessed algorithm. Please use it or an analogous one. void SkBlitRow::Color32(SkPMColor dst[], const SkPMColor src[], int count, SkPMColor color) { switch (SkGetPackedA32(color)) { case 0: memmove(dst, src, count * sizeof(SkPMColor)); return; case 255: sk_memset32(dst, color, count); return; } - - // This Sk4px impl works great on other platforms or when we have NEON. -#if defined(SK_CPU_ARM32) && !defined(SK_ARM_HAS_NEON) - if (auto proc = PlatformColor32Proc()) { return proc(dst, src, count, color); } -#endif - - unsigned invA = 255 - SkGetPackedA32(color); - invA += invA >> 7; - SkASSERT(invA < 256); // We've already handled alpha == 0 above. - - Sk16h colorHighAndRound = Sk4px::DupPMColor(color).widenHi() + Sk16h(128); - Sk16b invA_16x(invA); - - Sk4px::MapSrc(count, dst, src, [&](const Sk4px& src4) -> Sk4px { - return (src4 * invA_16x).addNarrowHi(colorHighAndRound); - }); + return SkOpts::blit_row_color32(dst, src, count, color); } |