When getting the number of bits necessary for addressing mode

ARMII::AddrModeT1_s, we need to take into account that if the frame register is
ARM::SP, then the number of bits is 8. If it's not ARM::SP, then the number of
bits is 5.

llvm-svn: 141529
This commit is contained in:
Bill Wendling 2011-10-10 07:24:23 +00:00
parent 251bbfb1f7
commit 94258753c7
2 changed files with 12 additions and 3 deletions

View File

@ -1109,11 +1109,20 @@ bool ARMBaseRegisterInfo::isFrameOffsetLegal(const MachineInstr *MI,
case ARMII::AddrMode3:
NumBits = 8;
break;
case ARMII::AddrModeT1_s:
NumBits = 5;
case ARMII::AddrModeT1_s: {
const MachineBasicBlock &MBB = *MI->getParent();
const MachineFunction &MF = *MBB.getParent();
unsigned FrameReg = ARM::SP;
if (MF.getFrameInfo()->hasVarSizedObjects())
// There are alloca()'s in this function, must reference off the frame
// pointer or base pointer instead.
FrameReg = (!hasBasePointer(MF) ?BasePtr : getFrameRegister(MF));
NumBits = (FrameReg == ARM::SP) ? 8 : 5;
Scale = 4;
isSigned = false;
break;
}
default:
llvm_unreachable("Unsupported addressing mode!");
break;

View File

@ -1,5 +1,5 @@
; RUN: llc < %s -march=thumb
; RUN: llc < %s -mtriple=thumb-linux | grep pop | count 2
; RUN: llc < %s -mtriple=thumb-linux | grep pop | count 1
; RUN: llc < %s -mtriple=thumb-darwin | grep pop | count 2
@str = internal constant [4 x i8] c"%d\0A\00" ; <[4 x i8]*> [#uses=1]