diff --git a/llvm/include/llvm/CodeGen/GlobalISel/RegBankSelect.h b/llvm/include/llvm/CodeGen/GlobalISel/RegBankSelect.h index b210a8e9f6e7c4c80ae97ce282919f1257f7e34c..7b8601fc3ae9ac4c52532b5ab518e8b096b002cf 100644 --- a/llvm/include/llvm/CodeGen/GlobalISel/RegBankSelect.h +++ b/llvm/include/llvm/CodeGen/GlobalISel/RegBankSelect.h @@ -525,11 +525,6 @@ private: RegBankSelect::RepairingPlacement &RepairPt, const iterator_range::iterator> &NewVRegs); - /// Set the insertion point of the MIRBuilder to a safe point - /// to insert instructions before (\p Before == true) or after - /// \p InsertPt. - void setSafeInsertionPoint(MachineInstr &InsertPt, bool Before); - /// Compute the cost of mapping \p MI with \p InstrMapping and /// compute the repairing placement for such mapping in \p /// RepairPts. diff --git a/llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp b/llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp index d8bc7b43c8103c7cf696f985f839600d5f12b61c..152fc0c306c73e476f541173953cd0ef33ebf79f 100644 --- a/llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp +++ b/llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp @@ -136,40 +136,6 @@ void RegBankSelect::repairReg( // Legalize NewInstrs if need be. } -void RegBankSelect::setSafeInsertionPoint(MachineInstr &InsertPt, bool Before) { - // Check that we are not looking to insert before a phi. - // Indeed, we would need more information on what to do. - // By default that should be all the predecessors, but this is - // probably not what we want in general. - assert((!Before || !InsertPt.isPHI()) && - "Insertion before phis not implemented"); - // The same kind of observation hold for terminators if we try to - // insert after them. - assert((Before || !InsertPt.isTerminator()) && - "Insertion after terminatos not implemented"); - if (InsertPt.isPHI()) { - assert(!Before && "Not supported!!"); - MachineBasicBlock *MBB = InsertPt.getParent(); - assert(MBB && "Insertion point is not in a basic block"); - MachineBasicBlock::iterator FirstNonPHIPt = MBB->getFirstNonPHI(); - if (FirstNonPHIPt == MBB->end()) { - // If there is not any non-phi instruction, insert at the end of MBB. - MIRBuilder.setMBB(*MBB, /*Beginning*/ false); - return; - } - // The insertion point before the first non-phi instruction. - MIRBuilder.setInstr(*FirstNonPHIPt, /*Before*/ true); - return; - } - if (InsertPt.isTerminator()) { - MachineBasicBlock *MBB = InsertPt.getParent(); - assert(MBB && "Insertion point is not in a basic block"); - MIRBuilder.setInstr(*MBB->getFirstTerminator(), /*Before*/ true); - return; - } - MIRBuilder.setInstr(InsertPt, /*Before*/ Before); -} - void RegBankSelect::tryAvoidingSplit( RegBankSelect::RepairingPlacement &RepairPt, const MachineOperand &MO, const RegisterBankInfo::ValueMapping &ValMapping) const {