[PATCH] D87136: [ARM][MachineOutliner] Add calls default handling.

[Yvan Roux via Phabricator via llvm-commits]

yroux added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp:5694
+        FrameDefault(target.isThumb() ? 2 : 4),
+        SaveRestoreLROnStack(target.isThumb() ? 8 : 8) {}
 };
----------------
samparker wrote:
> So this is the cost of the store and load, but don't we also need to consider the frame setup too?
The variable NumBytesToCreateFrame is initialized to the default cost of the frame setup and then incremented by the cost of the load/store if needed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87136