diff options
author | Bob Wilson <bob.wilson@apple.com> | 2012-03-20 19:28:25 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2012-03-20 19:28:25 +0000 |
commit | aaa1e2f820e61a2f4b878d97599b3ca093bc96ed (patch) | |
tree | ffa6b0424bcd8b730714a273c6f5590484759e7d /lib/Target | |
parent | 055a8127c9ffee287807fe7cc1b115d0f40162b0 (diff) | |
download | external_llvm-aaa1e2f820e61a2f4b878d97599b3ca093bc96ed.zip external_llvm-aaa1e2f820e61a2f4b878d97599b3ca093bc96ed.tar.gz external_llvm-aaa1e2f820e61a2f4b878d97599b3ca093bc96ed.tar.bz2 |
Require a base pointer for stack realignment when SP may vary dynamically.
ARMBaseRegisterInfo::canRealignStack was checking for variable-sized objects
but not for stack adjustments around calls. Use hasReservedCallFrame() to
check for both. The hasBasePointer function was already correctly checking
both conditions, so the effect of this was that a base pointer would be used
without checking whether the base pointer register could be reserved. I don't
have a small testcase for this.
<rdar://problem/11075906>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@153110 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target')
-rw-r--r-- | lib/Target/ARM/ARMBaseRegisterInfo.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/Target/ARM/ARMBaseRegisterInfo.cpp b/lib/Target/ARM/ARMBaseRegisterInfo.cpp index 8fd77e5..202da9b 100644 --- a/lib/Target/ARM/ARMBaseRegisterInfo.cpp +++ b/lib/Target/ARM/ARMBaseRegisterInfo.cpp @@ -532,8 +532,9 @@ bool ARMBaseRegisterInfo::canRealignStack(const MachineFunction &MF) const { // register allocation with frame pointer elimination, it is too late now. if (!MRI->canReserveReg(FramePtr)) return false; - // We may also need a base pointer if there are dynamic allocas. - if (!MFI->hasVarSizedObjects()) + // We may also need a base pointer if there are dynamic allocas or stack + // pointer adjustments around calls. + if (MF.getTarget().getFrameLowering()->hasReservedCallFrame(MF)) return true; if (!EnableBasePointer) return false; |