[llvm-commits] [llvm] r47522 - /llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
Evan Cheng
evan.cheng at apple.com
Fri Feb 22 17:47:44 PST 2008
Author: evancheng
Date: Fri Feb 22 19:47:44 2008
New Revision: 47522
URL: http://llvm.org/viewvc/llvm-project?rev=47522&view=rev
Log:
No need recognize load from a fixed argument slot as re-materializable. LiveIntervalAnalysis already handles it as a special case.
Modified:
llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
Modified: llvm/trunk/lib/Target/X86/X86InstrInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86InstrInfo.cpp?rev=47522&r1=47521&r2=47522&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/X86InstrInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86InstrInfo.cpp Fri Feb 22 19:47:44 2008
@@ -760,18 +760,6 @@
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
More information about the llvm-commits
mailing list