blob: 58f0347bf9f2341aa6e3349cab35caa5c61aa07f [file] [log] [blame]
Chris Lattnerb03832d2002-09-24 00:08:39 +00001//===- BreakCriticalEdges.cpp - Critical Edge Elimination Pass ------------===//
Misha Brukmanb1c93172005-04-21 23:48:37 +00002//
John Criswell482202a2003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
5// This file was developed by the LLVM research group and is distributed under
6// the University of Illinois Open Source License. See LICENSE.TXT for details.
Misha Brukmanb1c93172005-04-21 23:48:37 +00007//
John Criswell482202a2003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattnerb03832d2002-09-24 00:08:39 +00009//
10// BreakCriticalEdges pass - Break all of the critical edges in the CFG by
11// inserting a dummy basic block. This pass may be "required" by passes that
12// cannot deal with critical edges. For this usage, the structure type is
13// forward declared. This pass obviously invalidates the CFG, but can update
Chris Lattner1e6d3052003-11-10 04:42:42 +000014// forward dominator (set, immediate dominators, tree, and frontier)
15// information.
Chris Lattnerb03832d2002-09-24 00:08:39 +000016//
17//===----------------------------------------------------------------------===//
18
19#include "llvm/Transforms/Scalar.h"
Chris Lattner44743362003-11-10 04:10:50 +000020#include "llvm/Transforms/Utils/BasicBlockUtils.h"
Chris Lattnerb03832d2002-09-24 00:08:39 +000021#include "llvm/Analysis/Dominators.h"
Chris Lattner89c1dfc2005-08-13 01:38:43 +000022#include "llvm/Analysis/LoopInfo.h"
Chris Lattnerb03832d2002-09-24 00:08:39 +000023#include "llvm/Function.h"
Misha Brukman63b38bd2004-07-29 17:30:56 +000024#include "llvm/Instructions.h"
Chris Lattner7b9020a2005-03-17 15:38:16 +000025#include "llvm/Type.h"
Chris Lattner75f80bd2002-09-24 15:51:56 +000026#include "llvm/Support/CFG.h"
Chris Lattner3d27be12006-08-27 12:54:02 +000027#include "llvm/Support/Compiler.h"
Chris Lattner8aca0ee2006-10-03 07:02:02 +000028#include "llvm/ADT/SmallVector.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000029#include "llvm/ADT/Statistic.h"
Chris Lattnerdf3c3422004-01-09 06:12:26 +000030using namespace llvm;
Brian Gaeke960707c2003-11-11 22:41:34 +000031
Chris Lattner4bec6652002-09-24 15:43:12 +000032namespace {
Chris Lattnerbf3a0992002-10-01 22:38:41 +000033 Statistic<> NumBroken("break-crit-edges", "Number of blocks inserted");
Chris Lattnerb03832d2002-09-24 00:08:39 +000034
Chris Lattner996795b2006-06-28 23:17:24 +000035 struct VISIBILITY_HIDDEN BreakCriticalEdges : public FunctionPass {
Chris Lattner4bec6652002-09-24 15:43:12 +000036 virtual bool runOnFunction(Function &F);
Misha Brukmanb1c93172005-04-21 23:48:37 +000037
Chris Lattner4bec6652002-09-24 15:43:12 +000038 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
Chris Lattner0770d8e2006-01-11 05:11:13 +000039 AU.addPreserved<ETForest>();
Chris Lattner4bec6652002-09-24 15:43:12 +000040 AU.addPreserved<DominatorSet>();
41 AU.addPreserved<ImmediateDominators>();
42 AU.addPreserved<DominatorTree>();
Chris Lattner12764c82002-10-31 02:44:36 +000043 AU.addPreserved<DominanceFrontier>();
Chris Lattner89c1dfc2005-08-13 01:38:43 +000044 AU.addPreserved<LoopInfo>();
Chris Lattner72272a72003-10-12 21:52:28 +000045
46 // No loop canonicalization guarantees are broken by this pass.
47 AU.addPreservedID(LoopSimplifyID);
Chris Lattner4bec6652002-09-24 15:43:12 +000048 }
49 };
Chris Lattnerb03832d2002-09-24 00:08:39 +000050
Chris Lattnerc2d3d312006-08-27 22:42:52 +000051 RegisterPass<BreakCriticalEdges> X("break-crit-edges",
Chris Lattner4bec6652002-09-24 15:43:12 +000052 "Break critical edges in CFG");
53}
Chris Lattnerb03832d2002-09-24 00:08:39 +000054
Chris Lattner75f80bd2002-09-24 15:51:56 +000055// Publically exposed interface to pass...
Chris Lattnerdf3c3422004-01-09 06:12:26 +000056const PassInfo *llvm::BreakCriticalEdgesID = X.getPassInfo();
Chris Lattner7471b962004-07-31 10:01:58 +000057FunctionPass *llvm::createBreakCriticalEdgesPass() {
58 return new BreakCriticalEdges();
59}
Chris Lattnerb03832d2002-09-24 00:08:39 +000060
Chris Lattner1e6d3052003-11-10 04:42:42 +000061// runOnFunction - Loop over all of the edges in the CFG, breaking critical
62// edges as they are found.
63//
64bool BreakCriticalEdges::runOnFunction(Function &F) {
65 bool Changed = false;
66 for (Function::iterator I = F.begin(), E = F.end(); I != E; ++I) {
67 TerminatorInst *TI = I->getTerminator();
68 if (TI->getNumSuccessors() > 1)
69 for (unsigned i = 0, e = TI->getNumSuccessors(); i != e; ++i)
70 if (SplitCriticalEdge(TI, i, this)) {
71 ++NumBroken;
72 Changed = true;
73 }
74 }
75
76 return Changed;
77}
78
79//===----------------------------------------------------------------------===//
80// Implementation of the external critical edge manipulation functions
81//===----------------------------------------------------------------------===//
Chris Lattner75f80bd2002-09-24 15:51:56 +000082
83// isCriticalEdge - Return true if the specified edge is a critical edge.
84// Critical edges are edges from a block with multiple successors to a block
85// with multiple predecessors.
86//
Chris Lattner3e763f52006-10-28 06:58:17 +000087bool llvm::isCriticalEdge(const TerminatorInst *TI, unsigned SuccNum,
88 bool AllowIdenticalEdges) {
Chris Lattner75f80bd2002-09-24 15:51:56 +000089 assert(SuccNum < TI->getNumSuccessors() && "Illegal edge specification!");
Chris Lattner5ac72de2002-10-08 21:06:27 +000090 if (TI->getNumSuccessors() == 1) return false;
Chris Lattner75f80bd2002-09-24 15:51:56 +000091
92 const BasicBlock *Dest = TI->getSuccessor(SuccNum);
93 pred_const_iterator I = pred_begin(Dest), E = pred_end(Dest);
94
95 // If there is more than one predecessor, this is a critical edge...
96 assert(I != E && "No preds, but we have an edge to the block?");
Chris Lattner3e763f52006-10-28 06:58:17 +000097 const BasicBlock *FirstPred = *I;
Chris Lattner75f80bd2002-09-24 15:51:56 +000098 ++I; // Skip one edge due to the incoming arc from TI.
Chris Lattner3e763f52006-10-28 06:58:17 +000099 if (!AllowIdenticalEdges)
100 return I != E;
101
102 // If AllowIdenticalEdges is true, then we allow this edge to be considered
103 // non-critical iff all preds come from TI's block.
104 for (; I != E; ++I)
105 if (*I != FirstPred) return true;
106 return false;
Chris Lattner75f80bd2002-09-24 15:51:56 +0000107}
108
Chris Lattner44743362003-11-10 04:10:50 +0000109// SplitCriticalEdge - If this edge is a critical edge, insert a new node to
110// split the critical edge. This will update DominatorSet, ImmediateDominator,
111// DominatorTree, and DominatorFrontier information if it is available, thus
Chris Lattner80ea2072006-10-28 06:44:56 +0000112// calling this pass will not invalidate any of them. This returns true if
113// the edge was split, false otherwise. This ensures that all edges to that
114// dest go to one block instead of each going to a different block.
Chris Lattner75f80bd2002-09-24 15:51:56 +0000115//
Chris Lattner80ea2072006-10-28 06:44:56 +0000116bool llvm::SplitCriticalEdge(TerminatorInst *TI, unsigned SuccNum, Pass *P,
117 bool MergeIdenticalEdges) {
Chris Lattner3e763f52006-10-28 06:58:17 +0000118 if (!isCriticalEdge(TI, SuccNum, MergeIdenticalEdges)) return false;
Chris Lattner75f80bd2002-09-24 15:51:56 +0000119 BasicBlock *TIBB = TI->getParent();
Chris Lattner12764c82002-10-31 02:44:36 +0000120 BasicBlock *DestBB = TI->getSuccessor(SuccNum);
Chris Lattner75f80bd2002-09-24 15:51:56 +0000121
122 // Create a new basic block, linking it into the CFG.
Chris Lattner12764c82002-10-31 02:44:36 +0000123 BasicBlock *NewBB = new BasicBlock(TIBB->getName() + "." +
124 DestBB->getName() + "_crit_edge");
Chris Lattner75f80bd2002-09-24 15:51:56 +0000125 // Create our unconditional branch...
Chris Lattnera2960002003-11-21 16:52:05 +0000126 new BranchInst(DestBB, NewBB);
Misha Brukmanb1c93172005-04-21 23:48:37 +0000127
Chris Lattner80ea2072006-10-28 06:44:56 +0000128 // Branch to the new block, breaking the edge.
Chris Lattner75f80bd2002-09-24 15:51:56 +0000129 TI->setSuccessor(SuccNum, NewBB);
130
131 // Insert the block into the function... right after the block TI lives in.
132 Function &F = *TIBB->getParent();
133 F.getBasicBlockList().insert(TIBB->getNext(), NewBB);
Chris Lattner80ea2072006-10-28 06:44:56 +0000134
Chris Lattner75f80bd2002-09-24 15:51:56 +0000135 // If there are any PHI nodes in DestBB, we need to update them so that they
136 // merge incoming values from NewBB instead of from TIBB.
137 //
Reid Spencer66149462004-09-15 17:06:42 +0000138 for (BasicBlock::iterator I = DestBB->begin(); isa<PHINode>(I); ++I) {
139 PHINode *PN = cast<PHINode>(I);
Chris Lattner2de229f2004-02-29 22:24:41 +0000140 // We no longer enter through TIBB, now we come in through NewBB. Revector
141 // exactly one entry in the PHI node that used to come from TIBB to come
142 // from NewBB.
Chris Lattnerb7ebe652005-08-12 21:58:07 +0000143 int BBIdx = PN->getBasicBlockIndex(TIBB);
144 PN->setIncomingBlock(BBIdx, NewBB);
Chris Lattner75f80bd2002-09-24 15:51:56 +0000145 }
Chris Lattner80ea2072006-10-28 06:44:56 +0000146
147 // If there are any other edges from TIBB to DestBB, update those to go
148 // through the split block, making those edges non-critical as well (and
149 // reducing the number of phi entries in the DestBB if relevant).
150 if (MergeIdenticalEdges) {
151 for (unsigned i = SuccNum+1, e = TI->getNumSuccessors(); i != e; ++i) {
152 if (TI->getSuccessor(i) != DestBB) continue;
153
154 // Remove an entry for TIBB from DestBB phi nodes.
155 DestBB->removePredecessor(TIBB);
156
157 // We found another edge to DestBB, go to NewBB instead.
158 TI->setSuccessor(i, NewBB);
159 }
160 }
161
162
Chris Lattner75f80bd2002-09-24 15:51:56 +0000163
Chris Lattner5ac72de2002-10-08 21:06:27 +0000164 // If we don't have a pass object, we can't update anything...
Chris Lattner44743362003-11-10 04:10:50 +0000165 if (P == 0) return true;
Chris Lattner5ac72de2002-10-08 21:06:27 +0000166
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000167 // Now update analysis information. Since the only predecessor of NewBB is
168 // the TIBB, TIBB clearly dominates NewBB. TIBB usually doesn't dominate
169 // anything, as there are other successors of DestBB. However, if all other
170 // predecessors of DestBB are already dominated by DestBB (e.g. DestBB is a
171 // loop header) then NewBB dominates DestBB.
172 SmallVector<BasicBlock*, 8> OtherPreds;
Chris Lattner75f80bd2002-09-24 15:51:56 +0000173
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000174 for (pred_iterator I = pred_begin(DestBB), E = pred_end(DestBB); I != E; ++I)
175 if (*I != NewBB)
176 OtherPreds.push_back(*I);
177
178 // NewBBDominatesDestBB is valid if OtherPreds is empty, otherwise it isn't
179 // yet computed.
180 bool NewBBDominatesDestBB = true;
181
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000182 // Should we update DominatorSet information?
Chris Lattner5ac72de2002-10-08 21:06:27 +0000183 if (DominatorSet *DS = P->getAnalysisToUpdate<DominatorSet>()) {
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000184 // The blocks that dominate the new one are the blocks that dominate TIBB
185 // plus the new block itself.
186 DominatorSet::DomSetType DomSet = DS->getDominators(TIBB);
187 DomSet.insert(NewBB); // A block always dominates itself.
188 DS->addBasicBlock(NewBB, DomSet);
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000189
190 // If NewBBDominatesDestBB hasn't been computed yet, do so with DS.
191 if (!OtherPreds.empty()) {
192 while (!OtherPreds.empty() && NewBBDominatesDestBB) {
193 NewBBDominatesDestBB = DS->dominates(DestBB, OtherPreds.back());
194 OtherPreds.pop_back();
195 }
196 OtherPreds.clear();
197 }
198
199 // If NewBBDominatesDestBB, then NewBB dominates DestBB, otherwise it
200 // doesn't dominate anything. If NewBB does dominates DestBB, then it
201 // dominates everything that DestBB dominates.
202 if (NewBBDominatesDestBB) {
203 for (DominatorSet::iterator I = DS->begin(), E = DS->end(); I != E; ++I)
204 if (I->second.count(DestBB))
205 I->second.insert(NewBB);
206 }
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000207 }
Chris Lattner75f80bd2002-09-24 15:51:56 +0000208
Misha Brukman8b2bd4e2003-10-10 17:57:28 +0000209 // Should we update ImmediateDominator information?
Chris Lattner5ac72de2002-10-08 21:06:27 +0000210 if (ImmediateDominators *ID = P->getAnalysisToUpdate<ImmediateDominators>()) {
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000211 // TIBB is the new immediate dominator for NewBB.
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000212 ID->addNewBlock(NewBB, TIBB);
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000213
214 // If NewBBDominatesDestBB hasn't been computed yet, do so with ID.
215 if (!OtherPreds.empty()) {
216 while (!OtherPreds.empty() && NewBBDominatesDestBB) {
217 NewBBDominatesDestBB = ID->dominates(DestBB, OtherPreds.back());
218 OtherPreds.pop_back();
219 }
220 OtherPreds.clear();
221 }
222
223 // If NewBBDominatesDestBB, then NewBB dominates DestBB, otherwise it
224 // doesn't dominate anything.
225 if (NewBBDominatesDestBB)
226 ID->setImmediateDominator(DestBB, NewBB);
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000227 }
Misha Brukmanb1c93172005-04-21 23:48:37 +0000228
Chris Lattner0770d8e2006-01-11 05:11:13 +0000229 // Update the forest?
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000230 if (ETForest *EF = P->getAnalysisToUpdate<ETForest>()) {
231 // NewBB is dominated by TIBB.
Chris Lattner0770d8e2006-01-11 05:11:13 +0000232 EF->addNewBlock(NewBB, TIBB);
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000233
234 // If NewBBDominatesDestBB hasn't been computed yet, do so with EF.
235 if (!OtherPreds.empty()) {
236 while (!OtherPreds.empty() && NewBBDominatesDestBB) {
237 NewBBDominatesDestBB = EF->dominates(DestBB, OtherPreds.back());
238 OtherPreds.pop_back();
239 }
240 OtherPreds.clear();
241 }
242
243 // If NewBBDominatesDestBB, then NewBB dominates DestBB, otherwise it
244 // doesn't dominate anything.
245 if (NewBBDominatesDestBB)
246 EF->setImmediateDominator(DestBB, NewBB);
247 }
Chris Lattner0770d8e2006-01-11 05:11:13 +0000248
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000249 // Should we update DominatorTree information?
Chris Lattner5ac72de2002-10-08 21:06:27 +0000250 if (DominatorTree *DT = P->getAnalysisToUpdate<DominatorTree>()) {
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000251 DominatorTree::Node *TINode = DT->getNode(TIBB);
Misha Brukmanb1c93172005-04-21 23:48:37 +0000252
Chris Lattnerbedbd6b2002-09-26 16:18:51 +0000253 // The new block is not the immediate dominator for any other nodes, but
254 // TINode is the immediate dominator for the new node.
255 //
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000256 if (TINode) { // Don't break unreachable code!
257 DominatorTree::Node *NewBBNode = DT->createNewNode(NewBB, TINode);
258 DominatorTree::Node *DestBBNode = 0;
259
260 // If NewBBDominatesDestBB hasn't been computed yet, do so with DT.
261 if (!OtherPreds.empty()) {
262 DestBBNode = DT->getNode(DestBB);
263 while (!OtherPreds.empty() && NewBBDominatesDestBB) {
264 if (DominatorTree::Node *OPNode = DT->getNode(OtherPreds.back()))
265 NewBBDominatesDestBB = DestBBNode->dominates(OPNode);
266 OtherPreds.pop_back();
267 }
268 OtherPreds.clear();
269 }
270
271 // If NewBBDominatesDestBB, then NewBB dominates DestBB, otherwise it
272 // doesn't dominate anything.
273 if (NewBBDominatesDestBB) {
274 if (!DestBBNode) DestBBNode = DT->getNode(DestBB);
275 DT->changeImmediateDominator(DestBBNode, NewBBNode);
276 }
277 }
Chris Lattner75f80bd2002-09-24 15:51:56 +0000278 }
Chris Lattner12764c82002-10-31 02:44:36 +0000279
280 // Should we update DominanceFrontier information?
281 if (DominanceFrontier *DF = P->getAnalysisToUpdate<DominanceFrontier>()) {
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000282 // If NewBBDominatesDestBB hasn't been computed yet, do so with DF.
283 if (!OtherPreds.empty()) {
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000284 // FIXME: IMPLEMENT THIS!
Chris Lattner52886e72006-10-04 04:58:58 +0000285 assert(0 && "Requiring domfrontiers but not idom/domtree/domset."
286 " not implemented yet!");
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000287 }
288
Chris Lattner12764c82002-10-31 02:44:36 +0000289 // Since the new block is dominated by its only predecessor TIBB,
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000290 // it cannot be in any block's dominance frontier. If NewBB dominates
291 // DestBB, its dominance frontier is the same as DestBB's, otherwise it is
292 // just {DestBB}.
Chris Lattner12764c82002-10-31 02:44:36 +0000293 DominanceFrontier::DomSetType NewDFSet;
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000294 if (NewBBDominatesDestBB) {
295 DominanceFrontier::iterator I = DF->find(DestBB);
296 if (I != DF->end())
297 DF->addBasicBlock(NewBB, I->second);
298 else
299 DF->addBasicBlock(NewBB, DominanceFrontier::DomSetType());
300 } else {
301 DominanceFrontier::DomSetType NewDFSet;
302 NewDFSet.insert(DestBB);
303 DF->addBasicBlock(NewBB, NewDFSet);
304 }
Chris Lattner12764c82002-10-31 02:44:36 +0000305 }
Chris Lattner89c1dfc2005-08-13 01:38:43 +0000306
307 // Update LoopInfo if it is around.
308 if (LoopInfo *LI = P->getAnalysisToUpdate<LoopInfo>()) {
309 // If one or the other blocks were not in a loop, the new block is not
310 // either, and thus LI doesn't need to be updated.
311 if (Loop *TIL = LI->getLoopFor(TIBB))
312 if (Loop *DestLoop = LI->getLoopFor(DestBB)) {
313 if (TIL == DestLoop) {
314 // Both in the same loop, the NewBB joins loop.
315 DestLoop->addBasicBlockToLoop(NewBB, *LI);
316 } else if (TIL->contains(DestLoop->getHeader())) {
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000317 // Edge from an outer loop to an inner loop. Add to the outer loop.
Chris Lattner89c1dfc2005-08-13 01:38:43 +0000318 TIL->addBasicBlockToLoop(NewBB, *LI);
319 } else if (DestLoop->contains(TIL->getHeader())) {
Chris Lattner8aca0ee2006-10-03 07:02:02 +0000320 // Edge from an inner loop to an outer loop. Add to the outer loop.
Chris Lattner89c1dfc2005-08-13 01:38:43 +0000321 DestLoop->addBasicBlockToLoop(NewBB, *LI);
322 } else {
323 // Edge from two loops with no containment relation. Because these
324 // are natural loops, we know that the destination block must be the
325 // header of its loop (adding a branch into a loop elsewhere would
326 // create an irreducible loop).
327 assert(DestLoop->getHeader() == DestBB &&
328 "Should not create irreducible loops!");
329 if (Loop *P = DestLoop->getParentLoop())
330 P->addBasicBlockToLoop(NewBB, *LI);
331 }
332 }
Chris Lattner89c1dfc2005-08-13 01:38:43 +0000333 }
Chris Lattner44743362003-11-10 04:10:50 +0000334 return true;
Chris Lattner75f80bd2002-09-24 15:51:56 +0000335}