commit | 8b0f9d18ddf8af2ae620502af8ef4f64864ddaa5 | [log] [tgz] |
---|---|---|
author | Mike Klein <mtklein@google.com> | Thu Jan 03 11:26:57 2019 -0500 |
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Thu Jan 03 17:17:56 2019 +0000 |
tree | 829d5ea3d18e3a66ab33076c98a800f3a924d70e | |
parent | 3b7e65107f671e35674ab8bfc163d998039b81bf [diff] |
tear down one-big-switch SkRasterPipeline backend This was a pretty serious performance regression, especially on ARM. Code size was wash, almost exactly 1:1. While I like this one-big-switch approach in terms of maintainability and platform obliviousness, I'm not willing to take such a big hit. This'll just have to stay platform-aware. (The old code is also easier to debug and profile, but that's peanuts.) https://perf.skia.org/e/?begin=1546440652&end=1546459404&keys=X2ec5a42b0071f8848a5875f14400493d&xbaroffset=39028 Cq-Include-Trybots: luci.chromium.try:linux-blink-rel Change-Id: Ia47fd25bbe4145468efd67cbddef7e6f92de3f02 Reviewed-on: https://skia-review.googlesource.com/c/180940 Reviewed-by: Mike Klein <mtklein@google.com> Commit-Queue: Mike Klein <mtklein@google.com>