[PATCH] D96119: [Codegenprepare][X86] Use usub with overflow opt for IV increment

Mon Feb 8 11:38:36 PST 2021

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

In D96119#2547901 <https://reviews.llvm.org/D96119#2547901>, @mkazantsev wrote:

> I've filed https://bugs.llvm.org/show_bug.cgi?id=49087 to address it and merged relevant test as 0fc1738eb75d613b9e16143b83e7cb80512e84eb <https://reviews.llvm.org/rG0fc1738eb75d613b9e16143b83e7cb80512e84eb>. It fails even on the test not affected by the patch on current trunc, so I think it should not be an obstacle on this patch's way.

Thanks for filing the bug!
LGTM

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

https://reviews.llvm.org/D96119