[PATCH] D149510: [ValueTracking] add UGT/UGE and lshr case in imply icmp

[Zhu Siyuan via Phabricator via llvm-commits]

floatshadow added a comment.

In D149510#4307842 <https://reviews.llvm.org/D149510#4307842>, @goldstein.w.n wrote:

> LGTM. I'm not a maintainer to please wait a day or so to push so other have a chance to see.

Thanks. I am not surely why `isImpliedCondOperands` handles less case only. Do we expect instcombine will canonicalize this?


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

https://reviews.llvm.org/D149510