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

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 16:44:32 PST 2017


On Fri, Jan 27, 2017 at 5:12 PM, Davide Italiano <davide at freebsd.org> wrote:
> 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.

David, what do you think as instcombine owner?


More information about the llvm-commits mailing list