diff options
| author | Tim Northover <tnorthover@apple.com> | 2018-04-07 10:57:03 +0000 |
|---|---|---|
| committer | Tim Northover <tnorthover@apple.com> | 2018-04-07 10:57:03 +0000 |
| commit | e25e458d5233ef829f90110e1197aa3e1588faf4 (patch) | |
| tree | 2b917151fc8ad32f7574c9463a83843a411430a9 /llvm/lib/Target/ARM/ARMFrameLowering.h | |
| parent | 61061d69eacfb546283f33cba053c29bb1c1ef7e (diff) | |
| download | bcm5719-llvm-e25e458d5233ef829f90110e1197aa3e1588faf4.tar.gz bcm5719-llvm-e25e458d5233ef829f90110e1197aa3e1588faf4.zip | |
Reapply ARM: Do not spill CSR to stack on entry to noreturn functions
Should fix UBSan bot by also checking there's no "uwtable" attribute
before skipping. Otherwise the unwind table will be useless since its
moves expect CSRs to actually be preserved.
A noreturn nounwind function can be expected to never return in any way, and by
never returning it will also never have to restore any callee-saved registers
for its caller. This makes it possible to skip spills of those registers during
function entry, saving some stack space and time in the process. This is rather
useful for embedded targets with limited stack space.
Should fix PR9970.
Patch mostly by myeisha (pmb).
llvm-svn: 329494
Diffstat (limited to 'llvm/lib/Target/ARM/ARMFrameLowering.h')
| -rw-r--r-- | llvm/lib/Target/ARM/ARMFrameLowering.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/Target/ARM/ARMFrameLowering.h b/llvm/lib/Target/ARM/ARMFrameLowering.h index 1f18e2bf80c..e994cab28fe 100644 --- a/llvm/lib/Target/ARM/ARMFrameLowering.h +++ b/llvm/lib/Target/ARM/ARMFrameLowering.h @@ -44,6 +44,8 @@ public: bool noFramePointerElim(const MachineFunction &MF) const override; + bool enableCalleeSaveSkip(const MachineFunction &MF) const override; + bool hasFP(const MachineFunction &MF) const override; bool hasReservedCallFrame(const MachineFunction &MF) const override; bool canSimplifyCallFramePseudos(const MachineFunction &MF) const override; |
