From b2a0abce1ed97d6642f91e10114f22e824d933cb Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Sat, 23 Feb 2008 01:47:44 +0000 Subject: [PATCH] No need recognize load from a fixed argument slot as re-materializable. LiveIntervalAnalysis already handles it as a special case. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47522 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/X86/X86InstrInfo.cpp | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/lib/Target/X86/X86InstrInfo.cpp b/lib/Target/X86/X86InstrInfo.cpp index 082b896b37b..aae99bcfa1d 100644 --- 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 -- 2.34.1