diff options
author | Mike Klein <mtklein@chromium.org> | 2017-09-12 16:40:53 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-09-14 12:16:51 +0000 |
commit | 9b2f69b0aacc4819c3b4b9c5c8af8ef8ab35d572 (patch) | |
tree | 8c42dbdb70ab1ebcdfe331a53bd94e2e4a7244e5 /src/jumper/SkJumper.h | |
parent | 0f96bb303adfc712baba7830e036df1d0972d7f3 (diff) |
grand unifried lowp stages
I have text_16_AA_FF -> 8888 (forcing RP) faster than head now on my
laptop. I'm feeling confident that we can make this perform well.
After looking at performance a bit more today, it looks like everything
is within what I'd consider comparable in performance, especially on
ARM. On x86-64 it looks like big bulk blits get a little slower and
small mask blits get a little faster.
Quality looks good, and maybe improved for 565.
There are fewer platform-specific differences now in _lowp, and I think
they're few enough now that we could even consider completing the
unification by folding the 8-bit and float code together. Rename
"div255()" to "rebias()", slap on a few coats of paint...
Guarded for Chrome with SK_JUMPER_LEGACY_LOWP.
Change-Id: I36309c07cf736f3cb31952cca66030ad56026318
Reviewed-on: https://skia-review.googlesource.com/45982
Reviewed-by: Herb Derby <herb@google.com>
Reviewed-by: Florin Malita <fmalita@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/jumper/SkJumper.h')
-rw-r--r-- | src/jumper/SkJumper.h | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/jumper/SkJumper.h b/src/jumper/SkJumper.h index 32b56450ff..52e8e09f08 100644 --- a/src/jumper/SkJumper.h +++ b/src/jumper/SkJumper.h @@ -51,12 +51,8 @@ #endif // When compiled with Clang on ARM, we'll have 8-bit NEON stages. -#if defined(__clang__) - #if defined(__aarch64__) - #define JUMPER_HAS_NEON_8BIT - #elif defined(__arm__) && defined(__ARM_NEON__) - #define JUMPER_HAS_NEON_8BIT - #endif +#if defined(__clang__) && defined(__ARM_NEON) + #define JUMPER_HAS_NEON_LOWP #endif static const int SkJumper_kMaxStride = 8; |