blob: 72fea80e65a4f2f81e10afe82a14afdff23e5135 [file] [log] [blame]
Matt Arsenaultad966ea2013-06-19 20:18:24 +00001//===-- StructurizeCFG.cpp ------------------------------------------------===//
Tom Stellard6b7d99d2012-12-19 22:10:31 +00002//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
Tom Stellard6b7d99d2012-12-19 22:10:31 +00009
Matt Arsenaultad966ea2013-06-19 20:18:24 +000010#define DEBUG_TYPE "structurizecfg"
11#include "llvm/Transforms/Scalar.h"
Christian Konig43770272013-03-26 10:24:20 +000012#include "llvm/ADT/MapVector.h"
Benjamin Kramer5c352902013-05-23 17:10:37 +000013#include "llvm/ADT/SCCIterator.h"
Tom Stellard6b7d99d2012-12-19 22:10:31 +000014#include "llvm/Analysis/RegionInfo.h"
Chandler Carruth58a2cbe2013-01-02 10:22:59 +000015#include "llvm/Analysis/RegionIterator.h"
Tom Stellard6b7d99d2012-12-19 22:10:31 +000016#include "llvm/Analysis/RegionPass.h"
Chandler Carruth0b8c9a82013-01-02 11:36:10 +000017#include "llvm/IR/Module.h"
Christian Konigef6b2482013-02-16 11:27:50 +000018#include "llvm/Support/PatternMatch.h"
Benjamin Kramer5c352902013-05-23 17:10:37 +000019#include "llvm/Transforms/Utils/SSAUpdater.h"
Tom Stellard6b7d99d2012-12-19 22:10:31 +000020
21using namespace llvm;
Christian Konigef6b2482013-02-16 11:27:50 +000022using namespace llvm::PatternMatch;
Tom Stellard6b7d99d2012-12-19 22:10:31 +000023
24namespace {
25
26// Definition of the complex types used in this pass.
27
28typedef std::pair<BasicBlock *, Value *> BBValuePair;
Tom Stellard6b7d99d2012-12-19 22:10:31 +000029
30typedef SmallVector<RegionNode*, 8> RNVector;
31typedef SmallVector<BasicBlock*, 8> BBVector;
Tom Stellard27f5d062013-02-08 22:24:37 +000032typedef SmallVector<BranchInst*, 8> BranchVector;
Tom Stellard6b7d99d2012-12-19 22:10:31 +000033typedef SmallVector<BBValuePair, 2> BBValueVector;
34
Tom Stellard27f5d062013-02-08 22:24:37 +000035typedef SmallPtrSet<BasicBlock *, 8> BBSet;
36
Christian Konig43770272013-03-26 10:24:20 +000037typedef MapVector<PHINode *, BBValueVector> PhiMap;
38typedef MapVector<BasicBlock *, BBVector> BB2BBVecMap;
39
Christian Konigf0e469b2013-02-16 11:27:29 +000040typedef DenseMap<DomTreeNode *, unsigned> DTN2UnsignedMap;
Tom Stellard6b7d99d2012-12-19 22:10:31 +000041typedef DenseMap<BasicBlock *, PhiMap> BBPhiMap;
42typedef DenseMap<BasicBlock *, Value *> BBPredicates;
43typedef DenseMap<BasicBlock *, BBPredicates> PredMap;
Christian Konig623977d2013-02-16 11:27:45 +000044typedef DenseMap<BasicBlock *, BasicBlock*> BB2BBMap;
Tom Stellard6b7d99d2012-12-19 22:10:31 +000045
46// The name for newly created blocks.
47
Craig Topper4172a8a2013-07-16 01:17:10 +000048static const char *const FlowBlockName = "Flow";
Tom Stellard6b7d99d2012-12-19 22:10:31 +000049
Christian Konigf0e469b2013-02-16 11:27:29 +000050/// @brief Find the nearest common dominator for multiple BasicBlocks
51///
Matt Arsenaultad966ea2013-06-19 20:18:24 +000052/// Helper class for StructurizeCFG
Christian Konigf0e469b2013-02-16 11:27:29 +000053/// TODO: Maybe move into common code
54class NearestCommonDominator {
Christian Konigf0e469b2013-02-16 11:27:29 +000055 DominatorTree *DT;
56
57 DTN2UnsignedMap IndexMap;
58
59 BasicBlock *Result;
60 unsigned ResultIndex;
61 bool ExplicitMentioned;
62
63public:
64 /// \brief Start a new query
65 NearestCommonDominator(DominatorTree *DomTree) {
66 DT = DomTree;
67 Result = 0;
68 }
69
70 /// \brief Add BB to the resulting dominator
71 void addBlock(BasicBlock *BB, bool Remember = true) {
Christian Konigf0e469b2013-02-16 11:27:29 +000072 DomTreeNode *Node = DT->getNode(BB);
73
74 if (Result == 0) {
75 unsigned Numbering = 0;
76 for (;Node;Node = Node->getIDom())
77 IndexMap[Node] = ++Numbering;
78 Result = BB;
79 ResultIndex = 1;
80 ExplicitMentioned = Remember;
81 return;
82 }
83
84 for (;Node;Node = Node->getIDom())
85 if (IndexMap.count(Node))
86 break;
87 else
88 IndexMap[Node] = 0;
89
90 assert(Node && "Dominator tree invalid!");
91
92 unsigned Numbering = IndexMap[Node];
93 if (Numbering > ResultIndex) {
94 Result = Node->getBlock();
95 ResultIndex = Numbering;
96 ExplicitMentioned = Remember && (Result == BB);
97 } else if (Numbering == ResultIndex) {
98 ExplicitMentioned |= Remember;
99 }
100 }
101
102 /// \brief Is "Result" one of the BBs added with "Remember" = True?
103 bool wasResultExplicitMentioned() {
104 return ExplicitMentioned;
105 }
106
107 /// \brief Get the query result
108 BasicBlock *getResult() {
109 return Result;
110 }
111};
112
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000113/// @brief Transforms the control flow graph on one single entry/exit region
114/// at a time.
115///
116/// After the transform all "If"/"Then"/"Else" style control flow looks like
117/// this:
118///
119/// \verbatim
120/// 1
121/// ||
122/// | |
123/// 2 |
124/// | /
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000125/// |/
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000126/// 3
127/// || Where:
128/// | | 1 = "If" block, calculates the condition
129/// 4 | 2 = "Then" subregion, runs if the condition is true
130/// | / 3 = "Flow" blocks, newly inserted flow blocks, rejoins the flow
131/// |/ 4 = "Else" optional subregion, runs if the condition is false
132/// 5 5 = "End" block, also rejoins the control flow
133/// \endverbatim
134///
135/// Control flow is expressed as a branch where the true exit goes into the
136/// "Then"/"Else" region, while the false exit skips the region
137/// The condition for the optional "Else" region is expressed as a PHI node.
138/// The incomming values of the PHI node are true for the "If" edge and false
139/// for the "Then" edge.
140///
141/// Additionally to that even complicated loops look like this:
142///
143/// \verbatim
144/// 1
145/// ||
146/// | |
147/// 2 ^ Where:
148/// | / 1 = "Entry" block
149/// |/ 2 = "Loop" optional subregion, with all exits at "Flow" block
150/// 3 3 = "Flow" block, with back edge to entry block
151/// |
152/// \endverbatim
153///
154/// The back edge of the "Flow" block is always on the false side of the branch
155/// while the true side continues the general flow. So the loop condition
156/// consist of a network of PHI nodes where the true incoming values expresses
157/// breaks and the false values expresses continue states.
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000158class StructurizeCFG : public RegionPass {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000159 Type *Boolean;
160 ConstantInt *BoolTrue;
161 ConstantInt *BoolFalse;
162 UndefValue *BoolUndef;
163
164 Function *Func;
165 Region *ParentRegion;
166
167 DominatorTree *DT;
168
169 RNVector Order;
Tom Stellardf4e471a2013-02-08 22:24:38 +0000170 BBSet Visited;
Christian Konig623977d2013-02-16 11:27:45 +0000171
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000172 BBPhiMap DeletedPhis;
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000173 BB2BBVecMap AddedPhis;
Christian Konig623977d2013-02-16 11:27:45 +0000174
175 PredMap Predicates;
Tom Stellard27f5d062013-02-08 22:24:37 +0000176 BranchVector Conditions;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000177
Christian Konig623977d2013-02-16 11:27:45 +0000178 BB2BBMap Loops;
179 PredMap LoopPreds;
180 BranchVector LoopConds;
181
182 RegionNode *PrevNode;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000183
184 void orderNodes();
185
Christian Konig623977d2013-02-16 11:27:45 +0000186 void analyzeLoops(RegionNode *N);
187
Christian Konigef6b2482013-02-16 11:27:50 +0000188 Value *invert(Value *Condition);
189
Tom Stellard27f5d062013-02-08 22:24:37 +0000190 Value *buildCondition(BranchInst *Term, unsigned Idx, bool Invert);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000191
Christian Konig623977d2013-02-16 11:27:45 +0000192 void gatherPredicates(RegionNode *N);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000193
194 void collectInfos();
195
Christian Konig623977d2013-02-16 11:27:45 +0000196 void insertConditions(bool Loops);
Tom Stellard27f5d062013-02-08 22:24:37 +0000197
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000198 void delPhiValues(BasicBlock *From, BasicBlock *To);
199
200 void addPhiValues(BasicBlock *From, BasicBlock *To);
201
202 void setPhiValues();
203
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000204 void killTerminator(BasicBlock *BB);
205
Tom Stellardf4e471a2013-02-08 22:24:38 +0000206 void changeExit(RegionNode *Node, BasicBlock *NewExit,
207 bool IncludeDominator);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000208
Tom Stellardf4e471a2013-02-08 22:24:38 +0000209 BasicBlock *getNextFlow(BasicBlock *Dominator);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000210
Christian Konig623977d2013-02-16 11:27:45 +0000211 BasicBlock *needPrefix(bool NeedEmpty);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000212
Tom Stellardf4e471a2013-02-08 22:24:38 +0000213 BasicBlock *needPostfix(BasicBlock *Flow, bool ExitUseAllowed);
214
Christian Konig623977d2013-02-16 11:27:45 +0000215 void setPrevNode(BasicBlock *BB);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000216
217 bool dominatesPredicates(BasicBlock *BB, RegionNode *Node);
218
Christian Konig623977d2013-02-16 11:27:45 +0000219 bool isPredictableTrue(RegionNode *Node);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000220
Christian Konig623977d2013-02-16 11:27:45 +0000221 void wireFlow(bool ExitUseAllowed, BasicBlock *LoopEnd);
222
223 void handleLoops(bool ExitUseAllowed, BasicBlock *LoopEnd);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000224
225 void createFlow();
226
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000227 void rebuildSSA();
228
229public:
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000230 static char ID;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000231
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000232 StructurizeCFG() :
233 RegionPass(ID) {
Tom Stellardaf7ae9d2013-10-02 17:04:59 +0000234 initializeStructurizeCFGPass(*PassRegistry::getPassRegistry());
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000235 }
236
Christian Konig777962f2013-03-01 09:46:11 +0000237 using Pass::doInitialization;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000238 virtual bool doInitialization(Region *R, RGPassManager &RGM);
239
240 virtual bool runOnRegion(Region *R, RGPassManager &RGM);
241
242 virtual const char *getPassName() const {
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000243 return "Structurize control flow";
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000244 }
245
246 void getAnalysisUsage(AnalysisUsage &AU) const {
Tom Stellardaf7ae9d2013-10-02 17:04:59 +0000247 AU.addRequiredID(LowerSwitchID);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000248 AU.addRequired<DominatorTree>();
249 AU.addPreserved<DominatorTree>();
250 RegionPass::getAnalysisUsage(AU);
251 }
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000252};
253
254} // end anonymous namespace
255
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000256char StructurizeCFG::ID = 0;
257
258INITIALIZE_PASS_BEGIN(StructurizeCFG, "structurizecfg", "Structurize the CFG",
259 false, false)
Tom Stellardaf7ae9d2013-10-02 17:04:59 +0000260INITIALIZE_PASS_DEPENDENCY(LowerSwitch)
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000261INITIALIZE_PASS_DEPENDENCY(DominatorTree)
262INITIALIZE_PASS_DEPENDENCY(RegionInfo)
263INITIALIZE_PASS_END(StructurizeCFG, "structurizecfg", "Structurize the CFG",
264 false, false)
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000265
266/// \brief Initialize the types and constants used in the pass
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000267bool StructurizeCFG::doInitialization(Region *R, RGPassManager &RGM) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000268 LLVMContext &Context = R->getEntry()->getContext();
269
270 Boolean = Type::getInt1Ty(Context);
271 BoolTrue = ConstantInt::getTrue(Context);
272 BoolFalse = ConstantInt::getFalse(Context);
273 BoolUndef = UndefValue::get(Boolean);
274
275 return false;
276}
277
278/// \brief Build up the general order of nodes
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000279void StructurizeCFG::orderNodes() {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000280 scc_iterator<Region *> I = scc_begin(ParentRegion),
281 E = scc_end(ParentRegion);
282 for (Order.clear(); I != E; ++I) {
283 std::vector<RegionNode *> &Nodes = *I;
284 Order.append(Nodes.begin(), Nodes.end());
285 }
286}
287
Christian Konig623977d2013-02-16 11:27:45 +0000288/// \brief Determine the end of the loops
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000289void StructurizeCFG::analyzeLoops(RegionNode *N) {
Christian Konig623977d2013-02-16 11:27:45 +0000290 if (N->isSubRegion()) {
291 // Test for exit as back edge
292 BasicBlock *Exit = N->getNodeAs<Region>()->getExit();
293 if (Visited.count(Exit))
294 Loops[Exit] = N->getEntry();
295
296 } else {
297 // Test for sucessors as back edge
298 BasicBlock *BB = N->getNodeAs<BasicBlock>();
299 BranchInst *Term = cast<BranchInst>(BB->getTerminator());
300
301 for (unsigned i = 0, e = Term->getNumSuccessors(); i != e; ++i) {
302 BasicBlock *Succ = Term->getSuccessor(i);
303
304 if (Visited.count(Succ))
305 Loops[Succ] = BB;
306 }
307 }
308}
309
Christian Konigef6b2482013-02-16 11:27:50 +0000310/// \brief Invert the given condition
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000311Value *StructurizeCFG::invert(Value *Condition) {
Christian Konigef6b2482013-02-16 11:27:50 +0000312 // First: Check if it's a constant
313 if (Condition == BoolTrue)
314 return BoolFalse;
315
316 if (Condition == BoolFalse)
317 return BoolTrue;
318
319 if (Condition == BoolUndef)
320 return BoolUndef;
321
322 // Second: If the condition is already inverted, return the original value
323 if (match(Condition, m_Not(m_Value(Condition))))
324 return Condition;
325
326 // Third: Check all the users for an invert
327 BasicBlock *Parent = cast<Instruction>(Condition)->getParent();
328 for (Value::use_iterator I = Condition->use_begin(),
329 E = Condition->use_end(); I != E; ++I) {
330
331 Instruction *User = dyn_cast<Instruction>(*I);
332 if (!User || User->getParent() != Parent)
333 continue;
334
335 if (match(*I, m_Not(m_Specific(Condition))))
336 return *I;
337 }
338
339 // Last option: Create a new instruction
340 return BinaryOperator::CreateNot(Condition, "", Parent->getTerminator());
341}
342
Tom Stellard27f5d062013-02-08 22:24:37 +0000343/// \brief Build the condition for one edge
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000344Value *StructurizeCFG::buildCondition(BranchInst *Term, unsigned Idx,
345 bool Invert) {
Tom Stellard27f5d062013-02-08 22:24:37 +0000346 Value *Cond = Invert ? BoolFalse : BoolTrue;
347 if (Term->isConditional()) {
348 Cond = Term->getCondition();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000349
Aaron Ballmanf3d39522013-06-04 01:03:03 +0000350 if (Idx != (unsigned)Invert)
Christian Konigef6b2482013-02-16 11:27:50 +0000351 Cond = invert(Cond);
Tom Stellard27f5d062013-02-08 22:24:37 +0000352 }
353 return Cond;
354}
355
Tom Stellard27f5d062013-02-08 22:24:37 +0000356/// \brief Analyze the predecessors of each block and build up predicates
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000357void StructurizeCFG::gatherPredicates(RegionNode *N) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000358 RegionInfo *RI = ParentRegion->getRegionInfo();
Tom Stellard27f5d062013-02-08 22:24:37 +0000359 BasicBlock *BB = N->getEntry();
360 BBPredicates &Pred = Predicates[BB];
Christian Konig623977d2013-02-16 11:27:45 +0000361 BBPredicates &LPred = LoopPreds[BB];
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000362
Tom Stellard27f5d062013-02-08 22:24:37 +0000363 for (pred_iterator PI = pred_begin(BB), PE = pred_end(BB);
364 PI != PE; ++PI) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000365
Christian Konig623977d2013-02-16 11:27:45 +0000366 // Ignore it if it's a branch from outside into our region entry
367 if (!ParentRegion->contains(*PI))
Tom Stellard27f5d062013-02-08 22:24:37 +0000368 continue;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000369
Tom Stellard27f5d062013-02-08 22:24:37 +0000370 Region *R = RI->getRegionFor(*PI);
371 if (R == ParentRegion) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000372
Tom Stellard27f5d062013-02-08 22:24:37 +0000373 // It's a top level block in our region
374 BranchInst *Term = cast<BranchInst>((*PI)->getTerminator());
375 for (unsigned i = 0, e = Term->getNumSuccessors(); i != e; ++i) {
376 BasicBlock *Succ = Term->getSuccessor(i);
377 if (Succ != BB)
378 continue;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000379
Tom Stellard27f5d062013-02-08 22:24:37 +0000380 if (Visited.count(*PI)) {
381 // Normal forward edge
382 if (Term->isConditional()) {
383 // Try to treat it like an ELSE block
384 BasicBlock *Other = Term->getSuccessor(!i);
Christian Konig623977d2013-02-16 11:27:45 +0000385 if (Visited.count(Other) && !Loops.count(Other) &&
Tom Stellard27f5d062013-02-08 22:24:37 +0000386 !Pred.count(Other) && !Pred.count(*PI)) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000387
Tom Stellard27f5d062013-02-08 22:24:37 +0000388 Pred[Other] = BoolFalse;
389 Pred[*PI] = BoolTrue;
390 continue;
391 }
392 }
Christian Konig623977d2013-02-16 11:27:45 +0000393 Pred[*PI] = buildCondition(Term, i, false);
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000394
Tom Stellard27f5d062013-02-08 22:24:37 +0000395 } else {
396 // Back edge
Christian Konig623977d2013-02-16 11:27:45 +0000397 LPred[*PI] = buildCondition(Term, i, true);
Tom Stellard27f5d062013-02-08 22:24:37 +0000398 }
Tom Stellard27f5d062013-02-08 22:24:37 +0000399 }
400
401 } else {
402
403 // It's an exit from a sub region
404 while(R->getParent() != ParentRegion)
405 R = R->getParent();
406
407 // Edge from inside a subregion to its entry, ignore it
408 if (R == N)
409 continue;
410
411 BasicBlock *Entry = R->getEntry();
Christian Konig623977d2013-02-16 11:27:45 +0000412 if (Visited.count(Entry))
413 Pred[Entry] = BoolTrue;
414 else
415 LPred[Entry] = BoolFalse;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000416 }
417 }
418}
419
420/// \brief Collect various loop and predicate infos
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000421void StructurizeCFG::collectInfos() {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000422 // Reset predicate
423 Predicates.clear();
424
425 // and loop infos
Christian Konig623977d2013-02-16 11:27:45 +0000426 Loops.clear();
427 LoopPreds.clear();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000428
Tom Stellard27f5d062013-02-08 22:24:37 +0000429 // Reset the visited nodes
430 Visited.clear();
431
432 for (RNVector::reverse_iterator OI = Order.rbegin(), OE = Order.rend();
433 OI != OE; ++OI) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000434
435 // Analyze all the conditions leading to a node
Christian Konig623977d2013-02-16 11:27:45 +0000436 gatherPredicates(*OI);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000437
Tom Stellard27f5d062013-02-08 22:24:37 +0000438 // Remember that we've seen this node
Tom Stellardf4e471a2013-02-08 22:24:38 +0000439 Visited.insert((*OI)->getEntry());
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000440
Christian Konig623977d2013-02-16 11:27:45 +0000441 // Find the last back edges
442 analyzeLoops(*OI);
Tom Stellard27f5d062013-02-08 22:24:37 +0000443 }
Tom Stellard27f5d062013-02-08 22:24:37 +0000444}
445
446/// \brief Insert the missing branch conditions
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000447void StructurizeCFG::insertConditions(bool Loops) {
Christian Konig623977d2013-02-16 11:27:45 +0000448 BranchVector &Conds = Loops ? LoopConds : Conditions;
449 Value *Default = Loops ? BoolTrue : BoolFalse;
Tom Stellard27f5d062013-02-08 22:24:37 +0000450 SSAUpdater PhiInserter;
451
Christian Konig623977d2013-02-16 11:27:45 +0000452 for (BranchVector::iterator I = Conds.begin(),
453 E = Conds.end(); I != E; ++I) {
Tom Stellard27f5d062013-02-08 22:24:37 +0000454
455 BranchInst *Term = *I;
Tom Stellard27f5d062013-02-08 22:24:37 +0000456 assert(Term->isConditional());
457
Christian Konig623977d2013-02-16 11:27:45 +0000458 BasicBlock *Parent = Term->getParent();
459 BasicBlock *SuccTrue = Term->getSuccessor(0);
460 BasicBlock *SuccFalse = Term->getSuccessor(1);
Tom Stellard27f5d062013-02-08 22:24:37 +0000461
Christian Konig25bd8842013-02-16 11:27:40 +0000462 PhiInserter.Initialize(Boolean, "");
463 PhiInserter.AddAvailableValue(&Func->getEntryBlock(), Default);
Christian Konig623977d2013-02-16 11:27:45 +0000464 PhiInserter.AddAvailableValue(Loops ? SuccFalse : Parent, Default);
Christian Konig25bd8842013-02-16 11:27:40 +0000465
Christian Konig623977d2013-02-16 11:27:45 +0000466 BBPredicates &Preds = Loops ? LoopPreds[SuccFalse] : Predicates[SuccTrue];
Christian Konig25bd8842013-02-16 11:27:40 +0000467
468 NearestCommonDominator Dominator(DT);
469 Dominator.addBlock(Parent, false);
470
471 Value *ParentValue = 0;
Tom Stellard27f5d062013-02-08 22:24:37 +0000472 for (BBPredicates::iterator PI = Preds.begin(), PE = Preds.end();
473 PI != PE; ++PI) {
474
Christian Konig25bd8842013-02-16 11:27:40 +0000475 if (PI->first == Parent) {
476 ParentValue = PI->second;
477 break;
478 }
Tom Stellard27f5d062013-02-08 22:24:37 +0000479 PhiInserter.AddAvailableValue(PI->first, PI->second);
Christian Konig25bd8842013-02-16 11:27:40 +0000480 Dominator.addBlock(PI->first);
Tom Stellard27f5d062013-02-08 22:24:37 +0000481 }
482
Christian Konig25bd8842013-02-16 11:27:40 +0000483 if (ParentValue) {
484 Term->setCondition(ParentValue);
485 } else {
486 if (!Dominator.wasResultExplicitMentioned())
487 PhiInserter.AddAvailableValue(Dominator.getResult(), Default);
488
Tom Stellard27f5d062013-02-08 22:24:37 +0000489 Term->setCondition(PhiInserter.GetValueInMiddleOfBlock(Parent));
Christian Konig25bd8842013-02-16 11:27:40 +0000490 }
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000491 }
492}
493
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000494/// \brief Remove all PHI values coming from "From" into "To" and remember
495/// them in DeletedPhis
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000496void StructurizeCFG::delPhiValues(BasicBlock *From, BasicBlock *To) {
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000497 PhiMap &Map = DeletedPhis[To];
498 for (BasicBlock::iterator I = To->begin(), E = To->end();
499 I != E && isa<PHINode>(*I);) {
500
501 PHINode &Phi = cast<PHINode>(*I++);
502 while (Phi.getBasicBlockIndex(From) != -1) {
503 Value *Deleted = Phi.removeIncomingValue(From, false);
504 Map[&Phi].push_back(std::make_pair(From, Deleted));
505 }
506 }
507}
508
509/// \brief Add a dummy PHI value as soon as we knew the new predecessor
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000510void StructurizeCFG::addPhiValues(BasicBlock *From, BasicBlock *To) {
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000511 for (BasicBlock::iterator I = To->begin(), E = To->end();
512 I != E && isa<PHINode>(*I);) {
513
514 PHINode &Phi = cast<PHINode>(*I++);
515 Value *Undef = UndefValue::get(Phi.getType());
516 Phi.addIncoming(Undef, From);
517 }
518 AddedPhis[To].push_back(From);
519}
520
521/// \brief Add the real PHI value as soon as everything is set up
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000522void StructurizeCFG::setPhiValues() {
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000523 SSAUpdater Updater;
524 for (BB2BBVecMap::iterator AI = AddedPhis.begin(), AE = AddedPhis.end();
525 AI != AE; ++AI) {
526
527 BasicBlock *To = AI->first;
528 BBVector &From = AI->second;
529
530 if (!DeletedPhis.count(To))
531 continue;
532
533 PhiMap &Map = DeletedPhis[To];
534 for (PhiMap::iterator PI = Map.begin(), PE = Map.end();
535 PI != PE; ++PI) {
536
537 PHINode *Phi = PI->first;
538 Value *Undef = UndefValue::get(Phi->getType());
539 Updater.Initialize(Phi->getType(), "");
540 Updater.AddAvailableValue(&Func->getEntryBlock(), Undef);
541 Updater.AddAvailableValue(To, Undef);
542
Christian Konig4c79c712013-02-16 11:27:35 +0000543 NearestCommonDominator Dominator(DT);
544 Dominator.addBlock(To, false);
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000545 for (BBValueVector::iterator VI = PI->second.begin(),
546 VE = PI->second.end(); VI != VE; ++VI) {
547
548 Updater.AddAvailableValue(VI->first, VI->second);
Christian Konig4c79c712013-02-16 11:27:35 +0000549 Dominator.addBlock(VI->first);
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000550 }
551
Christian Konig4c79c712013-02-16 11:27:35 +0000552 if (!Dominator.wasResultExplicitMentioned())
553 Updater.AddAvailableValue(Dominator.getResult(), Undef);
554
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000555 for (BBVector::iterator FI = From.begin(), FE = From.end();
556 FI != FE; ++FI) {
557
558 int Idx = Phi->getBasicBlockIndex(*FI);
559 assert(Idx != -1);
560 Phi->setIncomingValue(Idx, Updater.GetValueAtEndOfBlock(*FI));
561 }
562 }
563
564 DeletedPhis.erase(To);
565 }
566 assert(DeletedPhis.empty());
567}
568
Tom Stellardf4e471a2013-02-08 22:24:38 +0000569/// \brief Remove phi values from all successors and then remove the terminator.
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000570void StructurizeCFG::killTerminator(BasicBlock *BB) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000571 TerminatorInst *Term = BB->getTerminator();
572 if (!Term)
573 return;
574
575 for (succ_iterator SI = succ_begin(BB), SE = succ_end(BB);
576 SI != SE; ++SI) {
577
578 delPhiValues(BB, *SI);
579 }
580
581 Term->eraseFromParent();
582}
583
Tom Stellardf4e471a2013-02-08 22:24:38 +0000584/// \brief Let node exit(s) point to NewExit
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000585void StructurizeCFG::changeExit(RegionNode *Node, BasicBlock *NewExit,
586 bool IncludeDominator) {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000587 if (Node->isSubRegion()) {
588 Region *SubRegion = Node->getNodeAs<Region>();
589 BasicBlock *OldExit = SubRegion->getExit();
590 BasicBlock *Dominator = 0;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000591
Tom Stellardf4e471a2013-02-08 22:24:38 +0000592 // Find all the edges from the sub region to the exit
593 for (pred_iterator I = pred_begin(OldExit), E = pred_end(OldExit);
594 I != E;) {
595
596 BasicBlock *BB = *I++;
597 if (!SubRegion->contains(BB))
598 continue;
599
600 // Modify the edges to point to the new exit
601 delPhiValues(BB, OldExit);
602 BB->getTerminator()->replaceUsesOfWith(OldExit, NewExit);
603 addPhiValues(BB, NewExit);
604
605 // Find the new dominator (if requested)
606 if (IncludeDominator) {
607 if (!Dominator)
608 Dominator = BB;
609 else
610 Dominator = DT->findNearestCommonDominator(Dominator, BB);
611 }
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000612 }
613
Tom Stellardf4e471a2013-02-08 22:24:38 +0000614 // Change the dominator (if requested)
615 if (Dominator)
616 DT->changeImmediateDominator(NewExit, Dominator);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000617
Tom Stellardf4e471a2013-02-08 22:24:38 +0000618 // Update the region info
619 SubRegion->replaceExit(NewExit);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000620
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000621 } else {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000622 BasicBlock *BB = Node->getNodeAs<BasicBlock>();
623 killTerminator(BB);
624 BranchInst::Create(NewExit, BB);
625 addPhiValues(BB, NewExit);
626 if (IncludeDominator)
627 DT->changeImmediateDominator(NewExit, BB);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000628 }
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000629}
630
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000631/// \brief Create a new flow node and update dominator tree and region info
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000632BasicBlock *StructurizeCFG::getNextFlow(BasicBlock *Dominator) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000633 LLVMContext &Context = Func->getContext();
634 BasicBlock *Insert = Order.empty() ? ParentRegion->getExit() :
635 Order.back()->getEntry();
636 BasicBlock *Flow = BasicBlock::Create(Context, FlowBlockName,
637 Func, Insert);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000638 DT->addNewBlock(Flow, Dominator);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000639 ParentRegion->getRegionInfo()->setRegionFor(Flow, ParentRegion);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000640 return Flow;
641}
642
Tom Stellardf4e471a2013-02-08 22:24:38 +0000643/// \brief Create a new or reuse the previous node as flow node
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000644BasicBlock *StructurizeCFG::needPrefix(bool NeedEmpty) {
Christian Konig623977d2013-02-16 11:27:45 +0000645 BasicBlock *Entry = PrevNode->getEntry();
Tom Stellardf4e471a2013-02-08 22:24:38 +0000646
Christian Konig623977d2013-02-16 11:27:45 +0000647 if (!PrevNode->isSubRegion()) {
648 killTerminator(Entry);
649 if (!NeedEmpty || Entry->getFirstInsertionPt() == Entry->end())
650 return Entry;
Tom Stellardf4e471a2013-02-08 22:24:38 +0000651
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000652 }
Tom Stellardf4e471a2013-02-08 22:24:38 +0000653
Christian Konig623977d2013-02-16 11:27:45 +0000654 // create a new flow node
655 BasicBlock *Flow = getNextFlow(Entry);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000656
Christian Konig623977d2013-02-16 11:27:45 +0000657 // and wire it up
658 changeExit(PrevNode, Flow, true);
659 PrevNode = ParentRegion->getBBNode(Flow);
660 return Flow;
Tom Stellardf4e471a2013-02-08 22:24:38 +0000661}
662
663/// \brief Returns the region exit if possible, otherwise just a new flow node
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000664BasicBlock *StructurizeCFG::needPostfix(BasicBlock *Flow,
665 bool ExitUseAllowed) {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000666 if (Order.empty() && ExitUseAllowed) {
667 BasicBlock *Exit = ParentRegion->getExit();
668 DT->changeImmediateDominator(Exit, Flow);
669 addPhiValues(Flow, Exit);
670 return Exit;
671 }
672 return getNextFlow(Flow);
673}
674
Christian Konig623977d2013-02-16 11:27:45 +0000675/// \brief Set the previous node
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000676void StructurizeCFG::setPrevNode(BasicBlock *BB) {
Christian Konig623977d2013-02-16 11:27:45 +0000677 PrevNode = ParentRegion->contains(BB) ? ParentRegion->getBBNode(BB) : 0;
Tom Stellardf4e471a2013-02-08 22:24:38 +0000678}
679
680/// \brief Does BB dominate all the predicates of Node ?
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000681bool StructurizeCFG::dominatesPredicates(BasicBlock *BB, RegionNode *Node) {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000682 BBPredicates &Preds = Predicates[Node->getEntry()];
683 for (BBPredicates::iterator PI = Preds.begin(), PE = Preds.end();
684 PI != PE; ++PI) {
685
686 if (!DT->dominates(BB, PI->first))
687 return false;
688 }
689 return true;
690}
691
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000692/// \brief Can we predict that this node will always be called?
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000693bool StructurizeCFG::isPredictableTrue(RegionNode *Node) {
Christian Konig623977d2013-02-16 11:27:45 +0000694 BBPredicates &Preds = Predicates[Node->getEntry()];
695 bool Dominated = false;
696
697 // Regionentry is always true
698 if (PrevNode == 0)
699 return true;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000700
701 for (BBPredicates::iterator I = Preds.begin(), E = Preds.end();
702 I != E; ++I) {
703
704 if (I->second != BoolTrue)
705 return false;
706
Christian Konig623977d2013-02-16 11:27:45 +0000707 if (!Dominated && DT->dominates(I->first, PrevNode->getEntry()))
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000708 Dominated = true;
709 }
Tom Stellardf4e471a2013-02-08 22:24:38 +0000710
711 // TODO: The dominator check is too strict
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000712 return Dominated;
713}
714
Tom Stellardf4e471a2013-02-08 22:24:38 +0000715/// Take one node from the order vector and wire it up
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000716void StructurizeCFG::wireFlow(bool ExitUseAllowed,
717 BasicBlock *LoopEnd) {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000718 RegionNode *Node = Order.pop_back_val();
Christian Konig623977d2013-02-16 11:27:45 +0000719 Visited.insert(Node->getEntry());
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000720
Christian Konig623977d2013-02-16 11:27:45 +0000721 if (isPredictableTrue(Node)) {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000722 // Just a linear flow
Christian Konig623977d2013-02-16 11:27:45 +0000723 if (PrevNode) {
724 changeExit(PrevNode, Node->getEntry(), true);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000725 }
Christian Konig623977d2013-02-16 11:27:45 +0000726 PrevNode = Node;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000727
728 } else {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000729 // Insert extra prefix node (or reuse last one)
Christian Konig623977d2013-02-16 11:27:45 +0000730 BasicBlock *Flow = needPrefix(false);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000731
Tom Stellardf4e471a2013-02-08 22:24:38 +0000732 // Insert extra postfix node (or use exit instead)
733 BasicBlock *Entry = Node->getEntry();
Christian Konig623977d2013-02-16 11:27:45 +0000734 BasicBlock *Next = needPostfix(Flow, ExitUseAllowed);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000735
736 // let it point to entry and next block
737 Conditions.push_back(BranchInst::Create(Entry, Next, BoolUndef, Flow));
738 addPhiValues(Flow, Entry);
739 DT->changeImmediateDominator(Entry, Flow);
740
Christian Konig623977d2013-02-16 11:27:45 +0000741 PrevNode = Node;
742 while (!Order.empty() && !Visited.count(LoopEnd) &&
Tom Stellardf4e471a2013-02-08 22:24:38 +0000743 dominatesPredicates(Entry, Order.back())) {
Christian Konig623977d2013-02-16 11:27:45 +0000744 handleLoops(false, LoopEnd);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000745 }
746
Christian Konig623977d2013-02-16 11:27:45 +0000747 changeExit(PrevNode, Next, false);
748 setPrevNode(Next);
749 }
750}
751
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000752void StructurizeCFG::handleLoops(bool ExitUseAllowed,
753 BasicBlock *LoopEnd) {
Christian Konig623977d2013-02-16 11:27:45 +0000754 RegionNode *Node = Order.back();
755 BasicBlock *LoopStart = Node->getEntry();
756
757 if (!Loops.count(LoopStart)) {
758 wireFlow(ExitUseAllowed, LoopEnd);
759 return;
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000760 }
761
Christian Konig623977d2013-02-16 11:27:45 +0000762 if (!isPredictableTrue(Node))
763 LoopStart = needPrefix(true);
764
765 LoopEnd = Loops[Node->getEntry()];
766 wireFlow(false, LoopEnd);
767 while (!Visited.count(LoopEnd)) {
768 handleLoops(false, LoopEnd);
769 }
770
771 // Create an extra loop end node
772 LoopEnd = needPrefix(false);
773 BasicBlock *Next = needPostfix(LoopEnd, ExitUseAllowed);
774 LoopConds.push_back(BranchInst::Create(Next, LoopStart,
775 BoolUndef, LoopEnd));
776 addPhiValues(LoopEnd, LoopStart);
777 setPrevNode(Next);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000778}
779
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000780/// After this function control flow looks like it should be, but
Tom Stellardf4e471a2013-02-08 22:24:38 +0000781/// branches and PHI nodes only have undefined conditions.
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000782void StructurizeCFG::createFlow() {
Tom Stellardf4e471a2013-02-08 22:24:38 +0000783 BasicBlock *Exit = ParentRegion->getExit();
784 bool EntryDominatesExit = DT->dominates(ParentRegion->getEntry(), Exit);
785
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000786 DeletedPhis.clear();
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000787 AddedPhis.clear();
Tom Stellardf4e471a2013-02-08 22:24:38 +0000788 Conditions.clear();
Christian Konig623977d2013-02-16 11:27:45 +0000789 LoopConds.clear();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000790
Christian Konig623977d2013-02-16 11:27:45 +0000791 PrevNode = 0;
792 Visited.clear();
793
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000794 while (!Order.empty()) {
Christian Konig623977d2013-02-16 11:27:45 +0000795 handleLoops(EntryDominatesExit, 0);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000796 }
797
Christian Konig623977d2013-02-16 11:27:45 +0000798 if (PrevNode)
799 changeExit(PrevNode, Exit, EntryDominatesExit);
Tom Stellardf4e471a2013-02-08 22:24:38 +0000800 else
801 assert(EntryDominatesExit);
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000802}
803
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000804/// Handle a rare case where the disintegrated nodes instructions
805/// no longer dominate all their uses. Not sure if this is really nessasary
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000806void StructurizeCFG::rebuildSSA() {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000807 SSAUpdater Updater;
808 for (Region::block_iterator I = ParentRegion->block_begin(),
809 E = ParentRegion->block_end();
810 I != E; ++I) {
811
812 BasicBlock *BB = *I;
813 for (BasicBlock::iterator II = BB->begin(), IE = BB->end();
814 II != IE; ++II) {
815
816 bool Initialized = false;
817 for (Use *I = &II->use_begin().getUse(), *Next; I; I = Next) {
818
819 Next = I->getNext();
820
821 Instruction *User = cast<Instruction>(I->getUser());
822 if (User->getParent() == BB) {
823 continue;
824
825 } else if (PHINode *UserPN = dyn_cast<PHINode>(User)) {
826 if (UserPN->getIncomingBlock(*I) == BB)
827 continue;
828 }
829
830 if (DT->dominates(II, User))
831 continue;
832
833 if (!Initialized) {
834 Value *Undef = UndefValue::get(II->getType());
835 Updater.Initialize(II->getType(), "");
836 Updater.AddAvailableValue(&Func->getEntryBlock(), Undef);
837 Updater.AddAvailableValue(BB, II);
838 Initialized = true;
839 }
840 Updater.RewriteUseAfterInsertions(*I);
841 }
842 }
843 }
844}
845
846/// \brief Run the transformation for each region found
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000847bool StructurizeCFG::runOnRegion(Region *R, RGPassManager &RGM) {
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000848 if (R->isTopLevelRegion())
849 return false;
850
851 Func = R->getEntry()->getParent();
852 ParentRegion = R;
853
854 DT = &getAnalysis<DominatorTree>();
855
856 orderNodes();
857 collectInfos();
858 createFlow();
Christian Konig623977d2013-02-16 11:27:45 +0000859 insertConditions(false);
860 insertConditions(true);
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000861 setPhiValues();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000862 rebuildSSA();
863
Tom Stellard27f5d062013-02-08 22:24:37 +0000864 // Cleanup
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000865 Order.clear();
866 Visited.clear();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000867 DeletedPhis.clear();
Tom Stellard13cf6cb2013-02-08 22:24:35 +0000868 AddedPhis.clear();
Christian Konig623977d2013-02-16 11:27:45 +0000869 Predicates.clear();
Tom Stellard27f5d062013-02-08 22:24:37 +0000870 Conditions.clear();
Christian Konig623977d2013-02-16 11:27:45 +0000871 Loops.clear();
872 LoopPreds.clear();
873 LoopConds.clear();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000874
875 return true;
876}
877
878/// \brief Create the pass
Matt Arsenaultad966ea2013-06-19 20:18:24 +0000879Pass *llvm::createStructurizeCFGPass() {
880 return new StructurizeCFG();
Tom Stellard6b7d99d2012-12-19 22:10:31 +0000881}