diff options
| author |  msarett <msarett@google.com> | 2016-07-22 06:23:00 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-07-22 06:23:00 -0700 |
| commit | a5ba329c81dd6e21f8d610b56416d9f45259fa42 (patch) | |
| tree | 86531b71d6ee4a608e765f85daa32fc3f700abfc /src/gpu/SkGr.cpp | |
| parent | bddb372ae9d2d7953defd3830b4e1fabd045c671 (diff) | |
Add clamp to sk_linear_to_srgb, reorder instructions
Improves performance for xforms toSRGB and to2Dot2. Seems
more optimal to save clamping until the end. That way we
don't stall the mul pipeline with a min/max.
toSRGB: 371us -> 346us
to2Dot2: 404us -> 387us
FWIW, it probably makes sense to clamp inside
sk_linear_to_srgb anyway. If not, we should potentially
provide two versions (one that clamps and one that
doesn't).
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2173803002
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/2173803002
Diffstat (limited to 'src/gpu/SkGr.cpp')
0 files changed, 0 insertions, 0 deletions