Make this comment clearer. Instead of using an ambiguous ~ (not) on an icmp
authorNick Lewycky <nicholas@mxc.ca>
Sun, 17 Aug 2008 20:02:02 +0000 (20:02 +0000)
committerNick Lewycky <nicholas@mxc.ca>
Sun, 17 Aug 2008 20:02:02 +0000 (20:02 +0000)
predicate, swap the order of the operands.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@54907 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/InstructionCombining.cpp

index defad26f8a1c33098e3af7a528b083e256754ee9..a517d941c9a9614dc7aaa63cc3d9ffed371ec545 100644 (file)
@@ -5830,7 +5830,7 @@ Instruction *InstCombiner::visitICmpInstWithInstAndIntCst(ICmpInst &ICI,
                             ConstantInt::get(RHSV ^ SignBit));
       }
 
-      // (icmp u/s (xor A ~SignBit), C) -> (icmp ~s/u A, (xor C ~SignBit))
+      // (icmp u/s (xor A ~SignBit), C) -> (icmp s/u (xor C ~SignBit), A)
       if (!ICI.isEquality() && (~XorCST->getValue()).isSignBit()) {
         const APInt &NotSignBit = XorCST->getValue();
         ICmpInst::Predicate Pred = ICI.isSignedPredicate()