diff options
| author |  mtklein <mtklein@chromium.org> | 2015-08-27 06:46:03 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-08-27 06:46:03 -0700 |
| commit | 435af2f736c85c3274a0c6760a3523810750d237 (patch) | |
| tree | d33d679033b2c9ae91a38f53a3d69c0b4e875724 /bench/BenchLogger.cpp | |
| parent | fe495f0bae96c732730a7d6b2667662c4d67bfa1 (diff) | |
SkColorCubeFilter_opts: start with a statically-initializable zero.
SkPMFloat(0) and SkPMFloat(0,0,0,0) end up with the same value,
but the first goes through math to get there. The second is a lot more
transparent to the compiler, and should compile all the way down to
just `xorps xmmN,xmmN` or even be optimized away.
Didn't measure any additional benefit from hoisting the zero outside
the loop and writing `SkPMFloat color = zero;`.
Perf win is <2%.
BUG=skia:
Review URL: https://codereview.chromium.org/1314763007
Diffstat (limited to 'bench/BenchLogger.cpp')
0 files changed, 0 insertions, 0 deletions