From ff2b99afc8cbc6cfa73181072888e0f9f07deb7e Mon Sep 17 00:00:00 2001 From: Jakob Stoklund Olesen Date: Thu, 9 Aug 2012 22:49:37 +0000 Subject: Move use list management into MachineRegisterInfo. Register MachineOperands are kept in linked lists accessible via MRI's reg_iterator interfaces. The linked list management was handled partly by MachineOperand methods, partly by MRI methods. Move all of the list management into MRI, delete MO::AddRegOperandToRegInfo() and MO::RemoveRegOperandFromRegInfo(). Be more explicit about handling the cases where an MRI pointer isn't available. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@161632 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/MachineRegisterInfo.cpp | 37 +++++++++++++++++++++++++++++++++++++ 1 file changed, 37 insertions(+) (limited to 'lib/CodeGen/MachineRegisterInfo.cpp') diff --git a/lib/CodeGen/MachineRegisterInfo.cpp b/lib/CodeGen/MachineRegisterInfo.cpp index 3a4eb8c967..6d05d35d7a 100644 --- a/lib/CodeGen/MachineRegisterInfo.cpp +++ b/lib/CodeGen/MachineRegisterInfo.cpp @@ -126,6 +126,43 @@ void MachineRegisterInfo::clearVirtRegs() { VRegInfo.clear(); } +/// Add MO to the linked list of operands for its register. +void MachineRegisterInfo::addRegOperandToUseList(MachineOperand *MO) { + assert(!MO->isOnRegUseList() && "Already on list"); + MachineOperand **Head = &getRegUseDefListHead(MO->getReg()); + + // For SSA values, we prefer to keep the definition at the start of the list. + // we do this by skipping over the definition if it is at the head of the + // list. + if (*Head && (*Head)->isDef()) + Head = &(*Head)->Contents.Reg.Next; + + MO->Contents.Reg.Next = *Head; + if (MO->Contents.Reg.Next) { + assert(MO->getReg() == MO->Contents.Reg.Next->getReg() && + "Different regs on the same list!"); + MO->Contents.Reg.Next->Contents.Reg.Prev = &MO->Contents.Reg.Next; + } + + MO->Contents.Reg.Prev = Head; + *Head = MO; +} + +/// Remove MO from its use-def list. +void MachineRegisterInfo::removeRegOperandFromUseList(MachineOperand *MO) { + assert(MO->isOnRegUseList() && "Operand not on use list"); + + // Unlink this from the doubly linked list of operands. + MachineOperand *NextOp = MO->Contents.Reg.Next; + *MO->Contents.Reg.Prev = NextOp; + if (NextOp) { + assert(NextOp->getReg() == MO->getReg() && "Corrupt reg use/def chain!"); + NextOp->Contents.Reg.Prev = MO->Contents.Reg.Prev; + } + MO->Contents.Reg.Prev = 0; + MO->Contents.Reg.Next = 0; +} + /// HandleVRegListReallocation - We just added a virtual register to the /// VRegInfo info list and it reallocated. Update the use/def lists info /// pointers. -- cgit v1.2.3-70-g09d2