diff options
| author |  mtklein <mtklein@chromium.org> | 2016-07-29 10:10:15 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-07-29 10:10:15 -0700 |
| commit | d05a8752738f84b0115678b3cdad89237173e904 (patch) | |
| tree | 65fa13ac0eec5bad711f84758eb98ff271c9b6ce /src/core/SkRasterPipelineBlitter.cpp | |
| parent | 4ef6dfa7089c092c67b0d5ec34e89c1e319af196 (diff) | |
SkNx: add Sk4u
This lets us get at logical >> in a nicely principled way.
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2197683002
CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Review-Url: https://codereview.chromium.org/2197683002
Diffstat (limited to 'src/core/SkRasterPipelineBlitter.cpp')
| -rw-r--r-- | src/core/SkRasterPipelineBlitter.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/core/SkRasterPipelineBlitter.cpp b/src/core/SkRasterPipelineBlitter.cpp index a27abbdc38..f3fc76ee79 100644 --- a/src/core/SkRasterPipelineBlitter.cpp +++ b/src/core/SkRasterPipelineBlitter.cpp @@ -297,8 +297,7 @@ static void SK_VECTORCALL load_d_srgb(SkRasterPipeline::Stage* st, size_t x, sk_linear_from_srgb[(ptr[2] >> SK_B32_SHIFT) & 0xff], sk_linear_from_srgb[(ptr[3] >> SK_B32_SHIFT) & 0xff] }; - // TODO: this >> doesn't really need mask if we make it logical instead of arithmetic. - da = SkNx_cast<float>((Sk4i::Load(ptr) >> SK_A32_SHIFT) & 0xff) * (1/255.0f); + da = SkNx_cast<float>(Sk4u::Load(ptr) >> SK_A32_SHIFT) * (1/255.0f); st->next(x, r,g,b,a, dr,dg,db,da); } |