Changes to files in the Targets to call sortUnique on MachineBasicBlock::LiveIns SortedVector after inserting regs in a given scope.

[qcolombet]

Hi Puyan,

I believe this change is not relevant anymore.
Could you close it?

Thanks,
-Quentin

lib/Target/AMDGPU/SIMachineFunctionInfo.cpp

     }
   }
 
+  for (auto BI = MF->begin(), BE = MF->end(); BI != BE; ++BI)
+    BI->sortUniqueLiveIns();
+
   Spill.VGPR = LaneVGPRs[LaneVGPRIdx];
   Spill.Lane = Lane;
   return Spill;

lib/Target/AMDGPU/SIPrepareScratchRegs.cpp

   if (!Entry->isLiveIn(ScratchOffsetPreloadReg))
     Entry->addLiveIn(ScratchOffsetPreloadReg);
 
+  Entry->sortUniqueLiveIns();
+
   // Load the scratch offset.
   unsigned ScratchOffsetReg =
       TRI->findUnusedRegister(MRI, &AMDGPU::SGPR_32RegClass);

     if (!MBB.isLiveIn(ScratchOffsetReg) &&
         ScratchOffsetReg != AMDGPU::NoRegister)
       MBB.addLiveIn(ScratchOffsetReg);
+    MBB.sortUniqueLiveIns();
     RS.enterBasicBlock(&MBB);
 
     for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end();

       MI.addOperand(MachineOperand::CreateReg(Rsrc2, false, true, true));
       MI.addOperand(MachineOperand::CreateReg(Rsrc3, false, true, true));
     }
+
+    MBB.sortUniqueLiveIns();
   }
   return true;
 }

lib/Target/Hexagon/HexagonFrameLowering.cpp

       if (!B.empty() && B.back().isReturn())
         insertEpilogueInBlock(B);
   }
+
+  for (auto &B : MF)
+    B.sortUniqueLiveIns();
 }
 
 

lib/Target/Mips/MipsSEFrameLowering.cpp

   if (MipsFI->callsEhReturn()) {
     // Insert instructions that spill eh data registers.
     for (int I = 0; I < 4; ++I) {
-      if (!MBB.isLiveIn(ABI.GetEhDataReg(I)))
-        MBB.addLiveIn(ABI.GetEhDataReg(I));
+      MBB.addLiveIn(ABI.GetEhDataReg(I));
       TII.storeRegToStackSlot(MBB, MBBI, ABI.GetEhDataReg(I), false,
                               MipsFI->getEhDataRegFI(I), RC, &RegInfo);
     }
+    MBB.sortUniqueLiveIns();
 
     // Emit .cfi_offset directives for eh data registers.
     for (int I = 0; I < 4; ++I) {

lib/Target/SystemZ/SystemZFrameLowering.cpp

     }
   }
 
+  MBB.sortUniqueLiveIns();
   return true;
 }
 

lib/Target/X86/X86FloatingPoint.cpp

     pushReg(Bundle.FixStack[i-1]);
   }
 
+  MBB->sortUniqueLiveIns();
+
   // Kill off unwanted live-ins. This can happen with a critical edge.
   // FIXME: We could keep these live registers around as zombies. They may need
   // to be revived at the end of a short block. It might save a few instrs.

lib/Target/X86/X86FrameLowering.cpp

     NumBytes = StackSize - X86FI->getCalleeSavedFrameSize();
   }
 
+  for (auto & I : MF)
+    I.sortUniqueLiveIns();
+
   // Skip the callee-saved push instructions.
   bool PushedRegs = false;
   int StackOffset = 2 * stackGrowth;

lib/Target/XCore/XCoreFrameLowering.cpp

                     SpillList[1].Offset);
     }
   }
+
+  MBB.sortUniqueLiveIns();
 }
 
 void XCoreFrameLowering::emitEpilogue(MachineFunction &MF,