No need recognize load from a fixed argument slot as re-materializable. LiveIntervalA...

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47522 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Target/X86/X86InstrInfo.cpp b/lib/Target/X86/X86InstrInfo.cpp
index 082b896b37b3199bc112008cb8377f7cf50b2fc9..aae99bcfa1dd269685d4538e8f7d81a2a4211bba 100644 (file)
--- a/lib/Target/X86/X86InstrInfo.cpp
+++ b/lib/Target/X86/X86InstrInfo.cpp
@@ -760,18 +760,6 @@ bool X86InstrInfo::isReallyTriviallyReMaterializable(MachineInstr *MI) const {
       return isPICBase;
     }
       
-    // If this is a load from a fixed argument slot, we know the value is
-    // invariant across the whole function, because we don't redefine argument
-    // values.
-#if 0
-    // FIXME: This is disabled due to a remat bug. rdar://5671644
-    if (MI->getOperand(1).isFI()) {
-      const MachineFrameInfo &MFI=*MI->getParent()->getParent()->getFrameInfo();
-      int Idx = MI->getOperand(1).getIndex();
-      return MFI.isFixedObjectIndex(Idx) && MFI.isImmutableObjectIndex(Idx);
-    }
-#endif
-      
     return false;
   }
   // All other instructions marked M_REMATERIALIZABLE are always trivially