Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 1 | //===-- PhiElimination.cpp - Eliminate PHI nodes by inserting copies ------===// |
Misha Brukman | edf128a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | 4ee451d | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | edf128a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 7 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 9 | // |
| 10 | // This pass eliminates machine instruction PHI nodes by inserting copy |
| 11 | // instructions. This destroys SSA information, but is the desired input for |
| 12 | // some register allocators. |
| 13 | // |
| 14 | //===----------------------------------------------------------------------===// |
| 15 | |
Chris Lattner | cd3245a | 2006-12-19 22:41:21 +0000 | [diff] [blame] | 16 | #define DEBUG_TYPE "phielim" |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 17 | #include "PHIElimination.h" |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 18 | #include "llvm/BasicBlock.h" |
| 19 | #include "llvm/Instructions.h" |
Misha Brukman | d7a10c8 | 2005-05-05 23:45:17 +0000 | [diff] [blame] | 20 | #include "llvm/CodeGen/LiveVariables.h" |
Chris Lattner | 0742b59 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 21 | #include "llvm/CodeGen/Passes.h" |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 23 | #include "llvm/CodeGen/MachineInstr.h" |
Evan Cheng | f870fbc | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 24 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
Chris Lattner | 84bc542 | 2007-12-31 04:13:23 +0000 | [diff] [blame] | 25 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 26 | #include "llvm/Target/TargetMachine.h" |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 27 | #include "llvm/ADT/SmallPtrSet.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 28 | #include "llvm/ADT/STLExtras.h" |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 29 | #include "llvm/ADT/Statistic.h" |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 30 | #include "llvm/Support/CommandLine.h" |
Chris Lattner | a4f0b3a | 2006-08-27 12:54:02 +0000 | [diff] [blame] | 31 | #include "llvm/Support/Compiler.h" |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 32 | #include "llvm/Support/Debug.h" |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 33 | #include <algorithm> |
Evan Cheng | 1088317 | 2008-04-02 17:23:50 +0000 | [diff] [blame] | 34 | #include <map> |
Chris Lattner | 0742b59 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 35 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 36 | |
Chris Lattner | cd3245a | 2006-12-19 22:41:21 +0000 | [diff] [blame] | 37 | STATISTIC(NumAtomic, "Number of atomic phis lowered"); |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 38 | STATISTIC(NumSplits, "Number of critical edges split on demand"); |
| 39 | |
| 40 | static cl::opt<bool> |
| 41 | SplitEdges("split-phi-edges", |
| 42 | cl::desc("Split critical edges during phi elimination"), |
| 43 | cl::init(false), cl::Hidden); |
Chris Lattner | cd3245a | 2006-12-19 22:41:21 +0000 | [diff] [blame] | 44 | |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 45 | char PHIElimination::ID = 0; |
| 46 | static RegisterPass<PHIElimination> |
Dan Gohman | 844731a | 2008-05-13 00:00:25 +0000 | [diff] [blame] | 47 | X("phi-node-elimination", "Eliminate PHI nodes for register allocation"); |
| 48 | |
Dan Gohman | 6ddba2b | 2008-05-13 02:05:11 +0000 | [diff] [blame] | 49 | const PassInfo *const llvm::PHIEliminationID = &X; |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 50 | |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 51 | void llvm::PHIElimination::getAnalysisUsage(AnalysisUsage &AU) const { |
Dan Gohman | 845012e | 2009-07-31 23:37:33 +0000 | [diff] [blame] | 52 | AU.addPreserved<LiveVariables>(); |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 53 | if (SplitEdges) { |
| 54 | AU.addRequired<LiveVariables>(); |
| 55 | } else { |
| 56 | AU.setPreservesCFG(); |
| 57 | AU.addPreservedID(MachineLoopInfoID); |
| 58 | AU.addPreservedID(MachineDominatorsID); |
| 59 | } |
Dan Gohman | 845012e | 2009-07-31 23:37:33 +0000 | [diff] [blame] | 60 | MachineFunctionPass::getAnalysisUsage(AU); |
| 61 | } |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 62 | |
| 63 | bool llvm::PHIElimination::runOnMachineFunction(MachineFunction &Fn) { |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 64 | MRI = &Fn.getRegInfo(); |
| 65 | |
Lang Hames | 2035463 | 2009-07-23 05:44:24 +0000 | [diff] [blame] | 66 | PHIDefs.clear(); |
| 67 | PHIKills.clear(); |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 68 | analyzePHINodes(Fn); |
| 69 | |
| 70 | bool Changed = false; |
| 71 | |
| 72 | // Eliminate PHI instructions by inserting copies into predecessor blocks. |
| 73 | for (MachineFunction::iterator I = Fn.begin(), E = Fn.end(); I != E; ++I) |
| 74 | Changed |= EliminatePHINodes(Fn, *I); |
| 75 | |
| 76 | // Remove dead IMPLICIT_DEF instructions. |
| 77 | for (SmallPtrSet<MachineInstr*,4>::iterator I = ImpDefs.begin(), |
| 78 | E = ImpDefs.end(); I != E; ++I) { |
| 79 | MachineInstr *DefMI = *I; |
| 80 | unsigned DefReg = DefMI->getOperand(0).getReg(); |
Evan Cheng | 1b38ec8 | 2008-06-19 01:21:26 +0000 | [diff] [blame] | 81 | if (MRI->use_empty(DefReg)) |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 82 | DefMI->eraseFromParent(); |
| 83 | } |
| 84 | |
| 85 | ImpDefs.clear(); |
| 86 | VRegPHIUseCount.clear(); |
| 87 | return Changed; |
| 88 | } |
| 89 | |
| 90 | |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 91 | /// EliminatePHINodes - Eliminate phi nodes by inserting copy instructions in |
| 92 | /// predecessor basic blocks. |
| 93 | /// |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 94 | bool llvm::PHIElimination::EliminatePHINodes(MachineFunction &MF, |
| 95 | MachineBasicBlock &MBB) { |
Alkis Evlogimenos | c0b9dc5 | 2004-02-12 02:27:10 +0000 | [diff] [blame] | 96 | if (MBB.empty() || MBB.front().getOpcode() != TargetInstrInfo::PHI) |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 97 | return false; // Quick exit for basic blocks without PHIs. |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 98 | |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 99 | if (SplitEdges) |
| 100 | SplitPHIEdges(MF, MBB); |
| 101 | |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 102 | // Get an iterator to the first instruction after the last PHI node (this may |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 103 | // also be the end of the basic block). |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 104 | MachineBasicBlock::iterator AfterPHIsIt = SkipPHIsAndLabels(MBB, MBB.begin()); |
Chris Lattner | 791f896 | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 105 | |
Bill Wendling | ca756d2 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 106 | while (MBB.front().getOpcode() == TargetInstrInfo::PHI) |
| 107 | LowerAtomicPHINode(MBB, AfterPHIsIt); |
| 108 | |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 109 | return true; |
| 110 | } |
Misha Brukman | edf128a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 111 | |
Evan Cheng | 1b38ec8 | 2008-06-19 01:21:26 +0000 | [diff] [blame] | 112 | /// isSourceDefinedByImplicitDef - Return true if all sources of the phi node |
| 113 | /// are implicit_def's. |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 114 | static bool isSourceDefinedByImplicitDef(const MachineInstr *MPhi, |
Evan Cheng | 1b38ec8 | 2008-06-19 01:21:26 +0000 | [diff] [blame] | 115 | const MachineRegisterInfo *MRI) { |
Evan Cheng | b3e0a6d | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 116 | for (unsigned i = 1; i != MPhi->getNumOperands(); i += 2) { |
| 117 | unsigned SrcReg = MPhi->getOperand(i).getReg(); |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 118 | const MachineInstr *DefMI = MRI->getVRegDef(SrcReg); |
Evan Cheng | b3e0a6d | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 119 | if (!DefMI || DefMI->getOpcode() != TargetInstrInfo::IMPLICIT_DEF) |
| 120 | return false; |
| 121 | } |
| 122 | return true; |
Evan Cheng | f870fbc | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 123 | } |
| 124 | |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 125 | // FindCopyInsertPoint - Find a safe place in MBB to insert a copy from SrcReg. |
| 126 | // This needs to be after any def or uses of SrcReg, but before any subsequent |
| 127 | // point where control flow might jump out of the basic block. |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 128 | MachineBasicBlock::iterator |
| 129 | llvm::PHIElimination::FindCopyInsertPoint(MachineBasicBlock &MBB, |
| 130 | unsigned SrcReg) { |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 131 | // Handle the trivial case trivially. |
| 132 | if (MBB.empty()) |
| 133 | return MBB.begin(); |
Evan Cheng | fc0b80d | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 134 | |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 135 | // If this basic block does not contain an invoke, then control flow always |
Evan Cheng | ddb03d8 | 2009-07-07 17:50:43 +0000 | [diff] [blame] | 136 | // reaches the end of it, so place the copy there. The logic below works in |
| 137 | // this case too, but is more expensive. |
| 138 | if (!isa<InvokeInst>(MBB.getBasicBlock()->getTerminator())) |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 139 | return MBB.getFirstTerminator(); |
| 140 | |
| 141 | // Discover any definition/uses in this basic block. |
| 142 | SmallPtrSet<MachineInstr*, 8> DefUsesInMBB; |
| 143 | for (MachineRegisterInfo::reg_iterator RI = MRI->reg_begin(SrcReg), |
| 144 | RE = MRI->reg_end(); RI != RE; ++RI) { |
| 145 | MachineInstr *DefUseMI = &*RI; |
| 146 | if (DefUseMI->getParent() == &MBB) |
| 147 | DefUsesInMBB.insert(DefUseMI); |
Evan Cheng | fc0b80d | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 148 | } |
| 149 | |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 150 | MachineBasicBlock::iterator InsertPoint; |
| 151 | if (DefUsesInMBB.empty()) { |
| 152 | // No def/uses. Insert the copy at the start of the basic block. |
| 153 | InsertPoint = MBB.begin(); |
| 154 | } else if (DefUsesInMBB.size() == 1) { |
| 155 | // Insert the copy immediately after the definition/use. |
| 156 | InsertPoint = *DefUsesInMBB.begin(); |
| 157 | ++InsertPoint; |
| 158 | } else { |
| 159 | // Insert the copy immediately after the last definition/use. |
| 160 | InsertPoint = MBB.end(); |
| 161 | while (!DefUsesInMBB.count(&*--InsertPoint)) {} |
| 162 | ++InsertPoint; |
Evan Cheng | fc0b80d | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 163 | } |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 164 | |
| 165 | // Make sure the copy goes after any phi nodes however. |
| 166 | return SkipPHIsAndLabels(MBB, InsertPoint); |
Evan Cheng | fc0b80d | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 167 | } |
| 168 | |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 169 | /// LowerAtomicPHINode - Lower the PHI node at the top of the specified block, |
| 170 | /// under the assuption that it needs to be lowered in a way that supports |
| 171 | /// atomic execution of PHIs. This lowering method is always correct all of the |
| 172 | /// time. |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 173 | /// |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 174 | void llvm::PHIElimination::LowerAtomicPHINode( |
| 175 | MachineBasicBlock &MBB, |
| 176 | MachineBasicBlock::iterator AfterPHIsIt) { |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 177 | // Unlink the PHI node from the basic block, but don't delete the PHI yet. |
| 178 | MachineInstr *MPhi = MBB.remove(MBB.begin()); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 179 | |
Evan Cheng | f870fbc | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 180 | unsigned NumSrcs = (MPhi->getNumOperands() - 1) / 2; |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 181 | unsigned DestReg = MPhi->getOperand(0).getReg(); |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 182 | bool isDead = MPhi->getOperand(0).isDead(); |
Misha Brukman | edf128a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 183 | |
Bill Wendling | ca756d2 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 184 | // Create a new register for the incoming PHI arguments. |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 185 | MachineFunction &MF = *MBB.getParent(); |
Chris Lattner | 84bc542 | 2007-12-31 04:13:23 +0000 | [diff] [blame] | 186 | const TargetRegisterClass *RC = MF.getRegInfo().getRegClass(DestReg); |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 187 | unsigned IncomingReg = 0; |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 188 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 189 | // Insert a register to register copy at the top of the current block (but |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 190 | // after any remaining phi nodes) which copies the new incoming register |
| 191 | // into the phi node destination. |
Owen Anderson | d10fd97 | 2007-12-31 06:32:00 +0000 | [diff] [blame] | 192 | const TargetInstrInfo *TII = MF.getTarget().getInstrInfo(); |
Evan Cheng | b3e0a6d | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 193 | if (isSourceDefinedByImplicitDef(MPhi, MRI)) |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 194 | // If all sources of a PHI node are implicit_def, just emit an |
| 195 | // implicit_def instead of a copy. |
Bill Wendling | d62e06c | 2009-02-03 02:29:34 +0000 | [diff] [blame] | 196 | BuildMI(MBB, AfterPHIsIt, MPhi->getDebugLoc(), |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 197 | TII->get(TargetInstrInfo::IMPLICIT_DEF), DestReg); |
| 198 | else { |
| 199 | IncomingReg = MF.getRegInfo().createVirtualRegister(RC); |
Evan Cheng | f870fbc | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 200 | TII->copyRegToReg(MBB, AfterPHIsIt, DestReg, IncomingReg, RC, RC); |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 201 | } |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 202 | |
Lang Hames | 287b8b0 | 2009-07-23 04:34:03 +0000 | [diff] [blame] | 203 | // Record PHI def. |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 204 | assert(!hasPHIDef(DestReg) && "Vreg has multiple phi-defs?"); |
Lang Hames | 2035463 | 2009-07-23 05:44:24 +0000 | [diff] [blame] | 205 | PHIDefs[DestReg] = &MBB; |
Lang Hames | 287b8b0 | 2009-07-23 04:34:03 +0000 | [diff] [blame] | 206 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 207 | // Update live variable information if there is any. |
Duncan Sands | 1465d61 | 2009-01-28 13:14:17 +0000 | [diff] [blame] | 208 | LiveVariables *LV = getAnalysisIfAvailable<LiveVariables>(); |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 209 | if (LV) { |
| 210 | MachineInstr *PHICopy = prior(AfterPHIsIt); |
| 211 | |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 212 | if (IncomingReg) { |
| 213 | // Increment use count of the newly created virtual register. |
| 214 | LV->getVarInfo(IncomingReg).NumUses++; |
Evan Cheng | 3fefc18 | 2007-04-18 00:36:11 +0000 | [diff] [blame] | 215 | |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 216 | // Add information to LiveVariables to know that the incoming value is |
| 217 | // killed. Note that because the value is defined in several places (once |
| 218 | // each for each incoming block), the "def" block and instruction fields |
| 219 | // for the VarInfo is not filled in. |
| 220 | LV->addVirtualRegisterKilled(IncomingReg, PHICopy); |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 221 | } |
Misha Brukman | edf128a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 222 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 223 | // Since we are going to be deleting the PHI node, if it is the last use of |
| 224 | // any registers, or if the value itself is dead, we need to move this |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 225 | // information over to the new copy we just inserted. |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 226 | LV->removeVirtualRegistersKilled(MPhi); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 227 | |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 228 | // If the result is dead, update LV. |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 229 | if (isDead) { |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 230 | LV->addVirtualRegisterDead(DestReg, PHICopy); |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 231 | LV->removeVirtualRegisterDead(DestReg, MPhi); |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 232 | } |
| 233 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 234 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 235 | // Adjust the VRegPHIUseCount map to account for the removal of this PHI node. |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 236 | for (unsigned i = 1; i != MPhi->getNumOperands(); i += 2) |
Chris Lattner | 8aa797a | 2007-12-30 23:10:15 +0000 | [diff] [blame] | 237 | --VRegPHIUseCount[BBVRegPair(MPhi->getOperand(i + 1).getMBB(), |
| 238 | MPhi->getOperand(i).getReg())]; |
Chris Lattner | 572c770 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 239 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 240 | // Now loop over all of the incoming arguments, changing them to copy into the |
| 241 | // IncomingReg register in the corresponding predecessor basic block. |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 242 | SmallPtrSet<MachineBasicBlock*, 8> MBBsInsertedInto; |
Evan Cheng | f870fbc | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 243 | for (int i = NumSrcs - 1; i >= 0; --i) { |
| 244 | unsigned SrcReg = MPhi->getOperand(i*2+1).getReg(); |
Dan Gohman | 6f0d024 | 2008-02-10 18:45:23 +0000 | [diff] [blame] | 245 | assert(TargetRegisterInfo::isVirtualRegister(SrcReg) && |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 246 | "Machine PHI Operands must all be virtual registers!"); |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 247 | |
Lang Hames | 287b8b0 | 2009-07-23 04:34:03 +0000 | [diff] [blame] | 248 | // Get the MachineBasicBlock equivalent of the BasicBlock that is the source |
| 249 | // path the PHI. |
| 250 | MachineBasicBlock &opBlock = *MPhi->getOperand(i*2+2).getMBB(); |
| 251 | |
| 252 | // Record the kill. |
Lang Hames | 2035463 | 2009-07-23 05:44:24 +0000 | [diff] [blame] | 253 | PHIKills[SrcReg].insert(&opBlock); |
Lang Hames | 287b8b0 | 2009-07-23 04:34:03 +0000 | [diff] [blame] | 254 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 255 | // If source is defined by an implicit def, there is no need to insert a |
Evan Cheng | 9f1c831 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 256 | // copy. |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 257 | MachineInstr *DefMI = MRI->getVRegDef(SrcReg); |
| 258 | if (DefMI->getOpcode() == TargetInstrInfo::IMPLICIT_DEF) { |
| 259 | ImpDefs.insert(DefMI); |
| 260 | continue; |
| 261 | } |
| 262 | |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 263 | // Check to make sure we haven't already emitted the copy for this block. |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 264 | // This can happen because PHI nodes may have multiple entries for the same |
| 265 | // basic block. |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 266 | if (!MBBsInsertedInto.insert(&opBlock)) |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 267 | continue; // If the copy has already been emitted, we're done. |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 268 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 269 | // Find a safe location to insert the copy, this may be the first terminator |
| 270 | // in the block (or end()). |
Duncan Sands | a5fec0d | 2009-03-17 09:46:22 +0000 | [diff] [blame] | 271 | MachineBasicBlock::iterator InsertPos = FindCopyInsertPoint(opBlock, SrcReg); |
Evan Cheng | fc0b80d | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 272 | |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 273 | // Insert the copy. |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 274 | TII->copyRegToReg(opBlock, InsertPos, IncomingReg, SrcReg, RC, RC); |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 275 | |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 276 | // Now update live variable information if we have it. Otherwise we're done |
| 277 | if (!LV) continue; |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 278 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 279 | // We want to be able to insert a kill of the register if this PHI (aka, the |
| 280 | // copy we just inserted) is the last use of the source value. Live |
| 281 | // variable analysis conservatively handles this by saying that the value is |
| 282 | // live until the end of the block the PHI entry lives in. If the value |
| 283 | // really is dead at the PHI copy, there will be no successor blocks which |
| 284 | // have the value live-in. |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 285 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 286 | // Also check to see if this register is in use by another PHI node which |
| 287 | // has not yet been eliminated. If so, it will be killed at an appropriate |
| 288 | // point later. |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 289 | |
| 290 | // Is it used by any PHI instructions in this block? |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 291 | bool ValueIsUsed = VRegPHIUseCount[BBVRegPair(&opBlock, SrcReg)] != 0; |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 292 | |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 293 | // Okay, if we now know that the value is not live out of the block, we can |
| 294 | // add a kill marker in this block saying that it kills the incoming value! |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 295 | // When SplitEdges is enabled, the value is never live out. |
| 296 | if (!ValueIsUsed && (SplitEdges || !isLiveOut(SrcReg, opBlock, *LV))) { |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 297 | // In our final twist, we have to decide which instruction kills the |
Bill Wendling | ae94dda | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 298 | // register. In most cases this is the copy, however, the first |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 299 | // terminator instruction at the end of the block may also use the value. |
| 300 | // In this case, we should mark *it* as being the killing block, not the |
| 301 | // copy. |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 302 | MachineBasicBlock::iterator KillInst = prior(InsertPos); |
| 303 | MachineBasicBlock::iterator Term = opBlock.getFirstTerminator(); |
| 304 | if (Term != opBlock.end()) { |
| 305 | if (Term->readsRegister(SrcReg)) |
| 306 | KillInst = Term; |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 307 | |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 308 | // Check that no other terminators use values. |
| 309 | #ifndef NDEBUG |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 310 | for (MachineBasicBlock::iterator TI = next(Term); TI != opBlock.end(); |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 311 | ++TI) { |
Evan Cheng | 576a270 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 312 | assert(!TI->readsRegister(SrcReg) && |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 313 | "Terminator instructions cannot use virtual registers unless" |
| 314 | "they are the first terminator in a block!"); |
| 315 | } |
| 316 | #endif |
| 317 | } |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 318 | |
Chris Lattner | 2adfa7e | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 319 | // Finally, mark it killed. |
| 320 | LV->addVirtualRegisterKilled(SrcReg, KillInst); |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 321 | |
| 322 | // This vreg no longer lives all of the way through opBlock. |
| 323 | unsigned opBlockNum = opBlock.getNumber(); |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 324 | LV->getVarInfo(SrcReg).AliveBlocks.reset(opBlockNum); |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 325 | } |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 326 | } |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 327 | |
Chris Lattner | 53a79aa | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 328 | // Really delete the PHI instruction now! |
Dan Gohman | 8e5f2c6 | 2008-07-07 23:14:23 +0000 | [diff] [blame] | 329 | MF.DeleteMachineInstr(MPhi); |
Chris Lattner | 6db0756 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 330 | ++NumAtomic; |
Chris Lattner | bc40e89 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 331 | } |
Bill Wendling | ca756d2 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 332 | |
| 333 | /// analyzePHINodes - Gather information about the PHI nodes in here. In |
| 334 | /// particular, we want to map the number of uses of a virtual register which is |
| 335 | /// used in a PHI node. We map that to the BB the vreg is coming from. This is |
| 336 | /// used later to determine when the vreg is killed in the BB. |
| 337 | /// |
Lang Hames | fae02a2 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 338 | void llvm::PHIElimination::analyzePHINodes(const MachineFunction& Fn) { |
Bill Wendling | ca756d2 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 339 | for (MachineFunction::const_iterator I = Fn.begin(), E = Fn.end(); |
| 340 | I != E; ++I) |
| 341 | for (MachineBasicBlock::const_iterator BBI = I->begin(), BBE = I->end(); |
| 342 | BBI != BBE && BBI->getOpcode() == TargetInstrInfo::PHI; ++BBI) |
| 343 | for (unsigned i = 1, e = BBI->getNumOperands(); i != e; i += 2) |
Chris Lattner | 8aa797a | 2007-12-30 23:10:15 +0000 | [diff] [blame] | 344 | ++VRegPHIUseCount[BBVRegPair(BBI->getOperand(i + 1).getMBB(), |
| 345 | BBI->getOperand(i).getReg())]; |
Bill Wendling | ca756d2 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 346 | } |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 347 | |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 348 | void llvm::PHIElimination::SplitPHIEdges(MachineFunction &MF, |
| 349 | MachineBasicBlock &MBB) { |
| 350 | LiveVariables &LV = getAnalysis<LiveVariables>(); |
| 351 | for (MachineBasicBlock::const_iterator BBI = MBB.begin(), BBE = MBB.end(); |
| 352 | BBI != BBE && BBI->getOpcode() == TargetInstrInfo::PHI; ++BBI) { |
| 353 | for (unsigned i = 1, e = BBI->getNumOperands(); i != e; i += 2) { |
| 354 | unsigned Reg = BBI->getOperand(i).getReg(); |
| 355 | MachineBasicBlock *PreMBB = BBI->getOperand(i+1).getMBB(); |
| 356 | // We break edges when registers are live out from the predecessor block |
| 357 | // (not considering PHI nodes). |
| 358 | if (isLiveOut(Reg, *PreMBB, LV)) |
| 359 | SplitCriticalEdge(PreMBB, &MBB); |
| 360 | } |
| 361 | } |
| 362 | } |
| 363 | |
Jakob Stoklund Olesen | e35e3c3 | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 364 | bool llvm::PHIElimination::isLiveOut(unsigned Reg, const MachineBasicBlock &MBB, |
| 365 | LiveVariables &LV) { |
| 366 | LiveVariables::VarInfo &InRegVI = LV.getVarInfo(Reg); |
| 367 | |
| 368 | // Loop over all of the successors of the basic block, checking to see if |
| 369 | // the value is either live in the block, or if it is killed in the block. |
| 370 | std::vector<MachineBasicBlock*> OpSuccBlocks; |
| 371 | |
| 372 | // Otherwise, scan successors, including the BB the PHI node lives in. |
| 373 | for (MachineBasicBlock::const_succ_iterator SI = MBB.succ_begin(), |
| 374 | E = MBB.succ_end(); SI != E; ++SI) { |
| 375 | MachineBasicBlock *SuccMBB = *SI; |
| 376 | |
| 377 | // Is it alive in this successor? |
| 378 | unsigned SuccIdx = SuccMBB->getNumber(); |
| 379 | if (InRegVI.AliveBlocks.test(SuccIdx)) |
| 380 | return true; |
| 381 | OpSuccBlocks.push_back(SuccMBB); |
| 382 | } |
| 383 | |
| 384 | // Check to see if this value is live because there is a use in a successor |
| 385 | // that kills it. |
| 386 | switch (OpSuccBlocks.size()) { |
| 387 | case 1: { |
| 388 | MachineBasicBlock *SuccMBB = OpSuccBlocks[0]; |
| 389 | for (unsigned i = 0, e = InRegVI.Kills.size(); i != e; ++i) |
| 390 | if (InRegVI.Kills[i]->getParent() == SuccMBB) |
| 391 | return true; |
| 392 | break; |
| 393 | } |
| 394 | case 2: { |
| 395 | MachineBasicBlock *SuccMBB1 = OpSuccBlocks[0], *SuccMBB2 = OpSuccBlocks[1]; |
| 396 | for (unsigned i = 0, e = InRegVI.Kills.size(); i != e; ++i) |
| 397 | if (InRegVI.Kills[i]->getParent() == SuccMBB1 || |
| 398 | InRegVI.Kills[i]->getParent() == SuccMBB2) |
| 399 | return true; |
| 400 | break; |
| 401 | } |
| 402 | default: |
| 403 | std::sort(OpSuccBlocks.begin(), OpSuccBlocks.end()); |
| 404 | for (unsigned i = 0, e = InRegVI.Kills.size(); i != e; ++i) |
| 405 | if (std::binary_search(OpSuccBlocks.begin(), OpSuccBlocks.end(), |
| 406 | InRegVI.Kills[i]->getParent())) |
| 407 | return true; |
| 408 | } |
| 409 | return false; |
| 410 | } |
Jakob Stoklund Olesen | f235f13 | 2009-11-10 22:01:05 +0000 | [diff] [blame^] | 411 | |
| 412 | MachineBasicBlock *PHIElimination::SplitCriticalEdge(MachineBasicBlock *A, |
| 413 | MachineBasicBlock *B) { |
| 414 | assert(A && B && "Missing MBB end point"); |
| 415 | ++NumSplits; |
| 416 | |
| 417 | MachineFunction *MF = A->getParent(); |
| 418 | MachineBasicBlock *NMBB = MF->CreateMachineBasicBlock(B->getBasicBlock()); |
| 419 | MF->push_back(NMBB); |
| 420 | const unsigned NewNum = NMBB->getNumber(); |
| 421 | DEBUG(errs() << "PHIElimination splitting critical edge:" |
| 422 | " BB#" << A->getNumber() |
| 423 | << " -- BB#" << NewNum |
| 424 | << " -- BB#" << B->getNumber() << '\n'); |
| 425 | |
| 426 | A->ReplaceUsesOfBlockWith(B, NMBB); |
| 427 | NMBB->addSuccessor(B); |
| 428 | |
| 429 | // Insert unconditional "jump B" instruction in NMBB. |
| 430 | SmallVector<MachineOperand, 4> Cond; |
| 431 | MF->getTarget().getInstrInfo()->InsertBranch(*NMBB, B, NULL, Cond); |
| 432 | |
| 433 | LiveVariables *LV = getAnalysisIfAvailable<LiveVariables>(); |
| 434 | if (LV) |
| 435 | LV->addNewBlock(NMBB, B); |
| 436 | |
| 437 | // Fix PHI nodes in B so they refer to NMBB instead of A |
| 438 | for (MachineBasicBlock::iterator i = B->begin(), e = B->end(); |
| 439 | i != e && i->getOpcode() == TargetInstrInfo::PHI; ++i) |
| 440 | for (unsigned ni = 1, ne = i->getNumOperands(); ni != ne; ni += 2) |
| 441 | if (i->getOperand(ni+1).getMBB() == A) { |
| 442 | i->getOperand(ni+1).setMBB(NMBB); |
| 443 | // Mark PHI sources as passing live through NMBB |
| 444 | if (LV) |
| 445 | LV->getVarInfo(i->getOperand(ni).getReg()).AliveBlocks.set(NewNum); |
| 446 | } |
| 447 | return NMBB; |
| 448 | } |
| 449 | |
| 450 | |