blob: f0285ea8f8eb352c3dbe7ebc5fae4817a20e8545 [file] [log] [blame]
Eugene Zelenko5df3d892017-08-24 21:21:39 +00001//===- MachineBlockPlacement.cpp - Basic Block Code Layout optimization ---===//
Chandler Carruth10281422011-10-21 06:46:38 +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//===----------------------------------------------------------------------===//
9//
Chandler Carruthbd1be4d2011-10-23 09:18:45 +000010// This file implements basic block placement transformations using the CFG
11// structure and branch probability estimates.
Chandler Carruth10281422011-10-21 06:46:38 +000012//
Chandler Carruthbd1be4d2011-10-23 09:18:45 +000013// The pass strives to preserve the structure of the CFG (that is, retain
Benjamin Kramerbde91762012-06-02 10:20:22 +000014// a topological ordering of basic blocks) in the absence of a *strong* signal
Chandler Carruthbd1be4d2011-10-23 09:18:45 +000015// 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 Carruth10281422011-10-21 06:46:38 +000025//
26//===----------------------------------------------------------------------===//
27
Haicheng Wu5b458cc2016-06-09 15:24:29 +000028#include "BranchFolding.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000029#include "llvm/ADT/ArrayRef.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000030#include "llvm/ADT/DenseMap.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000031#include "llvm/ADT/STLExtras.h"
32#include "llvm/ADT/SetVector.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000033#include "llvm/ADT/SmallPtrSet.h"
34#include "llvm/ADT/SmallVector.h"
35#include "llvm/ADT/Statistic.h"
Xinliang David Lifd3f6452017-01-29 01:57:02 +000036#include "llvm/Analysis/BlockFrequencyInfoImpl.h"
Chandler Carruth8b9737c2011-10-21 08:57:37 +000037#include "llvm/CodeGen/MachineBasicBlock.h"
Chandler Carruth10281422011-10-21 06:46:38 +000038#include "llvm/CodeGen/MachineBlockFrequencyInfo.h"
39#include "llvm/CodeGen/MachineBranchProbabilityInfo.h"
40#include "llvm/CodeGen/MachineFunction.h"
Chandler Carruth10281422011-10-21 06:46:38 +000041#include "llvm/CodeGen/MachineFunctionPass.h"
Chandler Carruth8b9737c2011-10-21 08:57:37 +000042#include "llvm/CodeGen/MachineLoopInfo.h"
43#include "llvm/CodeGen/MachineModuleInfo.h"
Kyle Buttb15c0662017-01-31 23:48:32 +000044#include "llvm/CodeGen/MachinePostDominators.h"
Kyle Butt0846e562016-10-11 20:36:43 +000045#include "llvm/CodeGen/TailDuplicator.h"
David Blaikie3f833ed2017-11-08 01:01:31 +000046#include "llvm/CodeGen/TargetInstrInfo.h"
David Blaikieb3bde2e2017-11-17 01:07:10 +000047#include "llvm/CodeGen/TargetLowering.h"
Chandler Carruth6bda14b2017-06-06 11:49:48 +000048#include "llvm/CodeGen/TargetPassConfig.h"
David Blaikieb3bde2e2017-11-17 01:07:10 +000049#include "llvm/CodeGen/TargetSubtargetInfo.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000050#include "llvm/IR/DebugLoc.h"
51#include "llvm/IR/Function.h"
52#include "llvm/Pass.h"
Chandler Carruth10281422011-10-21 06:46:38 +000053#include "llvm/Support/Allocator.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000054#include "llvm/Support/BlockFrequency.h"
55#include "llvm/Support/BranchProbability.h"
56#include "llvm/Support/CodeGen.h"
Nadav Rotemc3b0f502013-04-12 00:48:32 +000057#include "llvm/Support/CommandLine.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000058#include "llvm/Support/Compiler.h"
Chandler Carruthbd1be4d2011-10-23 09:18:45 +000059#include "llvm/Support/Debug.h"
Benjamin Kramer799003b2015-03-23 19:32:43 +000060#include "llvm/Support/raw_ostream.h"
Eugene Zelenko5df3d892017-08-24 21:21:39 +000061#include "llvm/Target/TargetMachine.h"
Chandler Carruth10281422011-10-21 06:46:38 +000062#include <algorithm>
Eugene Zelenko5df3d892017-08-24 21:21:39 +000063#include <cassert>
64#include <cstdint>
65#include <iterator>
66#include <memory>
67#include <string>
68#include <tuple>
Kyle Buttb15c0662017-01-31 23:48:32 +000069#include <utility>
Eugene Zelenko5df3d892017-08-24 21:21:39 +000070#include <vector>
71
Chandler Carruth10281422011-10-21 06:46:38 +000072using namespace llvm;
73
Chandler Carruthd0dced52015-03-05 02:28:25 +000074#define DEBUG_TYPE "block-placement"
Chandler Carruth1b9dde02014-04-22 02:02:50 +000075
Chandler Carruthae4e8002011-11-02 07:17:12 +000076STATISTIC(NumCondBranches, "Number of conditional branches");
Craig Topper77ec0772015-09-16 03:52:32 +000077STATISTIC(NumUncondBranches, "Number of unconditional branches");
Chandler Carruthae4e8002011-11-02 07:17:12 +000078STATISTIC(CondBranchTakenFreq,
79 "Potential frequency of taking conditional branches");
80STATISTIC(UncondBranchTakenFreq,
81 "Potential frequency of taking unconditional branches");
82
Nadav Rotemc3b0f502013-04-12 00:48:32 +000083static cl::opt<unsigned> AlignAllBlock("align-all-blocks",
84 cl::desc("Force the alignment of all "
85 "blocks in the function."),
86 cl::init(0), cl::Hidden);
87
Geoff Berry10494ac2016-01-21 17:25:52 +000088static cl::opt<unsigned> AlignAllNonFallThruBlocks(
89 "align-all-nofallthru-blocks",
90 cl::desc("Force the alignment of all "
91 "blocks that have no fall-through predecessors (i.e. don't add "
92 "nops that are executed)."),
93 cl::init(0), cl::Hidden);
94
Benjamin Kramerc8160d62013-11-20 19:08:44 +000095// FIXME: Find a good default for this flag and remove the flag.
Chandler Carruth2fc3fe12015-03-05 02:35:31 +000096static cl::opt<unsigned> ExitBlockBias(
97 "block-placement-exit-block-bias",
98 cl::desc("Block frequency percentage a loop exit block needs "
99 "over the original exit to be considered the new exit."),
100 cl::init(0), cl::Hidden);
Benjamin Kramerc8160d62013-11-20 19:08:44 +0000101
Sjoerd Meijer5e11a182016-07-27 08:49:23 +0000102// Definition:
103// - Outlining: placement of a basic block outside the chain or hot path.
104
Cong Houb90b9e02015-11-02 21:24:00 +0000105static cl::opt<unsigned> LoopToColdBlockRatio(
106 "loop-to-cold-block-ratio",
107 cl::desc("Outline loop blocks from loop chain if (frequency of loop) / "
108 "(frequency of block) is greater than this ratio"),
109 cl::init(5), cl::Hidden);
110
Kyle Butt74f61dd2017-08-04 21:13:41 +0000111static cl::opt<bool> ForceLoopColdBlock(
112 "force-loop-cold-block",
113 cl::desc("Force outlining cold blocks from loops."),
114 cl::init(false), cl::Hidden);
115
Cong Hou7745dbc2015-10-19 23:16:40 +0000116static cl::opt<bool>
117 PreciseRotationCost("precise-rotation-cost",
118 cl::desc("Model the cost of loop rotation more "
119 "precisely by using profile data."),
120 cl::init(false), cl::Hidden);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000121
Xinliang David Lif0ab6df2016-05-12 02:04:41 +0000122static cl::opt<bool>
123 ForcePreciseRotationCost("force-precise-rotation-cost",
Xinliang David Lib840bb82016-05-12 16:39:02 +0000124 cl::desc("Force the use of precise cost "
125 "loop rotation strategy."),
Xinliang David Lif0ab6df2016-05-12 02:04:41 +0000126 cl::init(false), cl::Hidden);
Cong Hou7745dbc2015-10-19 23:16:40 +0000127
128static cl::opt<unsigned> MisfetchCost(
129 "misfetch-cost",
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +0000130 cl::desc("Cost that models the probabilistic risk of an instruction "
Cong Hou7745dbc2015-10-19 23:16:40 +0000131 "misfetch due to a jump comparing to falling through, whose cost "
132 "is zero."),
133 cl::init(1), cl::Hidden);
134
135static cl::opt<unsigned> JumpInstCost("jump-inst-cost",
136 cl::desc("Cost of jump instructions."),
137 cl::init(1), cl::Hidden);
Kyle Butt0846e562016-10-11 20:36:43 +0000138static cl::opt<bool>
139TailDupPlacement("tail-dup-placement",
140 cl::desc("Perform tail duplication during placement. "
141 "Creates more fallthrough opportunites in "
142 "outline branches."),
143 cl::init(true), cl::Hidden);
Cong Hou7745dbc2015-10-19 23:16:40 +0000144
Haicheng Wu5b458cc2016-06-09 15:24:29 +0000145static cl::opt<bool>
146BranchFoldPlacement("branch-fold-placement",
147 cl::desc("Perform branch folding during placement. "
148 "Reduces code size."),
149 cl::init(true), cl::Hidden);
150
Kyle Butt0846e562016-10-11 20:36:43 +0000151// Heuristic for tail duplication.
Kyle Buttb15c0662017-01-31 23:48:32 +0000152static cl::opt<unsigned> TailDupPlacementThreshold(
Kyle Butt0846e562016-10-11 20:36:43 +0000153 "tail-dup-placement-threshold",
154 cl::desc("Instruction cutoff for tail duplication during layout. "
155 "Tail merging during layout is forced to have a threshold "
156 "that won't conflict."), cl::init(2),
157 cl::Hidden);
158
Kyle Butt7d531da2017-05-15 17:30:47 +0000159// Heuristic for aggressive tail duplication.
160static cl::opt<unsigned> TailDupPlacementAggressiveThreshold(
161 "tail-dup-placement-aggressive-threshold",
162 cl::desc("Instruction cutoff for aggressive tail duplication during "
163 "layout. Used at -O3. Tail merging during layout is forced to "
Richard Smithc0541df2017-08-17 23:38:41 +0000164 "have a threshold that won't conflict."), cl::init(4),
Kyle Butt7d531da2017-05-15 17:30:47 +0000165 cl::Hidden);
166
Kyle Buttb15c0662017-01-31 23:48:32 +0000167// Heuristic for tail duplication.
168static cl::opt<unsigned> TailDupPlacementPenalty(
169 "tail-dup-placement-penalty",
170 cl::desc("Cost penalty for blocks that can avoid breaking CFG by copying. "
171 "Copying can increase fallthrough, but it also increases icache "
172 "pressure. This parameter controls the penalty to account for that. "
173 "Percent as integer."),
174 cl::init(2),
175 cl::Hidden);
176
Kyle Butt1fa60302017-03-03 01:00:22 +0000177// Heuristic for triangle chains.
178static cl::opt<unsigned> TriangleChainCount(
179 "triangle-chain-count",
180 cl::desc("Number of triangle-shaped-CFG's that need to be in a row for the "
181 "triangle tail duplication heuristic to kick in. 0 to disable."),
Kyle Butt08655992017-03-16 01:32:29 +0000182 cl::init(2),
Kyle Butt1fa60302017-03-03 01:00:22 +0000183 cl::Hidden);
184
Xinliang David Liff287372016-06-03 23:48:36 +0000185extern cl::opt<unsigned> StaticLikelyProb;
Dehao Chen9f2bdfb2016-06-14 22:27:17 +0000186extern cl::opt<unsigned> ProfileLikelyProb;
Xinliang David Liff287372016-06-03 23:48:36 +0000187
Xinliang David Li58fcc9b2017-02-02 21:29:17 +0000188// Internal option used to control BFI display only after MBP pass.
189// Defined in CodeGen/MachineBlockFrequencyInfo.cpp:
190// -view-block-layout-with-bfi=
Xinliang David Lifd3f6452017-01-29 01:57:02 +0000191extern cl::opt<GVDAGType> ViewBlockLayoutWithBFI;
Xinliang David Li58fcc9b2017-02-02 21:29:17 +0000192
193// Command line option to specify the name of the function for CFG dump
194// Defined in Analysis/BlockFrequencyInfo.cpp: -view-bfi-func-name=
Xinliang David Lifd3f6452017-01-29 01:57:02 +0000195extern cl::opt<std::string> ViewBlockFreqFuncName;
Xinliang David Lifd3f6452017-01-29 01:57:02 +0000196
Chandler Carruth10281422011-10-21 06:46:38 +0000197namespace {
Chandler Carruth10281422011-10-21 06:46:38 +0000198
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000199class BlockChain;
200
201/// \brief Type for our function-wide basic block -> block chain mapping.
202using BlockToChainMapType = DenseMap<const MachineBasicBlock *, BlockChain *>;
203
Chandler Carruth10281422011-10-21 06:46:38 +0000204/// \brief A chain of blocks which will be laid out contiguously.
205///
206/// This is the datastructure representing a chain of consecutive blocks that
207/// are profitable to layout together in order to maximize fallthrough
Chandler Carruth9139f442012-06-26 05:16:37 +0000208/// probabilities and code locality. We also can use a block chain to represent
209/// a sequence of basic blocks which have some external (correctness)
210/// requirement for sequential layout.
Chandler Carruth10281422011-10-21 06:46:38 +0000211///
Chandler Carruth9139f442012-06-26 05:16:37 +0000212/// Chains can be built around a single basic block and can be merged to grow
213/// them. They participate in a block-to-chain mapping, which is updated
214/// automatically as chains are merged together.
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000215class BlockChain {
216 /// \brief The sequence of blocks belonging to this chain.
Chandler Carruth10281422011-10-21 06:46:38 +0000217 ///
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000218 /// This is the sequence of blocks for a particular chain. These will be laid
219 /// out in-order within the function.
220 SmallVector<MachineBasicBlock *, 4> Blocks;
Chandler Carruth10281422011-10-21 06:46:38 +0000221
222 /// \brief A handle to the function-wide basic block to block chain mapping.
223 ///
224 /// This is retained in each block chain to simplify the computation of child
225 /// block chains for SCC-formation and iteration. We store the edges to child
226 /// basic blocks, and map them back to their associated chains using this
227 /// structure.
228 BlockToChainMapType &BlockToChain;
229
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000230public:
Chandler Carruth10281422011-10-21 06:46:38 +0000231 /// \brief Construct a new BlockChain.
232 ///
233 /// This builds a new block chain representing a single basic block in the
234 /// function. It also registers itself as the chain that block participates
235 /// in with the BlockToChain mapping.
236 BlockChain(BlockToChainMapType &BlockToChain, MachineBasicBlock *BB)
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000237 : Blocks(1, BB), BlockToChain(BlockToChain) {
Chandler Carruth10281422011-10-21 06:46:38 +0000238 assert(BB && "Cannot create a chain with a null basic block");
239 BlockToChain[BB] = this;
240 }
241
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000242 /// \brief Iterator over blocks within the chain.
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000243 using iterator = SmallVectorImpl<MachineBasicBlock *>::iterator;
244 using const_iterator = SmallVectorImpl<MachineBasicBlock *>::const_iterator;
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000245
246 /// \brief Beginning of blocks within the chain.
Chandler Carruthccc7e422012-04-16 01:12:56 +0000247 iterator begin() { return Blocks.begin(); }
Kyle Butte9425c4f2017-02-04 02:26:32 +0000248 const_iterator begin() const { return Blocks.begin(); }
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000249
250 /// \brief End of blocks within the chain.
Chandler Carruthccc7e422012-04-16 01:12:56 +0000251 iterator end() { return Blocks.end(); }
Kyle Butte9425c4f2017-02-04 02:26:32 +0000252 const_iterator end() const { return Blocks.end(); }
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000253
Kyle Butt0846e562016-10-11 20:36:43 +0000254 bool remove(MachineBasicBlock* BB) {
255 for(iterator i = begin(); i != end(); ++i) {
256 if (*i == BB) {
257 Blocks.erase(i);
258 return true;
259 }
260 }
261 return false;
262 }
263
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000264 /// \brief Merge a block chain into this one.
Chandler Carruth10281422011-10-21 06:46:38 +0000265 ///
266 /// This routine merges a block chain into this one. It takes care of forming
267 /// a contiguous sequence of basic blocks, updating the edge list, and
268 /// updating the block -> chain mapping. It does not free or tear down the
269 /// old chain, but the old chain's block list is no longer valid.
Jakub Staszak90616162011-12-21 23:02:08 +0000270 void merge(MachineBasicBlock *BB, BlockChain *Chain) {
Kyle Butt0cf5b2f2017-05-17 23:44:41 +0000271 assert(BB && "Can't merge a null block.");
272 assert(!Blocks.empty() && "Can't merge into an empty chain.");
Chandler Carruth10281422011-10-21 06:46:38 +0000273
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000274 // Fast path in case we don't have a chain already.
275 if (!Chain) {
Kyle Butt0cf5b2f2017-05-17 23:44:41 +0000276 assert(!BlockToChain[BB] &&
277 "Passed chain is null, but BB has entry in BlockToChain.");
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000278 Blocks.push_back(BB);
279 BlockToChain[BB] = this;
280 return;
Chandler Carruth10281422011-10-21 06:46:38 +0000281 }
282
Kyle Butt0cf5b2f2017-05-17 23:44:41 +0000283 assert(BB == *Chain->begin() && "Passed BB is not head of Chain.");
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000284 assert(Chain->begin() != Chain->end());
Chandler Carruth10281422011-10-21 06:46:38 +0000285
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000286 // Update the incoming blocks to point to this chain, and add them to the
287 // chain structure.
Chandler Carruth7a715da2015-03-05 03:19:05 +0000288 for (MachineBasicBlock *ChainBB : *Chain) {
289 Blocks.push_back(ChainBB);
Kyle Butt0cf5b2f2017-05-17 23:44:41 +0000290 assert(BlockToChain[ChainBB] == Chain && "Incoming blocks not in chain.");
Chandler Carruth7a715da2015-03-05 03:19:05 +0000291 BlockToChain[ChainBB] = this;
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000292 }
Chandler Carruth10281422011-10-21 06:46:38 +0000293 }
Chandler Carruth8d150782011-11-13 11:20:44 +0000294
Chandler Carruth49158902012-04-08 14:37:01 +0000295#ifndef NDEBUG
296 /// \brief Dump the blocks in this chain.
Nico Weber7408c702014-01-03 22:53:37 +0000297 LLVM_DUMP_METHOD void dump() {
Chandler Carruth7a715da2015-03-05 03:19:05 +0000298 for (MachineBasicBlock *MBB : *this)
299 MBB->dump();
Chandler Carruth49158902012-04-08 14:37:01 +0000300 }
301#endif // NDEBUG
302
Philip Reamesae27b232016-03-03 00:58:43 +0000303 /// \brief Count of predecessors of any block within the chain which have not
304 /// yet been scheduled. In general, we will delay scheduling this chain
305 /// until those predecessors are scheduled (or we find a sufficiently good
306 /// reason to override this heuristic.) Note that when forming loop chains,
307 /// blocks outside the loop are ignored and treated as if they were already
308 /// scheduled.
Chandler Carruth8d150782011-11-13 11:20:44 +0000309 ///
Philip Reamesae27b232016-03-03 00:58:43 +0000310 /// Note: This field is reinitialized multiple times - once for each loop,
311 /// and then once for the function as a whole.
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000312 unsigned UnscheduledPredecessors = 0;
Chandler Carruth10281422011-10-21 06:46:38 +0000313};
Chandler Carruth10281422011-10-21 06:46:38 +0000314
Chandler Carruth10281422011-10-21 06:46:38 +0000315class MachineBlockPlacement : public MachineFunctionPass {
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000316 /// \brief A type for a block filter set.
317 using BlockFilterSet = SmallSetVector<const MachineBasicBlock *, 16>;
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000318
Kyle Buttb15c0662017-01-31 23:48:32 +0000319 /// Pair struct containing basic block and taildup profitiability
320 struct BlockAndTailDupResult {
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000321 MachineBasicBlock *BB;
Kyle Buttb15c0662017-01-31 23:48:32 +0000322 bool ShouldTailDup;
323 };
324
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000325 /// Triple struct containing edge weight and the edge.
326 struct WeightedEdge {
327 BlockFrequency Weight;
328 MachineBasicBlock *Src;
329 MachineBasicBlock *Dest;
330 };
331
Xinliang David Li93926ac2016-07-01 05:46:48 +0000332 /// \brief work lists of blocks that are ready to be laid out
333 SmallVector<MachineBasicBlock *, 16> BlockWorkList;
334 SmallVector<MachineBasicBlock *, 16> EHPadWorkList;
335
Kyle Buttebe6cc42017-02-23 21:22:24 +0000336 /// Edges that have already been computed as optimal.
337 DenseMap<const MachineBasicBlock *, BlockAndTailDupResult> ComputedEdges;
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000338
Xinliang David Li52530a72016-06-13 22:23:44 +0000339 /// \brief Machine Function
340 MachineFunction *F;
341
Chandler Carruth10281422011-10-21 06:46:38 +0000342 /// \brief A handle to the branch probability pass.
343 const MachineBranchProbabilityInfo *MBPI;
344
345 /// \brief A handle to the function-wide block frequency pass.
Haicheng Wu5b458cc2016-06-09 15:24:29 +0000346 std::unique_ptr<BranchFolder::MBFIWrapper> MBFI;
Chandler Carruth10281422011-10-21 06:46:38 +0000347
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000348 /// \brief A handle to the loop info.
Haicheng Wu5b458cc2016-06-09 15:24:29 +0000349 MachineLoopInfo *MLI;
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000350
Kyle Buttab9cca72016-10-27 21:37:20 +0000351 /// \brief Preferred loop exit.
352 /// Member variable for convenience. It may be removed by duplication deep
353 /// in the call stack.
354 MachineBasicBlock *PreferredLoopExit;
355
Chandler Carruth10281422011-10-21 06:46:38 +0000356 /// \brief A handle to the target's instruction info.
357 const TargetInstrInfo *TII;
358
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000359 /// \brief A handle to the target's lowering info.
Benjamin Kramer56b31bd2013-01-11 20:05:37 +0000360 const TargetLoweringBase *TLI;
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000361
Kyle Buttb15c0662017-01-31 23:48:32 +0000362 /// \brief A handle to the post dominator tree.
363 MachinePostDominatorTree *MPDT;
364
Kyle Butt0846e562016-10-11 20:36:43 +0000365 /// \brief Duplicator used to duplicate tails during placement.
366 ///
367 /// Placement decisions can open up new tail duplication opportunities, but
368 /// since tail duplication affects placement decisions of later blocks, it
369 /// must be done inline.
370 TailDuplicator TailDup;
371
Chandler Carruth10281422011-10-21 06:46:38 +0000372 /// \brief Allocator and owner of BlockChain structures.
373 ///
Chandler Carruth9139f442012-06-26 05:16:37 +0000374 /// We build BlockChains lazily while processing the loop structure of
375 /// a function. To reduce malloc traffic, we allocate them using this
376 /// slab-like allocator, and destroy them after the pass completes. An
377 /// important guarantee is that this allocator produces stable pointers to
378 /// the chains.
Chandler Carruth10281422011-10-21 06:46:38 +0000379 SpecificBumpPtrAllocator<BlockChain> ChainAllocator;
380
381 /// \brief Function wide BasicBlock to BlockChain mapping.
382 ///
383 /// This mapping allows efficiently moving from any given basic block to the
384 /// BlockChain it participates in, if any. We use it to, among other things,
385 /// allow implicitly defining edges between chains as the existing edges
386 /// between basic blocks.
Kyle Butte9425c4f2017-02-04 02:26:32 +0000387 DenseMap<const MachineBasicBlock *, BlockChain *> BlockToChain;
Chandler Carruth10281422011-10-21 06:46:38 +0000388
Sanjoy Dasd7389d62016-12-15 05:08:57 +0000389#ifndef NDEBUG
390 /// The set of basic blocks that have terminators that cannot be fully
391 /// analyzed. These basic blocks cannot be re-ordered safely by
392 /// MachineBlockPlacement, and we must preserve physical layout of these
393 /// blocks and their successors through the pass.
394 SmallPtrSet<MachineBasicBlock *, 4> BlocksWithUnanalyzableExits;
395#endif
396
Kyle Butt0846e562016-10-11 20:36:43 +0000397 /// Decrease the UnscheduledPredecessors count for all blocks in chain, and
398 /// if the count goes to 0, add them to the appropriate work list.
Kyle Butte9425c4f2017-02-04 02:26:32 +0000399 void markChainSuccessors(
400 const BlockChain &Chain, const MachineBasicBlock *LoopHeaderBB,
401 const BlockFilterSet *BlockFilter = nullptr);
Kyle Butt0846e562016-10-11 20:36:43 +0000402
403 /// Decrease the UnscheduledPredecessors count for a single block, and
404 /// if the count goes to 0, add them to the appropriate work list.
405 void markBlockSuccessors(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000406 const BlockChain &Chain, const MachineBasicBlock *BB,
407 const MachineBasicBlock *LoopHeaderBB,
Kyle Butt0846e562016-10-11 20:36:43 +0000408 const BlockFilterSet *BlockFilter = nullptr);
409
Xinliang David Li594ffa32016-06-11 18:35:40 +0000410 BranchProbability
Kyle Butte9425c4f2017-02-04 02:26:32 +0000411 collectViableSuccessors(
412 const MachineBasicBlock *BB, const BlockChain &Chain,
413 const BlockFilterSet *BlockFilter,
414 SmallVector<MachineBasicBlock *, 4> &Successors);
415 bool shouldPredBlockBeOutlined(
416 const MachineBasicBlock *BB, const MachineBasicBlock *Succ,
417 const BlockChain &Chain, const BlockFilterSet *BlockFilter,
418 BranchProbability SuccProb, BranchProbability HotProb);
Kyle Butt0846e562016-10-11 20:36:43 +0000419 bool repeatedlyTailDuplicateBlock(
420 MachineBasicBlock *BB, MachineBasicBlock *&LPred,
Kyle Butte9425c4f2017-02-04 02:26:32 +0000421 const MachineBasicBlock *LoopHeaderBB,
Kyle Butt0846e562016-10-11 20:36:43 +0000422 BlockChain &Chain, BlockFilterSet *BlockFilter,
423 MachineFunction::iterator &PrevUnplacedBlockIt);
Kyle Butte9425c4f2017-02-04 02:26:32 +0000424 bool maybeTailDuplicateBlock(
425 MachineBasicBlock *BB, MachineBasicBlock *LPred,
426 BlockChain &Chain, BlockFilterSet *BlockFilter,
427 MachineFunction::iterator &PrevUnplacedBlockIt,
428 bool &DuplicatedToPred);
429 bool hasBetterLayoutPredecessor(
430 const MachineBasicBlock *BB, const MachineBasicBlock *Succ,
431 const BlockChain &SuccChain, BranchProbability SuccProb,
432 BranchProbability RealSuccProb, const BlockChain &Chain,
433 const BlockFilterSet *BlockFilter);
434 BlockAndTailDupResult selectBestSuccessor(
435 const MachineBasicBlock *BB, const BlockChain &Chain,
436 const BlockFilterSet *BlockFilter);
437 MachineBasicBlock *selectBestCandidateBlock(
438 const BlockChain &Chain, SmallVectorImpl<MachineBasicBlock *> &WorkList);
439 MachineBasicBlock *getFirstUnplacedBlock(
440 const BlockChain &PlacedChain,
441 MachineFunction::iterator &PrevUnplacedBlockIt,
442 const BlockFilterSet *BlockFilter);
Amaury Secheteae09c22016-03-14 21:24:11 +0000443
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +0000444 /// \brief Add a basic block to the work list if it is appropriate.
Amaury Secheteae09c22016-03-14 21:24:11 +0000445 ///
446 /// If the optional parameter BlockFilter is provided, only MBB
447 /// present in the set will be added to the worklist. If nullptr
448 /// is provided, no filtering occurs.
Kyle Butte9425c4f2017-02-04 02:26:32 +0000449 void fillWorkLists(const MachineBasicBlock *MBB,
Amaury Secheteae09c22016-03-14 21:24:11 +0000450 SmallPtrSetImpl<BlockChain *> &UpdatedPreds,
Amaury Secheteae09c22016-03-14 21:24:11 +0000451 const BlockFilterSet *BlockFilter);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000452
Kyle Butte9425c4f2017-02-04 02:26:32 +0000453 void buildChain(const MachineBasicBlock *BB, BlockChain &Chain,
Kyle Butt0846e562016-10-11 20:36:43 +0000454 BlockFilterSet *BlockFilter = nullptr);
Kyle Butte9425c4f2017-02-04 02:26:32 +0000455 MachineBasicBlock *findBestLoopTop(
456 const MachineLoop &L, const BlockFilterSet &LoopBlockSet);
457 MachineBasicBlock *findBestLoopExit(
458 const MachineLoop &L, const BlockFilterSet &LoopBlockSet);
459 BlockFilterSet collectLoopBlockSet(const MachineLoop &L);
460 void buildLoopChains(const MachineLoop &L);
461 void rotateLoop(
462 BlockChain &LoopChain, const MachineBasicBlock *ExitingBB,
463 const BlockFilterSet &LoopBlockSet);
464 void rotateLoopWithProfile(
465 BlockChain &LoopChain, const MachineLoop &L,
466 const BlockFilterSet &LoopBlockSet);
Xinliang David Li52530a72016-06-13 22:23:44 +0000467 void buildCFGChains();
468 void optimizeBranches();
469 void alignBlocks();
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000470 /// Returns true if a block should be tail-duplicated to increase fallthrough
471 /// opportunities.
Kyle Buttb15c0662017-01-31 23:48:32 +0000472 bool shouldTailDuplicate(MachineBasicBlock *BB);
473 /// Check the edge frequencies to see if tail duplication will increase
474 /// fallthroughs.
475 bool isProfitableToTailDup(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000476 const MachineBasicBlock *BB, const MachineBasicBlock *Succ,
Kyle Buttb15c0662017-01-31 23:48:32 +0000477 BranchProbability AdjustedSumProb,
Kyle Butte9425c4f2017-02-04 02:26:32 +0000478 const BlockChain &Chain, const BlockFilterSet *BlockFilter);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000479
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000480 /// Check for a trellis layout.
481 bool isTrellis(const MachineBasicBlock *BB,
482 const SmallVectorImpl<MachineBasicBlock *> &ViableSuccs,
483 const BlockChain &Chain, const BlockFilterSet *BlockFilter);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000484
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000485 /// Get the best successor given a trellis layout.
486 BlockAndTailDupResult getBestTrellisSuccessor(
487 const MachineBasicBlock *BB,
488 const SmallVectorImpl<MachineBasicBlock *> &ViableSuccs,
489 BranchProbability AdjustedSumProb, const BlockChain &Chain,
490 const BlockFilterSet *BlockFilter);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000491
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000492 /// Get the best pair of non-conflicting edges.
493 static std::pair<WeightedEdge, WeightedEdge> getBestNonConflictingEdges(
494 const MachineBasicBlock *BB,
Benjamin Kramerd71461c2017-04-12 13:26:28 +0000495 MutableArrayRef<SmallVector<WeightedEdge, 8>> Edges);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000496
Kyle Buttb15c0662017-01-31 23:48:32 +0000497 /// Returns true if a block can tail duplicate into all unplaced
498 /// predecessors. Filters based on loop.
499 bool canTailDuplicateUnplacedPreds(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000500 const MachineBasicBlock *BB, MachineBasicBlock *Succ,
501 const BlockChain &Chain, const BlockFilterSet *BlockFilter);
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000502
Kyle Butt1fa60302017-03-03 01:00:22 +0000503 /// Find chains of triangles to tail-duplicate where a global analysis works,
504 /// but a local analysis would not find them.
505 void precomputeTriangleChains();
Chandler Carruth10281422011-10-21 06:46:38 +0000506
507public:
508 static char ID; // Pass identification, replacement for typeid
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000509
Chandler Carruth10281422011-10-21 06:46:38 +0000510 MachineBlockPlacement() : MachineFunctionPass(ID) {
511 initializeMachineBlockPlacementPass(*PassRegistry::getPassRegistry());
512 }
513
Craig Topper4584cd52014-03-07 09:26:03 +0000514 bool runOnMachineFunction(MachineFunction &F) override;
Chandler Carruth10281422011-10-21 06:46:38 +0000515
Craig Topper4584cd52014-03-07 09:26:03 +0000516 void getAnalysisUsage(AnalysisUsage &AU) const override {
Chandler Carruth10281422011-10-21 06:46:38 +0000517 AU.addRequired<MachineBranchProbabilityInfo>();
518 AU.addRequired<MachineBlockFrequencyInfo>();
Kyle Buttb15c0662017-01-31 23:48:32 +0000519 if (TailDupPlacement)
520 AU.addRequired<MachinePostDominatorTree>();
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000521 AU.addRequired<MachineLoopInfo>();
Haicheng Wu5b458cc2016-06-09 15:24:29 +0000522 AU.addRequired<TargetPassConfig>();
Chandler Carruth10281422011-10-21 06:46:38 +0000523 MachineFunctionPass::getAnalysisUsage(AU);
524 }
Chandler Carruth10281422011-10-21 06:46:38 +0000525};
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000526
527} // end anonymous namespace
Chandler Carruth10281422011-10-21 06:46:38 +0000528
529char MachineBlockPlacement::ID = 0;
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000530
Andrew Trick1fa5bcb2012-02-08 21:23:13 +0000531char &llvm::MachineBlockPlacementID = MachineBlockPlacement::ID;
Eugene Zelenko5df3d892017-08-24 21:21:39 +0000532
Matthias Braun1527baa2017-05-25 21:26:32 +0000533INITIALIZE_PASS_BEGIN(MachineBlockPlacement, DEBUG_TYPE,
Chandler Carruth10281422011-10-21 06:46:38 +0000534 "Branch Probability Basic Block Placement", false, false)
535INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)
536INITIALIZE_PASS_DEPENDENCY(MachineBlockFrequencyInfo)
Kyle Buttb15c0662017-01-31 23:48:32 +0000537INITIALIZE_PASS_DEPENDENCY(MachinePostDominatorTree)
Chandler Carruth8b9737c2011-10-21 08:57:37 +0000538INITIALIZE_PASS_DEPENDENCY(MachineLoopInfo)
Matthias Braun1527baa2017-05-25 21:26:32 +0000539INITIALIZE_PASS_END(MachineBlockPlacement, DEBUG_TYPE,
Chandler Carruth10281422011-10-21 06:46:38 +0000540 "Branch Probability Basic Block Placement", false, false)
541
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000542#ifndef NDEBUG
543/// \brief Helper to print the name of a MBB.
544///
545/// Only used by debug logging.
Kyle Butte9425c4f2017-02-04 02:26:32 +0000546static std::string getBlockName(const MachineBasicBlock *BB) {
Alp Tokere69170a2014-06-26 22:52:05 +0000547 std::string Result;
548 raw_string_ostream OS(Result);
Chandler Carruth2fc3fe12015-03-05 02:35:31 +0000549 OS << "BB#" << BB->getNumber();
Philip Reamesb9688f42016-03-02 21:45:13 +0000550 OS << " ('" << BB->getName() << "')";
Alp Tokere69170a2014-06-26 22:52:05 +0000551 OS.flush();
552 return Result;
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000553}
554#endif
555
Chandler Carrutheb4ec3a2011-11-13 11:34:55 +0000556/// \brief Mark a chain's successors as having one fewer preds.
557///
558/// When a chain is being merged into the "placed" chain, this routine will
559/// quickly walk the successors of each block in the chain and mark them as
560/// having one fewer active predecessor. It also adds any successors of this
Kyle Butt0846e562016-10-11 20:36:43 +0000561/// chain which reach the zero-predecessor state to the appropriate worklist.
Chandler Carruth8d150782011-11-13 11:20:44 +0000562void MachineBlockPlacement::markChainSuccessors(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000563 const BlockChain &Chain, const MachineBasicBlock *LoopHeaderBB,
Jakub Staszak90616162011-12-21 23:02:08 +0000564 const BlockFilterSet *BlockFilter) {
Chandler Carruth8d150782011-11-13 11:20:44 +0000565 // Walk all the blocks in this chain, marking their successors as having
566 // a predecessor placed.
Chandler Carruth7a715da2015-03-05 03:19:05 +0000567 for (MachineBasicBlock *MBB : Chain) {
Kyle Butt0846e562016-10-11 20:36:43 +0000568 markBlockSuccessors(Chain, MBB, LoopHeaderBB, BlockFilter);
569 }
570}
Chandler Carruth10281422011-10-21 06:46:38 +0000571
Kyle Butt0846e562016-10-11 20:36:43 +0000572/// \brief Mark a single block's successors as having one fewer preds.
573///
574/// Under normal circumstances, this is only called by markChainSuccessors,
575/// but if a block that was to be placed is completely tail-duplicated away,
576/// and was duplicated into the chain end, we need to redo markBlockSuccessors
577/// for just that block.
578void MachineBlockPlacement::markBlockSuccessors(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000579 const BlockChain &Chain, const MachineBasicBlock *MBB,
580 const MachineBasicBlock *LoopHeaderBB, const BlockFilterSet *BlockFilter) {
Kyle Butt0846e562016-10-11 20:36:43 +0000581 // Add any successors for which this is the only un-placed in-loop
582 // predecessor to the worklist as a viable candidate for CFG-neutral
583 // placement. No subsequent placement of this block will violate the CFG
584 // shape, so we get to use heuristics to choose a favorable placement.
585 for (MachineBasicBlock *Succ : MBB->successors()) {
586 if (BlockFilter && !BlockFilter->count(Succ))
587 continue;
588 BlockChain &SuccChain = *BlockToChain[Succ];
589 // Disregard edges within a fixed chain, or edges to the loop header.
590 if (&Chain == &SuccChain || Succ == LoopHeaderBB)
591 continue;
Amaury Sechetc53ad4f2016-04-07 21:29:39 +0000592
Kyle Butt0846e562016-10-11 20:36:43 +0000593 // This is a cross-chain edge that is within the loop, so decrement the
594 // loop predecessor count of the destination chain.
595 if (SuccChain.UnscheduledPredecessors == 0 ||
596 --SuccChain.UnscheduledPredecessors > 0)
597 continue;
598
599 auto *NewBB = *SuccChain.begin();
600 if (NewBB->isEHPad())
601 EHPadWorkList.push_back(NewBB);
602 else
603 BlockWorkList.push_back(NewBB);
Chandler Carruth10281422011-10-21 06:46:38 +0000604 }
Chandler Carruth8d150782011-11-13 11:20:44 +0000605}
Chandler Carruthbd1be4d2011-10-23 09:18:45 +0000606
Xinliang David Li594ffa32016-06-11 18:35:40 +0000607/// This helper function collects the set of successors of block
608/// \p BB that are allowed to be its layout successors, and return
609/// the total branch probability of edges from \p BB to those
610/// blocks.
611BranchProbability MachineBlockPlacement::collectViableSuccessors(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000612 const MachineBasicBlock *BB, const BlockChain &Chain,
613 const BlockFilterSet *BlockFilter,
Xinliang David Li594ffa32016-06-11 18:35:40 +0000614 SmallVector<MachineBasicBlock *, 4> &Successors) {
Cong Houd97c1002015-12-01 05:29:22 +0000615 // Adjust edge probabilities by excluding edges pointing to blocks that is
616 // either not in BlockFilter or is already in the current chain. Consider the
617 // following CFG:
Cong Hou41cf1a52015-11-18 00:52:52 +0000618 //
619 // --->A
620 // | / \
621 // | B C
622 // | \ / \
623 // ----D E
624 //
625 // Assume A->C is very hot (>90%), and C->D has a 50% probability, then after
626 // A->C is chosen as a fall-through, D won't be selected as a successor of C
627 // due to CFG constraint (the probability of C->D is not greater than
Hiroshi Inoue3c358f82017-06-16 12:23:04 +0000628 // HotProb to break topo-order). If we exclude E that is not in BlockFilter
629 // when calculating the probability of C->D, D will be selected and we
Xinliang David Li594ffa32016-06-11 18:35:40 +0000630 // will get A C D B as the layout of this loop.
Cong Houd97c1002015-12-01 05:29:22 +0000631 auto AdjustedSumProb = BranchProbability::getOne();
Cong Hou41cf1a52015-11-18 00:52:52 +0000632 for (MachineBasicBlock *Succ : BB->successors()) {
633 bool SkipSucc = false;
Amaury Sechetc53ad4f2016-04-07 21:29:39 +0000634 if (Succ->isEHPad() || (BlockFilter && !BlockFilter->count(Succ))) {
Cong Hou41cf1a52015-11-18 00:52:52 +0000635 SkipSucc = true;
636 } else {
637 BlockChain *SuccChain = BlockToChain[Succ];
638 if (SuccChain == &Chain) {
Cong Hou41cf1a52015-11-18 00:52:52 +0000639 SkipSucc = true;
640 } else if (Succ != *SuccChain->begin()) {
641 DEBUG(dbgs() << " " << getBlockName(Succ) << " -> Mid chain!\n");
642 continue;
643 }
644 }
645 if (SkipSucc)
Cong Houd97c1002015-12-01 05:29:22 +0000646 AdjustedSumProb -= MBPI->getEdgeProbability(BB, Succ);
Cong Hou41cf1a52015-11-18 00:52:52 +0000647 else
648 Successors.push_back(Succ);
649 }
650
Xinliang David Li594ffa32016-06-11 18:35:40 +0000651 return AdjustedSumProb;
652}
653
654/// The helper function returns the branch probability that is adjusted
655/// or normalized over the new total \p AdjustedSumProb.
Xinliang David Li594ffa32016-06-11 18:35:40 +0000656static BranchProbability
657getAdjustedProbability(BranchProbability OrigProb,
658 BranchProbability AdjustedSumProb) {
659 BranchProbability SuccProb;
660 uint32_t SuccProbN = OrigProb.getNumerator();
661 uint32_t SuccProbD = AdjustedSumProb.getNumerator();
662 if (SuccProbN >= SuccProbD)
663 SuccProb = BranchProbability::getOne();
664 else
665 SuccProb = BranchProbability(SuccProbN, SuccProbD);
666
667 return SuccProb;
668}
669
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000670/// Check if \p BB has exactly the successors in \p Successors.
671static bool
672hasSameSuccessors(MachineBasicBlock &BB,
673 SmallPtrSetImpl<const MachineBasicBlock *> &Successors) {
674 if (BB.succ_size() != Successors.size())
675 return false;
676 // We don't want to count self-loops
677 if (Successors.count(&BB))
678 return false;
679 for (MachineBasicBlock *Succ : BB.successors())
680 if (!Successors.count(Succ))
681 return false;
682 return true;
683}
684
685/// Check if a block should be tail duplicated to increase fallthrough
686/// opportunities.
Kyle Buttb15c0662017-01-31 23:48:32 +0000687/// \p BB Block to check.
688bool MachineBlockPlacement::shouldTailDuplicate(MachineBasicBlock *BB) {
689 // Blocks with single successors don't create additional fallthrough
690 // opportunities. Don't duplicate them. TODO: When conditional exits are
691 // analyzable, allow them to be duplicated.
692 bool IsSimple = TailDup.isSimpleBB(BB);
693
694 if (BB->succ_size() == 1)
695 return false;
696 return TailDup.shouldTailDuplicate(IsSimple, *BB);
697}
698
699/// Compare 2 BlockFrequency's with a small penalty for \p A.
700/// In order to be conservative, we apply a X% penalty to account for
701/// increased icache pressure and static heuristics. For small frequencies
702/// we use only the numerators to improve accuracy. For simplicity, we assume the
703/// penalty is less than 100%
704/// TODO(iteratee): Use 64-bit fixed point edge frequencies everywhere.
705static bool greaterWithBias(BlockFrequency A, BlockFrequency B,
706 uint64_t EntryFreq) {
707 BranchProbability ThresholdProb(TailDupPlacementPenalty, 100);
708 BlockFrequency Gain = A - B;
709 return (Gain / ThresholdProb).getFrequency() >= EntryFreq;
710}
711
712/// Check the edge frequencies to see if tail duplication will increase
713/// fallthroughs. It only makes sense to call this function when
714/// \p Succ would not be chosen otherwise. Tail duplication of \p Succ is
715/// always locally profitable if we would have picked \p Succ without
716/// considering duplication.
717bool MachineBlockPlacement::isProfitableToTailDup(
Kyle Butte9425c4f2017-02-04 02:26:32 +0000718 const MachineBasicBlock *BB, const MachineBasicBlock *Succ,
Kyle Buttb15c0662017-01-31 23:48:32 +0000719 BranchProbability QProb,
Kyle Butte9425c4f2017-02-04 02:26:32 +0000720 const BlockChain &Chain, const BlockFilterSet *BlockFilter) {
Kyle Buttb15c0662017-01-31 23:48:32 +0000721 // We need to do a probability calculation to make sure this is profitable.
722 // First: does succ have a successor that post-dominates? This affects the
723 // calculation. The 2 relevant cases are:
724 // BB BB
725 // | \Qout | \Qout
726 // P| C |P C
727 // = C' = C'
728 // | /Qin | /Qin
729 // | / | /
730 // Succ Succ
731 // / \ | \ V
732 // U/ =V |U \
733 // / \ = D
734 // D E | /
735 // | /
736 // |/
737 // PDom
738 // '=' : Branch taken for that CFG edge
739 // In the second case, Placing Succ while duplicating it into C prevents the
740 // fallthrough of Succ into either D or PDom, because they now have C as an
741 // unplaced predecessor
742
743 // Start by figuring out which case we fall into
744 MachineBasicBlock *PDom = nullptr;
745 SmallVector<MachineBasicBlock *, 4> SuccSuccs;
746 // Only scan the relevant successors
747 auto AdjustedSuccSumProb =
748 collectViableSuccessors(Succ, Chain, BlockFilter, SuccSuccs);
749 BranchProbability PProb = MBPI->getEdgeProbability(BB, Succ);
750 auto BBFreq = MBFI->getBlockFreq(BB);
751 auto SuccFreq = MBFI->getBlockFreq(Succ);
752 BlockFrequency P = BBFreq * PProb;
753 BlockFrequency Qout = BBFreq * QProb;
754 uint64_t EntryFreq = MBFI->getEntryFreq();
755 // If there are no more successors, it is profitable to copy, as it strictly
756 // increases fallthrough.
757 if (SuccSuccs.size() == 0)
758 return greaterWithBias(P, Qout, EntryFreq);
759
760 auto BestSuccSucc = BranchProbability::getZero();
761 // Find the PDom or the best Succ if no PDom exists.
762 for (MachineBasicBlock *SuccSucc : SuccSuccs) {
763 auto Prob = MBPI->getEdgeProbability(Succ, SuccSucc);
764 if (Prob > BestSuccSucc)
765 BestSuccSucc = Prob;
766 if (PDom == nullptr)
767 if (MPDT->dominates(SuccSucc, Succ)) {
768 PDom = SuccSucc;
769 break;
770 }
771 }
772 // For the comparisons, we need to know Succ's best incoming edge that isn't
773 // from BB.
774 auto SuccBestPred = BlockFrequency(0);
775 for (MachineBasicBlock *SuccPred : Succ->predecessors()) {
776 if (SuccPred == Succ || SuccPred == BB
777 || BlockToChain[SuccPred] == &Chain
778 || (BlockFilter && !BlockFilter->count(SuccPred)))
779 continue;
780 auto Freq = MBFI->getBlockFreq(SuccPred)
781 * MBPI->getEdgeProbability(SuccPred, Succ);
782 if (Freq > SuccBestPred)
783 SuccBestPred = Freq;
784 }
785 // Qin is Succ's best unplaced incoming edge that isn't BB
786 BlockFrequency Qin = SuccBestPred;
787 // If it doesn't have a post-dominating successor, here is the calculation:
788 // BB BB
789 // | \Qout | \
790 // P| C | =
791 // = C' | C
792 // | /Qin | |
793 // | / | C' (+Succ)
794 // Succ Succ /|
795 // / \ | \/ |
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000796 // U/ =V | == |
Kyle Buttb15c0662017-01-31 23:48:32 +0000797 // / \ | / \|
798 // D E D E
799 // '=' : Branch taken for that CFG edge
800 // Cost in the first case is: P + V
801 // For this calculation, we always assume P > Qout. If Qout > P
802 // The result of this function will be ignored at the caller.
Kyle Buttee51a202017-04-10 22:28:18 +0000803 // Let F = SuccFreq - Qin
804 // Cost in the second case is: Qout + min(Qin, F) * U + max(Qin, F) * V
Kyle Buttb15c0662017-01-31 23:48:32 +0000805
806 if (PDom == nullptr || !Succ->isSuccessor(PDom)) {
807 BranchProbability UProb = BestSuccSucc;
808 BranchProbability VProb = AdjustedSuccSumProb - UProb;
Kyle Buttee51a202017-04-10 22:28:18 +0000809 BlockFrequency F = SuccFreq - Qin;
Kyle Buttb15c0662017-01-31 23:48:32 +0000810 BlockFrequency V = SuccFreq * VProb;
Kyle Buttee51a202017-04-10 22:28:18 +0000811 BlockFrequency QinU = std::min(Qin, F) * UProb;
Kyle Buttb15c0662017-01-31 23:48:32 +0000812 BlockFrequency BaseCost = P + V;
Kyle Buttee51a202017-04-10 22:28:18 +0000813 BlockFrequency DupCost = Qout + QinU + std::max(Qin, F) * VProb;
Kyle Buttb15c0662017-01-31 23:48:32 +0000814 return greaterWithBias(BaseCost, DupCost, EntryFreq);
815 }
816 BranchProbability UProb = MBPI->getEdgeProbability(Succ, PDom);
817 BranchProbability VProb = AdjustedSuccSumProb - UProb;
818 BlockFrequency U = SuccFreq * UProb;
819 BlockFrequency V = SuccFreq * VProb;
Kyle Buttee51a202017-04-10 22:28:18 +0000820 BlockFrequency F = SuccFreq - Qin;
Kyle Buttb15c0662017-01-31 23:48:32 +0000821 // If there is a post-dominating successor, here is the calculation:
822 // BB BB BB BB
Kyle Buttee51a202017-04-10 22:28:18 +0000823 // | \Qout | \ | \Qout | \
824 // |P C | = |P C | =
825 // = C' |P C = C' |P C
826 // | /Qin | | | /Qin | |
827 // | / | C' (+Succ) | / | C' (+Succ)
828 // Succ Succ /| Succ Succ /|
829 // | \ V | \/ | | \ V | \/ |
830 // |U \ |U /\ =? |U = |U /\ |
831 // = D = = =?| | D | = =|
832 // | / |/ D | / |/ D
833 // | / | / | = | /
834 // |/ | / |/ | =
835 // Dom Dom Dom Dom
Kyle Buttb15c0662017-01-31 23:48:32 +0000836 // '=' : Branch taken for that CFG edge
837 // The cost for taken branches in the first case is P + U
Kyle Buttee51a202017-04-10 22:28:18 +0000838 // Let F = SuccFreq - Qin
Kyle Buttb15c0662017-01-31 23:48:32 +0000839 // The cost in the second case (assuming independence), given the layout:
Kyle Buttee51a202017-04-10 22:28:18 +0000840 // BB, Succ, (C+Succ), D, Dom or the layout:
841 // BB, Succ, D, Dom, (C+Succ)
842 // is Qout + max(F, Qin) * U + min(F, Qin)
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000843 // compare P + U vs Qout + P * U + Qin.
Kyle Buttb15c0662017-01-31 23:48:32 +0000844 //
845 // The 3rd and 4th cases cover when Dom would be chosen to follow Succ.
846 //
847 // For the 3rd case, the cost is P + 2 * V
Kyle Buttee51a202017-04-10 22:28:18 +0000848 // For the 4th case, the cost is Qout + min(Qin, F) * U + max(Qin, F) * V + V
849 // We choose 4 over 3 when (P + V) > Qout + min(Qin, F) * U + max(Qin, F) * V
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000850 if (UProb > AdjustedSuccSumProb / 2 &&
851 !hasBetterLayoutPredecessor(Succ, PDom, *BlockToChain[PDom], UProb, UProb,
852 Chain, BlockFilter))
Kyle Buttb15c0662017-01-31 23:48:32 +0000853 // Cases 3 & 4
Kyle Buttee51a202017-04-10 22:28:18 +0000854 return greaterWithBias(
855 (P + V), (Qout + std::max(Qin, F) * VProb + std::min(Qin, F) * UProb),
856 EntryFreq);
Kyle Buttb15c0662017-01-31 23:48:32 +0000857 // Cases 1 & 2
Kyle Buttee51a202017-04-10 22:28:18 +0000858 return greaterWithBias((P + U),
859 (Qout + std::min(Qin, F) * AdjustedSuccSumProb +
860 std::max(Qin, F) * UProb),
861 EntryFreq);
Kyle Buttb15c0662017-01-31 23:48:32 +0000862}
863
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000864/// Check for a trellis layout. \p BB is the upper part of a trellis if its
865/// successors form the lower part of a trellis. A successor set S forms the
866/// lower part of a trellis if all of the predecessors of S are either in S or
867/// have all of S as successors. We ignore trellises where BB doesn't have 2
868/// successors because for fewer than 2, it's trivial, and for 3 or greater they
869/// are very uncommon and complex to compute optimally. Allowing edges within S
870/// is not strictly a trellis, but the same algorithm works, so we allow it.
871bool MachineBlockPlacement::isTrellis(
872 const MachineBasicBlock *BB,
873 const SmallVectorImpl<MachineBasicBlock *> &ViableSuccs,
874 const BlockChain &Chain, const BlockFilterSet *BlockFilter) {
875 // Technically BB could form a trellis with branching factor higher than 2.
876 // But that's extremely uncommon.
877 if (BB->succ_size() != 2 || ViableSuccs.size() != 2)
878 return false;
879
880 SmallPtrSet<const MachineBasicBlock *, 2> Successors(BB->succ_begin(),
881 BB->succ_end());
882 // To avoid reviewing the same predecessors twice.
883 SmallPtrSet<const MachineBasicBlock *, 8> SeenPreds;
884
885 for (MachineBasicBlock *Succ : ViableSuccs) {
886 int PredCount = 0;
887 for (auto SuccPred : Succ->predecessors()) {
888 // Allow triangle successors, but don't count them.
Dehao Chenb197d5b2017-03-23 23:28:09 +0000889 if (Successors.count(SuccPred)) {
890 // Make sure that it is actually a triangle.
891 for (MachineBasicBlock *CheckSucc : SuccPred->successors())
892 if (!Successors.count(CheckSucc))
893 return false;
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000894 continue;
Dehao Chenb197d5b2017-03-23 23:28:09 +0000895 }
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000896 const BlockChain *PredChain = BlockToChain[SuccPred];
897 if (SuccPred == BB || (BlockFilter && !BlockFilter->count(SuccPred)) ||
898 PredChain == &Chain || PredChain == BlockToChain[Succ])
899 continue;
900 ++PredCount;
901 // Perform the successor check only once.
902 if (!SeenPreds.insert(SuccPred).second)
903 continue;
904 if (!hasSameSuccessors(*SuccPred, Successors))
905 return false;
906 }
907 // If one of the successors has only BB as a predecessor, it is not a
908 // trellis.
909 if (PredCount < 1)
910 return false;
911 }
912 return true;
913}
914
915/// Pick the highest total weight pair of edges that can both be laid out.
916/// The edges in \p Edges[0] are assumed to have a different destination than
917/// the edges in \p Edges[1]. Simple counting shows that the best pair is either
918/// the individual highest weight edges to the 2 different destinations, or in
919/// case of a conflict, one of them should be replaced with a 2nd best edge.
920std::pair<MachineBlockPlacement::WeightedEdge,
921 MachineBlockPlacement::WeightedEdge>
922MachineBlockPlacement::getBestNonConflictingEdges(
923 const MachineBasicBlock *BB,
Benjamin Kramerd71461c2017-04-12 13:26:28 +0000924 MutableArrayRef<SmallVector<MachineBlockPlacement::WeightedEdge, 8>>
925 Edges) {
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000926 // Sort the edges, and then for each successor, find the best incoming
927 // predecessor. If the best incoming predecessors aren't the same,
928 // then that is clearly the best layout. If there is a conflict, one of the
929 // successors will have to fallthrough from the second best predecessor. We
930 // compare which combination is better overall.
931
932 // Sort for highest frequency.
933 auto Cmp = [](WeightedEdge A, WeightedEdge B) { return A.Weight > B.Weight; };
934
935 std::stable_sort(Edges[0].begin(), Edges[0].end(), Cmp);
936 std::stable_sort(Edges[1].begin(), Edges[1].end(), Cmp);
937 auto BestA = Edges[0].begin();
938 auto BestB = Edges[1].begin();
939 // Arrange for the correct answer to be in BestA and BestB
940 // If the 2 best edges don't conflict, the answer is already there.
941 if (BestA->Src == BestB->Src) {
942 // Compare the total fallthrough of (Best + Second Best) for both pairs
943 auto SecondBestA = std::next(BestA);
944 auto SecondBestB = std::next(BestB);
945 BlockFrequency BestAScore = BestA->Weight + SecondBestB->Weight;
946 BlockFrequency BestBScore = BestB->Weight + SecondBestA->Weight;
947 if (BestAScore < BestBScore)
948 BestA = SecondBestA;
949 else
950 BestB = SecondBestB;
951 }
952 // Arrange for the BB edge to be in BestA if it exists.
953 if (BestB->Src == BB)
954 std::swap(BestA, BestB);
955 return std::make_pair(*BestA, *BestB);
956}
957
958/// Get the best successor from \p BB based on \p BB being part of a trellis.
959/// We only handle trellises with 2 successors, so the algorithm is
960/// straightforward: Find the best pair of edges that don't conflict. We find
961/// the best incoming edge for each successor in the trellis. If those conflict,
962/// we consider which of them should be replaced with the second best.
963/// Upon return the two best edges will be in \p BestEdges. If one of the edges
964/// comes from \p BB, it will be in \p BestEdges[0]
965MachineBlockPlacement::BlockAndTailDupResult
966MachineBlockPlacement::getBestTrellisSuccessor(
967 const MachineBasicBlock *BB,
968 const SmallVectorImpl<MachineBasicBlock *> &ViableSuccs,
969 BranchProbability AdjustedSumProb, const BlockChain &Chain,
970 const BlockFilterSet *BlockFilter) {
971
972 BlockAndTailDupResult Result = {nullptr, false};
973 SmallPtrSet<const MachineBasicBlock *, 4> Successors(BB->succ_begin(),
974 BB->succ_end());
975
976 // We assume size 2 because it's common. For general n, we would have to do
977 // the Hungarian algorithm, but it's not worth the complexity because more
978 // than 2 successors is fairly uncommon, and a trellis even more so.
979 if (Successors.size() != 2 || ViableSuccs.size() != 2)
980 return Result;
981
982 // Collect the edge frequencies of all edges that form the trellis.
Benjamin Kramerd71461c2017-04-12 13:26:28 +0000983 SmallVector<WeightedEdge, 8> Edges[2];
Kyle Butt7fbec9b2017-02-15 19:49:14 +0000984 int SuccIndex = 0;
985 for (auto Succ : ViableSuccs) {
986 for (MachineBasicBlock *SuccPred : Succ->predecessors()) {
987 // Skip any placed predecessors that are not BB
988 if (SuccPred != BB)
989 if ((BlockFilter && !BlockFilter->count(SuccPred)) ||
990 BlockToChain[SuccPred] == &Chain ||
991 BlockToChain[SuccPred] == BlockToChain[Succ])
992 continue;
993 BlockFrequency EdgeFreq = MBFI->getBlockFreq(SuccPred) *
994 MBPI->getEdgeProbability(SuccPred, Succ);
995 Edges[SuccIndex].push_back({EdgeFreq, SuccPred, Succ});
996 }
997 ++SuccIndex;
998 }
999
1000 // Pick the best combination of 2 edges from all the edges in the trellis.
1001 WeightedEdge BestA, BestB;
1002 std::tie(BestA, BestB) = getBestNonConflictingEdges(BB, Edges);
1003
1004 if (BestA.Src != BB) {
1005 // If we have a trellis, and BB doesn't have the best fallthrough edges,
1006 // we shouldn't choose any successor. We've already looked and there's a
1007 // better fallthrough edge for all the successors.
1008 DEBUG(dbgs() << "Trellis, but not one of the chosen edges.\n");
1009 return Result;
1010 }
1011
1012 // Did we pick the triangle edge? If tail-duplication is profitable, do
1013 // that instead. Otherwise merge the triangle edge now while we know it is
1014 // optimal.
1015 if (BestA.Dest == BestB.Src) {
1016 // The edges are BB->Succ1->Succ2, and we're looking to see if BB->Succ2
1017 // would be better.
1018 MachineBasicBlock *Succ1 = BestA.Dest;
1019 MachineBasicBlock *Succ2 = BestB.Dest;
1020 // Check to see if tail-duplication would be profitable.
1021 if (TailDupPlacement && shouldTailDuplicate(Succ2) &&
1022 canTailDuplicateUnplacedPreds(BB, Succ2, Chain, BlockFilter) &&
1023 isProfitableToTailDup(BB, Succ2, MBPI->getEdgeProbability(BB, Succ1),
1024 Chain, BlockFilter)) {
1025 DEBUG(BranchProbability Succ2Prob = getAdjustedProbability(
1026 MBPI->getEdgeProbability(BB, Succ2), AdjustedSumProb);
1027 dbgs() << " Selected: " << getBlockName(Succ2)
1028 << ", probability: " << Succ2Prob << " (Tail Duplicate)\n");
1029 Result.BB = Succ2;
1030 Result.ShouldTailDup = true;
1031 return Result;
1032 }
1033 }
1034 // We have already computed the optimal edge for the other side of the
1035 // trellis.
Kyle Buttebe6cc42017-02-23 21:22:24 +00001036 ComputedEdges[BestB.Src] = { BestB.Dest, false };
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001037
1038 auto TrellisSucc = BestA.Dest;
1039 DEBUG(BranchProbability SuccProb = getAdjustedProbability(
1040 MBPI->getEdgeProbability(BB, TrellisSucc), AdjustedSumProb);
1041 dbgs() << " Selected: " << getBlockName(TrellisSucc)
1042 << ", probability: " << SuccProb << " (Trellis)\n");
1043 Result.BB = TrellisSucc;
1044 return Result;
1045}
Kyle Buttb15c0662017-01-31 23:48:32 +00001046
1047/// When the option TailDupPlacement is on, this method checks if the
1048/// fallthrough candidate block \p Succ (of block \p BB) can be tail-duplicated
1049/// into all of its unplaced, unfiltered predecessors, that are not BB.
1050bool MachineBlockPlacement::canTailDuplicateUnplacedPreds(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001051 const MachineBasicBlock *BB, MachineBasicBlock *Succ,
1052 const BlockChain &Chain, const BlockFilterSet *BlockFilter) {
Kyle Buttb15c0662017-01-31 23:48:32 +00001053 if (!shouldTailDuplicate(Succ))
1054 return false;
1055
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001056 // For CFG checking.
1057 SmallPtrSet<const MachineBasicBlock *, 4> Successors(BB->succ_begin(),
1058 BB->succ_end());
Kyle Buttb15c0662017-01-31 23:48:32 +00001059 for (MachineBasicBlock *Pred : Succ->predecessors()) {
1060 // Make sure all unplaced and unfiltered predecessors can be
1061 // tail-duplicated into.
Kyle Butte9425c4f2017-02-04 02:26:32 +00001062 // Skip any blocks that are already placed or not in this loop.
Kyle Buttb15c0662017-01-31 23:48:32 +00001063 if (Pred == BB || (BlockFilter && !BlockFilter->count(Pred))
1064 || BlockToChain[Pred] == &Chain)
1065 continue;
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001066 if (!TailDup.canTailDuplicate(Succ, Pred)) {
1067 if (Successors.size() > 1 && hasSameSuccessors(*Pred, Successors))
1068 // This will result in a trellis after tail duplication, so we don't
1069 // need to copy Succ into this predecessor. In the presence
1070 // of a trellis tail duplication can continue to be profitable.
1071 // For example:
1072 // A A
1073 // |\ |\
1074 // | \ | \
1075 // | C | C+BB
1076 // | / | |
1077 // |/ | |
1078 // BB => BB |
1079 // |\ |\/|
1080 // | \ |/\|
1081 // | D | D
1082 // | / | /
1083 // |/ |/
1084 // Succ Succ
1085 //
1086 // After BB was duplicated into C, the layout looks like the one on the
1087 // right. BB and C now have the same successors. When considering
1088 // whether Succ can be duplicated into all its unplaced predecessors, we
1089 // ignore C.
1090 // We can do this because C already has a profitable fallthrough, namely
1091 // D. TODO(iteratee): ignore sufficiently cold predecessors for
1092 // duplication and for this test.
1093 //
1094 // This allows trellises to be laid out in 2 separate chains
1095 // (A,B,Succ,...) and later (C,D,...) This is a reasonable heuristic
1096 // because it allows the creation of 2 fallthrough paths with links
1097 // between them, and we correctly identify the best layout for these
1098 // CFGs. We want to extend trellises that the user created in addition
1099 // to trellises created by tail-duplication, so we just look for the
1100 // CFG.
1101 continue;
Kyle Buttb15c0662017-01-31 23:48:32 +00001102 return false;
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001103 }
Kyle Buttb15c0662017-01-31 23:48:32 +00001104 }
1105 return true;
1106}
1107
Kyle Butt1fa60302017-03-03 01:00:22 +00001108/// Find chains of triangles where we believe it would be profitable to
1109/// tail-duplicate them all, but a local analysis would not find them.
1110/// There are 3 ways this can be profitable:
1111/// 1) The post-dominators marked 50% are actually taken 55% (This shrinks with
1112/// longer chains)
1113/// 2) The chains are statically correlated. Branch probabilities have a very
1114/// U-shaped distribution.
1115/// [http://nrs.harvard.edu/urn-3:HUL.InstRepos:24015805]
1116/// If the branches in a chain are likely to be from the same side of the
1117/// distribution as their predecessor, but are independent at runtime, this
1118/// transformation is profitable. (Because the cost of being wrong is a small
1119/// fixed cost, unlike the standard triangle layout where the cost of being
1120/// wrong scales with the # of triangles.)
1121/// 3) The chains are dynamically correlated. If the probability that a previous
1122/// branch was taken positively influences whether the next branch will be
1123/// taken
1124/// We believe that 2 and 3 are common enough to justify the small margin in 1.
1125void MachineBlockPlacement::precomputeTriangleChains() {
1126 struct TriangleChain {
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001127 std::vector<MachineBasicBlock *> Edges;
Eugene Zelenko5df3d892017-08-24 21:21:39 +00001128
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001129 TriangleChain(MachineBasicBlock *src, MachineBasicBlock *dst)
1130 : Edges({src, dst}) {}
Kyle Butt1fa60302017-03-03 01:00:22 +00001131
1132 void append(MachineBasicBlock *dst) {
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001133 assert(getKey()->isSuccessor(dst) &&
Kyle Butt1fa60302017-03-03 01:00:22 +00001134 "Attempting to append a block that is not a successor.");
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001135 Edges.push_back(dst);
Kyle Butt1fa60302017-03-03 01:00:22 +00001136 }
1137
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001138 unsigned count() const { return Edges.size() - 1; }
1139
1140 MachineBasicBlock *getKey() const {
1141 return Edges.back();
Kyle Butt1fa60302017-03-03 01:00:22 +00001142 }
1143 };
1144
1145 if (TriangleChainCount == 0)
1146 return;
1147
1148 DEBUG(dbgs() << "Pre-computing triangle chains.\n");
1149 // Map from last block to the chain that contains it. This allows us to extend
1150 // chains as we find new triangles.
1151 DenseMap<const MachineBasicBlock *, TriangleChain> TriangleChainMap;
1152 for (MachineBasicBlock &BB : *F) {
1153 // If BB doesn't have 2 successors, it doesn't start a triangle.
1154 if (BB.succ_size() != 2)
1155 continue;
1156 MachineBasicBlock *PDom = nullptr;
1157 for (MachineBasicBlock *Succ : BB.successors()) {
1158 if (!MPDT->dominates(Succ, &BB))
1159 continue;
1160 PDom = Succ;
1161 break;
1162 }
1163 // If BB doesn't have a post-dominating successor, it doesn't form a
1164 // triangle.
1165 if (PDom == nullptr)
1166 continue;
1167 // If PDom has a hint that it is low probability, skip this triangle.
1168 if (MBPI->getEdgeProbability(&BB, PDom) < BranchProbability(50, 100))
1169 continue;
1170 // If PDom isn't eligible for duplication, this isn't the kind of triangle
1171 // we're looking for.
1172 if (!shouldTailDuplicate(PDom))
1173 continue;
1174 bool CanTailDuplicate = true;
1175 // If PDom can't tail-duplicate into it's non-BB predecessors, then this
1176 // isn't the kind of triangle we're looking for.
1177 for (MachineBasicBlock* Pred : PDom->predecessors()) {
1178 if (Pred == &BB)
1179 continue;
1180 if (!TailDup.canTailDuplicate(PDom, Pred)) {
1181 CanTailDuplicate = false;
1182 break;
1183 }
1184 }
1185 // If we can't tail-duplicate PDom to its predecessors, then skip this
1186 // triangle.
1187 if (!CanTailDuplicate)
1188 continue;
1189
1190 // Now we have an interesting triangle. Insert it if it's not part of an
Hiroshi Inoue3c358f82017-06-16 12:23:04 +00001191 // existing chain.
Kyle Butt1fa60302017-03-03 01:00:22 +00001192 // Note: This cannot be replaced with a call insert() or emplace() because
1193 // the find key is BB, but the insert/emplace key is PDom.
1194 auto Found = TriangleChainMap.find(&BB);
1195 // If it is, remove the chain from the map, grow it, and put it back in the
1196 // map with the end as the new key.
1197 if (Found != TriangleChainMap.end()) {
1198 TriangleChain Chain = std::move(Found->second);
1199 TriangleChainMap.erase(Found);
1200 Chain.append(PDom);
1201 TriangleChainMap.insert(std::make_pair(Chain.getKey(), std::move(Chain)));
1202 } else {
1203 auto InsertResult = TriangleChainMap.try_emplace(PDom, &BB, PDom);
Benjamin Kramer33580692017-04-12 13:26:31 +00001204 assert(InsertResult.second && "Block seen twice.");
1205 (void)InsertResult;
Kyle Butt1fa60302017-03-03 01:00:22 +00001206 }
1207 }
1208
Kyle Butt336c78f2017-04-12 18:30:32 +00001209 // Iterating over a DenseMap is safe here, because the only thing in the body
1210 // of the loop is inserting into another DenseMap (ComputedEdges).
1211 // ComputedEdges is never iterated, so this doesn't lead to non-determinism.
Kyle Butt1fa60302017-03-03 01:00:22 +00001212 for (auto &ChainPair : TriangleChainMap) {
1213 TriangleChain &Chain = ChainPair.second;
1214 // Benchmarking has shown that due to branch correlation duplicating 2 or
1215 // more triangles is profitable, despite the calculations assuming
1216 // independence.
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001217 if (Chain.count() < TriangleChainCount)
Kyle Butt1fa60302017-03-03 01:00:22 +00001218 continue;
Benjamin Kramerd71461c2017-04-12 13:26:28 +00001219 MachineBasicBlock *dst = Chain.Edges.back();
1220 Chain.Edges.pop_back();
1221 for (MachineBasicBlock *src : reverse(Chain.Edges)) {
Kyle Butt1fa60302017-03-03 01:00:22 +00001222 DEBUG(dbgs() << "Marking edge: " << getBlockName(src) << "->" <<
1223 getBlockName(dst) << " as pre-computed based on triangles.\n");
Benjamin Kramer33580692017-04-12 13:26:31 +00001224
1225 auto InsertResult = ComputedEdges.insert({src, {dst, true}});
1226 assert(InsertResult.second && "Block seen twice.");
1227 (void)InsertResult;
1228
Kyle Butt1fa60302017-03-03 01:00:22 +00001229 dst = src;
1230 }
1231 }
1232}
1233
Dehao Chen9f2bdfb2016-06-14 22:27:17 +00001234// When profile is not present, return the StaticLikelyProb.
1235// When profile is available, we need to handle the triangle-shape CFG.
1236static BranchProbability getLayoutSuccessorProbThreshold(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001237 const MachineBasicBlock *BB) {
Dehao Chen9f2bdfb2016-06-14 22:27:17 +00001238 if (!BB->getParent()->getFunction()->getEntryCount())
1239 return BranchProbability(StaticLikelyProb, 100);
1240 if (BB->succ_size() == 2) {
1241 const MachineBasicBlock *Succ1 = *BB->succ_begin();
1242 const MachineBasicBlock *Succ2 = *(BB->succ_begin() + 1);
Xinliang David Lie34ed832016-06-15 03:03:30 +00001243 if (Succ1->isSuccessor(Succ2) || Succ2->isSuccessor(Succ1)) {
1244 /* See case 1 below for the cost analysis. For BB->Succ to
1245 * be taken with smaller cost, the following needs to hold:
Kyle Buttb15c0662017-01-31 23:48:32 +00001246 * Prob(BB->Succ) > 2 * Prob(BB->Pred)
1247 * So the threshold T in the calculation below
1248 * (1-T) * Prob(BB->Succ) > T * Prob(BB->Pred)
1249 * So T / (1 - T) = 2, Yielding T = 2/3
1250 * Also adding user specified branch bias, we have
Xinliang David Lie34ed832016-06-15 03:03:30 +00001251 * T = (2/3)*(ProfileLikelyProb/50)
1252 * = (2*ProfileLikelyProb)/150)
1253 */
1254 return BranchProbability(2 * ProfileLikelyProb, 150);
1255 }
Dehao Chen9f2bdfb2016-06-14 22:27:17 +00001256 }
1257 return BranchProbability(ProfileLikelyProb, 100);
Xinliang David Licbf12142016-06-13 20:24:19 +00001258}
1259
1260/// Checks to see if the layout candidate block \p Succ has a better layout
1261/// predecessor than \c BB. If yes, returns true.
Kyle Buttb15c0662017-01-31 23:48:32 +00001262/// \p SuccProb: The probability adjusted for only remaining blocks.
1263/// Only used for logging
1264/// \p RealSuccProb: The un-adjusted probability.
1265/// \p Chain: The chain that BB belongs to and Succ is being considered for.
1266/// \p BlockFilter: if non-null, the set of blocks that make up the loop being
1267/// considered
Xinliang David Licbf12142016-06-13 20:24:19 +00001268bool MachineBlockPlacement::hasBetterLayoutPredecessor(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001269 const MachineBasicBlock *BB, const MachineBasicBlock *Succ,
1270 const BlockChain &SuccChain, BranchProbability SuccProb,
1271 BranchProbability RealSuccProb, const BlockChain &Chain,
1272 const BlockFilterSet *BlockFilter) {
Xinliang David Licbf12142016-06-13 20:24:19 +00001273
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001274 // There isn't a better layout when there are no unscheduled predecessors.
Xinliang David Licbf12142016-06-13 20:24:19 +00001275 if (SuccChain.UnscheduledPredecessors == 0)
1276 return false;
1277
1278 // There are two basic scenarios here:
1279 // -------------------------------------
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001280 // Case 1: triangular shape CFG (if-then):
Xinliang David Licbf12142016-06-13 20:24:19 +00001281 // BB
1282 // | \
1283 // | \
1284 // | Pred
1285 // | /
1286 // Succ
1287 // In this case, we are evaluating whether to select edge -> Succ, e.g.
1288 // set Succ as the layout successor of BB. Picking Succ as BB's
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001289 // successor breaks the CFG constraints (FIXME: define these constraints).
1290 // With this layout, Pred BB
Xinliang David Licbf12142016-06-13 20:24:19 +00001291 // is forced to be outlined, so the overall cost will be cost of the
1292 // branch taken from BB to Pred, plus the cost of back taken branch
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001293 // from Pred to Succ, as well as the additional cost associated
Xinliang David Licbf12142016-06-13 20:24:19 +00001294 // with the needed unconditional jump instruction from Pred To Succ.
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001295
Xinliang David Licbf12142016-06-13 20:24:19 +00001296 // The cost of the topological order layout is the taken branch cost
1297 // from BB to Succ, so to make BB->Succ a viable candidate, the following
1298 // must hold:
1299 // 2 * freq(BB->Pred) * taken_branch_cost + unconditional_jump_cost
1300 // < freq(BB->Succ) * taken_branch_cost.
1301 // Ignoring unconditional jump cost, we get
1302 // freq(BB->Succ) > 2 * freq(BB->Pred), i.e.,
1303 // prob(BB->Succ) > 2 * prob(BB->Pred)
1304 //
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001305 // When real profile data is available, we can precisely compute the
1306 // probability threshold that is needed for edge BB->Succ to be considered.
1307 // Without profile data, the heuristic requires the branch bias to be
Xinliang David Licbf12142016-06-13 20:24:19 +00001308 // a lot larger to make sure the signal is very strong (e.g. 80% default).
1309 // -----------------------------------------------------------------
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001310 // Case 2: diamond like CFG (if-then-else):
Xinliang David Licbf12142016-06-13 20:24:19 +00001311 // S
1312 // / \
1313 // | \
1314 // BB Pred
1315 // \ /
1316 // Succ
1317 // ..
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001318 //
1319 // The current block is BB and edge BB->Succ is now being evaluated.
1320 // Note that edge S->BB was previously already selected because
1321 // prob(S->BB) > prob(S->Pred).
1322 // At this point, 2 blocks can be placed after BB: Pred or Succ. If we
1323 // choose Pred, we will have a topological ordering as shown on the left
1324 // in the picture below. If we choose Succ, we have the solution as shown
1325 // on the right:
1326 //
1327 // topo-order:
1328 //
1329 // S----- ---S
1330 // | | | |
1331 // ---BB | | BB
1332 // | | | |
Hiroshi Inoue3c358f82017-06-16 12:23:04 +00001333 // | Pred-- | Succ--
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001334 // | | | |
Hiroshi Inoue3c358f82017-06-16 12:23:04 +00001335 // ---Succ ---Pred--
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001336 //
1337 // cost = freq(S->Pred) + freq(BB->Succ) cost = 2 * freq (S->Pred)
1338 // = freq(S->Pred) + freq(S->BB)
1339 //
1340 // If we have profile data (i.e, branch probabilities can be trusted), the
1341 // cost (number of taken branches) with layout S->BB->Succ->Pred is 2 *
1342 // freq(S->Pred) while the cost of topo order is freq(S->Pred) + freq(S->BB).
1343 // We know Prob(S->BB) > Prob(S->Pred), so freq(S->BB) > freq(S->Pred), which
1344 // means the cost of topological order is greater.
Xinliang David Licbf12142016-06-13 20:24:19 +00001345 // When profile data is not available, however, we need to be more
1346 // conservative. If the branch prediction is wrong, breaking the topo-order
1347 // will actually yield a layout with large cost. For this reason, we need
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001348 // strong biased branch at block S with Prob(S->BB) in order to select
1349 // BB->Succ. This is equivalent to looking the CFG backward with backward
Xinliang David Licbf12142016-06-13 20:24:19 +00001350 // edge: Prob(Succ->BB) needs to >= HotProb in order to be selected (without
1351 // profile data).
Kyle Butt02d8d052016-07-29 18:09:28 +00001352 // --------------------------------------------------------------------------
1353 // Case 3: forked diamond
1354 // S
1355 // / \
1356 // / \
1357 // BB Pred
1358 // | \ / |
1359 // | \ / |
1360 // | X |
1361 // | / \ |
1362 // | / \ |
1363 // S1 S2
1364 //
1365 // The current block is BB and edge BB->S1 is now being evaluated.
1366 // As above S->BB was already selected because
1367 // prob(S->BB) > prob(S->Pred). Assume that prob(BB->S1) >= prob(BB->S2).
1368 //
1369 // topo-order:
1370 //
1371 // S-------| ---S
1372 // | | | |
1373 // ---BB | | BB
1374 // | | | |
1375 // | Pred----| | S1----
1376 // | | | |
1377 // --(S1 or S2) ---Pred--
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001378 // |
1379 // S2
Kyle Butt02d8d052016-07-29 18:09:28 +00001380 //
1381 // topo-cost = freq(S->Pred) + freq(BB->S1) + freq(BB->S2)
1382 // + min(freq(Pred->S1), freq(Pred->S2))
1383 // Non-topo-order cost:
Kyle Butt02d8d052016-07-29 18:09:28 +00001384 // non-topo-cost = 2 * freq(S->Pred) + freq(BB->S2).
1385 // To be conservative, we can assume that min(freq(Pred->S1), freq(Pred->S2))
1386 // is 0. Then the non topo layout is better when
1387 // freq(S->Pred) < freq(BB->S1).
1388 // This is exactly what is checked below.
1389 // Note there are other shapes that apply (Pred may not be a single block,
1390 // but they all fit this general pattern.)
Dehao Chen9f2bdfb2016-06-14 22:27:17 +00001391 BranchProbability HotProb = getLayoutSuccessorProbThreshold(BB);
Xinliang David Licbf12142016-06-13 20:24:19 +00001392
Xinliang David Licbf12142016-06-13 20:24:19 +00001393 // Make sure that a hot successor doesn't have a globally more
1394 // important predecessor.
1395 BlockFrequency CandidateEdgeFreq = MBFI->getBlockFreq(BB) * RealSuccProb;
1396 bool BadCFGConflict = false;
1397
1398 for (MachineBasicBlock *Pred : Succ->predecessors()) {
1399 if (Pred == Succ || BlockToChain[Pred] == &SuccChain ||
1400 (BlockFilter && !BlockFilter->count(Pred)) ||
Kyle Buttb15c0662017-01-31 23:48:32 +00001401 BlockToChain[Pred] == &Chain ||
1402 // This check is redundant except for look ahead. This function is
1403 // called for lookahead by isProfitableToTailDup when BB hasn't been
1404 // placed yet.
1405 (Pred == BB))
Xinliang David Licbf12142016-06-13 20:24:19 +00001406 continue;
Kyle Butt02d8d052016-07-29 18:09:28 +00001407 // Do backward checking.
1408 // For all cases above, we need a backward checking to filter out edges that
Kyle Buttb15c0662017-01-31 23:48:32 +00001409 // are not 'strongly' biased.
Xinliang David Licbf12142016-06-13 20:24:19 +00001410 // BB Pred
1411 // \ /
1412 // Succ
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001413 // We select edge BB->Succ if
Xinliang David Licbf12142016-06-13 20:24:19 +00001414 // freq(BB->Succ) > freq(Succ) * HotProb
1415 // i.e. freq(BB->Succ) > freq(BB->Succ) * HotProb + freq(Pred->Succ) *
1416 // HotProb
1417 // i.e. freq((BB->Succ) * (1 - HotProb) > freq(Pred->Succ) * HotProb
Kyle Butt02d8d052016-07-29 18:09:28 +00001418 // Case 1 is covered too, because the first equation reduces to:
1419 // prob(BB->Succ) > HotProb. (freq(Succ) = freq(BB) for a triangle)
Xinliang David Licbf12142016-06-13 20:24:19 +00001420 BlockFrequency PredEdgeFreq =
1421 MBFI->getBlockFreq(Pred) * MBPI->getEdgeProbability(Pred, Succ);
1422 if (PredEdgeFreq * HotProb >= CandidateEdgeFreq * HotProb.getCompl()) {
1423 BadCFGConflict = true;
1424 break;
1425 }
1426 }
1427
1428 if (BadCFGConflict) {
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001429 DEBUG(dbgs() << " Not a candidate: " << getBlockName(Succ) << " -> " << SuccProb
Xinliang David Licbf12142016-06-13 20:24:19 +00001430 << " (prob) (non-cold CFG conflict)\n");
1431 return true;
1432 }
1433
1434 return false;
1435}
1436
Xinliang David Li594ffa32016-06-11 18:35:40 +00001437/// \brief Select the best successor for a block.
1438///
1439/// This looks across all successors of a particular block and attempts to
1440/// select the "best" one to be the layout successor. It only considers direct
1441/// successors which also pass the block filter. It will attempt to avoid
1442/// breaking CFG structure, but cave and break such structures in the case of
1443/// very hot successor edges.
1444///
Kyle Buttb15c0662017-01-31 23:48:32 +00001445/// \returns The best successor block found, or null if none are viable, along
1446/// with a boolean indicating if tail duplication is necessary.
1447MachineBlockPlacement::BlockAndTailDupResult
Kyle Butte9425c4f2017-02-04 02:26:32 +00001448MachineBlockPlacement::selectBestSuccessor(
1449 const MachineBasicBlock *BB, const BlockChain &Chain,
1450 const BlockFilterSet *BlockFilter) {
Xinliang David Li594ffa32016-06-11 18:35:40 +00001451 const BranchProbability HotProb(StaticLikelyProb, 100);
1452
Kyle Buttb15c0662017-01-31 23:48:32 +00001453 BlockAndTailDupResult BestSucc = { nullptr, false };
Xinliang David Li594ffa32016-06-11 18:35:40 +00001454 auto BestProb = BranchProbability::getZero();
1455
1456 SmallVector<MachineBasicBlock *, 4> Successors;
1457 auto AdjustedSumProb =
1458 collectViableSuccessors(BB, Chain, BlockFilter, Successors);
1459
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001460 DEBUG(dbgs() << "Selecting best successor for: " << getBlockName(BB) << "\n");
Kyle Buttb15c0662017-01-31 23:48:32 +00001461
Kyle Buttebe6cc42017-02-23 21:22:24 +00001462 // if we already precomputed the best successor for BB, return that if still
1463 // applicable.
1464 auto FoundEdge = ComputedEdges.find(BB);
1465 if (FoundEdge != ComputedEdges.end()) {
1466 MachineBasicBlock *Succ = FoundEdge->second.BB;
1467 ComputedEdges.erase(FoundEdge);
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001468 BlockChain *SuccChain = BlockToChain[Succ];
1469 if (BB->isSuccessor(Succ) && (!BlockFilter || BlockFilter->count(Succ)) &&
Kyle Buttebe6cc42017-02-23 21:22:24 +00001470 SuccChain != &Chain && Succ == *SuccChain->begin())
1471 return FoundEdge->second;
Kyle Butt7fbec9b2017-02-15 19:49:14 +00001472 }
1473
1474 // if BB is part of a trellis, Use the trellis to determine the optimal
1475 // fallthrough edges
1476 if (isTrellis(BB, Successors, Chain, BlockFilter))
1477 return getBestTrellisSuccessor(BB, Successors, AdjustedSumProb, Chain,
1478 BlockFilter);
1479
Kyle Buttb15c0662017-01-31 23:48:32 +00001480 // For blocks with CFG violations, we may be able to lay them out anyway with
1481 // tail-duplication. We keep this vector so we can perform the probability
1482 // calculations the minimum number of times.
1483 SmallVector<std::tuple<BranchProbability, MachineBasicBlock *>, 4>
1484 DupCandidates;
Cong Hou41cf1a52015-11-18 00:52:52 +00001485 for (MachineBasicBlock *Succ : Successors) {
Xinliang David Li594ffa32016-06-11 18:35:40 +00001486 auto RealSuccProb = MBPI->getEdgeProbability(BB, Succ);
1487 BranchProbability SuccProb =
1488 getAdjustedProbability(RealSuccProb, AdjustedSumProb);
Chandler Carruthb3361722011-11-13 11:34:53 +00001489
Cong Hou41cf1a52015-11-18 00:52:52 +00001490 BlockChain &SuccChain = *BlockToChain[Succ];
Xinliang David Licbf12142016-06-13 20:24:19 +00001491 // Skip the edge \c BB->Succ if block \c Succ has a better layout
1492 // predecessor that yields lower global cost.
1493 if (hasBetterLayoutPredecessor(BB, Succ, SuccChain, SuccProb, RealSuccProb,
Kyle Buttb15c0662017-01-31 23:48:32 +00001494 Chain, BlockFilter)) {
1495 // If tail duplication would make Succ profitable, place it.
1496 if (TailDupPlacement && shouldTailDuplicate(Succ))
1497 DupCandidates.push_back(std::make_tuple(SuccProb, Succ));
Xinliang David Licbf12142016-06-13 20:24:19 +00001498 continue;
Kyle Buttb15c0662017-01-31 23:48:32 +00001499 }
Chandler Carruth18dfac32011-11-20 11:22:06 +00001500
Xinliang David Licbf12142016-06-13 20:24:19 +00001501 DEBUG(
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001502 dbgs() << " Candidate: " << getBlockName(Succ) << ", probability: "
1503 << SuccProb
Xinliang David Licbf12142016-06-13 20:24:19 +00001504 << (SuccChain.UnscheduledPredecessors != 0 ? " (CFG break)" : "")
1505 << "\n");
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001506
Kyle Buttb15c0662017-01-31 23:48:32 +00001507 if (BestSucc.BB && BestProb >= SuccProb) {
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001508 DEBUG(dbgs() << " Not the best candidate, continuing\n");
Chandler Carruthb3361722011-11-13 11:34:53 +00001509 continue;
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001510 }
1511
1512 DEBUG(dbgs() << " Setting it as best candidate\n");
Kyle Buttb15c0662017-01-31 23:48:32 +00001513 BestSucc.BB = Succ;
Cong Houd97c1002015-12-01 05:29:22 +00001514 BestProb = SuccProb;
Chandler Carruthb3361722011-11-13 11:34:53 +00001515 }
Kyle Buttb15c0662017-01-31 23:48:32 +00001516 // Handle the tail duplication candidates in order of decreasing probability.
1517 // Stop at the first one that is profitable. Also stop if they are less
1518 // profitable than BestSucc. Position is important because we preserve it and
1519 // prefer first best match. Here we aren't comparing in order, so we capture
1520 // the position instead.
1521 if (DupCandidates.size() != 0) {
1522 auto cmp =
1523 [](const std::tuple<BranchProbability, MachineBasicBlock *> &a,
1524 const std::tuple<BranchProbability, MachineBasicBlock *> &b) {
1525 return std::get<0>(a) > std::get<0>(b);
1526 };
1527 std::stable_sort(DupCandidates.begin(), DupCandidates.end(), cmp);
1528 }
1529 for(auto &Tup : DupCandidates) {
1530 BranchProbability DupProb;
1531 MachineBasicBlock *Succ;
1532 std::tie(DupProb, Succ) = Tup;
1533 if (DupProb < BestProb)
1534 break;
1535 if (canTailDuplicateUnplacedPreds(BB, Succ, Chain, BlockFilter)
Kyle Butt7e8be282017-04-10 22:28:22 +00001536 && (isProfitableToTailDup(BB, Succ, BestProb, Chain, BlockFilter))) {
Kyle Buttb15c0662017-01-31 23:48:32 +00001537 DEBUG(
1538 dbgs() << " Candidate: " << getBlockName(Succ) << ", probability: "
1539 << DupProb
1540 << " (Tail Duplicate)\n");
1541 BestSucc.BB = Succ;
1542 BestSucc.ShouldTailDup = true;
1543 break;
1544 }
1545 }
1546
1547 if (BestSucc.BB)
1548 DEBUG(dbgs() << " Selected: " << getBlockName(BestSucc.BB) << "\n");
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001549
Chandler Carruthb3361722011-11-13 11:34:53 +00001550 return BestSucc;
1551}
1552
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001553/// \brief Select the best block from a worklist.
1554///
1555/// This looks through the provided worklist as a list of candidate basic
1556/// blocks and select the most profitable one to place. The definition of
1557/// profitable only really makes sense in the context of a loop. This returns
1558/// the most frequently visited block in the worklist, which in the case of
1559/// a loop, is the one most desirable to be physically close to the rest of the
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001560/// loop body in order to improve i-cache behavior.
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001561///
1562/// \returns The best block found, or null if none are viable.
1563MachineBasicBlock *MachineBlockPlacement::selectBestCandidateBlock(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001564 const BlockChain &Chain, SmallVectorImpl<MachineBasicBlock *> &WorkList) {
Chandler Carruth0af6a0b2011-11-14 09:46:33 +00001565 // Once we need to walk the worklist looking for a candidate, cleanup the
1566 // worklist of already placed entries.
1567 // FIXME: If this shows up on profiles, it could be folded (at the cost of
1568 // some code complexity) into the loop below.
Eugene Zelenko5df3d892017-08-24 21:21:39 +00001569 WorkList.erase(llvm::remove_if(WorkList,
1570 [&](MachineBasicBlock *BB) {
1571 return BlockToChain.lookup(BB) == &Chain;
1572 }),
Chandler Carruth0af6a0b2011-11-14 09:46:33 +00001573 WorkList.end());
1574
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001575 if (WorkList.empty())
1576 return nullptr;
1577
1578 bool IsEHPad = WorkList[0]->isEHPad();
1579
Craig Topperc0196b12014-04-14 00:51:57 +00001580 MachineBasicBlock *BestBlock = nullptr;
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001581 BlockFrequency BestFreq;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001582 for (MachineBasicBlock *MBB : WorkList) {
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00001583 assert(MBB->isEHPad() == IsEHPad &&
1584 "EHPad mismatch between block and work list.");
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001585
Chandler Carruth7a715da2015-03-05 03:19:05 +00001586 BlockChain &SuccChain = *BlockToChain[MBB];
Philip Reames02e11322016-03-02 22:40:51 +00001587 if (&SuccChain == &Chain)
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001588 continue;
Junmo Park4ba6cf62016-03-11 05:07:07 +00001589
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00001590 assert(SuccChain.UnscheduledPredecessors == 0 &&
1591 "Found CFG-violating block");
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001592
Chandler Carruth7a715da2015-03-05 03:19:05 +00001593 BlockFrequency CandidateFreq = MBFI->getBlockFreq(MBB);
1594 DEBUG(dbgs() << " " << getBlockName(MBB) << " -> ";
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00001595 MBFI->printBlockFreq(dbgs(), CandidateFreq) << " (freq)\n");
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001596
1597 // For ehpad, we layout the least probable first as to avoid jumping back
1598 // from least probable landingpads to more probable ones.
1599 //
1600 // FIXME: Using probability is probably (!) not the best way to achieve
1601 // this. We should probably have a more principled approach to layout
1602 // cleanup code.
1603 //
1604 // The goal is to get:
1605 //
1606 // +--------------------------+
1607 // | V
1608 // InnerLp -> InnerCleanup OuterLp -> OuterCleanup -> Resume
1609 //
1610 // Rather than:
1611 //
1612 // +-------------------------------------+
1613 // V |
1614 // OuterLp -> OuterCleanup -> Resume InnerLp -> InnerCleanup
1615 if (BestBlock && (IsEHPad ^ (BestFreq >= CandidateFreq)))
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001616 continue;
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001617
Chandler Carruth7a715da2015-03-05 03:19:05 +00001618 BestBlock = MBB;
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001619 BestFreq = CandidateFreq;
1620 }
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001621
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001622 return BestBlock;
1623}
1624
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001625/// \brief Retrieve the first unplaced basic block.
1626///
1627/// This routine is called when we are unable to use the CFG to walk through
1628/// all of the basic blocks and form a chain due to unnatural loops in the CFG.
Chandler Carruth9b548a7f2011-11-15 06:26:43 +00001629/// We walk through the function's blocks in order, starting from the
1630/// LastUnplacedBlockIt. We update this iterator on each call to avoid
1631/// re-scanning the entire sequence on repeated calls to this routine.
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001632MachineBasicBlock *MachineBlockPlacement::getFirstUnplacedBlock(
Xinliang David Li52530a72016-06-13 22:23:44 +00001633 const BlockChain &PlacedChain,
Chandler Carruth9b548a7f2011-11-15 06:26:43 +00001634 MachineFunction::iterator &PrevUnplacedBlockIt,
Jakub Staszak90616162011-12-21 23:02:08 +00001635 const BlockFilterSet *BlockFilter) {
Xinliang David Li52530a72016-06-13 22:23:44 +00001636 for (MachineFunction::iterator I = PrevUnplacedBlockIt, E = F->end(); I != E;
Chandler Carruth9b548a7f2011-11-15 06:26:43 +00001637 ++I) {
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00001638 if (BlockFilter && !BlockFilter->count(&*I))
Chandler Carruth9b548a7f2011-11-15 06:26:43 +00001639 continue;
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00001640 if (BlockToChain[&*I] != &PlacedChain) {
Chandler Carruth9b548a7f2011-11-15 06:26:43 +00001641 PrevUnplacedBlockIt = I;
Chandler Carruth4a87aa02011-11-23 03:03:21 +00001642 // Now select the head of the chain to which the unplaced block belongs
1643 // as the block to place. This will force the entire chain to be placed,
1644 // and satisfies the requirements of merging chains.
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00001645 return *BlockToChain[&*I]->begin();
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001646 }
1647 }
Craig Topperc0196b12014-04-14 00:51:57 +00001648 return nullptr;
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001649}
1650
Amaury Secheteae09c22016-03-14 21:24:11 +00001651void MachineBlockPlacement::fillWorkLists(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001652 const MachineBasicBlock *MBB,
Amaury Secheteae09c22016-03-14 21:24:11 +00001653 SmallPtrSetImpl<BlockChain *> &UpdatedPreds,
Amaury Secheteae09c22016-03-14 21:24:11 +00001654 const BlockFilterSet *BlockFilter = nullptr) {
1655 BlockChain &Chain = *BlockToChain[MBB];
1656 if (!UpdatedPreds.insert(&Chain).second)
1657 return;
1658
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00001659 assert(
1660 Chain.UnscheduledPredecessors == 0 &&
1661 "Attempting to place block with unscheduled predecessors in worklist.");
Amaury Secheteae09c22016-03-14 21:24:11 +00001662 for (MachineBasicBlock *ChainBB : Chain) {
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00001663 assert(BlockToChain[ChainBB] == &Chain &&
1664 "Block in chain doesn't match BlockToChain map.");
Amaury Secheteae09c22016-03-14 21:24:11 +00001665 for (MachineBasicBlock *Pred : ChainBB->predecessors()) {
1666 if (BlockFilter && !BlockFilter->count(Pred))
1667 continue;
1668 if (BlockToChain[Pred] == &Chain)
1669 continue;
1670 ++Chain.UnscheduledPredecessors;
1671 }
1672 }
1673
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001674 if (Chain.UnscheduledPredecessors != 0)
1675 return;
1676
Kyle Butte9425c4f2017-02-04 02:26:32 +00001677 MachineBasicBlock *BB = *Chain.begin();
1678 if (BB->isEHPad())
1679 EHPadWorkList.push_back(BB);
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001680 else
Kyle Butte9425c4f2017-02-04 02:26:32 +00001681 BlockWorkList.push_back(BB);
Amaury Secheteae09c22016-03-14 21:24:11 +00001682}
1683
Chandler Carruth8d150782011-11-13 11:20:44 +00001684void MachineBlockPlacement::buildChain(
Kyle Butte9425c4f2017-02-04 02:26:32 +00001685 const MachineBasicBlock *HeadBB, BlockChain &Chain,
Kyle Butt0846e562016-10-11 20:36:43 +00001686 BlockFilterSet *BlockFilter) {
Kyle Butte9425c4f2017-02-04 02:26:32 +00001687 assert(HeadBB && "BB must not be null.\n");
1688 assert(BlockToChain[HeadBB] == &Chain && "BlockToChainMap mis-match.\n");
Xinliang David Li52530a72016-06-13 22:23:44 +00001689 MachineFunction::iterator PrevUnplacedBlockIt = F->begin();
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001690
Kyle Butte9425c4f2017-02-04 02:26:32 +00001691 const MachineBasicBlock *LoopHeaderBB = HeadBB;
Xinliang David Li93926ac2016-07-01 05:46:48 +00001692 markChainSuccessors(Chain, LoopHeaderBB, BlockFilter);
Kyle Butte9425c4f2017-02-04 02:26:32 +00001693 MachineBasicBlock *BB = *std::prev(Chain.end());
Eugene Zelenko5df3d892017-08-24 21:21:39 +00001694 while (true) {
Kyle Butt82c22902016-06-28 22:50:54 +00001695 assert(BB && "null block found at end of chain in loop.");
1696 assert(BlockToChain[BB] == &Chain && "BlockToChainMap mis-match in loop.");
1697 assert(*std::prev(Chain.end()) == BB && "BB Not found at end of chain.");
1698
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00001699
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00001700 // Look for the best viable successor if there is one to place immediately
1701 // after this block.
Kyle Buttb15c0662017-01-31 23:48:32 +00001702 auto Result = selectBestSuccessor(BB, Chain, BlockFilter);
1703 MachineBasicBlock* BestSucc = Result.BB;
1704 bool ShouldTailDup = Result.ShouldTailDup;
1705 if (TailDupPlacement)
1706 ShouldTailDup |= (BestSucc && shouldTailDuplicate(BestSucc));
Chandler Carruth8d150782011-11-13 11:20:44 +00001707
1708 // If an immediate successor isn't available, look for the best viable
1709 // block among those we've identified as not violating the loop's CFG at
1710 // this point. This won't be a fallthrough, but it will increase locality.
Chandler Carruthf9213fe2011-11-13 11:42:26 +00001711 if (!BestSucc)
Amaury Sechet9ee4ddd2016-04-07 06:34:47 +00001712 BestSucc = selectBestCandidateBlock(Chain, BlockWorkList);
Amaury Sechetc53ad4f2016-04-07 21:29:39 +00001713 if (!BestSucc)
1714 BestSucc = selectBestCandidateBlock(Chain, EHPadWorkList);
Chandler Carruth8d150782011-11-13 11:20:44 +00001715
Chandler Carruth8d150782011-11-13 11:20:44 +00001716 if (!BestSucc) {
Xinliang David Li52530a72016-06-13 22:23:44 +00001717 BestSucc = getFirstUnplacedBlock(Chain, PrevUnplacedBlockIt, BlockFilter);
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001718 if (!BestSucc)
1719 break;
1720
1721 DEBUG(dbgs() << "Unnatural loop CFG detected, forcibly merging the "
1722 "layout successor until the CFG reduces\n");
Chandler Carruth8d150782011-11-13 11:20:44 +00001723 }
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00001724
Kyle Butt0846e562016-10-11 20:36:43 +00001725 // Placement may have changed tail duplication opportunities.
1726 // Check for that now.
Kyle Buttb15c0662017-01-31 23:48:32 +00001727 if (TailDupPlacement && BestSucc && ShouldTailDup) {
Kyle Butt0846e562016-10-11 20:36:43 +00001728 // If the chosen successor was duplicated into all its predecessors,
1729 // don't bother laying it out, just go round the loop again with BB as
1730 // the chain end.
1731 if (repeatedlyTailDuplicateBlock(BestSucc, BB, LoopHeaderBB, Chain,
1732 BlockFilter, PrevUnplacedBlockIt))
1733 continue;
1734 }
1735
Chandler Carruth8d150782011-11-13 11:20:44 +00001736 // Place this block, updating the datastructures to reflect its placement.
Jakub Staszak90616162011-12-21 23:02:08 +00001737 BlockChain &SuccChain = *BlockToChain[BestSucc];
Philip Reamesae27b232016-03-03 00:58:43 +00001738 // Zero out UnscheduledPredecessors for the successor we're about to merge in case
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001739 // we selected a successor that didn't fit naturally into the CFG.
Philip Reamesae27b232016-03-03 00:58:43 +00001740 SuccChain.UnscheduledPredecessors = 0;
Philip Reamesb9688f42016-03-02 21:45:13 +00001741 DEBUG(dbgs() << "Merging from " << getBlockName(BB) << " to "
1742 << getBlockName(BestSucc) << "\n");
Xinliang David Li93926ac2016-07-01 05:46:48 +00001743 markChainSuccessors(SuccChain, LoopHeaderBB, BlockFilter);
Chandler Carruth8d150782011-11-13 11:20:44 +00001744 Chain.merge(BestSucc, &SuccChain);
Benjamin Kramerb6d0bd42014-03-02 12:27:27 +00001745 BB = *std::prev(Chain.end());
Jakub Staszak190c7122011-12-07 19:46:10 +00001746 }
Chandler Carruth1071cfa2011-11-14 00:00:35 +00001747
1748 DEBUG(dbgs() << "Finished forming chain for header block "
Philip Reamesb9688f42016-03-02 21:45:13 +00001749 << getBlockName(*Chain.begin()) << "\n");
Chandler Carruth10281422011-10-21 06:46:38 +00001750}
1751
Chandler Carruth03adbd42011-11-27 13:34:33 +00001752/// \brief Find the best loop top block for layout.
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001753///
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001754/// Look for a block which is strictly better than the loop header for laying
1755/// out at the top of the loop. This looks for one and only one pattern:
1756/// a latch block with no conditional exit. This block will cause a conditional
1757/// jump around it or will be the bottom of the loop if we lay it out in place,
1758/// but if it it doesn't end up at the bottom of the loop for any reason,
1759/// rotation alone won't fix it. Because such a block will always result in an
1760/// unconditional jump (for the backedge) rotating it in front of the loop
1761/// header is always profitable.
1762MachineBasicBlock *
Kyle Butte9425c4f2017-02-04 02:26:32 +00001763MachineBlockPlacement::findBestLoopTop(const MachineLoop &L,
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001764 const BlockFilterSet &LoopBlockSet) {
Sjoerd Meijer15c81b02016-08-16 19:50:33 +00001765 // Placing the latch block before the header may introduce an extra branch
1766 // that skips this block the first time the loop is executed, which we want
1767 // to avoid when optimising for size.
1768 // FIXME: in theory there is a case that does not introduce a new branch,
1769 // i.e. when the layout predecessor does not fallthrough to the loop header.
1770 // In practice this never happens though: there always seems to be a preheader
1771 // that can fallthrough and that is also placed before the header.
1772 if (F->getFunction()->optForSize())
1773 return L.getHeader();
1774
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001775 // Check that the header hasn't been fused with a preheader block due to
1776 // crazy branches. If it has, we need to start with the header at the top to
1777 // prevent pulling the preheader into the loop body.
1778 BlockChain &HeaderChain = *BlockToChain[L.getHeader()];
1779 if (!LoopBlockSet.count(*HeaderChain.begin()))
1780 return L.getHeader();
1781
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00001782 DEBUG(dbgs() << "Finding best loop top for: " << getBlockName(L.getHeader())
1783 << "\n");
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001784
1785 BlockFrequency BestPredFreq;
Craig Topperc0196b12014-04-14 00:51:57 +00001786 MachineBasicBlock *BestPred = nullptr;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001787 for (MachineBasicBlock *Pred : L.getHeader()->predecessors()) {
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001788 if (!LoopBlockSet.count(Pred))
1789 continue;
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001790 DEBUG(dbgs() << " header pred: " << getBlockName(Pred) << ", has "
Michael Gottesmanb78dec82013-12-14 00:25:45 +00001791 << Pred->succ_size() << " successors, ";
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00001792 MBFI->printBlockFreq(dbgs(), Pred) << " freq\n");
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001793 if (Pred->succ_size() > 1)
1794 continue;
1795
1796 BlockFrequency PredFreq = MBFI->getBlockFreq(Pred);
1797 if (!BestPred || PredFreq > BestPredFreq ||
1798 (!(PredFreq < BestPredFreq) &&
1799 Pred->isLayoutSuccessor(L.getHeader()))) {
1800 BestPred = Pred;
1801 BestPredFreq = PredFreq;
1802 }
1803 }
1804
1805 // If no direct predecessor is fine, just use the loop header.
Philip Reamesb9688f42016-03-02 21:45:13 +00001806 if (!BestPred) {
1807 DEBUG(dbgs() << " final top unchanged\n");
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001808 return L.getHeader();
Philip Reamesb9688f42016-03-02 21:45:13 +00001809 }
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001810
1811 // Walk backwards through any straight line of predecessors.
1812 while (BestPred->pred_size() == 1 &&
1813 (*BestPred->pred_begin())->succ_size() == 1 &&
1814 *BestPred->pred_begin() != L.getHeader())
1815 BestPred = *BestPred->pred_begin();
1816
1817 DEBUG(dbgs() << " final top: " << getBlockName(BestPred) << "\n");
1818 return BestPred;
1819}
1820
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00001821/// \brief Find the best loop exiting block for layout.
1822///
Chandler Carruth03adbd42011-11-27 13:34:33 +00001823/// This routine implements the logic to analyze the loop looking for the best
1824/// block to layout at the top of the loop. Typically this is done to maximize
1825/// fallthrough opportunities.
1826MachineBasicBlock *
Kyle Butte9425c4f2017-02-04 02:26:32 +00001827MachineBlockPlacement::findBestLoopExit(const MachineLoop &L,
Chandler Carruthccc7e422012-04-16 01:12:56 +00001828 const BlockFilterSet &LoopBlockSet) {
Chandler Carruth68062612012-04-10 13:35:57 +00001829 // We don't want to layout the loop linearly in all cases. If the loop header
1830 // is just a normal basic block in the loop, we want to look for what block
1831 // within the loop is the best one to layout at the top. However, if the loop
1832 // header has be pre-merged into a chain due to predecessors not having
1833 // analyzable branches, *and* the predecessor it is merged with is *not* part
1834 // of the loop, rotating the header into the middle of the loop will create
1835 // a non-contiguous range of blocks which is Very Bad. So start with the
1836 // header and only rotate if safe.
1837 BlockChain &HeaderChain = *BlockToChain[L.getHeader()];
1838 if (!LoopBlockSet.count(*HeaderChain.begin()))
Craig Topperc0196b12014-04-14 00:51:57 +00001839 return nullptr;
Chandler Carruth68062612012-04-10 13:35:57 +00001840
Chandler Carruth03adbd42011-11-27 13:34:33 +00001841 BlockFrequency BestExitEdgeFreq;
Chandler Carruthccc7e422012-04-16 01:12:56 +00001842 unsigned BestExitLoopDepth = 0;
Craig Topperc0196b12014-04-14 00:51:57 +00001843 MachineBasicBlock *ExitingBB = nullptr;
Chandler Carruth4f567202011-11-27 20:18:00 +00001844 // If there are exits to outer loops, loop rotation can severely limit
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00001845 // fallthrough opportunities unless it selects such an exit. Keep a set of
Chandler Carruth4f567202011-11-27 20:18:00 +00001846 // blocks where rotating to exit with that block will reach an outer loop.
1847 SmallPtrSet<MachineBasicBlock *, 4> BlocksExitingToOuterLoop;
1848
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00001849 DEBUG(dbgs() << "Finding best loop exit for: " << getBlockName(L.getHeader())
1850 << "\n");
Chandler Carruth7a715da2015-03-05 03:19:05 +00001851 for (MachineBasicBlock *MBB : L.getBlocks()) {
1852 BlockChain &Chain = *BlockToChain[MBB];
Chandler Carruth03adbd42011-11-27 13:34:33 +00001853 // Ensure that this block is at the end of a chain; otherwise it could be
Chandler Carruth9a512a42015-04-15 13:19:54 +00001854 // mid-way through an inner loop or a successor of an unanalyzable branch.
Chandler Carruth7a715da2015-03-05 03:19:05 +00001855 if (MBB != *std::prev(Chain.end()))
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001856 continue;
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001857
Chandler Carruth03adbd42011-11-27 13:34:33 +00001858 // Now walk the successors. We need to establish whether this has a viable
1859 // exiting successor and whether it has a viable non-exiting successor.
1860 // We store the old exiting state and restore it if a viable looping
1861 // successor isn't found.
1862 MachineBasicBlock *OldExitingBB = ExitingBB;
1863 BlockFrequency OldBestExitEdgeFreq = BestExitEdgeFreq;
Chandler Carruthccc7e422012-04-16 01:12:56 +00001864 bool HasLoopingSucc = false;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001865 for (MachineBasicBlock *Succ : MBB->successors()) {
Reid Kleckner0e288232015-08-27 23:27:47 +00001866 if (Succ->isEHPad())
Chandler Carruth03adbd42011-11-27 13:34:33 +00001867 continue;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001868 if (Succ == MBB)
Chandler Carruth03adbd42011-11-27 13:34:33 +00001869 continue;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001870 BlockChain &SuccChain = *BlockToChain[Succ];
Chandler Carruth03adbd42011-11-27 13:34:33 +00001871 // Don't split chains, either this chain or the successor's chain.
Chandler Carruthccc7e422012-04-16 01:12:56 +00001872 if (&Chain == &SuccChain) {
Chandler Carruth7a715da2015-03-05 03:19:05 +00001873 DEBUG(dbgs() << " exiting: " << getBlockName(MBB) << " -> "
1874 << getBlockName(Succ) << " (chain conflict)\n");
Chandler Carruth03adbd42011-11-27 13:34:33 +00001875 continue;
1876 }
1877
Cong Houd97c1002015-12-01 05:29:22 +00001878 auto SuccProb = MBPI->getEdgeProbability(MBB, Succ);
Chandler Carruth7a715da2015-03-05 03:19:05 +00001879 if (LoopBlockSet.count(Succ)) {
1880 DEBUG(dbgs() << " looping: " << getBlockName(MBB) << " -> "
Cong Houd97c1002015-12-01 05:29:22 +00001881 << getBlockName(Succ) << " (" << SuccProb << ")\n");
Chandler Carruthccc7e422012-04-16 01:12:56 +00001882 HasLoopingSucc = true;
Chandler Carruth03adbd42011-11-27 13:34:33 +00001883 continue;
1884 }
1885
Chandler Carruthccc7e422012-04-16 01:12:56 +00001886 unsigned SuccLoopDepth = 0;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001887 if (MachineLoop *ExitLoop = MLI->getLoopFor(Succ)) {
Chandler Carruthccc7e422012-04-16 01:12:56 +00001888 SuccLoopDepth = ExitLoop->getLoopDepth();
1889 if (ExitLoop->contains(&L))
Chandler Carruth7a715da2015-03-05 03:19:05 +00001890 BlocksExitingToOuterLoop.insert(MBB);
Chandler Carruthccc7e422012-04-16 01:12:56 +00001891 }
1892
Chandler Carruth7a715da2015-03-05 03:19:05 +00001893 BlockFrequency ExitEdgeFreq = MBFI->getBlockFreq(MBB) * SuccProb;
1894 DEBUG(dbgs() << " exiting: " << getBlockName(MBB) << " -> "
1895 << getBlockName(Succ) << " [L:" << SuccLoopDepth << "] (";
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00001896 MBFI->printBlockFreq(dbgs(), ExitEdgeFreq) << ")\n");
Benjamin Kramerc8160d62013-11-20 19:08:44 +00001897 // Note that we bias this toward an existing layout successor to retain
1898 // incoming order in the absence of better information. The exit must have
1899 // a frequency higher than the current exit before we consider breaking
1900 // the layout.
1901 BranchProbability Bias(100 - ExitBlockBias, 100);
Chandler Carruth26d30172015-04-15 13:39:42 +00001902 if (!ExitingBB || SuccLoopDepth > BestExitLoopDepth ||
Chandler Carruthccc7e422012-04-16 01:12:56 +00001903 ExitEdgeFreq > BestExitEdgeFreq ||
Chandler Carruth7a715da2015-03-05 03:19:05 +00001904 (MBB->isLayoutSuccessor(Succ) &&
Benjamin Kramerc8160d62013-11-20 19:08:44 +00001905 !(ExitEdgeFreq < BestExitEdgeFreq * Bias))) {
Chandler Carruth03adbd42011-11-27 13:34:33 +00001906 BestExitEdgeFreq = ExitEdgeFreq;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001907 ExitingBB = MBB;
Chandler Carrutha0545802011-11-27 09:22:53 +00001908 }
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001909 }
Chandler Carruth03adbd42011-11-27 13:34:33 +00001910
Chandler Carruthccc7e422012-04-16 01:12:56 +00001911 if (!HasLoopingSucc) {
Chandler Carruthcfb2b9d2015-04-15 13:26:41 +00001912 // Restore the old exiting state, no viable looping successor was found.
Chandler Carruth03adbd42011-11-27 13:34:33 +00001913 ExitingBB = OldExitingBB;
1914 BestExitEdgeFreq = OldBestExitEdgeFreq;
Chandler Carruth03adbd42011-11-27 13:34:33 +00001915 }
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001916 }
Chandler Carruthccc7e422012-04-16 01:12:56 +00001917 // Without a candidate exiting block or with only a single block in the
Chandler Carruth03adbd42011-11-27 13:34:33 +00001918 // loop, just use the loop header to layout the loop.
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001919 if (!ExitingBB) {
1920 DEBUG(dbgs() << " No other candidate exit blocks, using loop header\n");
Craig Topperc0196b12014-04-14 00:51:57 +00001921 return nullptr;
Sjoerd Meijer5e11a182016-07-27 08:49:23 +00001922 }
1923 if (L.getNumBlocks() == 1) {
1924 DEBUG(dbgs() << " Loop has 1 block, using loop header as exit\n");
1925 return nullptr;
1926 }
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001927
Chandler Carruth4f567202011-11-27 20:18:00 +00001928 // Also, if we have exit blocks which lead to outer loops but didn't select
1929 // one of them as the exiting block we are rotating toward, disable loop
1930 // rotation altogether.
1931 if (!BlocksExitingToOuterLoop.empty() &&
1932 !BlocksExitingToOuterLoop.count(ExitingBB))
Craig Topperc0196b12014-04-14 00:51:57 +00001933 return nullptr;
Chandler Carruth4f567202011-11-27 20:18:00 +00001934
Chandler Carruth03adbd42011-11-27 13:34:33 +00001935 DEBUG(dbgs() << " Best exiting block: " << getBlockName(ExitingBB) << "\n");
Chandler Carruthccc7e422012-04-16 01:12:56 +00001936 return ExitingBB;
Chandler Carruth9ffb97e2011-11-27 00:38:03 +00001937}
1938
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001939/// \brief Attempt to rotate an exiting block to the bottom of the loop.
1940///
1941/// Once we have built a chain, try to rotate it to line up the hot exit block
1942/// with fallthrough out of the loop if doing so doesn't introduce unnecessary
1943/// branches. For example, if the loop has fallthrough into its header and out
1944/// of its bottom already, don't rotate it.
1945void MachineBlockPlacement::rotateLoop(BlockChain &LoopChain,
Kyle Butte9425c4f2017-02-04 02:26:32 +00001946 const MachineBasicBlock *ExitingBB,
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001947 const BlockFilterSet &LoopBlockSet) {
1948 if (!ExitingBB)
1949 return;
1950
1951 MachineBasicBlock *Top = *LoopChain.begin();
Serguei Katkov0e831c92017-07-11 08:34:58 +00001952 MachineBasicBlock *Bottom = *std::prev(LoopChain.end());
1953
1954 // If ExitingBB is already the last one in a chain then nothing to do.
1955 if (Bottom == ExitingBB)
1956 return;
1957
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001958 bool ViableTopFallthrough = false;
Chandler Carruth7a715da2015-03-05 03:19:05 +00001959 for (MachineBasicBlock *Pred : Top->predecessors()) {
1960 BlockChain *PredChain = BlockToChain[Pred];
1961 if (!LoopBlockSet.count(Pred) &&
1962 (!PredChain || Pred == *std::prev(PredChain->end()))) {
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001963 ViableTopFallthrough = true;
1964 break;
1965 }
1966 }
1967
1968 // If the header has viable fallthrough, check whether the current loop
1969 // bottom is a viable exiting block. If so, bail out as rotating will
1970 // introduce an unnecessary branch.
1971 if (ViableTopFallthrough) {
Chandler Carruth7a715da2015-03-05 03:19:05 +00001972 for (MachineBasicBlock *Succ : Bottom->successors()) {
1973 BlockChain *SuccChain = BlockToChain[Succ];
1974 if (!LoopBlockSet.count(Succ) &&
1975 (!SuccChain || Succ == *SuccChain->begin()))
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001976 return;
1977 }
1978 }
1979
Eugene Zelenko5df3d892017-08-24 21:21:39 +00001980 BlockChain::iterator ExitIt = llvm::find(LoopChain, ExitingBB);
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00001981 if (ExitIt == LoopChain.end())
1982 return;
1983
Serguei Katkov0e831c92017-07-11 08:34:58 +00001984 // Rotating a loop exit to the bottom when there is a fallthrough to top
1985 // trades the entry fallthrough for an exit fallthrough.
1986 // If there is no bottom->top edge, but the chosen exit block does have
1987 // a fallthrough, we break that fallthrough for nothing in return.
1988
1989 // Let's consider an example. We have a built chain of basic blocks
1990 // B1, B2, ..., Bn, where Bk is a ExitingBB - chosen exit block.
1991 // By doing a rotation we get
1992 // Bk+1, ..., Bn, B1, ..., Bk
1993 // Break of fallthrough to B1 is compensated by a fallthrough from Bk.
1994 // If we had a fallthrough Bk -> Bk+1 it is broken now.
1995 // It might be compensated by fallthrough Bn -> B1.
1996 // So we have a condition to avoid creation of extra branch by loop rotation.
1997 // All below must be true to avoid loop rotation:
1998 // If there is a fallthrough to top (B1)
1999 // There was fallthrough from chosen exit block (Bk) to next one (Bk+1)
2000 // There is no fallthrough from bottom (Bn) to top (B1).
2001 // Please note that there is no exit fallthrough from Bn because we checked it
2002 // above.
2003 if (ViableTopFallthrough) {
2004 assert(std::next(ExitIt) != LoopChain.end() &&
2005 "Exit should not be last BB");
2006 MachineBasicBlock *NextBlockInChain = *std::next(ExitIt);
2007 if (ExitingBB->isSuccessor(NextBlockInChain))
2008 if (!Bottom->isSuccessor(Top))
2009 return;
2010 }
2011
2012 DEBUG(dbgs() << "Rotating loop to put exit " << getBlockName(ExitingBB)
2013 << " at bottom\n");
Benjamin Kramerb6d0bd42014-03-02 12:27:27 +00002014 std::rotate(LoopChain.begin(), std::next(ExitIt), LoopChain.end());
Chandler Carruth8c74c7b2012-04-16 09:31:23 +00002015}
2016
Cong Hou7745dbc2015-10-19 23:16:40 +00002017/// \brief Attempt to rotate a loop based on profile data to reduce branch cost.
2018///
2019/// With profile data, we can determine the cost in terms of missed fall through
2020/// opportunities when rotating a loop chain and select the best rotation.
2021/// Basically, there are three kinds of cost to consider for each rotation:
2022/// 1. The possibly missed fall through edge (if it exists) from BB out of
2023/// the loop to the loop header.
2024/// 2. The possibly missed fall through edges (if they exist) from the loop
2025/// exits to BB out of the loop.
2026/// 3. The missed fall through edge (if it exists) from the last BB to the
2027/// first BB in the loop chain.
2028/// Therefore, the cost for a given rotation is the sum of costs listed above.
2029/// We select the best rotation with the smallest cost.
2030void MachineBlockPlacement::rotateLoopWithProfile(
Kyle Butte9425c4f2017-02-04 02:26:32 +00002031 BlockChain &LoopChain, const MachineLoop &L,
2032 const BlockFilterSet &LoopBlockSet) {
Cong Hou7745dbc2015-10-19 23:16:40 +00002033 auto HeaderBB = L.getHeader();
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002034 auto HeaderIter = llvm::find(LoopChain, HeaderBB);
Cong Hou7745dbc2015-10-19 23:16:40 +00002035 auto RotationPos = LoopChain.end();
2036
2037 BlockFrequency SmallestRotationCost = BlockFrequency::getMaxFrequency();
2038
2039 // A utility lambda that scales up a block frequency by dividing it by a
2040 // branch probability which is the reciprocal of the scale.
2041 auto ScaleBlockFrequency = [](BlockFrequency Freq,
2042 unsigned Scale) -> BlockFrequency {
2043 if (Scale == 0)
2044 return 0;
2045 // Use operator / between BlockFrequency and BranchProbability to implement
2046 // saturating multiplication.
2047 return Freq / BranchProbability(1, Scale);
2048 };
2049
2050 // Compute the cost of the missed fall-through edge to the loop header if the
2051 // chain head is not the loop header. As we only consider natural loops with
2052 // single header, this computation can be done only once.
2053 BlockFrequency HeaderFallThroughCost(0);
2054 for (auto *Pred : HeaderBB->predecessors()) {
2055 BlockChain *PredChain = BlockToChain[Pred];
2056 if (!LoopBlockSet.count(Pred) &&
2057 (!PredChain || Pred == *std::prev(PredChain->end()))) {
2058 auto EdgeFreq =
2059 MBFI->getBlockFreq(Pred) * MBPI->getEdgeProbability(Pred, HeaderBB);
2060 auto FallThruCost = ScaleBlockFrequency(EdgeFreq, MisfetchCost);
2061 // If the predecessor has only an unconditional jump to the header, we
2062 // need to consider the cost of this jump.
2063 if (Pred->succ_size() == 1)
2064 FallThruCost += ScaleBlockFrequency(EdgeFreq, JumpInstCost);
2065 HeaderFallThroughCost = std::max(HeaderFallThroughCost, FallThruCost);
2066 }
2067 }
2068
2069 // Here we collect all exit blocks in the loop, and for each exit we find out
2070 // its hottest exit edge. For each loop rotation, we define the loop exit cost
2071 // as the sum of frequencies of exit edges we collect here, excluding the exit
2072 // edge from the tail of the loop chain.
2073 SmallVector<std::pair<MachineBasicBlock *, BlockFrequency>, 4> ExitsWithFreq;
2074 for (auto BB : LoopChain) {
Cong Houd97c1002015-12-01 05:29:22 +00002075 auto LargestExitEdgeProb = BranchProbability::getZero();
Cong Hou7745dbc2015-10-19 23:16:40 +00002076 for (auto *Succ : BB->successors()) {
2077 BlockChain *SuccChain = BlockToChain[Succ];
2078 if (!LoopBlockSet.count(Succ) &&
2079 (!SuccChain || Succ == *SuccChain->begin())) {
Cong Houd97c1002015-12-01 05:29:22 +00002080 auto SuccProb = MBPI->getEdgeProbability(BB, Succ);
2081 LargestExitEdgeProb = std::max(LargestExitEdgeProb, SuccProb);
Cong Hou7745dbc2015-10-19 23:16:40 +00002082 }
2083 }
Cong Houd97c1002015-12-01 05:29:22 +00002084 if (LargestExitEdgeProb > BranchProbability::getZero()) {
2085 auto ExitFreq = MBFI->getBlockFreq(BB) * LargestExitEdgeProb;
Cong Hou7745dbc2015-10-19 23:16:40 +00002086 ExitsWithFreq.emplace_back(BB, ExitFreq);
2087 }
2088 }
2089
2090 // In this loop we iterate every block in the loop chain and calculate the
2091 // cost assuming the block is the head of the loop chain. When the loop ends,
2092 // we should have found the best candidate as the loop chain's head.
2093 for (auto Iter = LoopChain.begin(), TailIter = std::prev(LoopChain.end()),
2094 EndIter = LoopChain.end();
2095 Iter != EndIter; Iter++, TailIter++) {
2096 // TailIter is used to track the tail of the loop chain if the block we are
2097 // checking (pointed by Iter) is the head of the chain.
2098 if (TailIter == LoopChain.end())
2099 TailIter = LoopChain.begin();
2100
2101 auto TailBB = *TailIter;
2102
2103 // Calculate the cost by putting this BB to the top.
2104 BlockFrequency Cost = 0;
2105
2106 // If the current BB is the loop header, we need to take into account the
2107 // cost of the missed fall through edge from outside of the loop to the
2108 // header.
2109 if (Iter != HeaderIter)
2110 Cost += HeaderFallThroughCost;
2111
2112 // Collect the loop exit cost by summing up frequencies of all exit edges
2113 // except the one from the chain tail.
2114 for (auto &ExitWithFreq : ExitsWithFreq)
2115 if (TailBB != ExitWithFreq.first)
2116 Cost += ExitWithFreq.second;
2117
2118 // The cost of breaking the once fall-through edge from the tail to the top
2119 // of the loop chain. Here we need to consider three cases:
2120 // 1. If the tail node has only one successor, then we will get an
2121 // additional jmp instruction. So the cost here is (MisfetchCost +
2122 // JumpInstCost) * tail node frequency.
2123 // 2. If the tail node has two successors, then we may still get an
2124 // additional jmp instruction if the layout successor after the loop
2125 // chain is not its CFG successor. Note that the more frequently executed
2126 // jmp instruction will be put ahead of the other one. Assume the
2127 // frequency of those two branches are x and y, where x is the frequency
2128 // of the edge to the chain head, then the cost will be
2129 // (x * MisfetechCost + min(x, y) * JumpInstCost) * tail node frequency.
2130 // 3. If the tail node has more than two successors (this rarely happens),
2131 // we won't consider any additional cost.
2132 if (TailBB->isSuccessor(*Iter)) {
2133 auto TailBBFreq = MBFI->getBlockFreq(TailBB);
2134 if (TailBB->succ_size() == 1)
2135 Cost += ScaleBlockFrequency(TailBBFreq.getFrequency(),
2136 MisfetchCost + JumpInstCost);
2137 else if (TailBB->succ_size() == 2) {
2138 auto TailToHeadProb = MBPI->getEdgeProbability(TailBB, *Iter);
2139 auto TailToHeadFreq = TailBBFreq * TailToHeadProb;
2140 auto ColderEdgeFreq = TailToHeadProb > BranchProbability(1, 2)
2141 ? TailBBFreq * TailToHeadProb.getCompl()
2142 : TailToHeadFreq;
2143 Cost += ScaleBlockFrequency(TailToHeadFreq, MisfetchCost) +
2144 ScaleBlockFrequency(ColderEdgeFreq, JumpInstCost);
2145 }
2146 }
2147
Philip Reamesb9688f42016-03-02 21:45:13 +00002148 DEBUG(dbgs() << "The cost of loop rotation by making " << getBlockName(*Iter)
Cong Hou7745dbc2015-10-19 23:16:40 +00002149 << " to the top: " << Cost.getFrequency() << "\n");
2150
2151 if (Cost < SmallestRotationCost) {
2152 SmallestRotationCost = Cost;
2153 RotationPos = Iter;
2154 }
2155 }
2156
2157 if (RotationPos != LoopChain.end()) {
Philip Reamesb9688f42016-03-02 21:45:13 +00002158 DEBUG(dbgs() << "Rotate loop by making " << getBlockName(*RotationPos)
Cong Hou7745dbc2015-10-19 23:16:40 +00002159 << " to the top\n");
2160 std::rotate(LoopChain.begin(), RotationPos, LoopChain.end());
2161 }
2162}
2163
Cong Houb90b9e02015-11-02 21:24:00 +00002164/// \brief Collect blocks in the given loop that are to be placed.
2165///
2166/// When profile data is available, exclude cold blocks from the returned set;
2167/// otherwise, collect all blocks in the loop.
2168MachineBlockPlacement::BlockFilterSet
Kyle Butte9425c4f2017-02-04 02:26:32 +00002169MachineBlockPlacement::collectLoopBlockSet(const MachineLoop &L) {
Cong Houb90b9e02015-11-02 21:24:00 +00002170 BlockFilterSet LoopBlockSet;
2171
2172 // Filter cold blocks off from LoopBlockSet when profile data is available.
2173 // Collect the sum of frequencies of incoming edges to the loop header from
2174 // outside. If we treat the loop as a super block, this is the frequency of
2175 // the loop. Then for each block in the loop, we calculate the ratio between
2176 // its frequency and the frequency of the loop block. When it is too small,
2177 // don't add it to the loop chain. If there are outer loops, then this block
2178 // will be merged into the first outer loop chain for which this block is not
2179 // cold anymore. This needs precise profile data and we only do this when
2180 // profile data is available.
Kyle Butt74f61dd2017-08-04 21:13:41 +00002181 if (F->getFunction()->getEntryCount() || ForceLoopColdBlock) {
Cong Houb90b9e02015-11-02 21:24:00 +00002182 BlockFrequency LoopFreq(0);
2183 for (auto LoopPred : L.getHeader()->predecessors())
2184 if (!L.contains(LoopPred))
2185 LoopFreq += MBFI->getBlockFreq(LoopPred) *
2186 MBPI->getEdgeProbability(LoopPred, L.getHeader());
2187
2188 for (MachineBasicBlock *LoopBB : L.getBlocks()) {
2189 auto Freq = MBFI->getBlockFreq(LoopBB).getFrequency();
2190 if (Freq == 0 || LoopFreq.getFrequency() / Freq > LoopToColdBlockRatio)
2191 continue;
2192 LoopBlockSet.insert(LoopBB);
2193 }
2194 } else
2195 LoopBlockSet.insert(L.block_begin(), L.block_end());
2196
2197 return LoopBlockSet;
2198}
2199
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00002200/// \brief Forms basic block chains from the natural loop structures.
Chandler Carruth10281422011-10-21 06:46:38 +00002201///
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00002202/// These chains are designed to preserve the existing *structure* of the code
2203/// as much as possible. We can then stitch the chains together in a way which
2204/// both preserves the topological structure and minimizes taken conditional
2205/// branches.
Kyle Butte9425c4f2017-02-04 02:26:32 +00002206void MachineBlockPlacement::buildLoopChains(const MachineLoop &L) {
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00002207 // First recurse through any nested loops, building chains for those inner
2208 // loops.
Kyle Butte9425c4f2017-02-04 02:26:32 +00002209 for (const MachineLoop *InnerLoop : L)
Xinliang David Li52530a72016-06-13 22:23:44 +00002210 buildLoopChains(*InnerLoop);
Chandler Carruth10281422011-10-21 06:46:38 +00002211
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00002212 assert(BlockWorkList.empty() &&
2213 "BlockWorkList not empty when starting to build loop chains.");
2214 assert(EHPadWorkList.empty() &&
2215 "EHPadWorkList not empty when starting to build loop chains.");
Xinliang David Li52530a72016-06-13 22:23:44 +00002216 BlockFilterSet LoopBlockSet = collectLoopBlockSet(L);
Chandler Carruth03adbd42011-11-27 13:34:33 +00002217
Cong Hou7745dbc2015-10-19 23:16:40 +00002218 // Check if we have profile data for this function. If yes, we will rotate
2219 // this loop by modeling costs more precisely which requires the profile data
2220 // for better layout.
2221 bool RotateLoopWithProfile =
Xinliang David Lif0ab6df2016-05-12 02:04:41 +00002222 ForcePreciseRotationCost ||
Xinliang David Li52530a72016-06-13 22:23:44 +00002223 (PreciseRotationCost && F->getFunction()->getEntryCount());
Cong Hou7745dbc2015-10-19 23:16:40 +00002224
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00002225 // First check to see if there is an obviously preferable top block for the
2226 // loop. This will default to the header, but may end up as one of the
2227 // predecessors to the header if there is one which will result in strictly
2228 // fewer branches in the loop body.
Cong Hou7745dbc2015-10-19 23:16:40 +00002229 // When we use profile data to rotate the loop, this is unnecessary.
2230 MachineBasicBlock *LoopTop =
2231 RotateLoopWithProfile ? L.getHeader() : findBestLoopTop(L, LoopBlockSet);
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00002232
2233 // If we selected just the header for the loop top, look for a potentially
2234 // profitable exit block in the event that rotating the loop can eliminate
2235 // branches by placing an exit edge at the bottom.
Xin Tongd8d97972017-10-04 21:39:25 +00002236 //
2237 // Loops are processed innermost to uttermost, make sure we clear
2238 // PreferredLoopExit before processing a new loop.
2239 PreferredLoopExit = nullptr;
Cong Hou7745dbc2015-10-19 23:16:40 +00002240 if (!RotateLoopWithProfile && LoopTop == L.getHeader())
Kyle Buttab9cca72016-10-27 21:37:20 +00002241 PreferredLoopExit = findBestLoopExit(L, LoopBlockSet);
Chandler Carruth8c0b41d2012-04-16 13:33:36 +00002242
2243 BlockChain &LoopChain = *BlockToChain[LoopTop];
Chandler Carruth10281422011-10-21 06:46:38 +00002244
Chandler Carruth8d150782011-11-13 11:20:44 +00002245 // FIXME: This is a really lame way of walking the chains in the loop: we
2246 // walk the blocks, and use a set to prevent visiting a particular chain
2247 // twice.
Jakub Staszak90616162011-12-21 23:02:08 +00002248 SmallPtrSet<BlockChain *, 4> UpdatedPreds;
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00002249 assert(LoopChain.UnscheduledPredecessors == 0 &&
2250 "LoopChain should not have unscheduled predecessors.");
Jakub Staszak190c7122011-12-07 19:46:10 +00002251 UpdatedPreds.insert(&LoopChain);
Cong Houb90b9e02015-11-02 21:24:00 +00002252
Kyle Butte9425c4f2017-02-04 02:26:32 +00002253 for (const MachineBasicBlock *LoopBB : LoopBlockSet)
Xinliang David Li93926ac2016-07-01 05:46:48 +00002254 fillWorkLists(LoopBB, UpdatedPreds, &LoopBlockSet);
Chandler Carruth8d150782011-11-13 11:20:44 +00002255
Xinliang David Li93926ac2016-07-01 05:46:48 +00002256 buildChain(LoopTop, LoopChain, &LoopBlockSet);
Cong Hou7745dbc2015-10-19 23:16:40 +00002257
2258 if (RotateLoopWithProfile)
2259 rotateLoopWithProfile(LoopChain, L, LoopBlockSet);
2260 else
Kyle Buttab9cca72016-10-27 21:37:20 +00002261 rotateLoop(LoopChain, PreferredLoopExit, LoopBlockSet);
Chandler Carruth8d150782011-11-13 11:20:44 +00002262
2263 DEBUG({
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002264 // Crash at the end so we get all of the debugging output first.
2265 bool BadLoop = false;
Philip Reamesae27b232016-03-03 00:58:43 +00002266 if (LoopChain.UnscheduledPredecessors) {
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002267 BadLoop = true;
Chandler Carruth8d150782011-11-13 11:20:44 +00002268 dbgs() << "Loop chain contains a block without its preds placed!\n"
2269 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
2270 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n";
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002271 }
Chandler Carruth7a715da2015-03-05 03:19:05 +00002272 for (MachineBasicBlock *ChainBB : LoopChain) {
2273 dbgs() << " ... " << getBlockName(ChainBB) << "\n";
Rong Xu66827422016-11-16 20:50:06 +00002274 if (!LoopBlockSet.remove(ChainBB)) {
Chandler Carruth0a31d142011-11-14 10:55:53 +00002275 // We don't mark the loop as bad here because there are real situations
2276 // where this can occur. For example, with an unanalyzable fallthrough
Chandler Carruth99fe42f2011-11-23 10:35:36 +00002277 // from a loop block to a non-loop block or vice versa.
Chandler Carruth8d150782011-11-13 11:20:44 +00002278 dbgs() << "Loop chain contains a block not contained by the loop!\n"
2279 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
2280 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n"
Chandler Carruth7a715da2015-03-05 03:19:05 +00002281 << " Bad block: " << getBlockName(ChainBB) << "\n";
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002282 }
Chandler Carruthccc7e422012-04-16 01:12:56 +00002283 }
Chandler Carruth8d150782011-11-13 11:20:44 +00002284
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002285 if (!LoopBlockSet.empty()) {
2286 BadLoop = true;
Kyle Butte9425c4f2017-02-04 02:26:32 +00002287 for (const MachineBasicBlock *LoopBB : LoopBlockSet)
Chandler Carruth8d150782011-11-13 11:20:44 +00002288 dbgs() << "Loop contains blocks never placed into a chain!\n"
2289 << " Loop header: " << getBlockName(*L.block_begin()) << "\n"
2290 << " Chain header: " << getBlockName(*LoopChain.begin()) << "\n"
Chandler Carruth7a715da2015-03-05 03:19:05 +00002291 << " Bad block: " << getBlockName(LoopBB) << "\n";
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002292 }
2293 assert(!BadLoop && "Detected problems with the placement of this loop.");
Chandler Carruth8d150782011-11-13 11:20:44 +00002294 });
Xinliang David Li93926ac2016-07-01 05:46:48 +00002295
2296 BlockWorkList.clear();
2297 EHPadWorkList.clear();
Chandler Carruth10281422011-10-21 06:46:38 +00002298}
2299
Xinliang David Li52530a72016-06-13 22:23:44 +00002300void MachineBlockPlacement::buildCFGChains() {
Chandler Carruth8d150782011-11-13 11:20:44 +00002301 // Ensure that every BB in the function has an associated chain to simplify
2302 // the assumptions of the remaining algorithm.
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002303 SmallVector<MachineOperand, 4> Cond; // For AnalyzeBranch.
Xinliang David Li52530a72016-06-13 22:23:44 +00002304 for (MachineFunction::iterator FI = F->begin(), FE = F->end(); FI != FE;
2305 ++FI) {
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00002306 MachineBasicBlock *BB = &*FI;
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00002307 BlockChain *Chain =
2308 new (ChainAllocator.Allocate()) BlockChain(BlockToChain, BB);
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002309 // Also, merge any blocks which we cannot reason about and must preserve
2310 // the exact fallthrough behavior for.
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002311 while (true) {
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002312 Cond.clear();
Craig Topperc0196b12014-04-14 00:51:57 +00002313 MachineBasicBlock *TBB = nullptr, *FBB = nullptr; // For AnalyzeBranch.
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002314 if (!TII->analyzeBranch(*BB, TBB, FBB, Cond) || !FI->canFallThrough())
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002315 break;
2316
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00002317 MachineFunction::iterator NextFI = std::next(FI);
2318 MachineBasicBlock *NextBB = &*NextFI;
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002319 // Ensure that the layout successor is a viable block, as we know that
2320 // fallthrough is a possibility.
2321 assert(NextFI != FE && "Can't fallthrough past the last block.");
2322 DEBUG(dbgs() << "Pre-merging due to unanalyzable fallthrough: "
2323 << getBlockName(BB) << " -> " << getBlockName(NextBB)
2324 << "\n");
Craig Topperc0196b12014-04-14 00:51:57 +00002325 Chain->merge(NextBB, nullptr);
Hal Finkel34f9d6a2016-12-15 05:33:19 +00002326#ifndef NDEBUG
Sanjoy Dasd7389d62016-12-15 05:08:57 +00002327 BlocksWithUnanalyzableExits.insert(&*BB);
Hal Finkel34f9d6a2016-12-15 05:33:19 +00002328#endif
Chandler Carruthf3dc9ef2011-11-19 10:26:02 +00002329 FI = NextFI;
2330 BB = NextBB;
2331 }
2332 }
Chandler Carruth8d150782011-11-13 11:20:44 +00002333
2334 // Build any loop-based chains.
Sam McCall2a36eee2016-11-01 22:02:14 +00002335 PreferredLoopExit = nullptr;
Chandler Carruth7a715da2015-03-05 03:19:05 +00002336 for (MachineLoop *L : *MLI)
Xinliang David Li52530a72016-06-13 22:23:44 +00002337 buildLoopChains(*L);
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00002338
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00002339 assert(BlockWorkList.empty() &&
2340 "BlockWorkList should be empty before building final chain.");
2341 assert(EHPadWorkList.empty() &&
2342 "EHPadWorkList should be empty before building final chain.");
Chandler Carruthbd1be4d2011-10-23 09:18:45 +00002343
Chandler Carruth8d150782011-11-13 11:20:44 +00002344 SmallPtrSet<BlockChain *, 4> UpdatedPreds;
Xinliang David Li52530a72016-06-13 22:23:44 +00002345 for (MachineBasicBlock &MBB : *F)
Xinliang David Li93926ac2016-07-01 05:46:48 +00002346 fillWorkLists(&MBB, UpdatedPreds);
Chandler Carruth8d150782011-11-13 11:20:44 +00002347
Xinliang David Li52530a72016-06-13 22:23:44 +00002348 BlockChain &FunctionChain = *BlockToChain[&F->front()];
Xinliang David Li93926ac2016-07-01 05:46:48 +00002349 buildChain(&F->front(), FunctionChain);
Chandler Carruth8d150782011-11-13 11:20:44 +00002350
Matt Arsenault0f5f0152013-12-10 18:55:37 +00002351#ifndef NDEBUG
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002352 using FunctionBlockSetType = SmallPtrSet<MachineBasicBlock *, 16>;
Matt Arsenault0f5f0152013-12-10 18:55:37 +00002353#endif
Chandler Carruth8d150782011-11-13 11:20:44 +00002354 DEBUG({
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002355 // Crash at the end so we get all of the debugging output first.
2356 bool BadFunc = false;
Chandler Carruth8d150782011-11-13 11:20:44 +00002357 FunctionBlockSetType FunctionBlockSet;
Xinliang David Li52530a72016-06-13 22:23:44 +00002358 for (MachineBasicBlock &MBB : *F)
Chandler Carruth7a715da2015-03-05 03:19:05 +00002359 FunctionBlockSet.insert(&MBB);
Chandler Carruth8d150782011-11-13 11:20:44 +00002360
Chandler Carruth7a715da2015-03-05 03:19:05 +00002361 for (MachineBasicBlock *ChainBB : FunctionChain)
2362 if (!FunctionBlockSet.erase(ChainBB)) {
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002363 BadFunc = true;
Chandler Carruth8d150782011-11-13 11:20:44 +00002364 dbgs() << "Function chain contains a block not in the function!\n"
Chandler Carruth7a715da2015-03-05 03:19:05 +00002365 << " Bad block: " << getBlockName(ChainBB) << "\n";
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002366 }
Chandler Carruth8d150782011-11-13 11:20:44 +00002367
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002368 if (!FunctionBlockSet.empty()) {
2369 BadFunc = true;
Chandler Carruth7a715da2015-03-05 03:19:05 +00002370 for (MachineBasicBlock *RemainingBB : FunctionBlockSet)
Chandler Carruth8d150782011-11-13 11:20:44 +00002371 dbgs() << "Function contains blocks never placed into a chain!\n"
Chandler Carruth7a715da2015-03-05 03:19:05 +00002372 << " Bad block: " << getBlockName(RemainingBB) << "\n";
Chandler Carruth8e1d9062011-11-13 21:39:51 +00002373 }
2374 assert(!BadFunc && "Detected problems with the block placement.");
Chandler Carruth8d150782011-11-13 11:20:44 +00002375 });
2376
2377 // Splice the blocks into place.
Xinliang David Li52530a72016-06-13 22:23:44 +00002378 MachineFunction::iterator InsertPos = F->begin();
Xinliang David Li449cdfd2016-06-24 22:54:21 +00002379 DEBUG(dbgs() << "[MBP] Function: "<< F->getName() << "\n");
Chandler Carruth7a715da2015-03-05 03:19:05 +00002380 for (MachineBasicBlock *ChainBB : FunctionChain) {
2381 DEBUG(dbgs() << (ChainBB == *FunctionChain.begin() ? "Placing chain "
2382 : " ... ")
2383 << getBlockName(ChainBB) << "\n");
2384 if (InsertPos != MachineFunction::iterator(ChainBB))
Xinliang David Li52530a72016-06-13 22:23:44 +00002385 F->splice(InsertPos, ChainBB);
Chandler Carruth8d150782011-11-13 11:20:44 +00002386 else
2387 ++InsertPos;
2388
2389 // Update the terminator of the previous block.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002390 if (ChainBB == *FunctionChain.begin())
Chandler Carruth8d150782011-11-13 11:20:44 +00002391 continue;
Duncan P. N. Exon Smith6ac07fd2015-10-09 19:36:12 +00002392 MachineBasicBlock *PrevBB = &*std::prev(MachineFunction::iterator(ChainBB));
Chandler Carruth8d150782011-11-13 11:20:44 +00002393
Chandler Carruth10281422011-10-21 06:46:38 +00002394 // FIXME: It would be awesome of updateTerminator would just return rather
2395 // than assert when the branch cannot be analyzed in order to remove this
2396 // boiler plate.
2397 Cond.clear();
Craig Topperc0196b12014-04-14 00:51:57 +00002398 MachineBasicBlock *TBB = nullptr, *FBB = nullptr; // For AnalyzeBranch.
Shuxin Yang8b8fd212013-06-04 01:00:57 +00002399
Sanjoy Dasd7389d62016-12-15 05:08:57 +00002400#ifndef NDEBUG
2401 if (!BlocksWithUnanalyzableExits.count(PrevBB)) {
2402 // Given the exact block placement we chose, we may actually not _need_ to
2403 // be able to edit PrevBB's terminator sequence, but not being _able_ to
2404 // do that at this point is a bug.
2405 assert((!TII->analyzeBranch(*PrevBB, TBB, FBB, Cond) ||
2406 !PrevBB->canFallThrough()) &&
2407 "Unexpected block with un-analyzable fallthrough!");
2408 Cond.clear();
2409 TBB = FBB = nullptr;
2410 }
2411#endif
2412
Haicheng Wu90a55652016-05-24 22:16:14 +00002413 // The "PrevBB" is not yet updated to reflect current code layout, so,
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00002414 // o. it may fall-through to a block without explicit "goto" instruction
Haicheng Wu90a55652016-05-24 22:16:14 +00002415 // before layout, and no longer fall-through it after layout; or
2416 // o. just opposite.
2417 //
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002418 // analyzeBranch() may return erroneous value for FBB when these two
Haicheng Wu90a55652016-05-24 22:16:14 +00002419 // situations take place. For the first scenario FBB is mistakenly set NULL;
2420 // for the 2nd scenario, the FBB, which is expected to be NULL, is
2421 // mistakenly pointing to "*BI".
2422 // Thus, if the future change needs to use FBB before the layout is set, it
2423 // has to correct FBB first by using the code similar to the following:
2424 //
2425 // if (!Cond.empty() && (!FBB || FBB == ChainBB)) {
2426 // PrevBB->updateTerminator();
2427 // Cond.clear();
2428 // TBB = FBB = nullptr;
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002429 // if (TII->analyzeBranch(*PrevBB, TBB, FBB, Cond)) {
Haicheng Wu90a55652016-05-24 22:16:14 +00002430 // // FIXME: This should never take place.
2431 // TBB = FBB = nullptr;
2432 // }
2433 // }
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002434 if (!TII->analyzeBranch(*PrevBB, TBB, FBB, Cond))
Haicheng Wu90a55652016-05-24 22:16:14 +00002435 PrevBB->updateTerminator();
Chandler Carruth10281422011-10-21 06:46:38 +00002436 }
Chandler Carruth8d150782011-11-13 11:20:44 +00002437
2438 // Fixup the last block.
2439 Cond.clear();
Craig Topperc0196b12014-04-14 00:51:57 +00002440 MachineBasicBlock *TBB = nullptr, *FBB = nullptr; // For AnalyzeBranch.
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002441 if (!TII->analyzeBranch(F->back(), TBB, FBB, Cond))
Xinliang David Li52530a72016-06-13 22:23:44 +00002442 F->back().updateTerminator();
Xinliang David Li93926ac2016-07-01 05:46:48 +00002443
2444 BlockWorkList.clear();
2445 EHPadWorkList.clear();
Haicheng Wu90a55652016-05-24 22:16:14 +00002446}
2447
Xinliang David Li52530a72016-06-13 22:23:44 +00002448void MachineBlockPlacement::optimizeBranches() {
2449 BlockChain &FunctionChain = *BlockToChain[&F->front()];
Haicheng Wu90a55652016-05-24 22:16:14 +00002450 SmallVector<MachineOperand, 4> Cond; // For AnalyzeBranch.
Quentin Colombet776e6de2016-05-02 22:58:59 +00002451
2452 // Now that all the basic blocks in the chain have the proper layout,
2453 // make a final call to AnalyzeBranch with AllowModify set.
2454 // Indeed, the target may be able to optimize the branches in a way we
2455 // cannot because all branches may not be analyzable.
2456 // E.g., the target may be able to remove an unconditional branch to
2457 // a fallthrough when it occurs after predicated terminators.
2458 for (MachineBasicBlock *ChainBB : FunctionChain) {
2459 Cond.clear();
Haicheng Wu90a55652016-05-24 22:16:14 +00002460 MachineBasicBlock *TBB = nullptr, *FBB = nullptr; // For AnalyzeBranch.
Jacques Pienaar71c30a12016-07-15 14:41:04 +00002461 if (!TII->analyzeBranch(*ChainBB, TBB, FBB, Cond, /*AllowModify*/ true)) {
Haicheng Wu90a55652016-05-24 22:16:14 +00002462 // If PrevBB has a two-way branch, try to re-order the branches
2463 // such that we branch to the successor with higher probability first.
2464 if (TBB && !Cond.empty() && FBB &&
2465 MBPI->getEdgeProbability(ChainBB, FBB) >
2466 MBPI->getEdgeProbability(ChainBB, TBB) &&
Matt Arsenault1b9fc8e2016-09-14 20:43:16 +00002467 !TII->reverseBranchCondition(Cond)) {
Haicheng Wu90a55652016-05-24 22:16:14 +00002468 DEBUG(dbgs() << "Reverse order of the two branches: "
2469 << getBlockName(ChainBB) << "\n");
2470 DEBUG(dbgs() << " Edge probability: "
2471 << MBPI->getEdgeProbability(ChainBB, FBB) << " vs "
2472 << MBPI->getEdgeProbability(ChainBB, TBB) << "\n");
2473 DebugLoc dl; // FIXME: this is nowhere
Matt Arsenault1b9fc8e2016-09-14 20:43:16 +00002474 TII->removeBranch(*ChainBB);
Matt Arsenaulte8e0f5c2016-09-14 17:24:15 +00002475 TII->insertBranch(*ChainBB, FBB, TBB, Cond, dl);
Haicheng Wu90a55652016-05-24 22:16:14 +00002476 ChainBB->updateTerminator();
2477 }
2478 }
Quentin Colombet776e6de2016-05-02 22:58:59 +00002479 }
Haicheng Wue749ce52016-04-29 17:06:44 +00002480}
Chandler Carruth10281422011-10-21 06:46:38 +00002481
Xinliang David Li52530a72016-06-13 22:23:44 +00002482void MachineBlockPlacement::alignBlocks() {
Chandler Carruthccc7e422012-04-16 01:12:56 +00002483 // Walk through the backedges of the function now that we have fully laid out
2484 // the basic blocks and align the destination of each backedge. We don't rely
Chandler Carruth881d0a72012-08-07 09:45:24 +00002485 // exclusively on the loop info here so that we can align backedges in
2486 // unnatural CFGs and backedges that were introduced purely because of the
2487 // loop rotations done during this layout pass.
Xinliang David Li52530a72016-06-13 22:23:44 +00002488 if (F->getFunction()->optForSize())
Chandler Carruth8b9737c2011-10-21 08:57:37 +00002489 return;
Xinliang David Li52530a72016-06-13 22:23:44 +00002490 BlockChain &FunctionChain = *BlockToChain[&F->front()];
Chandler Carruth881d0a72012-08-07 09:45:24 +00002491 if (FunctionChain.begin() == FunctionChain.end())
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00002492 return; // Empty chain.
Chandler Carruth8b9737c2011-10-21 08:57:37 +00002493
Chandler Carruth881d0a72012-08-07 09:45:24 +00002494 const BranchProbability ColdProb(1, 5); // 20%
Xinliang David Li52530a72016-06-13 22:23:44 +00002495 BlockFrequency EntryFreq = MBFI->getBlockFreq(&F->front());
Chandler Carruth881d0a72012-08-07 09:45:24 +00002496 BlockFrequency WeightedEntryFreq = EntryFreq * ColdProb;
Chandler Carruth7a715da2015-03-05 03:19:05 +00002497 for (MachineBasicBlock *ChainBB : FunctionChain) {
2498 if (ChainBB == *FunctionChain.begin())
2499 continue;
2500
Chandler Carruth881d0a72012-08-07 09:45:24 +00002501 // Don't align non-looping basic blocks. These are unlikely to execute
2502 // enough times to matter in practice. Note that we'll still handle
2503 // unnatural CFGs inside of a natural outer loop (the common case) and
2504 // rotated loops.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002505 MachineLoop *L = MLI->getLoopFor(ChainBB);
Chandler Carruth881d0a72012-08-07 09:45:24 +00002506 if (!L)
2507 continue;
2508
Hal Finkel57725662015-01-03 17:58:24 +00002509 unsigned Align = TLI->getPrefLoopAlignment(L);
2510 if (!Align)
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00002511 continue; // Don't care about loop alignment.
Hal Finkel57725662015-01-03 17:58:24 +00002512
Chandler Carruth881d0a72012-08-07 09:45:24 +00002513 // If the block is cold relative to the function entry don't waste space
2514 // aligning it.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002515 BlockFrequency Freq = MBFI->getBlockFreq(ChainBB);
Chandler Carruth881d0a72012-08-07 09:45:24 +00002516 if (Freq < WeightedEntryFreq)
2517 continue;
2518
2519 // If the block is cold relative to its loop header, don't align it
2520 // regardless of what edges into the block exist.
2521 MachineBasicBlock *LoopHeader = L->getHeader();
2522 BlockFrequency LoopHeaderFreq = MBFI->getBlockFreq(LoopHeader);
2523 if (Freq < (LoopHeaderFreq * ColdProb))
2524 continue;
2525
2526 // Check for the existence of a non-layout predecessor which would benefit
2527 // from aligning this block.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002528 MachineBasicBlock *LayoutPred =
2529 &*std::prev(MachineFunction::iterator(ChainBB));
Chandler Carruth881d0a72012-08-07 09:45:24 +00002530
2531 // Force alignment if all the predecessors are jumps. We already checked
2532 // that the block isn't cold above.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002533 if (!LayoutPred->isSuccessor(ChainBB)) {
2534 ChainBB->setAlignment(Align);
Chandler Carruth881d0a72012-08-07 09:45:24 +00002535 continue;
2536 }
2537
2538 // Align this block if the layout predecessor's edge into this block is
Nadav Rotem6036f582013-03-29 16:34:23 +00002539 // cold relative to the block. When this is true, other predecessors make up
Chandler Carruth881d0a72012-08-07 09:45:24 +00002540 // all of the hot entries into the block and thus alignment is likely to be
2541 // important.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002542 BranchProbability LayoutProb =
2543 MBPI->getEdgeProbability(LayoutPred, ChainBB);
Chandler Carruth881d0a72012-08-07 09:45:24 +00002544 BlockFrequency LayoutEdgeFreq = MBFI->getBlockFreq(LayoutPred) * LayoutProb;
2545 if (LayoutEdgeFreq <= (Freq * ColdProb))
Chandler Carruth7a715da2015-03-05 03:19:05 +00002546 ChainBB->setAlignment(Align);
Chandler Carruthccc7e422012-04-16 01:12:56 +00002547 }
Chandler Carruth8b9737c2011-10-21 08:57:37 +00002548}
2549
Kyle Butt0846e562016-10-11 20:36:43 +00002550/// Tail duplicate \p BB into (some) predecessors if profitable, repeating if
2551/// it was duplicated into its chain predecessor and removed.
2552/// \p BB - Basic block that may be duplicated.
2553///
2554/// \p LPred - Chosen layout predecessor of \p BB.
2555/// Updated to be the chain end if LPred is removed.
2556/// \p Chain - Chain to which \p LPred belongs, and \p BB will belong.
2557/// \p BlockFilter - Set of blocks that belong to the loop being laid out.
2558/// Used to identify which blocks to update predecessor
2559/// counts.
2560/// \p PrevUnplacedBlockIt - Iterator pointing to the last block that was
2561/// chosen in the given order due to unnatural CFG
2562/// only needed if \p BB is removed and
2563/// \p PrevUnplacedBlockIt pointed to \p BB.
2564/// @return true if \p BB was removed.
2565bool MachineBlockPlacement::repeatedlyTailDuplicateBlock(
2566 MachineBasicBlock *BB, MachineBasicBlock *&LPred,
Kyle Butte9425c4f2017-02-04 02:26:32 +00002567 const MachineBasicBlock *LoopHeaderBB,
Kyle Butt0846e562016-10-11 20:36:43 +00002568 BlockChain &Chain, BlockFilterSet *BlockFilter,
2569 MachineFunction::iterator &PrevUnplacedBlockIt) {
2570 bool Removed, DuplicatedToLPred;
2571 bool DuplicatedToOriginalLPred;
2572 Removed = maybeTailDuplicateBlock(BB, LPred, Chain, BlockFilter,
2573 PrevUnplacedBlockIt,
2574 DuplicatedToLPred);
2575 if (!Removed)
2576 return false;
2577 DuplicatedToOriginalLPred = DuplicatedToLPred;
2578 // Iteratively try to duplicate again. It can happen that a block that is
2579 // duplicated into is still small enough to be duplicated again.
2580 // No need to call markBlockSuccessors in this case, as the blocks being
2581 // duplicated from here on are already scheduled.
2582 // Note that DuplicatedToLPred always implies Removed.
2583 while (DuplicatedToLPred) {
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002584 assert(Removed && "Block must have been removed to be duplicated into its "
2585 "layout predecessor.");
Kyle Butt0846e562016-10-11 20:36:43 +00002586 MachineBasicBlock *DupBB, *DupPred;
2587 // The removal callback causes Chain.end() to be updated when a block is
2588 // removed. On the first pass through the loop, the chain end should be the
2589 // same as it was on function entry. On subsequent passes, because we are
2590 // duplicating the block at the end of the chain, if it is removed the
2591 // chain will have shrunk by one block.
2592 BlockChain::iterator ChainEnd = Chain.end();
2593 DupBB = *(--ChainEnd);
2594 // Now try to duplicate again.
2595 if (ChainEnd == Chain.begin())
2596 break;
2597 DupPred = *std::prev(ChainEnd);
2598 Removed = maybeTailDuplicateBlock(DupBB, DupPred, Chain, BlockFilter,
2599 PrevUnplacedBlockIt,
2600 DuplicatedToLPred);
2601 }
2602 // If BB was duplicated into LPred, it is now scheduled. But because it was
2603 // removed, markChainSuccessors won't be called for its chain. Instead we
2604 // call markBlockSuccessors for LPred to achieve the same effect. This must go
2605 // at the end because repeating the tail duplication can increase the number
2606 // of unscheduled predecessors.
2607 LPred = *std::prev(Chain.end());
2608 if (DuplicatedToOriginalLPred)
2609 markBlockSuccessors(Chain, LPred, LoopHeaderBB, BlockFilter);
2610 return true;
2611}
2612
2613/// Tail duplicate \p BB into (some) predecessors if profitable.
2614/// \p BB - Basic block that may be duplicated
2615/// \p LPred - Chosen layout predecessor of \p BB
2616/// \p Chain - Chain to which \p LPred belongs, and \p BB will belong.
2617/// \p BlockFilter - Set of blocks that belong to the loop being laid out.
2618/// Used to identify which blocks to update predecessor
2619/// counts.
2620/// \p PrevUnplacedBlockIt - Iterator pointing to the last block that was
2621/// chosen in the given order due to unnatural CFG
2622/// only needed if \p BB is removed and
2623/// \p PrevUnplacedBlockIt pointed to \p BB.
2624/// \p DuplicatedToLPred - True if the block was duplicated into LPred. Will
2625/// only be true if the block was removed.
2626/// \return - True if the block was duplicated into all preds and removed.
2627bool MachineBlockPlacement::maybeTailDuplicateBlock(
2628 MachineBasicBlock *BB, MachineBasicBlock *LPred,
Kyle Butte9425c4f2017-02-04 02:26:32 +00002629 BlockChain &Chain, BlockFilterSet *BlockFilter,
Kyle Butt0846e562016-10-11 20:36:43 +00002630 MachineFunction::iterator &PrevUnplacedBlockIt,
2631 bool &DuplicatedToLPred) {
Kyle Butt0846e562016-10-11 20:36:43 +00002632 DuplicatedToLPred = false;
Kyle Buttc7d67eef2017-02-04 02:26:34 +00002633 if (!shouldTailDuplicate(BB))
2634 return false;
2635
Kyle Butt0846e562016-10-11 20:36:43 +00002636 DEBUG(dbgs() << "Redoing tail duplication for Succ#"
2637 << BB->getNumber() << "\n");
Kyle Buttb15c0662017-01-31 23:48:32 +00002638
Kyle Butt0846e562016-10-11 20:36:43 +00002639 // This has to be a callback because none of it can be done after
2640 // BB is deleted.
2641 bool Removed = false;
2642 auto RemovalCallback =
2643 [&](MachineBasicBlock *RemBB) {
2644 // Signal to outer function
2645 Removed = true;
2646
2647 // Conservative default.
2648 bool InWorkList = true;
2649 // Remove from the Chain and Chain Map
2650 if (BlockToChain.count(RemBB)) {
2651 BlockChain *Chain = BlockToChain[RemBB];
2652 InWorkList = Chain->UnscheduledPredecessors == 0;
2653 Chain->remove(RemBB);
2654 BlockToChain.erase(RemBB);
2655 }
2656
2657 // Handle the unplaced block iterator
2658 if (&(*PrevUnplacedBlockIt) == RemBB) {
2659 PrevUnplacedBlockIt++;
2660 }
2661
2662 // Handle the Work Lists
2663 if (InWorkList) {
2664 SmallVectorImpl<MachineBasicBlock *> &RemoveList = BlockWorkList;
2665 if (RemBB->isEHPad())
2666 RemoveList = EHPadWorkList;
2667 RemoveList.erase(
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002668 llvm::remove_if(RemoveList,
2669 [RemBB](MachineBasicBlock *BB) {
2670 return BB == RemBB;
2671 }),
Kyle Butt0846e562016-10-11 20:36:43 +00002672 RemoveList.end());
2673 }
2674
2675 // Handle the filter set
2676 if (BlockFilter) {
Rong Xu66827422016-11-16 20:50:06 +00002677 BlockFilter->remove(RemBB);
Kyle Butt0846e562016-10-11 20:36:43 +00002678 }
2679
2680 // Remove the block from loop info.
2681 MLI->removeBlock(RemBB);
Kyle Buttab9cca72016-10-27 21:37:20 +00002682 if (RemBB == PreferredLoopExit)
2683 PreferredLoopExit = nullptr;
Kyle Butt0846e562016-10-11 20:36:43 +00002684
Kyle Butt0846e562016-10-11 20:36:43 +00002685 DEBUG(dbgs() << "TailDuplicator deleted block: "
2686 << getBlockName(RemBB) << "\n");
2687 };
2688 auto RemovalCallbackRef =
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002689 function_ref<void(MachineBasicBlock*)>(RemovalCallback);
Kyle Butt0846e562016-10-11 20:36:43 +00002690
2691 SmallVector<MachineBasicBlock *, 8> DuplicatedPreds;
Kyle Buttb15c0662017-01-31 23:48:32 +00002692 bool IsSimple = TailDup.isSimpleBB(BB);
Kyle Butt0846e562016-10-11 20:36:43 +00002693 TailDup.tailDuplicateAndUpdate(IsSimple, BB, LPred,
2694 &DuplicatedPreds, &RemovalCallbackRef);
2695
2696 // Update UnscheduledPredecessors to reflect tail-duplication.
2697 DuplicatedToLPred = false;
2698 for (MachineBasicBlock *Pred : DuplicatedPreds) {
2699 // We're only looking for unscheduled predecessors that match the filter.
2700 BlockChain* PredChain = BlockToChain[Pred];
2701 if (Pred == LPred)
2702 DuplicatedToLPred = true;
2703 if (Pred == LPred || (BlockFilter && !BlockFilter->count(Pred))
2704 || PredChain == &Chain)
2705 continue;
2706 for (MachineBasicBlock *NewSucc : Pred->successors()) {
2707 if (BlockFilter && !BlockFilter->count(NewSucc))
2708 continue;
2709 BlockChain *NewChain = BlockToChain[NewSucc];
2710 if (NewChain != &Chain && NewChain != PredChain)
2711 NewChain->UnscheduledPredecessors++;
2712 }
2713 }
2714 return Removed;
2715}
2716
Xinliang David Li52530a72016-06-13 22:23:44 +00002717bool MachineBlockPlacement::runOnMachineFunction(MachineFunction &MF) {
2718 if (skipFunction(*MF.getFunction()))
Andrew Kaylor50271f72016-05-03 22:32:30 +00002719 return false;
2720
Chandler Carruth10281422011-10-21 06:46:38 +00002721 // Check for single-block functions and skip them.
Xinliang David Li52530a72016-06-13 22:23:44 +00002722 if (std::next(MF.begin()) == MF.end())
Chandler Carruth10281422011-10-21 06:46:38 +00002723 return false;
2724
Xinliang David Li52530a72016-06-13 22:23:44 +00002725 F = &MF;
Chandler Carruth10281422011-10-21 06:46:38 +00002726 MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002727 MBFI = llvm::make_unique<BranchFolder::MBFIWrapper>(
2728 getAnalysis<MachineBlockFrequencyInfo>());
Chandler Carruth8b9737c2011-10-21 08:57:37 +00002729 MLI = &getAnalysis<MachineLoopInfo>();
Xinliang David Li52530a72016-06-13 22:23:44 +00002730 TII = MF.getSubtarget().getInstrInfo();
2731 TLI = MF.getSubtarget().getTargetLowering();
Kyle Buttb15c0662017-01-31 23:48:32 +00002732 MPDT = nullptr;
Eric Christopher690f8e52016-11-01 22:15:50 +00002733
2734 // Initialize PreferredLoopExit to nullptr here since it may never be set if
2735 // there are no MachineLoops.
2736 PreferredLoopExit = nullptr;
2737
Kyle Butt0cf5b2f2017-05-17 23:44:41 +00002738 assert(BlockToChain.empty() &&
2739 "BlockToChain map should be empty before starting placement.");
2740 assert(ComputedEdges.empty() &&
2741 "Computed Edge map should be empty before starting placement.");
Kyle Butt04300b032017-04-12 03:18:20 +00002742
Kyle Butt7d531da2017-05-15 17:30:47 +00002743 unsigned TailDupSize = TailDupPlacementThreshold;
2744 // If only the aggressive threshold is explicitly set, use it.
2745 if (TailDupPlacementAggressiveThreshold.getNumOccurrences() != 0 &&
2746 TailDupPlacementThreshold.getNumOccurrences() == 0)
2747 TailDupSize = TailDupPlacementAggressiveThreshold;
2748
2749 TargetPassConfig *PassConfig = &getAnalysis<TargetPassConfig>();
2750 // For agressive optimization, we can adjust some thresholds to be less
2751 // conservative.
2752 if (PassConfig->getOptLevel() >= CodeGenOpt::Aggressive) {
2753 // At O3 we should be more willing to copy blocks for tail duplication. This
2754 // increases size pressure, so we only do it at O3
2755 // Do this unless only the regular threshold is explicitly set.
2756 if (TailDupPlacementThreshold.getNumOccurrences() == 0 ||
2757 TailDupPlacementAggressiveThreshold.getNumOccurrences() != 0)
2758 TailDupSize = TailDupPlacementAggressiveThreshold;
2759 }
2760
Kyle Butt0846e562016-10-11 20:36:43 +00002761 if (TailDupPlacement) {
Kyle Buttb15c0662017-01-31 23:48:32 +00002762 MPDT = &getAnalysis<MachinePostDominatorTree>();
Kyle Butt0846e562016-10-11 20:36:43 +00002763 if (MF.getFunction()->optForSize())
2764 TailDupSize = 1;
Matthias Braun8426d132017-08-23 03:17:59 +00002765 bool PreRegAlloc = false;
2766 TailDup.initMF(MF, PreRegAlloc, MBPI, /* LayoutMode */ true, TailDupSize);
Kyle Butt1fa60302017-03-03 01:00:22 +00002767 precomputeTriangleChains();
Kyle Butt0846e562016-10-11 20:36:43 +00002768 }
2769
Xinliang David Li52530a72016-06-13 22:23:44 +00002770 buildCFGChains();
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002771
2772 // Changing the layout can create new tail merging opportunities.
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002773 // TailMerge can create jump into if branches that make CFG irreducible for
Sjoerd Meijerfd0ad4e2016-07-15 18:41:56 +00002774 // HW that requires structured CFG.
Xinliang David Li52530a72016-06-13 22:23:44 +00002775 bool EnableTailMerge = !MF.getTarget().requiresStructuredCFG() &&
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002776 PassConfig->getEnableTailMerge() &&
2777 BranchFoldPlacement;
2778 // No tail merging opportunities if the block number is less than four.
Xinliang David Li52530a72016-06-13 22:23:44 +00002779 if (MF.size() > 3 && EnableTailMerge) {
Kyle Butt7d531da2017-05-15 17:30:47 +00002780 unsigned TailMergeSize = TailDupSize + 1;
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002781 BranchFolder BF(/*EnableTailMerge=*/true, /*CommonHoist=*/false, *MBFI,
Kyle Butt64e42812016-08-18 18:57:29 +00002782 *MBPI, TailMergeSize);
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002783
Xinliang David Li52530a72016-06-13 22:23:44 +00002784 if (BF.OptimizeFunction(MF, TII, MF.getSubtarget().getRegisterInfo(),
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002785 getAnalysisIfAvailable<MachineModuleInfo>(), MLI,
2786 /*AfterBlockPlacement=*/true)) {
2787 // Redo the layout if tail merging creates/removes/moves blocks.
2788 BlockToChain.clear();
Kyle Butt04300b032017-04-12 03:18:20 +00002789 ComputedEdges.clear();
Kyle Butt13937612017-03-02 21:44:24 +00002790 // Must redo the post-dominator tree if blocks were changed.
Kyle Buttb15c0662017-01-31 23:48:32 +00002791 if (MPDT)
2792 MPDT->runOnMachineFunction(MF);
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002793 ChainAllocator.DestroyAll();
Xinliang David Li52530a72016-06-13 22:23:44 +00002794 buildCFGChains();
Haicheng Wu5b458cc2016-06-09 15:24:29 +00002795 }
2796 }
2797
Xinliang David Li52530a72016-06-13 22:23:44 +00002798 optimizeBranches();
2799 alignBlocks();
Chandler Carruth10281422011-10-21 06:46:38 +00002800
Chandler Carruth10281422011-10-21 06:46:38 +00002801 BlockToChain.clear();
Kyle Butt04300b032017-04-12 03:18:20 +00002802 ComputedEdges.clear();
Chandler Carruthfd9b4d92011-11-14 10:57:23 +00002803 ChainAllocator.DestroyAll();
Chandler Carruth10281422011-10-21 06:46:38 +00002804
Nadav Rotemc0adc9f2013-04-12 01:24:16 +00002805 if (AlignAllBlock)
2806 // Align all of the blocks in the function to a specific alignment.
Xinliang David Li52530a72016-06-13 22:23:44 +00002807 for (MachineBasicBlock &MBB : MF)
Chandler Carruth7a715da2015-03-05 03:19:05 +00002808 MBB.setAlignment(AlignAllBlock);
Geoff Berry10494ac2016-01-21 17:25:52 +00002809 else if (AlignAllNonFallThruBlocks) {
2810 // Align all of the blocks that have no fall-through predecessors to a
2811 // specific alignment.
Xinliang David Li52530a72016-06-13 22:23:44 +00002812 for (auto MBI = std::next(MF.begin()), MBE = MF.end(); MBI != MBE; ++MBI) {
Geoff Berry10494ac2016-01-21 17:25:52 +00002813 auto LayoutPred = std::prev(MBI);
2814 if (!LayoutPred->isSuccessor(&*MBI))
2815 MBI->setAlignment(AlignAllNonFallThruBlocks);
2816 }
2817 }
Xinliang David Lifd3f6452017-01-29 01:57:02 +00002818 if (ViewBlockLayoutWithBFI != GVDT_None &&
2819 (ViewBlockFreqFuncName.empty() ||
2820 F->getFunction()->getName().equals(ViewBlockFreqFuncName))) {
Xinliang David Li538d6662017-02-15 19:21:04 +00002821 MBFI->view("MBP." + MF.getName(), false);
Xinliang David Lifd3f6452017-01-29 01:57:02 +00002822 }
Xinliang David Lifd3f6452017-01-29 01:57:02 +00002823
Nadav Rotemc0adc9f2013-04-12 01:24:16 +00002824
Chandler Carruth10281422011-10-21 06:46:38 +00002825 // We always return true as we have no way to track whether the final order
2826 // differs from the original order.
2827 return true;
2828}
Chandler Carruthae4e8002011-11-02 07:17:12 +00002829
2830namespace {
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002831
Chandler Carruthae4e8002011-11-02 07:17:12 +00002832/// \brief A pass to compute block placement statistics.
2833///
2834/// A separate pass to compute interesting statistics for evaluating block
2835/// placement. This is separate from the actual placement pass so that they can
Benjamin Kramerbde91762012-06-02 10:20:22 +00002836/// be computed in the absence of any placement transformations or when using
Chandler Carruthae4e8002011-11-02 07:17:12 +00002837/// alternative placement strategies.
2838class MachineBlockPlacementStats : public MachineFunctionPass {
2839 /// \brief A handle to the branch probability pass.
2840 const MachineBranchProbabilityInfo *MBPI;
2841
2842 /// \brief A handle to the function-wide block frequency pass.
2843 const MachineBlockFrequencyInfo *MBFI;
2844
2845public:
2846 static char ID; // Pass identification, replacement for typeid
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002847
Chandler Carruthae4e8002011-11-02 07:17:12 +00002848 MachineBlockPlacementStats() : MachineFunctionPass(ID) {
2849 initializeMachineBlockPlacementStatsPass(*PassRegistry::getPassRegistry());
2850 }
2851
Craig Topper4584cd52014-03-07 09:26:03 +00002852 bool runOnMachineFunction(MachineFunction &F) override;
Chandler Carruthae4e8002011-11-02 07:17:12 +00002853
Craig Topper4584cd52014-03-07 09:26:03 +00002854 void getAnalysisUsage(AnalysisUsage &AU) const override {
Chandler Carruthae4e8002011-11-02 07:17:12 +00002855 AU.addRequired<MachineBranchProbabilityInfo>();
2856 AU.addRequired<MachineBlockFrequencyInfo>();
2857 AU.setPreservesAll();
2858 MachineFunctionPass::getAnalysisUsage(AU);
2859 }
Chandler Carruthae4e8002011-11-02 07:17:12 +00002860};
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002861
2862} // end anonymous namespace
Chandler Carruthae4e8002011-11-02 07:17:12 +00002863
2864char MachineBlockPlacementStats::ID = 0;
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002865
Andrew Trick1fa5bcb2012-02-08 21:23:13 +00002866char &llvm::MachineBlockPlacementStatsID = MachineBlockPlacementStats::ID;
Eugene Zelenko5df3d892017-08-24 21:21:39 +00002867
Chandler Carruthae4e8002011-11-02 07:17:12 +00002868INITIALIZE_PASS_BEGIN(MachineBlockPlacementStats, "block-placement-stats",
2869 "Basic Block Placement Stats", false, false)
2870INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)
2871INITIALIZE_PASS_DEPENDENCY(MachineBlockFrequencyInfo)
2872INITIALIZE_PASS_END(MachineBlockPlacementStats, "block-placement-stats",
2873 "Basic Block Placement Stats", false, false)
2874
Chandler Carruthae4e8002011-11-02 07:17:12 +00002875bool MachineBlockPlacementStats::runOnMachineFunction(MachineFunction &F) {
2876 // Check for single-block functions and skip them.
Benjamin Kramerb6d0bd42014-03-02 12:27:27 +00002877 if (std::next(F.begin()) == F.end())
Chandler Carruthae4e8002011-11-02 07:17:12 +00002878 return false;
2879
2880 MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
2881 MBFI = &getAnalysis<MachineBlockFrequencyInfo>();
2882
Chandler Carruth7a715da2015-03-05 03:19:05 +00002883 for (MachineBasicBlock &MBB : F) {
2884 BlockFrequency BlockFreq = MBFI->getBlockFreq(&MBB);
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00002885 Statistic &NumBranches =
Chandler Carruth7a715da2015-03-05 03:19:05 +00002886 (MBB.succ_size() > 1) ? NumCondBranches : NumUncondBranches;
Chandler Carruth2fc3fe12015-03-05 02:35:31 +00002887 Statistic &BranchTakenFreq =
Chandler Carruth7a715da2015-03-05 03:19:05 +00002888 (MBB.succ_size() > 1) ? CondBranchTakenFreq : UncondBranchTakenFreq;
2889 for (MachineBasicBlock *Succ : MBB.successors()) {
Chandler Carruthae4e8002011-11-02 07:17:12 +00002890 // Skip if this successor is a fallthrough.
Chandler Carruth7a715da2015-03-05 03:19:05 +00002891 if (MBB.isLayoutSuccessor(Succ))
Chandler Carruthae4e8002011-11-02 07:17:12 +00002892 continue;
2893
Chandler Carruth7a715da2015-03-05 03:19:05 +00002894 BlockFrequency EdgeFreq =
2895 BlockFreq * MBPI->getEdgeProbability(&MBB, Succ);
Chandler Carruthae4e8002011-11-02 07:17:12 +00002896 ++NumBranches;
2897 BranchTakenFreq += EdgeFreq.getFrequency();
2898 }
2899 }
2900
2901 return false;
2902}