diff options
author | Chad Rosier <mcrosier@apple.com> | 2013-02-16 01:25:28 +0000 |
---|---|---|
committer | Chad Rosier <mcrosier@apple.com> | 2013-02-16 01:25:28 +0000 |
commit | b56606274d43c7a3e01b18a08d1115fbf2889996 (patch) | |
tree | ea89f8db120a3b54de9e5e6d3ff083272ed9ebd6 /lib/Target/X86 | |
parent | 42edeb1ba8aabcbe0e5cc846d9e5f2a9e2261292 (diff) | |
download | external_llvm-b56606274d43c7a3e01b18a08d1115fbf2889996.zip external_llvm-b56606274d43c7a3e01b18a08d1115fbf2889996.tar.gz external_llvm-b56606274d43c7a3e01b18a08d1115fbf2889996.tar.bz2 |
[ms-inline asm] Do not omit the frame pointer if we have ms-inline assembly.
If the frame pointer is omitted, and any stack changes occur in the inline
assembly, e.g.: "pusha", then any C local variable or C argument references
will be incorrect.
I pass no judgement on anyone who would do such a thing. ;)
rdar://13218191
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@175334 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/X86')
-rw-r--r-- | lib/Target/X86/X86FrameLowering.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/Target/X86/X86FrameLowering.cpp b/lib/Target/X86/X86FrameLowering.cpp index 84b1c10..950fd39 100644 --- a/lib/Target/X86/X86FrameLowering.cpp +++ b/lib/Target/X86/X86FrameLowering.cpp @@ -50,7 +50,7 @@ bool X86FrameLowering::hasFP(const MachineFunction &MF) const { return (MF.getTarget().Options.DisableFramePointerElim(MF) || RegInfo->needsStackRealignment(MF) || MFI->hasVarSizedObjects() || - MFI->isFrameAddressTaken() || + MFI->isFrameAddressTaken() || MF.hasMSInlineAsm() || MF.getInfo<X86MachineFunctionInfo>()->getForceFramePointer() || MMI.callsUnwindInit() || MMI.callsEHReturn()); } |