From 01cecbffd677df87fceb1675ce6bd29056f7421e Mon Sep 17 00:00:00 2001 From: Chad Rosier Date: Thu, 15 Mar 2012 21:40:23 +0000 Subject: [PATCH] [fast-isel] Don't try to encode LONG_MIN using cmn instructions. rdar://11038907 llvm-svn: 152847 --- llvm/lib/Target/ARM/ARMFastISel.cpp | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/llvm/lib/Target/ARM/ARMFastISel.cpp b/llvm/lib/Target/ARM/ARMFastISel.cpp index 71de52fcaaa4..4651a3ac929e 100644 --- a/llvm/lib/Target/ARM/ARMFastISel.cpp +++ b/llvm/lib/Target/ARM/ARMFastISel.cpp @@ -1384,12 +1384,16 @@ bool ARMFastISel::ARMEmitCmp(const Value *Src1Value, const Value *Src2Value, SrcVT == MVT::i1) { const APInt &CIVal = ConstInt->getValue(); Imm = (isZExt) ? (int)CIVal.getZExtValue() : (int)CIVal.getSExtValue(); - if (Imm < 0) { - isNegativeImm = true; - Imm = -Imm; + // We can't encode LONG_MIN (i.e., 0x80000000) as an immediate because + // there is no way to represent 2147483648 as a signed 32-bit int. + if (Imm != (int)0x80000000) { + if (Imm < 0) { + isNegativeImm = true; + Imm = -Imm; + } + UseImm = isThumb2 ? (ARM_AM::getT2SOImmVal(Imm) != -1) : + (ARM_AM::getSOImmVal(Imm) != -1); } - UseImm = isThumb2 ? (ARM_AM::getT2SOImmVal(Imm) != -1) : - (ARM_AM::getSOImmVal(Imm) != -1); } } else if (const ConstantFP *ConstFP = dyn_cast(Src2Value)) { if (SrcVT == MVT::f32 || SrcVT == MVT::f64)