[PATCH] D38531: Improve clamp recognition in ValueTracking.

[Artur Gainullin via Phabricator via llvm-commits]

ArturGainullin added a comment.

Created separate patch to move folding of cmp instruction: https://reviews.llvm.org/D38934. I will be able to update this differential revision when it will be committed.
I have renamed function so that we will be able to add an smax sibling for this fold with separate commit.

As far as I understood efriedma proposes to implement more general transformation (changed it a little bit so that it is more evident that it is related to current transformation):

Precondition: C2 s> C1

  icmp sgt smin(a, C2), C1 -> icmp sgt a, C1  

https://rise4fun.com/Alive/oFF
I agree that this should be done as a separate commit.


https://reviews.llvm.org/D38531