Revert "Reorder saturation calculation for ClampAddOp and ClampSupOp"
This reverts commit ffc71c032540fc188f8b8104b1b26387d3e18a1b.
Reason for revert: Didn't improve the perf issue.
Original change's description:
> Reorder saturation calculation for ClampAddOp and ClampSupOp
>
> The order of operations changed in crrev.com/505494 with C++14
> constexpr optimizations. However, that change appears to have regressed
> performance in some cases. So, this CL tweaks the code to match the
> old structure and trigger the same compiler optimization heuristics.
>
> Bug: 770832
> Change-Id: I28f57ddf53825619eb73c9a92c663fc8463e16a7
> Reviewed-on: https://chromium-review.googlesource.com/696721
> Reviewed-by: Tom Sepez <tsepez@chromium.org>
> Commit-Queue: Justin Schuh <jschuh@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#506078}
TBR=jschuh@chromium.org,tsepez@chromium.org
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: 770832
Change-Id: I076c93215b422e321ad4bf39a72bab144cc0195d
Reviewed-on: https://chromium-review.googlesource.com/700775
Reviewed-by: Justin Schuh <jschuh@chromium.org>
Commit-Queue: Justin Schuh <jschuh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506559}
CrOS-Libchrome-Original-Commit: 95a989c109f41b856fba698bd0c96bfcd69f78d9
1 file changed