[llvm] [ARM] Fix -mno-omit-leaf-frame-pointer flag doesn't works on 32-bit ARM (PR #109628)
Eli Friedman via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 7 15:40:23 PDT 2024
================
@@ -22,10 +22,6 @@ 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 the target want to forcibly keep frame pointer.
- if (MF.getSubtarget().getFrameLowering()->keepFramePointer(MF))
----------------
efriedma-quic wrote:
keepFramePointer is implemented as a target independent method... if you're going to get rid of the call here, you should also get rid of the interface from TargetFrameLowering. (No other in-tree target implements it anyway, but we want to keep the code clean, and give an obvious error in case there is someone with an out-of-tree dependency.)
https://github.com/llvm/llvm-project/pull/109628
More information about the llvm-commits
mailing list