[PATCH] D71168: [AArch64] Save FP for leaf functions when disabling frame pointer elimination

kamlesh kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 7 20:04:23 PST 2019


kamleshbhalui added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64FrameLowering.cpp:247
   // Retain behavior of always omitting the FP for leaf functions when possible.
-  if (MFI.hasCalls() && MF.getTarget().Options.DisableFramePointerElim(MF))
     return true;
----------------
MFI.hasCalls() was returning here false, Even for function who has call inside. May be because it is called earlier,before generating Full frame Info .  It needs to be removed from here. I could that it is done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71168/new/

https://reviews.llvm.org/D71168





More information about the llvm-commits mailing list