!fixup address comments, thanks!

Author: fhahn

llvm/lib/Analysis/ScalarEvolution.cpp

@@ -3220,11 +3220,10 @@ const SCEV *ScalarEvolution::getMulExpr(SmallVectorImpl<const SCEV *> &Ops,
       // D is a multiple of C2, and C1 is a multiple of C1.
       const SCEV *D;
       const SCEVConstant *C2;
-      APInt LHSV = LHSC->getAPInt();
+      const APInt &LHSV = LHSC->getAPInt();
       if (LHSV.isPowerOf2() &&
           match(Ops[1], m_scev_UDiv(m_SCEV(D), m_SCEVConstant(C2))) &&
-          C2->getAPInt().isPowerOf2() &&
-          LHSV.logBase2() >= C2->getAPInt().logBase2() &&
+          C2->getAPInt().isPowerOf2() && LHSV.uge(C2->getAPInt()) &&
           LHSV.logBase2() <= getMinTrailingZeros(D)) {
         return getMulExpr(getUDivExpr(LHSC, C2), D);
       }