[llvm] [InstCombine] Make the `(icmp eq/ne (and X, Y), X)` canonicalization work for non-const operands (PR #84688)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 10 13:00:54 PDT 2024


================
@@ -4651,6 +4651,22 @@ static Instruction *foldICmpAndXX(ICmpInst &I, const SimplifyQuery &Q,
   if (Pred == ICmpInst::ICMP_UGE)
     return new ICmpInst(ICmpInst::ICMP_EQ, Op0, Op1);
 
+  if (ICmpInst::isEquality(Pred) && Op0->hasOneUse()) {
+    // icmp (X & Y) eq/ne Y --> (X | ~Y) eq/ne -1 if Y is freely invertible and
+    // Y is non-constant. If Y is constant this form is preferable (and
+    // canonicalize too it elsewhere).
+    if (!match(Op1, m_ImmConstant()))
+      if (auto *NotOp1 = IC.getFreelyInverted(
+              Op1, Op1->hasOneUse() || Op1->hasNUses(2), &IC.Builder))
----------------
nikic wrote:

`!hasNUsesOrMore`.

https://github.com/llvm/llvm-project/pull/84688


More information about the llvm-commits mailing list