[PATCH] D59378: [InstCombine] Prevent icmp transform that can cause inf loop if part of min/max

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 14:07:31 PDT 2019


spatel added a comment.

In D59378#1433602 <https://reviews.llvm.org/D59378#1433602>, @nikic wrote:

> @spatel I've created D59506 <https://reviews.llvm.org/D59506> for the InstSimplify change.


Thanks! I think we can safely abandon this patch...until we hit the next infinite loop (although that extra regression test in this patch should still be committed).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59378





More information about the llvm-commits mailing list