Remove the FrameSizeLocals field which appears to be unused (write-only).

Might have gotten replaced by some other field, but don't quite remember. Spotted while looking for ways to share the addProlog() code between targets. BUG=none R=stichnot@chromium.org Review URL: https://codereview.chromium.org/1158713005

Remove the FrameSizeLocals field which appears to be unused (write-only).
0d9faeac · Jan Voung · 9a05aea8 · 0d9faeac · 0d9faeac
Commit 0d9faeac authored May 27, 2015 by Jan Voung
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

IceTargetLoweringX8632.cpp src/IceTargetLoweringX8632.cpp +1 -2

IceTargetLoweringX8632.h src/IceTargetLoweringX8632.h +0 -1

No files found.
--- a/src/IceTargetLoweringX8632.cpp
+++ b/src/IceTargetLoweringX8632.cpp
@@ -403,7 +403,7 @@ TargetX8632::TargetX8632(Cfg *Func)
      InstructionSet(static_cast<X86InstructionSet>(
          Func->getContext()->getFlags().getTargetInstructionSet() -
          TargetInstructionSet::X86InstructionSet_Begin)),
-      IsEbpBasedFrame(false), NeedsStackAlignment(false), FrameSizeLocals(0),
+      IsEbpBasedFrame(false), NeedsStackAlignment(false),
      SpillAreaSizeBytes(0) {
  static_assert((X86InstructionSet::End - X86InstructionSet::Begin) ==
                    (TargetInstructionSet::X86InstructionSet_End -
@@ -1023,7 +1023,6 @@ void TargetX8632::addProlog(CfgNode *Node) {
    else
      Var->setStackOffset(SpillAreaSizeBytes - NextStackOffset);
  }
-  this->FrameSizeLocals = NextStackOffset - SpillAreaPaddingBytes;
  this->HasComputedFrame = true;

  // Assign stack offsets to variables that have been linked to spilled

--- a/src/IceTargetLoweringX8632.h
+++ b/src/IceTargetLoweringX8632.h
@@ -567,7 +567,6 @@ protected:
  const X86InstructionSet InstructionSet;
  bool IsEbpBasedFrame;
  bool NeedsStackAlignment;
-  size_t FrameSizeLocals;
  size_t SpillAreaSizeBytes;
  llvm::SmallBitVector TypeToRegisterSet[IceType_NUM];
  llvm::SmallBitVector ScratchRegs;