summaryrefslogtreecommitdiffstats
path: root/llvm/lib/CodeGen/TargetOptionsImpl.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'llvm/lib/CodeGen/TargetOptionsImpl.cpp')
-rw-r--r--llvm/lib/CodeGen/TargetOptionsImpl.cpp31
1 files changed, 25 insertions, 6 deletions
diff --git a/llvm/lib/CodeGen/TargetOptionsImpl.cpp b/llvm/lib/CodeGen/TargetOptionsImpl.cpp
index 853e71d0efa..3c133fb8594 100644
--- a/llvm/lib/CodeGen/TargetOptionsImpl.cpp
+++ b/llvm/lib/CodeGen/TargetOptionsImpl.cpp
@@ -23,15 +23,34 @@ using namespace llvm;
/// DisableFramePointerElim - This returns true if frame pointer elimination
/// optimization should be disabled for the given machine function.
bool TargetOptions::DisableFramePointerElim(const MachineFunction &MF) const {
- // Check to see if we should eliminate all frame pointers.
- if (MF.getSubtarget().getFrameLowering()->noFramePointerElim(MF))
+ // Check to see if the target want to forcably keep frame pointer.
+ if (MF.getSubtarget().getFrameLowering()->keepFramePointer(MF))
return true;
- // Check to see if we should eliminate non-leaf frame pointers.
- if (MF.getFunction().hasFnAttribute("no-frame-pointer-elim-non-leaf"))
- return MF.getFrameInfo().hasCalls();
+ const Function &F = MF.getFunction();
+
+ // TODO: Remove support for old `fp elim` function attributes after fully
+ // migrate to use "frame-pointer"
+ if (!F.hasFnAttribute("frame-pointer")) {
+ // Check to see if we should eliminate all frame pointers.
+ if (F.getFnAttribute("no-frame-pointer-elim").getValueAsString() == "true")
+ return true;
+
+ // Check to see if we should eliminate non-leaf frame pointers.
+ if (F.hasFnAttribute("no-frame-pointer-elim-non-leaf"))
+ return MF.getFrameInfo().hasCalls();
- return false;
+ return false;
+ }
+
+ StringRef FP = F.getFnAttribute("frame-pointer").getValueAsString();
+ if (FP == "all")
+ return true;
+ if (FP == "non-leaf")
+ return MF.getFrameInfo().hasCalls();
+ if (FP == "none")
+ return false;
+ llvm_unreachable("unknown frame pointer flag");
}
/// HonorSignDependentRoundingFPMath - Return true if the codegen must assume
OpenPOWER on IntegriCloud