[llvm] r203883 - Fix issue with r203865. The old behaviour would get a MachineOperand then find the MI for the bundle the MI was in. The new behaviour was failing to get the parent bundle and instead just used the MI from the MachineOperand

Pete Cooper peter_cooper at apple.com
Thu Mar 13 19:28:05 PDT 2014


Author: pete
Date: Thu Mar 13 21:28:05 2014
New Revision: 203883

URL: http://llvm.org/viewvc/llvm-project?rev=203883&view=rev
Log:
Fix issue with r203865.  The old behaviour would get a MachineOperand then find the MI for the bundle the MI was in.  The new behaviour was failing to get the parent bundle and instead just used the MI from the MachineOperand

Modified:
    llvm/trunk/lib/CodeGen/InlineSpiller.cpp

Modified: llvm/trunk/lib/CodeGen/InlineSpiller.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/InlineSpiller.cpp?rev=203883&r1=203882&r2=203883&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/InlineSpiller.cpp (original)
+++ llvm/trunk/lib/CodeGen/InlineSpiller.cpp Thu Mar 13 21:28:05 2014
@@ -1196,7 +1196,7 @@ void InlineSpiller::spillAroundUses(unsi
   for (MachineRegisterInfo::reg_bundle_iterator
        RegI = MRI.reg_bundle_begin(Reg), E = MRI.reg_bundle_end();
        RegI != E; ) {
-    MachineInstr *MI = &*(RegI++);
+    MachineInstr *MI = getBundleStart(&*(RegI++));
 
     // Debug values are not allowed to affect codegen.
     if (MI->isDebugValue()) {





More information about the llvm-commits mailing list