From e749ce53d48b5b355137b8b81db62852e1223014 Mon Sep 17 00:00:00 2001 From: Haicheng Wu Date: Fri, 29 Apr 2016 17:06:44 +0000 Subject: [PATCH] [MBP] Split placement and alignment into two functions. NFC. Cut and Paste. llvm-svn: 268067 --- llvm/lib/CodeGen/MachineBlockPlacement.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/llvm/lib/CodeGen/MachineBlockPlacement.cpp b/llvm/lib/CodeGen/MachineBlockPlacement.cpp index 6a9078a..970be7f 100644 --- a/llvm/lib/CodeGen/MachineBlockPlacement.cpp +++ b/llvm/lib/CodeGen/MachineBlockPlacement.cpp @@ -300,6 +300,7 @@ class MachineBlockPlacement : public MachineFunctionPass { void rotateLoopWithProfile(BlockChain &LoopChain, MachineLoop &L, const BlockFilterSet &LoopBlockSet); void buildCFGChains(MachineFunction &F); + void alignBlocks(MachineFunction &F); public: static char ID; // Pass identification, replacement for typeid @@ -1357,7 +1358,9 @@ void MachineBlockPlacement::buildCFGChains(MachineFunction &F) { MachineBasicBlock *TBB = nullptr, *FBB = nullptr; // For AnalyzeBranch. if (!TII->AnalyzeBranch(F.back(), TBB, FBB, Cond)) F.back().updateTerminator(); +} +void MachineBlockPlacement::alignBlocks(MachineFunction &F) { // Walk through the backedges of the function now that we have fully laid out // the basic blocks and align the destination of each backedge. We don't rely // exclusively on the loop info here so that we can align backedges in @@ -1366,6 +1369,7 @@ void MachineBlockPlacement::buildCFGChains(MachineFunction &F) { // FIXME: Use Function::optForSize(). if (F.getFunction()->hasFnAttribute(Attribute::OptimizeForSize)) return; + BlockChain &FunctionChain = *BlockToChain[&F.front()]; if (FunctionChain.begin() == FunctionChain.end()) return; // Empty chain. @@ -1442,6 +1446,7 @@ bool MachineBlockPlacement::runOnMachineFunction(MachineFunction &F) { assert(BlockToChain.empty()); buildCFGChains(F); + alignBlocks(F); BlockToChain.clear(); ChainAllocator.DestroyAll(); -- 2.7.4