diff options
| author | Chris Lattner <sabre@nondot.org> | 2003-05-12 14:24:00 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2003-05-12 14:24:00 +0000 |
| commit | fb2cb69dc55c1e1d7754186cd1c8a9d543f54bdd (patch) | |
| tree | f1f6ca6c860200e417e543295f48175669c15a27 /lib | |
| parent | bc4a15f6faa6c32b2b84205fcdac9042728cf365 (diff) | |
| download | external_llvm-fb2cb69dc55c1e1d7754186cd1c8a9d543f54bdd.zip external_llvm-fb2cb69dc55c1e1d7754186cd1c8a9d543f54bdd.tar.gz external_llvm-fb2cb69dc55c1e1d7754186cd1c8a9d543f54bdd.tar.bz2 | |
* Keep the BBMap around as long as the pass is live
* Change getVarInfo to take real virtual register numbers and offset them
itself. This has caused me so much grief, it's not even funny.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@6115 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/CodeGen/LiveVariables.cpp | 31 |
1 files changed, 23 insertions, 8 deletions
diff --git a/lib/CodeGen/LiveVariables.cpp b/lib/CodeGen/LiveVariables.cpp index 81fdbe0..3d97581 100644 --- a/lib/CodeGen/LiveVariables.cpp +++ b/lib/CodeGen/LiveVariables.cpp @@ -28,6 +28,26 @@ static RegisterAnalysis<LiveVariables> X("livevars", "Live Variable Analysis"); +const std::pair<MachineBasicBlock*, unsigned> & +LiveVariables::getMachineBasicBlockInfo(MachineBasicBlock *MBB) const{ + return BBMap.find(MBB->getBasicBlock())->second; +} + +LiveVariables::VarInfo &LiveVariables::getVarInfo(unsigned RegIdx) { + assert(RegIdx >= MRegisterInfo::FirstVirtualRegister && + "getVarInfo: not a virtual register!"); + RegIdx -= MRegisterInfo::FirstVirtualRegister; + if (RegIdx >= VirtRegInfo.size()) { + if (RegIdx >= 2*VirtRegInfo.size()) + VirtRegInfo.resize(RegIdx*2); + else + VirtRegInfo.resize(2*VirtRegInfo.size()); + } + return VirtRegInfo[RegIdx]; +} + + + void LiveVariables::MarkVirtRegAliveInBlock(VarInfo &VRInfo, const BasicBlock *BB) { const std::pair<MachineBasicBlock*,unsigned> &Info = BBMap.find(BB)->second; @@ -195,8 +215,7 @@ bool LiveVariables::runOnMachineFunction(MachineFunction &MF) { MachineOperand &MO = MI->getOperand(i); if (MO.opIsUse() || MO.opIsDefAndUse()) { if (MO.isVirtualRegister() && !MO.getVRegValueOrNull()) { - unsigned RegIdx = MO.getReg()-MRegisterInfo::FirstVirtualRegister; - HandleVirtRegUse(getVarInfo(RegIdx), MBB, MI); + HandleVirtRegUse(getVarInfo(MO.getReg()), MBB, MI); } else if (MO.isPhysicalRegister() && AllocatablePhysicalRegisters[MO.getReg()]) { HandlePhysRegUse(MO.getReg(), MI); @@ -214,8 +233,7 @@ bool LiveVariables::runOnMachineFunction(MachineFunction &MF) { MachineOperand &MO = MI->getOperand(i); if (MO.opIsDef() || MO.opIsDefAndUse()) { if (MO.isVirtualRegister()) { - unsigned RegIdx = MO.getReg()-MRegisterInfo::FirstVirtualRegister; - VarInfo &VRInfo = getVarInfo(RegIdx); + VarInfo &VRInfo = getVarInfo(MO.getReg()); assert(VRInfo.DefBlock == 0 && "Variable multiply defined!"); VRInfo.DefBlock = MBB; // Created here... @@ -245,8 +263,7 @@ bool LiveVariables::runOnMachineFunction(MachineFunction &MF) { if (MI->getOperand(i+1).getMachineBasicBlock() == MBB) { MachineOperand &MO = MI->getOperand(i); if (!MO.getVRegValueOrNull()) { - unsigned RegIdx = MO.getReg()-MRegisterInfo::FirstVirtualRegister; - VarInfo &VRInfo = getVarInfo(RegIdx); + VarInfo &VRInfo = getVarInfo(MO.getReg()); // Only mark it alive only in the block we are representing... MarkVirtRegAliveInBlock(VRInfo, BB); @@ -263,8 +280,6 @@ bool LiveVariables::runOnMachineFunction(MachineFunction &MF) { HandlePhysRegDef(i, 0); } - BBMap.clear(); - // Convert the information we have gathered into VirtRegInfo and transform it // into a form usable by RegistersKilled. // |