PEI: Support for non-zero SPAdj at beginning of a basic block.
authorManman Ren <mren@apple.com>
Mon, 15 Jul 2013 23:47:29 +0000 (23:47 +0000)
committerManman Ren <mren@apple.com>
Mon, 15 Jul 2013 23:47:29 +0000 (23:47 +0000)
commit519127f758fc4515adb06d52a4c2c1ae1a968879
treea24664b8e708c20707181dcad240d765a111a2e2
parent135e81efe3c1848a308c96dfd65e4d88b0d8667b
PEI: Support for non-zero SPAdj at beginning of a basic block.

We can have a FrameSetup in one basic block and the matching FrameDestroy
in a different basic block when we have struct byval. In that case, SPAdj
is not zero at beginning of the basic block.

Modify PEI to correctly set SPAdj at beginning of each basic block using
DFS traversal. We used to assume SPAdj is 0 at beginning of each basic block.

PEI had an assert SPAdjCount || SPAdj == 0.
If we have a Destroy <n> followed by a Setup <m>, PEI will assert failure.
We can add an extra condition to make sure the pairs are matched:
  The pairs start with a FrameSetup.
But since we are doing a much better job in the verifier, this patch removes
the check in PEI.

PR16393

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186364 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/PrologEpilogInserter.cpp
test/CodeGen/ARM/2013-06-03-ByVal-2Kbytes.ll [new file with mode: 0644]
test/CodeGen/ARM/struct-byval-frame-index.ll [new file with mode: 0644]