From 63e34f690c511a146b936435f84ee76fda154f7c Mon Sep 17 00:00:00 2001 From: Jakob Stoklund Olesen Date: Thu, 13 May 2010 00:19:39 +0000 Subject: [PATCH] More asserts around physreg uses git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@103685 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/RegAllocFast.cpp | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/lib/CodeGen/RegAllocFast.cpp b/lib/CodeGen/RegAllocFast.cpp index eac8407fff3..304162a303f 100644 --- a/lib/CodeGen/RegAllocFast.cpp +++ b/lib/CodeGen/RegAllocFast.cpp @@ -487,7 +487,7 @@ unsigned RAFast::reloadVirtReg(MachineBasicBlock &MBB, MachineInstr *MI, } /// reservePhysReg - Mark PhysReg as reserved. This is very similar to -/// defineVirtReg except the physreg is reverved instead of allocated. +/// defineVirtReg except the physreg is reserved instead of allocated. void RAFast::reservePhysReg(MachineBasicBlock &MBB, MachineInstr *MI, unsigned PhysReg) { UsedInInstr.set(PhysReg); @@ -623,6 +623,16 @@ void RAFast::AllocateBasicBlock(MachineBasicBlock &MBB) { if (!Reg || !TargetRegisterInfo::isPhysicalRegister(Reg) || ReservedRegs.test(Reg)) continue; if (MO.isUse()) { +#ifndef NDEBUG + // We are using a physreg directly. It had better not be clobbered by a + // virtreg. + assert(PhysRegState[Reg] <= regReserved && "Using clobbered physreg"); + if (PhysRegState[Reg] == regDisabled) + for (const unsigned *AS = TRI->getAliasSet(Reg); + unsigned Alias = *AS; ++AS) + assert(PhysRegState[Alias] <= regReserved && + "Physreg alias was clobbered"); +#endif PhysKills.push_back(Reg); // Any clean physreg use is a kill. UsedInInstr.set(Reg); } else if (MO.isEarlyClobber()) {