diff options
author | mtklein <mtklein@chromium.org> | 2016-01-22 07:42:53 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-22 07:42:53 -0800 |
commit | 8bf7b79cf9776b4edb3f6810e5ab8c80c49d3480 (patch) | |
tree | 65f114dce236fffbbc0e4605a6f1223523281e41 /bench/SwizzleBench.cpp | |
parent | 6c9cd55f00beeba3ded3f28bcbdd6ef030c4dac7 (diff) |
Refactor swizzle names and types.
- Plant a flag to say "pretend all the inputs are RGBA".
This is how libpng thinks.
This is the opposite of what the implementation had been doing,
so I've rearranged everything to reflect the new orientation.
- Rewrite the names to be less mysterious looking. No more Xs.
- Make the src type uniformly const void*, to allow for 888 (RGB) srcs.
This should be performance and pixel neutral. (Please revert if it's not.)
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1626463002
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Review URL: https://codereview.chromium.org/1626463002
Diffstat (limited to 'bench/SwizzleBench.cpp')
-rw-r--r-- | bench/SwizzleBench.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/bench/SwizzleBench.cpp b/bench/SwizzleBench.cpp index 13b2003f9a..922c276dbc 100644 --- a/bench/SwizzleBench.cpp +++ b/bench/SwizzleBench.cpp @@ -10,7 +10,7 @@ class SwizzleBench : public Benchmark { public: - SwizzleBench(const char* name, SkOpts::Swizzle_8888_8888 fn) : fName(name), fFn(fn) {} + SwizzleBench(const char* name, SkOpts::Swizzle_8888 fn) : fName(name), fFn(fn) {} bool isSuitableFor(Backend backend) override { return backend == kNonRendering_Backend; } const char* onGetName() override { return fName; } @@ -23,10 +23,10 @@ public: } private: const char* fName; - SkOpts::Swizzle_8888_8888 fFn; + SkOpts::Swizzle_8888 fFn; }; -DEF_BENCH(return new SwizzleBench("SkOpts::premul_xxxa", SkOpts::premul_xxxa)); -DEF_BENCH(return new SwizzleBench("SkOpts::swaprb_xxxa", SkOpts::swaprb_xxxa)); -DEF_BENCH(return new SwizzleBench("SkOpts::premul_swaprb_xxxa", SkOpts::premul_swaprb_xxxa)); +DEF_BENCH(return new SwizzleBench("SkOpts::RGBA_to_rgbA", SkOpts::RGBA_to_rgbA)); +DEF_BENCH(return new SwizzleBench("SkOpts::RGBA_to_bgrA", SkOpts::RGBA_to_bgrA)); +DEF_BENCH(return new SwizzleBench("SkOpts::RGBA_to_BGRA", SkOpts::RGBA_to_BGRA)); |