diff options
| author |  fmalita <fmalita@chromium.org> | 2016-02-03 05:44:21 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-02-03 05:44:21 -0800 |
| commit | 2404f03413bc9b4fa3f556c64642e57507092d8d (patch) | |
| tree | de08b8a947abd698523fb3712b27edf265cd4d59 /src/opts/SkBitmapProcState_opts_SSE2.h | |
| parent | 15691a055db9b68c9b48f589e48d8a85888cf83f (diff) | |
Use SkBitmapProcStateAutoMapper for filter samplers also
Observation: filter procs are also biased by s.fFilterOne{X,Y} / 2. They all do
something along these lines:
s.fInvProc(s.fInvMatrix,
SkIntToScalar(x) + SK_ScalarHalf,
SkIntToScalar(y) + SK_ScalarHalf, &srcPt);
SkFixed fx = SkScalarToFixed(srcPt.fX) - (s.fFilterOneX >> 1);
SkFixed fy = SkScalarToFixed(srcPt.fY) - (s.fFilterOneX >> 1);
It's trivial to extend SkBitmapProcStateAutoMapper to handle this internally, and
convert everyone off explicit mapping.
R=reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1661613002
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot
Review URL: https://codereview.chromium.org/1661613002
Diffstat (limited to 'src/opts/SkBitmapProcState_opts_SSE2.h')
0 files changed, 0 insertions, 0 deletions