[PowerPC] Fix IsDarwin arg in PPCFrameLowering:: calls
authorUlrich Weigand <ulrich.weigand@de.ibm.com>
Mon, 23 Jun 2014 13:21:43 +0000 (13:21 +0000)
committerUlrich Weigand <ulrich.weigand@de.ibm.com>
Mon, 23 Jun 2014 13:21:43 +0000 (13:21 +0000)
As remarked in the commit message to r211493, in several places
throughout the 64-bit SVR4 ABI code there are calls to
PPCFrameLowering::getLinkageSize and getMinCallFrameSize
using an incorrect IsDarwin argument of "true".

(Some of those were made explicit by the above refactoring patch, others
have been there all along.)

This patch fixes those places to pass "false" for IsDarwin.

No change in generated code expected.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@211494 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/PowerPC/PPCISelLowering.cpp

index 74c16cc8ce48d176057e31e3b5a4a49a0f497c18..3b910ca01d2413eef8fb61e29e9e7cf0b307a3d3 100644 (file)
@@ -2420,7 +2420,7 @@ PPCTargetLowering::LowerFormalArguments_64SVR4(
                        (CallConv == CallingConv::Fast));
   unsigned PtrByteSize = 8;
 
-  unsigned ArgOffset = PPCFrameLowering::getLinkageSize(true, true);
+  unsigned ArgOffset = PPCFrameLowering::getLinkageSize(true, false);
   // Area that is at least reserved in caller of this function.
   unsigned MinReservedArea = ArgOffset;
 
@@ -2659,7 +2659,7 @@ PPCTargetLowering::LowerFormalArguments_64SVR4(
   // Area that is at least reserved in the caller of this function.
   MinReservedArea =
     std::max(MinReservedArea,
-             PPCFrameLowering::getMinCallFrameSize(true, true));
+             PPCFrameLowering::getMinCallFrameSize(true, false));
 
   // Set the size that is at least reserved in caller of this function.  Tail
   // call optimized functions' reserved stack space needs to be aligned so that
@@ -3965,7 +3965,7 @@ PPCTargetLowering::LowerCall_64SVR4(SDValue Chain, SDValue Callee,
   // Count how many bytes are to be pushed on the stack, including the linkage
   // area, and parameter passing area.  We start with at least 48 bytes, which
   // is reserved space for [SP][CR][LR][3 x unused].
-  unsigned NumBytes = PPCFrameLowering::getLinkageSize(true, true);
+  unsigned NumBytes = PPCFrameLowering::getLinkageSize(true, false);
 
   // Add up all the space actually used.
   for (unsigned i = 0; i != NumOps; ++i) {
@@ -3987,7 +3987,7 @@ PPCTargetLowering::LowerCall_64SVR4(SDValue Chain, SDValue Callee,
   // conservatively assume that it is needed.  As such, make sure we have at
   // least enough stack space for the caller to store the 8 GPRs.
   NumBytes = std::max(NumBytes,
-                      PPCFrameLowering::getMinCallFrameSize(true, true));
+                      PPCFrameLowering::getMinCallFrameSize(true, false));
 
   // Tail call needs the stack to be aligned.
   if (getTargetMachine().Options.GuaranteedTailCallOpt &&
@@ -4024,7 +4024,7 @@ PPCTargetLowering::LowerCall_64SVR4(SDValue Chain, SDValue Callee,
   // memory.  Also, if this is a vararg function, floating point operations
   // must be stored to our stack, and loaded into integer regs as well, if
   // any integer regs are available for argument passing.
-  unsigned ArgOffset = PPCFrameLowering::getLinkageSize(true, true);
+  unsigned ArgOffset = PPCFrameLowering::getLinkageSize(true, false);
   unsigned GPR_idx = 0, FPR_idx = 0, VR_idx = 0;
 
   static const MCPhysReg GPR[] = {