diff options
author | Mike Klein <mtklein@chromium.org> | 2017-06-05 11:02:15 -0400 |
---|---|---|
committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-06-05 15:29:11 +0000 |
commit | eeccbf735deedb88313c480dde29bfd8193ee5f3 (patch) | |
tree | 6e6c4667bc16f6d29c31e63a32f6e1a0c9cb3a98 /.clang-format | |
parent | 42bb6acf56471e7308926e6b242fbbdd4103f306 (diff) |
Real fix for stage perf regression.
When we made start_pipeline() return void, the call into the tail!=0 run
of the pipeline became eligble to be a tail-call, and Clang made that
choice. This had the side effect of not going through vzeroupper on
those tails.
We now mark start_pipeline() as inelligible for tail calls when
targeting AVX+. All paths go through the vzeroupper at the end.
BUG=chromium:729237
Change-Id: I2099931284214f24c67b38979b3ad4b4d10e8bba
Reviewed-on: https://skia-review.googlesource.com/18591
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to '.clang-format')
0 files changed, 0 insertions, 0 deletions