[PATCH] D118061: [InstCombine] fold one-bit set/clear icmp with mul operand

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 12:15:51 PST 2022


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

LGTM, thanks.
The proof looks correct to me, but a fear that we are missing things due to extra artificial uses due to the preconditions has struck me before.



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCompares.cpp:1847
                                                    const APInt &C) {
   if (Instruction *I = foldICmpAndConstConst(Cmp, And, C))
     return I;
----------------
This fold should probably be in `foldICmpAndConstConst()`


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

https://reviews.llvm.org/D118061



More information about the llvm-commits mailing list