[llvm] r302014 - Revert "[AVR] Enable the frame pointer for all functions"

Dylan McKay via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 04:36:44 PDT 2017


Author: dylanmckay
Date: Wed May  3 06:36:42 2017
New Revision: 302014

URL: http://llvm.org/viewvc/llvm-project?rev=302014&view=rev
Log:
Revert "[AVR] Enable the frame pointer for all functions"

This reverts commit 358ad02d999e88853d2cfc954bd2f668308a51f7.

Modified:
    llvm/trunk/lib/Target/AVR/AVRFrameLowering.cpp

Modified: llvm/trunk/lib/Target/AVR/AVRFrameLowering.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AVR/AVRFrameLowering.cpp?rev=302014&r1=302013&r2=302014&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AVR/AVRFrameLowering.cpp (original)
+++ llvm/trunk/lib/Target/AVR/AVRFrameLowering.cpp Wed May  3 06:36:42 2017
@@ -228,9 +228,8 @@ void AVRFrameLowering::emitEpilogue(Mach
 bool AVRFrameLowering::hasFP(const MachineFunction &MF) const {
   const AVRMachineFunctionInfo *FuncInfo = MF.getInfo<AVRMachineFunctionInfo>();
 
-  // TODO: We do not always need a frame pointer.
-  // This can be optimised.
-  return true;
+  return (FuncInfo->getHasSpills() || FuncInfo->getHasAllocas() ||
+          FuncInfo->getHasStackArgs());
 }
 
 bool AVRFrameLowering::spillCalleeSavedRegisters(




More information about the llvm-commits mailing list