[PATCH] D16434: Fix LivePhysRegs::addLiveOuts

Weiming Zhao via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 14:09:52 PST 2016


weimingz retitled this revision from " Fix ARM load/store opt live reg computing" to " Fix LivePhysRegs::addLiveOuts".
weimingz updated the summary for this revision.
weimingz removed rL LLVM as the repository for this revision.
weimingz updated this revision to Diff 45741.
weimingz added a comment.

I think originally the problem was covered because epilogue will do the job. Now with shrink-wrap, it is exposed.


http://reviews.llvm.org/D16434

Files:
  lib/CodeGen/LivePhysRegs.cpp

Index: lib/CodeGen/LivePhysRegs.cpp
===================================================================
--- lib/CodeGen/LivePhysRegs.cpp
+++ lib/CodeGen/LivePhysRegs.cpp
@@ -151,7 +151,7 @@
   if (AddPristinesAndCSRs) {
     const MachineFunction &MF = *MBB->getParent();
     addPristines(*this, MF, *TRI);
-    if (!MBB->isReturnBlock()) {
+    if (MBB->isReturnBlock()) {
       // The return block has no successors whose live-ins we could merge
       // below. So instead we add the callee saved registers manually.
       for (const MCPhysReg *I = TRI->getCalleeSavedRegs(&MF); *I; ++I)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16434.45741.patch
Type: text/x-patch
Size: 594 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160122/f33ff67c/attachment.bin>


More information about the llvm-commits mailing list