[llvm-commits] [llvm] r167546 - in /llvm/trunk/lib/Target/Mips: MipsISelLowering.cpp MipsMachineFunction.h

[Akira Hatanaka]

Author: ahatanak
Date: Wed Nov  7 13:04:26 2012
New Revision: 167546

URL: http://llvm.org/viewvc/llvm-project?rev=167546&view=rev
Log:
Delete MipsFunctionInfo::NextStackOffset. No functionality change intended. 


Modified:
    llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
    llvm/trunk/lib/Target/Mips/MipsMachineFunction.h

Modified: llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp?rev=167546&r1=167545&r2=167546&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp (original)
+++ llvm/trunk/lib/Target/Mips/MipsISelLowering.cpp Wed Nov  7 13:04:26 2012
@@ -2674,9 +2674,9 @@
   if (MipsCCInfo.hasByValArg() || FI.hasByvalArg())
     return false;
 
-  // Return true if the callee's next stack offset is no larger than the
+  // Return true if the callee's argument area is no larger than the
   // caller's.
-  return NextStackOffset <= FI.nextStackOffset();
+  return NextStackOffset <= FI.getIncomingArgSize();
 }
 
 SDValue
@@ -3027,7 +3027,6 @@
   MipsCCInfo.analyzeFormalArguments(Ins);
   MipsFI->setFormalArgInfo(CCInfo.getNextStackOffset(),
                            MipsCCInfo.hasByValArg());
-  MipsFI->setIncomingArgSize(CCInfo.getNextStackOffset());
 
   Function::const_arg_iterator FuncArg =
     DAG.getMachineFunction().getFunction()->arg_begin();

Modified: llvm/trunk/lib/Target/Mips/MipsMachineFunction.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MipsMachineFunction.h?rev=167546&r1=167545&r2=167546&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/MipsMachineFunction.h (original)
+++ llvm/trunk/lib/Target/Mips/MipsMachineFunction.h Wed Nov  7 13:04:26 2012
@@ -47,11 +47,10 @@
   /// VarArgsFrameIndex - FrameIndex for start of varargs area.
   int VarArgsFrameIndex;
 
-  // Formal argument information obtained during call to LowerFormalArguments.
-  unsigned NextStackOffset;
+  /// True if function has a byval argument.
   bool HasByvalArg;
 
-  // Size of incoming argument area.
+  /// Size of incoming argument area.
   unsigned IncomingArgSize;
 
 public:
@@ -72,15 +71,13 @@
   int getVarArgsFrameIndex() const { return VarArgsFrameIndex; }
   void setVarArgsFrameIndex(int Index) { VarArgsFrameIndex = Index; }
 
-  unsigned nextStackOffset() const { return NextStackOffset; }
   bool hasByvalArg() const { return HasByvalArg; }
-  void setFormalArgInfo(unsigned Offset, bool HasByval) {
-    NextStackOffset = Offset;
+  void setFormalArgInfo(unsigned Size, bool HasByval) {
+    IncomingArgSize = Size;
     HasByvalArg = HasByval;
   }
 
   unsigned getIncomingArgSize() const { return IncomingArgSize; }
-  void setIncomingArgSize(unsigned S) { IncomingArgSize = S; }
 };
 
 } // end of namespace llvm