[ValueTracking] Check likely to be canonicalized constant operands first in ; NFC

[goldstein.w.n]

Summary

Checking operands that a likely to be canonicalized constants first makes sense
from a compile time perspective as it puts whats expected to be a cheaper check first.

[nikic]

llvm/lib/Analysis/ValueTracking.cpp

2108

Does this canonicalization actually exist?

[goldstein.w.n]

llvm/lib/Analysis/ValueTracking.cpp

2108

No. Happy to drop but think its not difference either way.

[nikic]

LGTM

[goldstein.w.n]

Drop changes to select

llvm/lib/Analysis/ValueTracking.cpp

2108

I dropped the select changes.

[goldstein.w.n]

Rebase before isKnownNonZero patch

llvm/lib/Analysis/ValueTracking.cpp

case Instruction::UDiv:
     return false;
   case Instruction::Mul:
     return OrZero &&
            isKnownToBeAPowerOfTwo(I->getOperand(1), OrZero, Depth, Q) &&
            isKnownToBeAPowerOfTwo(I->getOperand(0), OrZero, Depth, Q);
   case Instruction::And:
     if (OrZero) {
       // A power of two and'd with anything is a power of two or zero.
-      if (isKnownToBeAPowerOfTwo(I->getOperand(0), /*OrZero*/ true, Depth, Q) ||
-          isKnownToBeAPowerOfTwo(I->getOperand(1), /*OrZero*/ true, Depth, Q))
+      if (isKnownToBeAPowerOfTwo(I->getOperand(1), /*OrZero*/ true, Depth, Q) ||
+          isKnownToBeAPowerOfTwo(I->getOperand(0), /*OrZero*/ true, Depth, Q))
         return true;
       // X & (-X) is always a power of two or zero.
       if (match(I->getOperand(0), m_Neg(m_Specific(I->getOperand(1)))) ||
           match(I->getOperand(1), m_Neg(m_Specific(I->getOperand(0)))))
         return true;
     }
     return false;
   case Instruction::Add: {
if (OrZero || Q.IIQ.hasNoUnsignedWrap(VOBO) ||
         if (OrZero || RHSBits.One.getBoolValue() || LHSBits.One.getBoolValue())
           return true;
     }
     return false;
   }
   case Instruction::Select:
     return isKnownToBeAPowerOfTwo(I->getOperand(1), OrZero, Depth, Q) &&
            isKnownToBeAPowerOfTwo(I->getOperand(2), OrZero, Depth, Q);
   case Instruction::PHI: {
     // A PHI node is power of two if all incoming values are power of two, or if
     // it is an induction variable where in each step its value is a power of
     // two.
     auto *PN = cast<PHINode>(I);
     SimplifyQuery RecQ = Q;
 
     // Check if it is an induction variable and always power of two.
     if (isPowerOfTwoRecurrence(PN, OrZero, Depth, RecQ))