mirror of
https://github.com/RPCSX/llvm.git
synced 2024-12-05 02:16:46 +00:00
9a154bfe94
The PPCFrameLowering::determineFrameLayout routine currently ensures that every function that allocates a stack frame provides space for the parameter save area (via PPCFrameLowering::getMinCallFrameSize). This is actually not necessary. There may be functions that never call another routine but still allocate a frame; those do not require the parameter save area. In the future, with the ELFv2 ABI, even some routines that do call other functions do not need to allocate the parameter save area. While it is not a bug to allocate the parameter area when it is not needed, it is better to avoid it to save stack space. Note that when any particular function call requires the parameter save area, this space will already have been included by ABI code in the size the CALLSEQ_START insn is annotated with, and therefore included in the size returned by MFI->getMaxCallFrameSize(). This means that determineFrameLayout simply does not need to care about the parameter save area. (It still needs to ensure that every frame provides the linkage area.) This is implemented by this patch. Note that this exposed a bug in the new fast-isel code where the parameter area was *not* included in the CALLSEQ_START size; this is also fixed. A couple of test cases needed to be adapted for the new (smaller) stack frame size those tests now see. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@211495 91177308-0d34-0410-b5e6-96231b3b80d8
30 lines
1.0 KiB
LLVM
30 lines
1.0 KiB
LLVM
; RUN: llc < %s -march=ppc32 -mtriple=powerpc-apple-darwin8 -o %t1
|
|
; RUN: not grep "stw r31, -4(r1)" %t1
|
|
; RUN: grep "stwu r1, -16416(r1)" %t1
|
|
; RUN: grep "addi r1, r1, 16416" %t1
|
|
; RUN: llc < %s -march=ppc32 | \
|
|
; RUN: not grep "lwz r31, -4(r1)"
|
|
; RUN: llc < %s -march=ppc32 -mtriple=powerpc-apple-darwin8 -disable-fp-elim \
|
|
; RUN: -o %t2
|
|
; RUN: grep "stw r31, -4(r1)" %t2
|
|
; RUN: grep "stwu r1, -16416(r1)" %t2
|
|
; RUN: grep "addi r1, r1, 16416" %t2
|
|
; RUN: grep "lwz r31, -4(r1)" %t2
|
|
; RUN: llc < %s -march=ppc64 -mtriple=powerpc-apple-darwin8 -o %t3
|
|
; RUN: not grep "std r31, -8(r1)" %t3
|
|
; RUN: grep "stdu r1, -16432(r1)" %t3
|
|
; RUN: grep "addi r1, r1, 16432" %t3
|
|
; RUN: not grep "ld r31, -8(r1)" %t3
|
|
; RUN: llc < %s -march=ppc64 -mtriple=powerpc-apple-darwin8 -disable-fp-elim \
|
|
; RUN: -o %t4
|
|
; RUN: grep "std r31, -8(r1)" %t4
|
|
; RUN: grep "stdu r1, -16448(r1)" %t4
|
|
; RUN: grep "addi r1, r1, 16448" %t4
|
|
; RUN: grep "ld r31, -8(r1)" %t4
|
|
|
|
define i32* @f1() {
|
|
%tmp = alloca i32, i32 4095 ; <i32*> [#uses=1]
|
|
ret i32* %tmp
|
|
}
|
|
|