blob: 304f16717ba1889fc886d7bd46bd35f72a120807 [file] [log] [blame]
Chandler Carruthdb350872011-10-21 06:46:38 +00001//===-- MachineBlockPlacement.cpp - Basic Block Code Layout optimization --===//
2//
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//===----------------------------------------------------------------------===//
9//
Chandler Carruth30713632011-10-23 09:18:45 +000010// This file implements basic block placement transformations using the CFG
11// structure and branch probability estimates.
Chandler Carruthdb350872011-10-21 06:46:38 +000012//
Chandler Carruth30713632011-10-23 09:18:45 +000013// The pass strives to preserve the structure of the CFG (that is, retain
14// a topological ordering of basic blocks) in the absense of a *strong* signal
15// to the contrary from probabilities. However, within the CFG structure, it
16// attempts to choose an ordering which favors placing more likely sequences of
17// blocks adjacent to each other.
18//
19// The algorithm works from the inner-most loop within a function outward, and
20// at each stage walks through the basic blocks, trying to coalesce them into
21// sequential chains where allowed by the CFG (or demanded by heavy
22// probabilities). Finally, it walks the blocks in topological order, and the
23// first time it reaches a chain of basic blocks, it schedules them in the
24// function in-order.
Chandler Carruthdb350872011-10-21 06:46:38 +000025//
26//===----------------------------------------------------------------------===//
27
28#define DEBUG_TYPE "block-placement2"
Chandler Carruth4a85cc92011-10-21 08:57:37 +000029#include "llvm/CodeGen/MachineBasicBlock.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000030#include "llvm/CodeGen/MachineBlockFrequencyInfo.h"
31#include "llvm/CodeGen/MachineBranchProbabilityInfo.h"
32#include "llvm/CodeGen/MachineFunction.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000033#include "llvm/CodeGen/MachineFunctionPass.h"
Chandler Carruth4a85cc92011-10-21 08:57:37 +000034#include "llvm/CodeGen/MachineLoopInfo.h"
35#include "llvm/CodeGen/MachineModuleInfo.h"
36#include "llvm/CodeGen/Passes.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000037#include "llvm/Support/Allocator.h"
Chandler Carruth30713632011-10-23 09:18:45 +000038#include "llvm/Support/Debug.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000039#include "llvm/Support/ErrorHandling.h"
40#include "llvm/ADT/DenseMap.h"
Chandler Carruth30713632011-10-23 09:18:45 +000041#include "llvm/ADT/PostOrderIterator.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000042#include "llvm/ADT/SCCIterator.h"
43#include "llvm/ADT/SmallPtrSet.h"
44#include "llvm/ADT/SmallVector.h"
45#include "llvm/ADT/Statistic.h"
46#include "llvm/Target/TargetInstrInfo.h"
Chandler Carruth4a85cc92011-10-21 08:57:37 +000047#include "llvm/Target/TargetLowering.h"
Chandler Carruthdb350872011-10-21 06:46:38 +000048#include <algorithm>
49using namespace llvm;
50
Chandler Carruth37efc9f2011-11-02 07:17:12 +000051STATISTIC(NumCondBranches, "Number of conditional branches");
52STATISTIC(NumUncondBranches, "Number of uncondittional branches");
53STATISTIC(CondBranchTakenFreq,
54 "Potential frequency of taking conditional branches");
55STATISTIC(UncondBranchTakenFreq,
56 "Potential frequency of taking unconditional branches");
57
Chandler Carruthdb350872011-10-21 06:46:38 +000058namespace {
59/// \brief A structure for storing a weighted edge.
60///
61/// This stores an edge and its weight, computed as the product of the
62/// frequency that the starting block is entered with the probability of
63/// a particular exit block.
64struct WeightedEdge {
65 BlockFrequency EdgeFrequency;
66 MachineBasicBlock *From, *To;
67
68 bool operator<(const WeightedEdge &RHS) const {
69 return EdgeFrequency < RHS.EdgeFrequency;
70 }
71};
72}
73
74namespace {
Chandler Carruth30713632011-10-23 09:18:45 +000075class BlockChain;
Chandler Carruthdb350872011-10-21 06:46:38 +000076/// \brief Type for our function-wide basic block -> block chain mapping.
77typedef DenseMap<MachineBasicBlock *, BlockChain *> BlockToChainMapType;
78}
79
80namespace {
81/// \brief A chain of blocks which will be laid out contiguously.
82///
83/// This is the datastructure representing a chain of consecutive blocks that
84/// are profitable to layout together in order to maximize fallthrough
85/// probabilities. We also can use a block chain to represent a sequence of
86/// basic blocks which have some external (correctness) requirement for
87/// sequential layout.
88///
89/// Eventually, the block chains will form a directed graph over the function.
90/// We provide an SCC-supporting-iterator in order to quicky build and walk the
91/// SCCs of block chains within a function.
92///
93/// The block chains also have support for calculating and caching probability
94/// information related to the chain itself versus other chains. This is used
95/// for ranking during the final layout of block chains.
Chandler Carruth30713632011-10-23 09:18:45 +000096class BlockChain {
97 /// \brief The sequence of blocks belonging to this chain.
Chandler Carruthdb350872011-10-21 06:46:38 +000098 ///
Chandler Carruth30713632011-10-23 09:18:45 +000099 /// This is the sequence of blocks for a particular chain. These will be laid
100 /// out in-order within the function.
101 SmallVector<MachineBasicBlock *, 4> Blocks;
Chandler Carruthdb350872011-10-21 06:46:38 +0000102
103 /// \brief A handle to the function-wide basic block to block chain mapping.
104 ///
105 /// This is retained in each block chain to simplify the computation of child
106 /// block chains for SCC-formation and iteration. We store the edges to child
107 /// basic blocks, and map them back to their associated chains using this
108 /// structure.
109 BlockToChainMapType &BlockToChain;
110
Chandler Carruth30713632011-10-23 09:18:45 +0000111public:
Chandler Carruthdb350872011-10-21 06:46:38 +0000112 /// \brief Construct a new BlockChain.
113 ///
114 /// This builds a new block chain representing a single basic block in the
115 /// function. It also registers itself as the chain that block participates
116 /// in with the BlockToChain mapping.
117 BlockChain(BlockToChainMapType &BlockToChain, MachineBasicBlock *BB)
Chandler Carruthdf234352011-11-13 11:20:44 +0000118 : Blocks(1, BB), BlockToChain(BlockToChain), LoopPredecessors(0) {
Chandler Carruthdb350872011-10-21 06:46:38 +0000119 assert(BB && "Cannot create a chain with a null basic block");
120 BlockToChain[BB] = this;
121 }
122
Chandler Carruth30713632011-10-23 09:18:45 +0000123 /// \brief Iterator over blocks within the chain.
124 typedef SmallVectorImpl<MachineBasicBlock *>::const_iterator iterator;
125
126 /// \brief Beginning of blocks within the chain.
127 iterator begin() const { return Blocks.begin(); }
128
129 /// \brief End of blocks within the chain.
130 iterator end() const { return Blocks.end(); }
131
132 /// \brief Merge a block chain into this one.
Chandler Carruthdb350872011-10-21 06:46:38 +0000133 ///
134 /// This routine merges a block chain into this one. It takes care of forming
135 /// a contiguous sequence of basic blocks, updating the edge list, and
136 /// updating the block -> chain mapping. It does not free or tear down the
137 /// old chain, but the old chain's block list is no longer valid.
Chandler Carruth30713632011-10-23 09:18:45 +0000138 void merge(MachineBasicBlock *BB, BlockChain *Chain) {
139 assert(BB);
140 assert(!Blocks.empty());
Chandler Carruthdb350872011-10-21 06:46:38 +0000141
Chandler Carruth30713632011-10-23 09:18:45 +0000142 // Fast path in case we don't have a chain already.
143 if (!Chain) {
144 assert(!BlockToChain[BB]);
145 Blocks.push_back(BB);
146 BlockToChain[BB] = this;
147 return;
Chandler Carruthdb350872011-10-21 06:46:38 +0000148 }
149
Chandler Carruth30713632011-10-23 09:18:45 +0000150 assert(BB == *Chain->begin());
151 assert(Chain->begin() != Chain->end());
Chandler Carruthdb350872011-10-21 06:46:38 +0000152
Chandler Carruth30713632011-10-23 09:18:45 +0000153 // Update the incoming blocks to point to this chain, and add them to the
154 // chain structure.
155 for (BlockChain::iterator BI = Chain->begin(), BE = Chain->end();
156 BI != BE; ++BI) {
157 Blocks.push_back(*BI);
158 assert(BlockToChain[*BI] == Chain && "Incoming blocks not in chain");
159 BlockToChain[*BI] = this;
160 }
Chandler Carruthdb350872011-10-21 06:46:38 +0000161 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000162
163 /// \brief Count of predecessors within the loop currently being processed.
164 ///
165 /// This count is updated at each loop we process to represent the number of
166 /// in-loop predecessors of this chain.
167 unsigned LoopPredecessors;
Chandler Carruthdb350872011-10-21 06:46:38 +0000168};
169}
170
171namespace {
172class MachineBlockPlacement : public MachineFunctionPass {
Chandler Carruth30713632011-10-23 09:18:45 +0000173 /// \brief A typedef for a block filter set.
174 typedef SmallPtrSet<MachineBasicBlock *, 16> BlockFilterSet;
175
Chandler Carruthdb350872011-10-21 06:46:38 +0000176 /// \brief A handle to the branch probability pass.
177 const MachineBranchProbabilityInfo *MBPI;
178
179 /// \brief A handle to the function-wide block frequency pass.
180 const MachineBlockFrequencyInfo *MBFI;
181
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000182 /// \brief A handle to the loop info.
183 const MachineLoopInfo *MLI;
184
Chandler Carruthdb350872011-10-21 06:46:38 +0000185 /// \brief A handle to the target's instruction info.
186 const TargetInstrInfo *TII;
187
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000188 /// \brief A handle to the target's lowering info.
189 const TargetLowering *TLI;
190
Chandler Carruthdb350872011-10-21 06:46:38 +0000191 /// \brief Allocator and owner of BlockChain structures.
192 ///
193 /// We build BlockChains lazily by merging together high probability BB
194 /// sequences acording to the "Algo2" in the paper mentioned at the top of
195 /// the file. To reduce malloc traffic, we allocate them using this slab-like
196 /// allocator, and destroy them after the pass completes.
197 SpecificBumpPtrAllocator<BlockChain> ChainAllocator;
198
199 /// \brief Function wide BasicBlock to BlockChain mapping.
200 ///
201 /// This mapping allows efficiently moving from any given basic block to the
202 /// BlockChain it participates in, if any. We use it to, among other things,
203 /// allow implicitly defining edges between chains as the existing edges
204 /// between basic blocks.
205 DenseMap<MachineBasicBlock *, BlockChain *> BlockToChain;
206
Chandler Carruthdf234352011-11-13 11:20:44 +0000207 void markChainSuccessors(BlockChain &Chain,
208 MachineBasicBlock *LoopHeaderBB,
Chandler Carruthb5856c82011-11-14 00:00:35 +0000209 SmallVectorImpl<MachineBasicBlock *> &BlockWorkList,
Chandler Carruthdf234352011-11-13 11:20:44 +0000210 const BlockFilterSet *BlockFilter = 0);
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000211 MachineBasicBlock *selectBestSuccessor(MachineBasicBlock *BB,
212 BlockChain &Chain,
213 const BlockFilterSet *BlockFilter);
Chandler Carruthf3fc0052011-11-13 11:42:26 +0000214 MachineBasicBlock *selectBestCandidateBlock(
215 BlockChain &Chain, SmallVectorImpl<MachineBasicBlock *> &WorkList,
216 const BlockFilterSet *BlockFilter);
Chandler Carruthb5856c82011-11-14 00:00:35 +0000217 MachineBasicBlock *getFirstUnplacedBlock(const BlockChain &PlacedChain,
218 ArrayRef<MachineBasicBlock *> Blocks,
219 unsigned &PrevUnplacedBlockIdx);
Chandler Carruthdf234352011-11-13 11:20:44 +0000220 void buildChain(MachineBasicBlock *BB, BlockChain &Chain,
Chandler Carruthb5856c82011-11-14 00:00:35 +0000221 ArrayRef<MachineBasicBlock *> Blocks,
222 SmallVectorImpl<MachineBasicBlock *> &BlockWorkList,
Chandler Carruthdf234352011-11-13 11:20:44 +0000223 const BlockFilterSet *BlockFilter = 0);
Chandler Carruth30713632011-10-23 09:18:45 +0000224 void buildLoopChains(MachineFunction &F, MachineLoop &L);
225 void buildCFGChains(MachineFunction &F);
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000226 void AlignLoops(MachineFunction &F);
Chandler Carruthdb350872011-10-21 06:46:38 +0000227
228public:
229 static char ID; // Pass identification, replacement for typeid
230 MachineBlockPlacement() : MachineFunctionPass(ID) {
231 initializeMachineBlockPlacementPass(*PassRegistry::getPassRegistry());
232 }
233
234 bool runOnMachineFunction(MachineFunction &F);
235
236 void getAnalysisUsage(AnalysisUsage &AU) const {
237 AU.addRequired<MachineBranchProbabilityInfo>();
238 AU.addRequired<MachineBlockFrequencyInfo>();
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000239 AU.addRequired<MachineLoopInfo>();
Chandler Carruthdb350872011-10-21 06:46:38 +0000240 MachineFunctionPass::getAnalysisUsage(AU);
241 }
242
243 const char *getPassName() const { return "Block Placement"; }
244};
245}
246
247char MachineBlockPlacement::ID = 0;
248INITIALIZE_PASS_BEGIN(MachineBlockPlacement, "block-placement2",
249 "Branch Probability Basic Block Placement", false, false)
250INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)
251INITIALIZE_PASS_DEPENDENCY(MachineBlockFrequencyInfo)
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000252INITIALIZE_PASS_DEPENDENCY(MachineLoopInfo)
Chandler Carruthdb350872011-10-21 06:46:38 +0000253INITIALIZE_PASS_END(MachineBlockPlacement, "block-placement2",
254 "Branch Probability Basic Block Placement", false, false)
255
256FunctionPass *llvm::createMachineBlockPlacementPass() {
257 return new MachineBlockPlacement();
258}
259
Chandler Carruth30713632011-10-23 09:18:45 +0000260#ifndef NDEBUG
261/// \brief Helper to print the name of a MBB.
262///
263/// Only used by debug logging.
264static std::string getBlockName(MachineBasicBlock *BB) {
265 std::string Result;
266 raw_string_ostream OS(Result);
267 OS << "BB#" << BB->getNumber()
268 << " (derived from LLVM BB '" << BB->getName() << "')";
269 OS.flush();
270 return Result;
Chandler Carruthdb350872011-10-21 06:46:38 +0000271}
272
Chandler Carruth30713632011-10-23 09:18:45 +0000273/// \brief Helper to print the number of a MBB.
274///
275/// Only used by debug logging.
276static std::string getBlockNum(MachineBasicBlock *BB) {
277 std::string Result;
278 raw_string_ostream OS(Result);
279 OS << "BB#" << BB->getNumber();
280 OS.flush();
281 return Result;
282}
283#endif
284
Chandler Carruth729bec82011-11-13 11:34:55 +0000285/// \brief Mark a chain's successors as having one fewer preds.
286///
287/// When a chain is being merged into the "placed" chain, this routine will
288/// quickly walk the successors of each block in the chain and mark them as
289/// having one fewer active predecessor. It also adds any successors of this
290/// chain which reach the zero-predecessor state to the worklist passed in.
Chandler Carruthdf234352011-11-13 11:20:44 +0000291void MachineBlockPlacement::markChainSuccessors(
292 BlockChain &Chain,
293 MachineBasicBlock *LoopHeaderBB,
294 SmallVectorImpl<MachineBasicBlock *> &BlockWorkList,
295 const BlockFilterSet *BlockFilter) {
296 // Walk all the blocks in this chain, marking their successors as having
297 // a predecessor placed.
298 for (BlockChain::iterator CBI = Chain.begin(), CBE = Chain.end();
299 CBI != CBE; ++CBI) {
300 // Add any successors for which this is the only un-placed in-loop
301 // predecessor to the worklist as a viable candidate for CFG-neutral
302 // placement. No subsequent placement of this block will violate the CFG
303 // shape, so we get to use heuristics to choose a favorable placement.
304 for (MachineBasicBlock::succ_iterator SI = (*CBI)->succ_begin(),
305 SE = (*CBI)->succ_end();
306 SI != SE; ++SI) {
307 if (BlockFilter && !BlockFilter->count(*SI))
308 continue;
309 BlockChain &SuccChain = *BlockToChain[*SI];
310 // Disregard edges within a fixed chain, or edges to the loop header.
311 if (&Chain == &SuccChain || *SI == LoopHeaderBB)
312 continue;
Chandler Carruthdb350872011-10-21 06:46:38 +0000313
Chandler Carruthdf234352011-11-13 11:20:44 +0000314 // This is a cross-chain edge that is within the loop, so decrement the
315 // loop predecessor count of the destination chain.
316 if (SuccChain.LoopPredecessors > 0 && --SuccChain.LoopPredecessors == 0)
317 BlockWorkList.push_back(*SI);
Chandler Carruthdb350872011-10-21 06:46:38 +0000318 }
319 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000320}
Chandler Carruth30713632011-10-23 09:18:45 +0000321
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000322/// \brief Select the best successor for a block.
323///
324/// This looks across all successors of a particular block and attempts to
325/// select the "best" one to be the layout successor. It only considers direct
326/// successors which also pass the block filter. It will attempt to avoid
327/// breaking CFG structure, but cave and break such structures in the case of
328/// very hot successor edges.
329///
330/// \returns The best successor block found, or null if none are viable.
331MachineBasicBlock *MachineBlockPlacement::selectBestSuccessor(
332 MachineBasicBlock *BB, BlockChain &Chain,
333 const BlockFilterSet *BlockFilter) {
334 const BranchProbability HotProb(4, 5); // 80%
335
336 MachineBasicBlock *BestSucc = 0;
Chandler Carruth340d5962011-11-14 09:12:57 +0000337 // FIXME: Due to the performance of the probability and weight routines in
338 // the MBPI analysis, we manually compute probabilities using the edge
339 // weights. This is suboptimal as it means that the somewhat subtle
340 // definition of edge weight semantics is encoded here as well. We should
341 // improve the MBPI interface to effeciently support query patterns such as
342 // this.
343 uint32_t BestWeight = 0;
344 uint32_t WeightScale = 0;
345 uint32_t SumWeight = MBPI->getSumForBlock(BB, WeightScale);
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000346 DEBUG(dbgs() << "Attempting merge from: " << getBlockName(BB) << "\n");
347 for (MachineBasicBlock::succ_iterator SI = BB->succ_begin(),
348 SE = BB->succ_end();
349 SI != SE; ++SI) {
350 if (BlockFilter && !BlockFilter->count(*SI))
351 continue;
352 BlockChain &SuccChain = *BlockToChain[*SI];
353 if (&SuccChain == &Chain) {
354 DEBUG(dbgs() << " " << getBlockName(*SI) << " -> Already merged!\n");
355 continue;
356 }
357
Chandler Carruth340d5962011-11-14 09:12:57 +0000358 uint32_t SuccWeight = MBPI->getEdgeWeight(BB, *SI);
359 BranchProbability SuccProb(SuccWeight / WeightScale, SumWeight);
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000360
361 // Only consider successors which are either "hot", or wouldn't violate
362 // any CFG constraints.
363 if (SuccChain.LoopPredecessors != 0 && SuccProb < HotProb) {
364 DEBUG(dbgs() << " " << getBlockName(*SI) << " -> CFG conflict\n");
365 continue;
366 }
367
368 DEBUG(dbgs() << " " << getBlockName(*SI) << " -> " << SuccProb
369 << " (prob)"
370 << (SuccChain.LoopPredecessors != 0 ? " (CFG break)" : "")
371 << "\n");
Chandler Carruth340d5962011-11-14 09:12:57 +0000372 if (BestSucc && BestWeight >= SuccWeight)
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000373 continue;
374 BestSucc = *SI;
Chandler Carruth340d5962011-11-14 09:12:57 +0000375 BestWeight = SuccWeight;
Chandler Carruth9fd4e052011-11-13 11:34:53 +0000376 }
377 return BestSucc;
378}
379
Chandler Carruthfa976582011-11-14 09:46:33 +0000380namespace {
381/// \brief Predicate struct to detect blocks already placed.
382class IsBlockPlaced {
383 const BlockChain &PlacedChain;
384 const BlockToChainMapType &BlockToChain;
385
386public:
387 IsBlockPlaced(const BlockChain &PlacedChain,
388 const BlockToChainMapType &BlockToChain)
389 : PlacedChain(PlacedChain), BlockToChain(BlockToChain) {}
390
391 bool operator()(MachineBasicBlock *BB) const {
392 return BlockToChain.lookup(BB) == &PlacedChain;
393 }
394};
395}
396
Chandler Carruthf3fc0052011-11-13 11:42:26 +0000397/// \brief Select the best block from a worklist.
398///
399/// This looks through the provided worklist as a list of candidate basic
400/// blocks and select the most profitable one to place. The definition of
401/// profitable only really makes sense in the context of a loop. This returns
402/// the most frequently visited block in the worklist, which in the case of
403/// a loop, is the one most desirable to be physically close to the rest of the
404/// loop body in order to improve icache behavior.
405///
406/// \returns The best block found, or null if none are viable.
407MachineBasicBlock *MachineBlockPlacement::selectBestCandidateBlock(
408 BlockChain &Chain, SmallVectorImpl<MachineBasicBlock *> &WorkList,
409 const BlockFilterSet *BlockFilter) {
Chandler Carruthfa976582011-11-14 09:46:33 +0000410 // Once we need to walk the worklist looking for a candidate, cleanup the
411 // worklist of already placed entries.
412 // FIXME: If this shows up on profiles, it could be folded (at the cost of
413 // some code complexity) into the loop below.
414 WorkList.erase(std::remove_if(WorkList.begin(), WorkList.end(),
415 IsBlockPlaced(Chain, BlockToChain)),
416 WorkList.end());
417
Chandler Carruthf3fc0052011-11-13 11:42:26 +0000418 MachineBasicBlock *BestBlock = 0;
419 BlockFrequency BestFreq;
420 for (SmallVectorImpl<MachineBasicBlock *>::iterator WBI = WorkList.begin(),
421 WBE = WorkList.end();
422 WBI != WBE; ++WBI) {
Chandler Carruthfa976582011-11-14 09:46:33 +0000423 assert(!BlockFilter || BlockFilter->count(*WBI));
Chandler Carruthf3fc0052011-11-13 11:42:26 +0000424 BlockChain &SuccChain = *BlockToChain[*WBI];
425 if (&SuccChain == &Chain) {
426 DEBUG(dbgs() << " " << getBlockName(*WBI)
427 << " -> Already merged!\n");
428 continue;
429 }
430 assert(SuccChain.LoopPredecessors == 0 && "Found CFG-violating block");
431
432 BlockFrequency CandidateFreq = MBFI->getBlockFreq(*WBI);
433 DEBUG(dbgs() << " " << getBlockName(*WBI) << " -> " << CandidateFreq
434 << " (freq)\n");
435 if (BestBlock && BestFreq >= CandidateFreq)
436 continue;
437 BestBlock = *WBI;
438 BestFreq = CandidateFreq;
439 }
440 return BestBlock;
441}
442
Chandler Carruthb5856c82011-11-14 00:00:35 +0000443/// \brief Retrieve the first unplaced basic block.
444///
445/// This routine is called when we are unable to use the CFG to walk through
446/// all of the basic blocks and form a chain due to unnatural loops in the CFG.
447/// We walk through the sequence of blocks, starting from the
448/// LastUnplacedBlockIdx. We update this index to avoid re-scanning the entire
449/// sequence on repeated calls to this routine.
450MachineBasicBlock *MachineBlockPlacement::getFirstUnplacedBlock(
451 const BlockChain &PlacedChain,
452 ArrayRef<MachineBasicBlock *> Blocks,
453 unsigned &PrevUnplacedBlockIdx) {
454 for (unsigned i = PrevUnplacedBlockIdx, e = Blocks.size(); i != e; ++i) {
455 MachineBasicBlock *BB = Blocks[i];
456 if (BlockToChain[BB] != &PlacedChain) {
457 PrevUnplacedBlockIdx = i;
458 return BB;
459 }
460 }
461 return 0;
462}
463
Chandler Carruthdf234352011-11-13 11:20:44 +0000464void MachineBlockPlacement::buildChain(
465 MachineBasicBlock *BB,
466 BlockChain &Chain,
Chandler Carruthb5856c82011-11-14 00:00:35 +0000467 ArrayRef<MachineBasicBlock *> Blocks,
Chandler Carruthdf234352011-11-13 11:20:44 +0000468 SmallVectorImpl<MachineBasicBlock *> &BlockWorkList,
469 const BlockFilterSet *BlockFilter) {
Chandler Carruthdf234352011-11-13 11:20:44 +0000470 assert(BB);
471 assert(BlockToChain[BB] == &Chain);
472 assert(*Chain.begin() == BB);
Chandler Carruthb5856c82011-11-14 00:00:35 +0000473 SmallVector<MachineOperand, 4> Cond; // For AnalyzeBranch.
474 unsigned PrevUnplacedBlockIdx = 0;
475
Chandler Carruthdf234352011-11-13 11:20:44 +0000476 MachineBasicBlock *LoopHeaderBB = BB;
477 markChainSuccessors(Chain, LoopHeaderBB, BlockWorkList, BlockFilter);
478 BB = *llvm::prior(Chain.end());
479 for (;;) {
480 assert(BB);
481 assert(BlockToChain[BB] == &Chain);
482 assert(*llvm::prior(Chain.end()) == BB);
Chandler Carruth6527ecc2011-11-13 12:17:28 +0000483 MachineBasicBlock *BestSucc = 0;
Chandler Carruth30713632011-10-23 09:18:45 +0000484
Chandler Carruth6527ecc2011-11-13 12:17:28 +0000485 // Check for unreasonable branches, and forcibly merge the existing layout
486 // successor for them. We can handle cases that AnalyzeBranch can't: jump
487 // tables etc are fine. The case we want to handle specially is when there
488 // is potential fallthrough, but the branch cannot be analyzed. This
489 // includes blocks without terminators as well as other cases.
490 Cond.clear();
491 MachineBasicBlock *TBB = 0, *FBB = 0; // For AnalyzeBranch.
492 if (TII->AnalyzeBranch(*BB, TBB, FBB, Cond) && BB->canFallThrough()) {
Chandler Carruthb5856c82011-11-14 00:00:35 +0000493 MachineFunction::iterator I(BB), NextI(llvm::next(I));
494 // Ensure that the layout successor is a viable block, as we know that
Chandler Carruthbc83fcd2011-11-14 10:55:53 +0000495 // fallthrough is a possibility. Note that this may not be a valid block
496 // in the loop, but we allow that to cope with degenerate situations.
Chandler Carruthb5856c82011-11-14 00:00:35 +0000497 assert(NextI != BB->getParent()->end());
Chandler Carruthb5856c82011-11-14 00:00:35 +0000498 BestSucc = NextI;
Chandler Carruth6527ecc2011-11-13 12:17:28 +0000499 }
500
501 // Otherwise, look for the best viable successor if there is one to place
502 // immediately after this block.
503 if (!BestSucc)
504 BestSucc = selectBestSuccessor(BB, Chain, BlockFilter);
Chandler Carruthdf234352011-11-13 11:20:44 +0000505
506 // If an immediate successor isn't available, look for the best viable
507 // block among those we've identified as not violating the loop's CFG at
508 // this point. This won't be a fallthrough, but it will increase locality.
Chandler Carruthf3fc0052011-11-13 11:42:26 +0000509 if (!BestSucc)
510 BestSucc = selectBestCandidateBlock(Chain, BlockWorkList, BlockFilter);
Chandler Carruthdf234352011-11-13 11:20:44 +0000511
Chandler Carruthdf234352011-11-13 11:20:44 +0000512 if (!BestSucc) {
Chandler Carruthb5856c82011-11-14 00:00:35 +0000513 BestSucc = getFirstUnplacedBlock(Chain, Blocks, PrevUnplacedBlockIdx);
514 if (!BestSucc)
515 break;
516
517 DEBUG(dbgs() << "Unnatural loop CFG detected, forcibly merging the "
518 "layout successor until the CFG reduces\n");
Chandler Carruthdf234352011-11-13 11:20:44 +0000519 }
Chandler Carruth30713632011-10-23 09:18:45 +0000520
Chandler Carruthdf234352011-11-13 11:20:44 +0000521 // Place this block, updating the datastructures to reflect its placement.
522 BlockChain &SuccChain = *BlockToChain[BestSucc];
Chandler Carruthb5856c82011-11-14 00:00:35 +0000523 // Zero out LoopPredecessors for the successor we're about to merge in case
524 // we selected a successor that didn't fit naturally into the CFG.
525 SuccChain.LoopPredecessors = 0;
Chandler Carruthdf234352011-11-13 11:20:44 +0000526 DEBUG(dbgs() << "Merging from " << getBlockNum(BB)
527 << " to " << getBlockNum(BestSucc) << "\n");
528 markChainSuccessors(SuccChain, LoopHeaderBB, BlockWorkList, BlockFilter);
529 Chain.merge(BestSucc, &SuccChain);
530 BB = *llvm::prior(Chain.end());
Chandler Carruthb5856c82011-11-14 00:00:35 +0000531 };
532
533 DEBUG(dbgs() << "Finished forming chain for header block "
534 << getBlockNum(*Chain.begin()) << "\n");
Chandler Carruthdb350872011-10-21 06:46:38 +0000535}
536
Chandler Carruth30713632011-10-23 09:18:45 +0000537/// \brief Forms basic block chains from the natural loop structures.
Chandler Carruthdb350872011-10-21 06:46:38 +0000538///
Chandler Carruth30713632011-10-23 09:18:45 +0000539/// These chains are designed to preserve the existing *structure* of the code
540/// as much as possible. We can then stitch the chains together in a way which
541/// both preserves the topological structure and minimizes taken conditional
542/// branches.
Chandler Carruthdf234352011-11-13 11:20:44 +0000543void MachineBlockPlacement::buildLoopChains(MachineFunction &F,
544 MachineLoop &L) {
Chandler Carruth30713632011-10-23 09:18:45 +0000545 // First recurse through any nested loops, building chains for those inner
546 // loops.
547 for (MachineLoop::iterator LI = L.begin(), LE = L.end(); LI != LE; ++LI)
548 buildLoopChains(F, **LI);
Chandler Carruthdb350872011-10-21 06:46:38 +0000549
Chandler Carruthdf234352011-11-13 11:20:44 +0000550 SmallVector<MachineBasicBlock *, 16> BlockWorkList;
551 BlockFilterSet LoopBlockSet(L.block_begin(), L.block_end());
Chandler Carruthb5856c82011-11-14 00:00:35 +0000552 BlockChain &LoopChain = *BlockToChain[L.getHeader()];
Chandler Carruthdb350872011-10-21 06:46:38 +0000553
Chandler Carruthdf234352011-11-13 11:20:44 +0000554 // FIXME: This is a really lame way of walking the chains in the loop: we
555 // walk the blocks, and use a set to prevent visiting a particular chain
556 // twice.
557 SmallPtrSet<BlockChain *, 4> UpdatedPreds;
558 for (MachineLoop::block_iterator BI = L.block_begin(),
559 BE = L.block_end();
Chandler Carruth30713632011-10-23 09:18:45 +0000560 BI != BE; ++BI) {
Chandler Carruthdf234352011-11-13 11:20:44 +0000561 BlockChain &Chain = *BlockToChain[*BI];
562 if (!UpdatedPreds.insert(&Chain) || BI == L.block_begin())
563 continue;
564
565 assert(Chain.LoopPredecessors == 0);
566 for (BlockChain::iterator BCI = Chain.begin(), BCE = Chain.end();
567 BCI != BCE; ++BCI) {
568 assert(BlockToChain[*BCI] == &Chain);
569 for (MachineBasicBlock::pred_iterator PI = (*BCI)->pred_begin(),
570 PE = (*BCI)->pred_end();
571 PI != PE; ++PI) {
572 if (BlockToChain[*PI] == &Chain || !LoopBlockSet.count(*PI))
573 continue;
574 ++Chain.LoopPredecessors;
575 }
576 }
577
578 if (Chain.LoopPredecessors == 0)
579 BlockWorkList.push_back(*BI);
Chandler Carruthdb350872011-10-21 06:46:38 +0000580 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000581
Chandler Carruthb5856c82011-11-14 00:00:35 +0000582 buildChain(*L.block_begin(), LoopChain, L.getBlocks(), BlockWorkList,
583 &LoopBlockSet);
Chandler Carruthdf234352011-11-13 11:20:44 +0000584
585 DEBUG({
Chandler Carruth10252db2011-11-13 21:39:51 +0000586 // Crash at the end so we get all of the debugging output first.
587 bool BadLoop = false;
588 if (LoopChain.LoopPredecessors) {
589 BadLoop = true;
Chandler Carruthdf234352011-11-13 11:20:44 +0000590 dbgs() << "Loop chain contains a block without its preds placed!\n"
591 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
592 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n";
Chandler Carruth10252db2011-11-13 21:39:51 +0000593 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000594 for (BlockChain::iterator BCI = LoopChain.begin(), BCE = LoopChain.end();
595 BCI != BCE; ++BCI)
Chandler Carruth10252db2011-11-13 21:39:51 +0000596 if (!LoopBlockSet.erase(*BCI)) {
Chandler Carruthbc83fcd2011-11-14 10:55:53 +0000597 // We don't mark the loop as bad here because there are real situations
598 // where this can occur. For example, with an unanalyzable fallthrough
599 // from a loop block to a non-loop block.
600 // FIXME: Such constructs shouldn't exist. Track them down and fix them.
Chandler Carruthdf234352011-11-13 11:20:44 +0000601 dbgs() << "Loop chain contains a block not contained by the loop!\n"
602 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
603 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n"
604 << " Bad block: " << getBlockName(*BCI) << "\n";
Chandler Carruth10252db2011-11-13 21:39:51 +0000605 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000606
Chandler Carruth10252db2011-11-13 21:39:51 +0000607 if (!LoopBlockSet.empty()) {
608 BadLoop = true;
Chandler Carruthc0f05b32011-11-13 22:50:09 +0000609 for (BlockFilterSet::iterator LBI = LoopBlockSet.begin(),
610 LBE = LoopBlockSet.end();
Chandler Carruthdf234352011-11-13 11:20:44 +0000611 LBI != LBE; ++LBI)
612 dbgs() << "Loop contains blocks never placed into a chain!\n"
613 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
614 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n"
615 << " Bad block: " << getBlockName(*LBI) << "\n";
Chandler Carruth10252db2011-11-13 21:39:51 +0000616 }
617 assert(!BadLoop && "Detected problems with the placement of this loop.");
Chandler Carruthdf234352011-11-13 11:20:44 +0000618 });
Chandler Carruthdb350872011-10-21 06:46:38 +0000619}
620
Chandler Carruth30713632011-10-23 09:18:45 +0000621void MachineBlockPlacement::buildCFGChains(MachineFunction &F) {
Chandler Carruthdf234352011-11-13 11:20:44 +0000622 // Ensure that every BB in the function has an associated chain to simplify
623 // the assumptions of the remaining algorithm.
624 for (MachineFunction::iterator FI = F.begin(), FE = F.end(); FI != FE; ++FI)
625 BlockToChain[&*FI] =
626 new (ChainAllocator.Allocate()) BlockChain(BlockToChain, &*FI);
627
628 // Build any loop-based chains.
Chandler Carruth30713632011-10-23 09:18:45 +0000629 for (MachineLoopInfo::iterator LI = MLI->begin(), LE = MLI->end(); LI != LE;
630 ++LI)
631 buildLoopChains(F, **LI);
632
Chandler Carruthb5856c82011-11-14 00:00:35 +0000633 // We need a vector of blocks so that buildChain can handle unnatural CFG
634 // constructs by searching for unplaced blocks and just concatenating them.
635 SmallVector<MachineBasicBlock *, 16> Blocks;
636 Blocks.reserve(F.size());
637
Chandler Carruthdf234352011-11-13 11:20:44 +0000638 SmallVector<MachineBasicBlock *, 16> BlockWorkList;
Chandler Carruth30713632011-10-23 09:18:45 +0000639
Chandler Carruthdf234352011-11-13 11:20:44 +0000640 SmallPtrSet<BlockChain *, 4> UpdatedPreds;
Chandler Carruthdb350872011-10-21 06:46:38 +0000641 for (MachineFunction::iterator FI = F.begin(), FE = F.end(); FI != FE; ++FI) {
Chandler Carruthdf234352011-11-13 11:20:44 +0000642 MachineBasicBlock *BB = &*FI;
Chandler Carruthb5856c82011-11-14 00:00:35 +0000643 Blocks.push_back(BB);
Chandler Carruthdf234352011-11-13 11:20:44 +0000644 BlockChain &Chain = *BlockToChain[BB];
645 if (!UpdatedPreds.insert(&Chain))
646 continue;
647
648 assert(Chain.LoopPredecessors == 0);
649 for (BlockChain::iterator BCI = Chain.begin(), BCE = Chain.end();
650 BCI != BCE; ++BCI) {
651 assert(BlockToChain[*BCI] == &Chain);
652 for (MachineBasicBlock::pred_iterator PI = (*BCI)->pred_begin(),
653 PE = (*BCI)->pred_end();
654 PI != PE; ++PI) {
655 if (BlockToChain[*PI] == &Chain)
656 continue;
657 ++Chain.LoopPredecessors;
658 }
659 }
660
661 if (Chain.LoopPredecessors == 0)
662 BlockWorkList.push_back(BB);
663 }
664
665 BlockChain &FunctionChain = *BlockToChain[&F.front()];
Chandler Carruthb5856c82011-11-14 00:00:35 +0000666 buildChain(&F.front(), FunctionChain, Blocks, BlockWorkList);
Chandler Carruthdf234352011-11-13 11:20:44 +0000667
668 typedef SmallPtrSet<MachineBasicBlock *, 16> FunctionBlockSetType;
669 DEBUG({
Chandler Carruth10252db2011-11-13 21:39:51 +0000670 // Crash at the end so we get all of the debugging output first.
671 bool BadFunc = false;
Chandler Carruthdf234352011-11-13 11:20:44 +0000672 FunctionBlockSetType FunctionBlockSet;
673 for (MachineFunction::iterator FI = F.begin(), FE = F.end(); FI != FE; ++FI)
674 FunctionBlockSet.insert(FI);
675
Chandler Carruthc0f05b32011-11-13 22:50:09 +0000676 for (BlockChain::iterator BCI = FunctionChain.begin(),
677 BCE = FunctionChain.end();
Chandler Carruthdf234352011-11-13 11:20:44 +0000678 BCI != BCE; ++BCI)
Chandler Carruth10252db2011-11-13 21:39:51 +0000679 if (!FunctionBlockSet.erase(*BCI)) {
680 BadFunc = true;
Chandler Carruthdf234352011-11-13 11:20:44 +0000681 dbgs() << "Function chain contains a block not in the function!\n"
682 << " Bad block: " << getBlockName(*BCI) << "\n";
Chandler Carruth10252db2011-11-13 21:39:51 +0000683 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000684
Chandler Carruth10252db2011-11-13 21:39:51 +0000685 if (!FunctionBlockSet.empty()) {
686 BadFunc = true;
Chandler Carruthc0f05b32011-11-13 22:50:09 +0000687 for (FunctionBlockSetType::iterator FBI = FunctionBlockSet.begin(),
688 FBE = FunctionBlockSet.end();
689 FBI != FBE; ++FBI)
Chandler Carruthdf234352011-11-13 11:20:44 +0000690 dbgs() << "Function contains blocks never placed into a chain!\n"
691 << " Bad block: " << getBlockName(*FBI) << "\n";
Chandler Carruth10252db2011-11-13 21:39:51 +0000692 }
693 assert(!BadFunc && "Detected problems with the block placement.");
Chandler Carruthdf234352011-11-13 11:20:44 +0000694 });
695
696 // Splice the blocks into place.
697 MachineFunction::iterator InsertPos = F.begin();
698 SmallVector<MachineOperand, 4> Cond; // For AnalyzeBranch.
Chandler Carruthc0f05b32011-11-13 22:50:09 +0000699 for (BlockChain::iterator BI = FunctionChain.begin(),
700 BE = FunctionChain.end();
Chandler Carruthdf234352011-11-13 11:20:44 +0000701 BI != BE; ++BI) {
702 DEBUG(dbgs() << (BI == FunctionChain.begin() ? "Placing chain "
703 : " ... ")
704 << getBlockName(*BI) << "\n");
705 if (InsertPos != MachineFunction::iterator(*BI))
706 F.splice(InsertPos, *BI);
707 else
708 ++InsertPos;
709
710 // Update the terminator of the previous block.
711 if (BI == FunctionChain.begin())
712 continue;
713 MachineBasicBlock *PrevBB = llvm::prior(MachineFunction::iterator(*BI));
714
Chandler Carruthdb350872011-10-21 06:46:38 +0000715 // FIXME: It would be awesome of updateTerminator would just return rather
716 // than assert when the branch cannot be analyzed in order to remove this
717 // boiler plate.
718 Cond.clear();
719 MachineBasicBlock *TBB = 0, *FBB = 0; // For AnalyzeBranch.
Chandler Carruthdf234352011-11-13 11:20:44 +0000720 if (!TII->AnalyzeBranch(*PrevBB, TBB, FBB, Cond))
721 PrevBB->updateTerminator();
Chandler Carruthdb350872011-10-21 06:46:38 +0000722 }
Chandler Carruthdf234352011-11-13 11:20:44 +0000723
724 // Fixup the last block.
725 Cond.clear();
726 MachineBasicBlock *TBB = 0, *FBB = 0; // For AnalyzeBranch.
727 if (!TII->AnalyzeBranch(F.back(), TBB, FBB, Cond))
728 F.back().updateTerminator();
Chandler Carruthdb350872011-10-21 06:46:38 +0000729}
730
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000731/// \brief Recursive helper to align a loop and any nested loops.
732static void AlignLoop(MachineFunction &F, MachineLoop *L, unsigned Align) {
733 // Recurse through nested loops.
734 for (MachineLoop::iterator I = L->begin(), E = L->end(); I != E; ++I)
735 AlignLoop(F, *I, Align);
736
737 L->getTopBlock()->setAlignment(Align);
738}
739
740/// \brief Align loop headers to target preferred alignments.
741void MachineBlockPlacement::AlignLoops(MachineFunction &F) {
742 if (F.getFunction()->hasFnAttr(Attribute::OptimizeForSize))
743 return;
744
745 unsigned Align = TLI->getPrefLoopAlignment();
746 if (!Align)
747 return; // Don't care about loop alignment.
748
749 for (MachineLoopInfo::iterator I = MLI->begin(), E = MLI->end(); I != E; ++I)
750 AlignLoop(F, *I, Align);
751}
752
Chandler Carruthdb350872011-10-21 06:46:38 +0000753bool MachineBlockPlacement::runOnMachineFunction(MachineFunction &F) {
754 // Check for single-block functions and skip them.
755 if (llvm::next(F.begin()) == F.end())
756 return false;
757
758 MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
759 MBFI = &getAnalysis<MachineBlockFrequencyInfo>();
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000760 MLI = &getAnalysis<MachineLoopInfo>();
Chandler Carruthdb350872011-10-21 06:46:38 +0000761 TII = F.getTarget().getInstrInfo();
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000762 TLI = F.getTarget().getTargetLowering();
Chandler Carruthdb350872011-10-21 06:46:38 +0000763 assert(BlockToChain.empty());
Chandler Carruthdb350872011-10-21 06:46:38 +0000764
Chandler Carruth30713632011-10-23 09:18:45 +0000765 buildCFGChains(F);
Chandler Carruth4a85cc92011-10-21 08:57:37 +0000766 AlignLoops(F);
Chandler Carruthdb350872011-10-21 06:46:38 +0000767
Chandler Carruthdb350872011-10-21 06:46:38 +0000768 BlockToChain.clear();
Chandler Carruthf5e47ac2011-11-14 10:57:23 +0000769 ChainAllocator.DestroyAll();
Chandler Carruthdb350872011-10-21 06:46:38 +0000770
771 // We always return true as we have no way to track whether the final order
772 // differs from the original order.
773 return true;
774}
Chandler Carruth37efc9f2011-11-02 07:17:12 +0000775
776namespace {
777/// \brief A pass to compute block placement statistics.
778///
779/// A separate pass to compute interesting statistics for evaluating block
780/// placement. This is separate from the actual placement pass so that they can
781/// be computed in the absense of any placement transformations or when using
782/// alternative placement strategies.
783class MachineBlockPlacementStats : public MachineFunctionPass {
784 /// \brief A handle to the branch probability pass.
785 const MachineBranchProbabilityInfo *MBPI;
786
787 /// \brief A handle to the function-wide block frequency pass.
788 const MachineBlockFrequencyInfo *MBFI;
789
790public:
791 static char ID; // Pass identification, replacement for typeid
792 MachineBlockPlacementStats() : MachineFunctionPass(ID) {
793 initializeMachineBlockPlacementStatsPass(*PassRegistry::getPassRegistry());
794 }
795
796 bool runOnMachineFunction(MachineFunction &F);
797
798 void getAnalysisUsage(AnalysisUsage &AU) const {
799 AU.addRequired<MachineBranchProbabilityInfo>();
800 AU.addRequired<MachineBlockFrequencyInfo>();
801 AU.setPreservesAll();
802 MachineFunctionPass::getAnalysisUsage(AU);
803 }
804
805 const char *getPassName() const { return "Block Placement Stats"; }
806};
807}
808
809char MachineBlockPlacementStats::ID = 0;
810INITIALIZE_PASS_BEGIN(MachineBlockPlacementStats, "block-placement-stats",
811 "Basic Block Placement Stats", false, false)
812INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)
813INITIALIZE_PASS_DEPENDENCY(MachineBlockFrequencyInfo)
814INITIALIZE_PASS_END(MachineBlockPlacementStats, "block-placement-stats",
815 "Basic Block Placement Stats", false, false)
816
817FunctionPass *llvm::createMachineBlockPlacementStatsPass() {
818 return new MachineBlockPlacementStats();
819}
820
821bool MachineBlockPlacementStats::runOnMachineFunction(MachineFunction &F) {
822 // Check for single-block functions and skip them.
823 if (llvm::next(F.begin()) == F.end())
824 return false;
825
826 MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
827 MBFI = &getAnalysis<MachineBlockFrequencyInfo>();
828
829 for (MachineFunction::iterator I = F.begin(), E = F.end(); I != E; ++I) {
830 BlockFrequency BlockFreq = MBFI->getBlockFreq(I);
831 Statistic &NumBranches = (I->succ_size() > 1) ? NumCondBranches
832 : NumUncondBranches;
833 Statistic &BranchTakenFreq = (I->succ_size() > 1) ? CondBranchTakenFreq
834 : UncondBranchTakenFreq;
835 for (MachineBasicBlock::succ_iterator SI = I->succ_begin(),
836 SE = I->succ_end();
837 SI != SE; ++SI) {
838 // Skip if this successor is a fallthrough.
839 if (I->isLayoutSuccessor(*SI))
840 continue;
841
842 BlockFrequency EdgeFreq = BlockFreq * MBPI->getEdgeProbability(I, *SI);
843 ++NumBranches;
844 BranchTakenFreq += EdgeFreq.getFrequency();
845 }
846 }
847
848 return false;
849}
850