[PATCH] D114272: [InstCombine] Add two optimizations for mul-and-icmp patterns

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

The test coverage still seems lacking to me, and I'm not seeing why we need to create a new multiply. If we assume the right constraints, the first transform should be only a low-bit check? 
https://alive2.llvm.org/ce/z/YB_4pn

I'll add some tests and post a draft that handles that part of the problem.


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

https://reviews.llvm.org/D114272