diff options
author | Mike Klein <mtklein@chromium.org> | 2017-01-07 22:04:51 +0000 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-01-07 22:05:05 +0000 |
commit | 86d55b312a2649d80890ccf75f24571ada0265f1 (patch) | |
tree | 8e25adaa99e9dfb8f7d0f88cc4c00099b7b04e8a /src/opts | |
parent | c3063e54cb1274359f36d9720d4c1b3acf37c203 (diff) |
Revert "Retry "SkRasterPipelineBlitter: support A8"..."
This reverts commit f55ea6a1deb21120944d406124a2984b5009260a.
Reason for revert: crbug.com/679147
Original change's description:
> Retry "SkRasterPipelineBlitter: support A8"...
>
> ...preferring SkA8_Coverage_Blitter over SkRasterPipelineBlitter.
>
> I think we could make this work with SkRasterPipelineBlitter (tell it, draw white in Src mode with this mask), but the existing blitter is pretty hard to beat in efficiency and correctness.
>
> CQ_INCLUDE_TRYBOTS=skia.primary:Perf-Ubuntu-Clang-GCE-CPU-AVX2-x86_64-Debug-MSAN,Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD
>
>
> Change-Id: I72df9995c63f3334d8111c59711818cb5ed1e63c
> Reviewed-on: https://skia-review.googlesource.com/6627
> Reviewed-by: Mike Klein <mtklein@chromium.org>
> Commit-Queue: Mike Klein <mtklein@chromium.org>
>
TBR=mtklein@chromium.org,brianosman@google.com,reed@google.com,reviews@skia.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Change-Id: I6a36b4c087a52e54f4d591ded40e6a202fb77068
Reviewed-on: https://skia-review.googlesource.com/6760
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src/opts')
-rw-r--r-- | src/opts/SkNx_sse.h | 4 | ||||
-rw-r--r-- | src/opts/SkRasterPipeline_opts.h | 10 |
2 files changed, 0 insertions, 14 deletions
diff --git a/src/opts/SkNx_sse.h b/src/opts/SkNx_sse.h index 78b6d39030..3dc60f746f 100644 --- a/src/opts/SkNx_sse.h +++ b/src/opts/SkNx_sse.h @@ -650,10 +650,6 @@ public: hi = _mm256_extractf128_si256(src.fVec, 1); return _mm_packus_epi32(lo, hi); } - template<> AI /*static*/ Sk8b SkNx_cast<uint8_t>(const Sk8i& src) { - auto _16 = SkNx_cast<uint16_t>(src); - return _mm_packus_epi16(_16.fVec, _16.fVec); - } #endif diff --git a/src/opts/SkRasterPipeline_opts.h b/src/opts/SkRasterPipeline_opts.h index 3fe43e9838..143a9ece79 100644 --- a/src/opts/SkRasterPipeline_opts.h +++ b/src/opts/SkRasterPipeline_opts.h @@ -497,16 +497,6 @@ STAGE_CTX(lerp_565, const uint16_t**) { a = 1.0f; } -STAGE_CTX(load_a8, const uint8_t**) { - auto ptr = *ctx + x; - r = g = b = 0.0f; - a = SkNf_from_byte(load(tail, ptr)); -} -STAGE_CTX(store_a8, uint8_t**) { - auto ptr = *ctx + x; - store(tail, SkNx_cast<uint8_t>(SkNf_round(255.0f, a)), ptr); -} - STAGE_CTX(load_565, const uint16_t**) { auto ptr = *ctx + x; from_565(load(tail, ptr), &r,&g,&b); |