Make sure to handle small negative values hiding as large unsigned longs --
this is a common case created by the front-end.
llvm-svn: 4127
diff --git a/llvm/lib/Target/Sparc/SparcInstrInfo.cpp b/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
index 993f2a2..148b76e 100644
--- a/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
+++ b/llvm/lib/Target/Sparc/SparcInstrInfo.cpp
@@ -266,7 +266,7 @@
// Entry == 0 ==> no immediate constant field exists at all.
// Entry > 0 ==> abs(immediate constant) <= Entry
//
-vector<unsigned int> MaxConstantsTable(Instruction::NumOtherOps);
+vector<int> MaxConstantsTable(Instruction::NumOtherOps);
static int
MaxConstantForInstr(unsigned llvmOpCode)
@@ -343,13 +343,14 @@
return false;
if (const ConstantUInt* U = dyn_cast<ConstantUInt>(CV))
- return (U->getValue() > MaxConstantsTable[I->getOpcode()]);
+ /* Large unsigned longs may really just be small negative signed longs */
+ return (labs((int64_t) U->getValue()) > MaxConstantsTable[I->getOpcode()]);
if (const ConstantSInt* S = dyn_cast<ConstantSInt>(CV))
- return (labs(S->getValue()) > (int) MaxConstantsTable[I->getOpcode()]);
+ return (labs(S->getValue()) > MaxConstantsTable[I->getOpcode()]);
if (isa<ConstantBool>(CV))
- return (1U > MaxConstantsTable[I->getOpcode()]);
+ return (1 > MaxConstantsTable[I->getOpcode()]);
return true;
}
@@ -380,6 +381,11 @@
//
const Type* valType = val->getType();
+ // Unfortunate special case: a ConstantPointerRef is just a
+ // reference to GlobalValue.
+ if (isa<ConstantPointerRef>(val))
+ val = cast<ConstantPointerRef>(val)->getValue();
+
if (isa<GlobalValue>(val))
{
TmpInstruction* tmpReg =