diff options
author | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-03-28 20:11:44 +0000 |
---|---|---|
committer | Jakob Stoklund Olesen <stoklund@2pi.dk> | 2012-03-28 20:11:44 +0000 |
commit | 8d813de71e18dd73ed98e8d58407caba0265e38e (patch) | |
tree | 2313b53a8a5312b54adf7cc8affb93f3042e778f | |
parent | 78836f0bb2bf5958c1b9f904b0ad0057c77ab75f (diff) |
Revert r153516: "Invalidate liveness in Thumb2ITBlockPass."
Revert r153519: "ARMLoadStoreOptimizer invalidates register liveness."
These patches caused miscompilations in povray by turning off branch
folding's updating of live-in lists.
It turns out the the late scheduler depends on the live-in lists, even
if it doesn't need correct kill flags.
<rdar://problem/11139228>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@153593 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Target/ARM/ARMLoadStoreOptimizer.cpp | 4 | ||||
-rw-r--r-- | lib/Target/ARM/Thumb2ITBlockPass.cpp | 4 |
2 files changed, 0 insertions, 8 deletions
diff --git a/lib/Target/ARM/ARMLoadStoreOptimizer.cpp b/lib/Target/ARM/ARMLoadStoreOptimizer.cpp index 5b729a642b..51dfa65da6 100644 --- a/lib/Target/ARM/ARMLoadStoreOptimizer.cpp +++ b/lib/Target/ARM/ARMLoadStoreOptimizer.cpp @@ -1411,10 +1411,6 @@ bool ARMLoadStoreOpt::runOnMachineFunction(MachineFunction &Fn) { } delete RS; - - // Kill flags aren't updated accurately by this pass. - Fn.getRegInfo().invalidateLiveness(); - return Modified; } diff --git a/lib/Target/ARM/Thumb2ITBlockPass.cpp b/lib/Target/ARM/Thumb2ITBlockPass.cpp index e2166ffc32..ecb4c2f0e5 100644 --- a/lib/Target/ARM/Thumb2ITBlockPass.cpp +++ b/lib/Target/ARM/Thumb2ITBlockPass.cpp @@ -15,7 +15,6 @@ #include "llvm/CodeGen/MachineInstrBuilder.h" #include "llvm/CodeGen/MachineInstrBundle.h" #include "llvm/CodeGen/MachineFunctionPass.h" -#include "llvm/CodeGen/MachineRegisterInfo.h" #include "llvm/ADT/SmallSet.h" #include "llvm/ADT/Statistic.h" using namespace llvm; @@ -259,9 +258,6 @@ bool Thumb2ITBlockPass::runOnMachineFunction(MachineFunction &Fn) { if (!AFI->isThumbFunction()) return false; - // IT block insertion invalidates accurate register liveness. - Fn.getRegInfo().invalidateLiveness(); - bool Modified = false; for (MachineFunction::iterator MFI = Fn.begin(), E = Fn.end(); MFI != E; ) { MachineBasicBlock &MBB = *MFI; |