From 05227d88aff1223cdaa28ff401a3c88b36d285eb Mon Sep 17 00:00:00 2001 From: Reid Spencer Date: Mon, 19 Mar 2007 21:29:50 +0000 Subject: [PATCH] APIntify the isHighOnes utility function. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@35190 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/InstructionCombining.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 1d44f6b9840..c87a351e63b 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -3491,14 +3491,15 @@ static bool isLowOnes(const ConstantInt *CI) { // isHighOnes - Return true if the constant is of the form 1+0+. // This is the same as lowones(~X). static bool isHighOnes(const ConstantInt *CI) { - uint64_t V = ~CI->getZExtValue(); - if (~V == 0) return false; // 0's does not match "1+" + if (CI->getValue() == 0) return false; // 0's does not match "1+" + + APInt V(~CI->getValue()); // There won't be bits set in parts that the type doesn't contain. - V &= ConstantInt::getAllOnesValue(CI->getType())->getZExtValue(); + V &= APInt::getAllOnesValue(CI->getType()->getBitWidth()); - uint64_t U = V+1; // If it is low ones, this should be a power of two. - return U && V && (U & V) == 0; + APInt U(V+1); // If it is low ones, this should be a power of two. + return (U!=0) && (V!=0) && (U & V) == 0; } /// getICmpCode - Encode a icmp predicate into a three bit mask. These bits