diff options
| author |  Mike Klein <mtklein@chromium.org> | 2016-10-19 21:05:17 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2016-10-20 16:59:23 +0000 |
| commit | 2878e76247fd335dee755cfd4c9cd026804f9655 (patch) | |
| tree | f99ff1d17f73373b503a0e4fce1b41dc54b44d9b /src/opts/SkOpts_hsw.cpp | |
| parent | d2fe3bce079671b0210d5c7cddcc712988d0b686 (diff) | |
SkRasterPipeline refactor
- Give body and tail functions separate types. This frees a register in body functions, especially important for Windows.
- Fill out default, SSE4.1, and HSW versions of all functions. This means we don't have to mess around with SkNf_abi... all functions come from the same compilation unit where SkNf is a single consistent type.
- Move Stage::next() into SkRasterPipeline_opts.h as a static inline function.
- Remove Stage::ctx() entirely... fCtx is literally the same thing.
This is a step along the way toward building the entire pipeline in src/opts, removing the need for all the stages to be functions living in SkOpts.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3680
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot,Test-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-ASAN-Trybot
Change-Id: I7de78ffebc15b9bad4eda187c9f50369cd7e5e42
Reviewed-on: https://skia-review.googlesource.com/3680
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/opts/SkOpts_hsw.cpp')
| -rw-r--r-- | src/opts/SkOpts_hsw.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/opts/SkOpts_hsw.cpp b/src/opts/SkOpts_hsw.cpp index ecbb3e72b5..3c5d19d47c 100644 --- a/src/opts/SkOpts_hsw.cpp +++ b/src/opts/SkOpts_hsw.cpp @@ -35,11 +35,6 @@ namespace SkOpts { STAGE(lerp_u8); STAGE(lerp_565); - #undef STAGE - - #define STAGE(stage) \ - body[SkRasterPipeline::stage] = (SkOpts::VoidFn)SK_OPTS_NS::stage; \ - tail[SkRasterPipeline::stage] = (SkOpts::VoidFn)SK_OPTS_NS::stage STAGE(just_return); STAGE(swap_src_dst); |