Planar ramp() for S32 linear gradients

Planar ramp() for S32 linear gradients

R=mtklein@google.com,reed@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1825333003

Committed: https://skia.googlesource.com/skia/+/83aa9208dab96f1b6affa8167522e9a94afd70a5

[f(malita), 2016-03-23 18:10:04]

Description was changed from

==========
Planar ramp() for S32 linear gradients

R=mtklein@google.com,reed@google.com
==========

to

==========
Planar ramp() for S32 linear gradients

R=mtklein@google.com,reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[f(malita), 2016-03-23 18:19:10]

Speeds up GM:gradient_many_stops_4f by ~10% in S32 mode.

[f(malita), 2016-03-23 18:19:16]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-23 18:19:26]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1825333003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1825333003/20001

[mtklein, 2016-03-23 18:27:27]

lgtm

[commit-bot: I haz the power, 2016-03-23 18:34:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-23 18:34:39]

Dry run: This issue passed the CQ dry run.

[mtklein, 2016-03-23 19:05:27]

https://codereview.chromium.org/1825333003/diff/20001/src/effects/gradients/S...
File src/effects/gradients/Sk4fLinearGradient.cpp (right):

https://codereview.chromium.org/1825333003/diff/20001/src/effects/gradients/S...
src/effects/gradients/Sk4fLinearGradient.cpp:77: ::store(Sk4f(c4x.r[n & 2],
c4x.g[n & 2], c4x.b[n & 2], c4x.a[n & 2]), dst);
Wait, walk me through this one again?

[mtklein, 2016-03-23 19:12:57]

https://codereview.chromium.org/1825333003/diff/20001/src/effects/gradients/S...
File src/effects/gradients/Sk4fLinearGradient.cpp (right):

https://codereview.chromium.org/1825333003/diff/20001/src/effects/gradients/S...
src/effects/gradients/Sk4fLinearGradient.cpp:77: ::store(Sk4f(c4x.r[n & 2],
c4x.g[n & 2], c4x.b[n & 2], c4x.a[n & 2]), dst);
On 2016/03/23 19:05:27, mtklein wrote:
> Wait, walk me through this one again?

ok, cute!

[f(malita), 2016-03-23 19:27:17]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-03-23 19:27:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1825333003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1825333003/20001

[commit-bot: I haz the power, 2016-03-23 19:28:16]

Description was changed from

==========
Planar ramp() for S32 linear gradients

R=mtklein@google.com,reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Planar ramp() for S32 linear gradients

R=mtklein@google.com,reed@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/83aa9208dab96f1b6affa8167522e9a94afd70a5
==========

[commit-bot: I haz the power, 2016-03-23 19:28:17]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/83aa9208dab96f1b6affa8167522e9a94afd70a5