[PATCH] D98152: [InstCombine] Canonicalize SPF to min/max intrinsics

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 23 07:51:41 PST 2022


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

In D98152#3340385 <https://reviews.llvm.org/D98152#3340385>, @nikic wrote:

> Rebase. The three remaining regressions are fixed now.

That was fast! I was going to look at the remaining bits, but too late. :)

Thank you for pushing it forward. LGTM.

For reference, these were at least some of the commits to handle the last round of missing folds:
e2f627e5e385 <https://reviews.llvm.org/rGe2f627e5e3855309f3a7421f6786b401efb6b7c7>
6777ec9e4df7 <https://reviews.llvm.org/rG6777ec9e4df79d88f179593e20d6cb58a3effba4>
5ccb0582c2b1 <https://reviews.llvm.org/rG5ccb0582c2b199913829d75a1dbbc866a707f400>
03e6efb8c23f <https://reviews.llvm.org/rG03e6efb8c23f489e45353b6b6d941628d9c49ca2>
587c7ff15c26 <https://reviews.llvm.org/rG587c7ff15c26d3a751fb6a10b2af60d4a25640c9>


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98152/new/

https://reviews.llvm.org/D98152



More information about the llvm-commits mailing list