[PATCH] D49459: [ARM] Avoid spilling lr with Thumb1 tail calls.

[Eli Friedman via Phabricator via llvm-commits]

efriedma marked 2 inline comments as done.
efriedma added inline comments.


================
Comment at: lib/Target/ARM/ARMFrameLowering.cpp:1927
+    if (!LRSpilled && AFI->isThumb1OnlyFunction()) {
+      for (MachineBasicBlock &MBB : MF) {
+        if (!MBB.empty() &&
----------------
chill wrote:
> chill wrote:
> > Instead of traversing all the blocks, would it be possible (or make sense) to set a flag in `ARMMachineFunctionInfo`, indicating that the function contains tail calls, whenever we create a `TC_RETURN` in `LowerCall`, 
> > 
> Maybe `const MachineBasicBlock &` ?
> 
Turns out MachineFrameInfo already has a flag to track this. Patch updated.


Repository:
  rL LLVM

https://reviews.llvm.org/D49459