diff options
| author |  Mike Klein <mtklein@chromium.org> | 2017-09-27 11:08:02 -0400 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2017-09-27 15:40:58 +0000 |
| commit | aaab3c73575d5502ae345dd71cf8748c2070ffda (patch) | |
| tree | 96dd80d8b00f9a5c3869553b6c1b25f5de9a6a1e /src | |
| parent | 0e4d096051f5861f8b2f8fc9097dd25b74674bd3 (diff) | |
remove __attribute__((disable_tail_calls))
This is a no-op in terms of generated code.
There is no longer a tail call here to be disabled,
not since we changed start_pipeline() to operate in 2D.
Change-Id: Ife92590eb059e28e4a84e3729180c7410a93b410
Reviewed-on: https://skia-review.googlesource.com/52020
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/jumper/SkJumper_stages.cpp | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/jumper/SkJumper_stages.cpp b/src/jumper/SkJumper_stages.cpp index f8e454d3ab..c9896653c6 100644 --- a/src/jumper/SkJumper_stages.cpp +++ b/src/jumper/SkJumper_stages.cpp @@ -55,11 +55,6 @@ static const size_t N = sizeof(F) / sizeof(float); using Stage = void(size_t tail, void** program, size_t x, size_t y, F,F,F,F, F,F,F,F); #endif -#if defined(JUMPER_IS_AVX) || defined(JUMPER_IS_AVX2) - // We really want to make sure all paths go through this function's (implicit) vzeroupper. - // If they don't, we'll experience severe slowdowns when we first use SSE instructions again. - __attribute__((disable_tail_calls)) -#endif MAYBE_MSABI extern "C" void WRAP(start_pipeline)(size_t x, size_t y, size_t xlimit, size_t ylimit, void** program) { |