[X86] Fix emitPrologue() to make less assumptions about pushes
authorMichael Kuperstein <michael.m.kuperstein@intel.com>
Thu, 16 Jul 2015 12:27:59 +0000 (12:27 +0000)
committerMichael Kuperstein <michael.m.kuperstein@intel.com>
Thu, 16 Jul 2015 12:27:59 +0000 (12:27 +0000)
commit72400f8d50869f50a54b67146b17acaab0fcb16f
treef938e5748ec7fea1b8cc837fb4251ec3ee4c35b2
parent4fceffb0d6cc4ee7fa066270dddbc780b5382465
[X86] Fix emitPrologue() to make less assumptions about pushes

When X86FrameLowering::emitPrologue() looks for where to insert the %esp subtraction
to allocate stack space for local allocations, it assumes that any sequence of push
instructions that starts at function entry consists purely of spills of callee-save
registers.
This may be false, since from some point forward, the pushes may pushing arguments
to a subsequent function call.

This caused a miscompile that was exposed by r240257, and is not easily testable
since r240257 was reverted. A test will be committed separately after r240257 is
reapplied.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@242395 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/X86/X86FrameLowering.cpp