[PATCH] D36160: Liveness issues in tail merging, and the ARM::LR saved-but-not-restored

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 12:52:24 PDT 2017


efriedma added a comment.

> Oh, I see what you mean... we don't actually decide whether to "pop {pc}" until ARMFrameLowering::emitPopInst.

Wait, that's not right; we don't need to include lr in the list of live-outs either way.  If we generate "pop {pc}", lr isn't used; if we generate "bx lr", the explicit use of lr should allow us to generate correct liveness.


Repository:
  rL LLVM

https://reviews.llvm.org/D36160





More information about the llvm-commits mailing list