[llvm] r293345 - [InstCombine] move icmp transforms that might be recognized as min/max and inf-loop (PR31751)

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 27 17:12:04 PST 2017


On Fri, Jan 27, 2017 at 5:11 PM, Davide Italiano <davide at freebsd.org> wrote:
> On Fri, Jan 27, 2017 at 5:02 PM, Sanjay Patel <spatel at rotateright.com> wrote:
>> Understood - my POV is from trunk.
>>
>> If people think it's safer to revert patches for 4.0, I'd suggest reverting
>> everything that was added to value tracking's matchMinMax() since 3.9. That
>> should remove doubt (but miss quite a few min/max improvements).
>>
>
> I'm afraid that's not really feasible. At least this patch should be
> merged back to 4.0, FWIW.
>

+ Hans.


-- 
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare


More information about the llvm-commits mailing list