12cab237bSDimitry Andric //===- MachineSink.cpp - Sinking for machine instructions -----------------===//
2f22ef01cSRoman Divacky //
3f22ef01cSRoman Divacky // The LLVM Compiler Infrastructure
4f22ef01cSRoman Divacky //
5f22ef01cSRoman Divacky // This file is distributed under the University of Illinois Open Source
6f22ef01cSRoman Divacky // License. See LICENSE.TXT for details.
7f22ef01cSRoman Divacky //
8f22ef01cSRoman Divacky //===----------------------------------------------------------------------===//
9f22ef01cSRoman Divacky //
10ffd1746dSEd Schouten // This pass moves instructions into successor blocks when possible, so that
11f22ef01cSRoman Divacky // they aren't executed on paths where their results aren't needed.
12f22ef01cSRoman Divacky //
13f22ef01cSRoman Divacky // This pass is not intended to be a replacement or a complete alternative
14f22ef01cSRoman Divacky // for an LLVM-IR-level sinking pass. It is only designed to sink simple
15f22ef01cSRoman Divacky // constructs that are not exposed before lowering and instruction selection.
16f22ef01cSRoman Divacky //
17f22ef01cSRoman Divacky //===----------------------------------------------------------------------===//
18f22ef01cSRoman Divacky
1939d628a0SDimitry Andric #include "llvm/ADT/SetVector.h"
202754fe60SDimitry Andric #include "llvm/ADT/SmallSet.h"
212cab237bSDimitry Andric #include "llvm/ADT/SmallVector.h"
22ff0cc061SDimitry Andric #include "llvm/ADT/SparseBitVector.h"
23f22ef01cSRoman Divacky #include "llvm/ADT/Statistic.h"
24139f7f9bSDimitry Andric #include "llvm/Analysis/AliasAnalysis.h"
25d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineBasicBlock.h"
2639d628a0SDimitry Andric #include "llvm/CodeGen/MachineBlockFrequencyInfo.h"
27d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineBranchProbabilityInfo.h"
28139f7f9bSDimitry Andric #include "llvm/CodeGen/MachineDominators.h"
29d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineFunction.h"
30d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineFunctionPass.h"
31d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineInstr.h"
32139f7f9bSDimitry Andric #include "llvm/CodeGen/MachineLoopInfo.h"
33d88c1a5aSDimitry Andric #include "llvm/CodeGen/MachineOperand.h"
3439d628a0SDimitry Andric #include "llvm/CodeGen/MachinePostDominators.h"
35139f7f9bSDimitry Andric #include "llvm/CodeGen/MachineRegisterInfo.h"
362cab237bSDimitry Andric #include "llvm/CodeGen/TargetInstrInfo.h"
372cab237bSDimitry Andric #include "llvm/CodeGen/TargetRegisterInfo.h"
382cab237bSDimitry Andric #include "llvm/CodeGen/TargetSubtargetInfo.h"
392cab237bSDimitry Andric #include "llvm/IR/BasicBlock.h"
403ca95b02SDimitry Andric #include "llvm/IR/LLVMContext.h"
412cab237bSDimitry Andric #include "llvm/IR/DebugInfoMetadata.h"
422cab237bSDimitry Andric #include "llvm/Pass.h"
432cab237bSDimitry Andric #include "llvm/Support/BranchProbability.h"
44e580952dSDimitry Andric #include "llvm/Support/CommandLine.h"
45f22ef01cSRoman Divacky #include "llvm/Support/Debug.h"
46f22ef01cSRoman Divacky #include "llvm/Support/raw_ostream.h"
47d88c1a5aSDimitry Andric #include <algorithm>
48d88c1a5aSDimitry Andric #include <cassert>
49d88c1a5aSDimitry Andric #include <cstdint>
50d88c1a5aSDimitry Andric #include <map>
51d88c1a5aSDimitry Andric #include <utility>
52d88c1a5aSDimitry Andric #include <vector>
53d88c1a5aSDimitry Andric
54f22ef01cSRoman Divacky using namespace llvm;
55f22ef01cSRoman Divacky
5691bc56edSDimitry Andric #define DEBUG_TYPE "machine-sink"
5791bc56edSDimitry Andric
58e580952dSDimitry Andric static cl::opt<bool>
59e580952dSDimitry Andric SplitEdges("machine-sink-split",
60e580952dSDimitry Andric cl::desc("Split critical edges during machine sinking"),
612754fe60SDimitry Andric cl::init(true), cl::Hidden);
62e580952dSDimitry Andric
6339d628a0SDimitry Andric static cl::opt<bool>
6439d628a0SDimitry Andric UseBlockFreqInfo("machine-sink-bfi",
6539d628a0SDimitry Andric cl::desc("Use block frequency info to find successors to sink"),
6639d628a0SDimitry Andric cl::init(true), cl::Hidden);
6739d628a0SDimitry Andric
68d88c1a5aSDimitry Andric static cl::opt<unsigned> SplitEdgeProbabilityThreshold(
69d88c1a5aSDimitry Andric "machine-sink-split-probability-threshold",
70d88c1a5aSDimitry Andric cl::desc(
71d88c1a5aSDimitry Andric "Percentage threshold for splitting single-instruction critical edge. "
72d88c1a5aSDimitry Andric "If the branch threshold is higher than this threshold, we allow "
73d88c1a5aSDimitry Andric "speculative execution of up to 1 instruction to avoid branching to "
74d88c1a5aSDimitry Andric "splitted critical edge"),
75d88c1a5aSDimitry Andric cl::init(40), cl::Hidden);
7639d628a0SDimitry Andric
77f22ef01cSRoman Divacky STATISTIC(NumSunk, "Number of machine instructions sunk");
78e580952dSDimitry Andric STATISTIC(NumSplit, "Number of critical edges split");
792754fe60SDimitry Andric STATISTIC(NumCoalesces, "Number of copies coalesced");
804ba319b5SDimitry Andric STATISTIC(NumPostRACopySink, "Number of copies sunk after RA");
81f22ef01cSRoman Divacky
82f22ef01cSRoman Divacky namespace {
83d88c1a5aSDimitry Andric
84f22ef01cSRoman Divacky class MachineSinking : public MachineFunctionPass {
85f22ef01cSRoman Divacky const TargetInstrInfo *TII;
86f22ef01cSRoman Divacky const TargetRegisterInfo *TRI;
872754fe60SDimitry Andric MachineRegisterInfo *MRI; // Machine register information
88f22ef01cSRoman Divacky MachineDominatorTree *DT; // Machine dominator tree
8939d628a0SDimitry Andric MachinePostDominatorTree *PDT; // Machine post dominator tree
90f22ef01cSRoman Divacky MachineLoopInfo *LI;
9139d628a0SDimitry Andric const MachineBlockFrequencyInfo *MBFI;
92d88c1a5aSDimitry Andric const MachineBranchProbabilityInfo *MBPI;
93f22ef01cSRoman Divacky AliasAnalysis *AA;
94f22ef01cSRoman Divacky
952754fe60SDimitry Andric // Remember which edges have been considered for breaking.
962754fe60SDimitry Andric SmallSet<std::pair<MachineBasicBlock*, MachineBasicBlock*>, 8>
972754fe60SDimitry Andric CEBCandidates;
9839d628a0SDimitry Andric // Remember which edges we are about to split.
9939d628a0SDimitry Andric // This is different from CEBCandidates since those edges
10039d628a0SDimitry Andric // will be split.
10139d628a0SDimitry Andric SetVector<std::pair<MachineBasicBlock *, MachineBasicBlock *>> ToSplit;
1022754fe60SDimitry Andric
103ff0cc061SDimitry Andric SparseBitVector<> RegsToClearKillFlags;
104ff0cc061SDimitry Andric
1052cab237bSDimitry Andric using AllSuccsCache =
1062cab237bSDimitry Andric std::map<MachineBasicBlock *, SmallVector<MachineBasicBlock *, 4>>;
1078f0fd8f6SDimitry Andric
108f22ef01cSRoman Divacky public:
109f22ef01cSRoman Divacky static char ID; // Pass identification
110d88c1a5aSDimitry Andric
MachineSinking()1112754fe60SDimitry Andric MachineSinking() : MachineFunctionPass(ID) {
1122754fe60SDimitry Andric initializeMachineSinkingPass(*PassRegistry::getPassRegistry());
1132754fe60SDimitry Andric }
114f22ef01cSRoman Divacky
11591bc56edSDimitry Andric bool runOnMachineFunction(MachineFunction &MF) override;
116f22ef01cSRoman Divacky
getAnalysisUsage(AnalysisUsage & AU) const11791bc56edSDimitry Andric void getAnalysisUsage(AnalysisUsage &AU) const override {
118f22ef01cSRoman Divacky AU.setPreservesCFG();
119f22ef01cSRoman Divacky MachineFunctionPass::getAnalysisUsage(AU);
1207d523365SDimitry Andric AU.addRequired<AAResultsWrapperPass>();
121f22ef01cSRoman Divacky AU.addRequired<MachineDominatorTree>();
12239d628a0SDimitry Andric AU.addRequired<MachinePostDominatorTree>();
123f22ef01cSRoman Divacky AU.addRequired<MachineLoopInfo>();
124d88c1a5aSDimitry Andric AU.addRequired<MachineBranchProbabilityInfo>();
125f22ef01cSRoman Divacky AU.addPreserved<MachineDominatorTree>();
12639d628a0SDimitry Andric AU.addPreserved<MachinePostDominatorTree>();
127f22ef01cSRoman Divacky AU.addPreserved<MachineLoopInfo>();
12839d628a0SDimitry Andric if (UseBlockFreqInfo)
12939d628a0SDimitry Andric AU.addRequired<MachineBlockFrequencyInfo>();
130f22ef01cSRoman Divacky }
1312754fe60SDimitry Andric
releaseMemory()13291bc56edSDimitry Andric void releaseMemory() override {
1332754fe60SDimitry Andric CEBCandidates.clear();
1342754fe60SDimitry Andric }
1352754fe60SDimitry Andric
136f22ef01cSRoman Divacky private:
137f22ef01cSRoman Divacky bool ProcessBlock(MachineBasicBlock &MBB);
1383ca95b02SDimitry Andric bool isWorthBreakingCriticalEdge(MachineInstr &MI,
1392754fe60SDimitry Andric MachineBasicBlock *From,
140e580952dSDimitry Andric MachineBasicBlock *To);
1412cab237bSDimitry Andric
1424ba319b5SDimitry Andric /// Postpone the splitting of the given critical
14339d628a0SDimitry Andric /// edge (\p From, \p To).
14439d628a0SDimitry Andric ///
14539d628a0SDimitry Andric /// We do not split the edges on the fly. Indeed, this invalidates
14639d628a0SDimitry Andric /// the dominance information and thus triggers a lot of updates
14739d628a0SDimitry Andric /// of that information underneath.
14839d628a0SDimitry Andric /// Instead, we postpone all the splits after each iteration of
14939d628a0SDimitry Andric /// the main loop. That way, the information is at least valid
15039d628a0SDimitry Andric /// for the lifetime of an iteration.
15139d628a0SDimitry Andric ///
15239d628a0SDimitry Andric /// \return True if the edge is marked as toSplit, false otherwise.
15339d628a0SDimitry Andric /// False can be returned if, for instance, this is not profitable.
1543ca95b02SDimitry Andric bool PostponeSplitCriticalEdge(MachineInstr &MI,
1552754fe60SDimitry Andric MachineBasicBlock *From,
1562754fe60SDimitry Andric MachineBasicBlock *To,
1572754fe60SDimitry Andric bool BreakPHIEdge);
1583ca95b02SDimitry Andric bool SinkInstruction(MachineInstr &MI, bool &SawStore,
1592cab237bSDimitry Andric
1608f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors);
161e580952dSDimitry Andric bool AllUsesDominatedByBlock(unsigned Reg, MachineBasicBlock *MBB,
1622754fe60SDimitry Andric MachineBasicBlock *DefMBB,
1632754fe60SDimitry Andric bool &BreakPHIEdge, bool &LocalUse) const;
1643ca95b02SDimitry Andric MachineBasicBlock *FindSuccToSinkTo(MachineInstr &MI, MachineBasicBlock *MBB,
1658f0fd8f6SDimitry Andric bool &BreakPHIEdge, AllSuccsCache &AllSuccessors);
1663ca95b02SDimitry Andric bool isProfitableToSinkTo(unsigned Reg, MachineInstr &MI,
167dff0c46cSDimitry Andric MachineBasicBlock *MBB,
1688f0fd8f6SDimitry Andric MachineBasicBlock *SuccToSinkTo,
1698f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors);
170dff0c46cSDimitry Andric
1713ca95b02SDimitry Andric bool PerformTrivialForwardCoalescing(MachineInstr &MI,
1722754fe60SDimitry Andric MachineBasicBlock *MBB);
1738f0fd8f6SDimitry Andric
1748f0fd8f6SDimitry Andric SmallVector<MachineBasicBlock *, 4> &
1753ca95b02SDimitry Andric GetAllSortedSuccessors(MachineInstr &MI, MachineBasicBlock *MBB,
1768f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors) const;
177f22ef01cSRoman Divacky };
178d88c1a5aSDimitry Andric
179f22ef01cSRoman Divacky } // end anonymous namespace
180f22ef01cSRoman Divacky
181f22ef01cSRoman Divacky char MachineSinking::ID = 0;
1822cab237bSDimitry Andric
183dff0c46cSDimitry Andric char &llvm::MachineSinkingID = MachineSinking::ID;
1842cab237bSDimitry Andric
185302affcbSDimitry Andric INITIALIZE_PASS_BEGIN(MachineSinking, DEBUG_TYPE,
1862754fe60SDimitry Andric "Machine code sinking", false, false)
INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)187d88c1a5aSDimitry Andric INITIALIZE_PASS_DEPENDENCY(MachineBranchProbabilityInfo)
1882754fe60SDimitry Andric INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree)
1892754fe60SDimitry Andric INITIALIZE_PASS_DEPENDENCY(MachineLoopInfo)
1907d523365SDimitry Andric INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass)
191302affcbSDimitry Andric INITIALIZE_PASS_END(MachineSinking, DEBUG_TYPE,
1922754fe60SDimitry Andric "Machine code sinking", false, false)
193f22ef01cSRoman Divacky
1943ca95b02SDimitry Andric bool MachineSinking::PerformTrivialForwardCoalescing(MachineInstr &MI,
1952754fe60SDimitry Andric MachineBasicBlock *MBB) {
1963ca95b02SDimitry Andric if (!MI.isCopy())
1972754fe60SDimitry Andric return false;
1982754fe60SDimitry Andric
1993ca95b02SDimitry Andric unsigned SrcReg = MI.getOperand(1).getReg();
2003ca95b02SDimitry Andric unsigned DstReg = MI.getOperand(0).getReg();
2012754fe60SDimitry Andric if (!TargetRegisterInfo::isVirtualRegister(SrcReg) ||
2022754fe60SDimitry Andric !TargetRegisterInfo::isVirtualRegister(DstReg) ||
2032754fe60SDimitry Andric !MRI->hasOneNonDBGUse(SrcReg))
2042754fe60SDimitry Andric return false;
2052754fe60SDimitry Andric
2062754fe60SDimitry Andric const TargetRegisterClass *SRC = MRI->getRegClass(SrcReg);
2072754fe60SDimitry Andric const TargetRegisterClass *DRC = MRI->getRegClass(DstReg);
2082754fe60SDimitry Andric if (SRC != DRC)
2092754fe60SDimitry Andric return false;
2102754fe60SDimitry Andric
2112754fe60SDimitry Andric MachineInstr *DefMI = MRI->getVRegDef(SrcReg);
2122754fe60SDimitry Andric if (DefMI->isCopyLike())
2132754fe60SDimitry Andric return false;
2144ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << "Coalescing: " << *DefMI);
2154ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << "*** to: " << MI);
2162754fe60SDimitry Andric MRI->replaceRegWith(DstReg, SrcReg);
2173ca95b02SDimitry Andric MI.eraseFromParent();
21839d628a0SDimitry Andric
21939d628a0SDimitry Andric // Conservatively, clear any kill flags, since it's possible that they are no
22039d628a0SDimitry Andric // longer correct.
22139d628a0SDimitry Andric MRI->clearKillFlags(SrcReg);
22239d628a0SDimitry Andric
2232754fe60SDimitry Andric ++NumCoalesces;
2242754fe60SDimitry Andric return true;
2252754fe60SDimitry Andric }
2262754fe60SDimitry Andric
227f22ef01cSRoman Divacky /// AllUsesDominatedByBlock - Return true if all uses of the specified register
228e580952dSDimitry Andric /// occur in blocks dominated by the specified block. If any use is in the
229e580952dSDimitry Andric /// definition block, then return false since it is never legal to move def
230e580952dSDimitry Andric /// after uses.
2312754fe60SDimitry Andric bool
AllUsesDominatedByBlock(unsigned Reg,MachineBasicBlock * MBB,MachineBasicBlock * DefMBB,bool & BreakPHIEdge,bool & LocalUse) const2322754fe60SDimitry Andric MachineSinking::AllUsesDominatedByBlock(unsigned Reg,
233e580952dSDimitry Andric MachineBasicBlock *MBB,
234e580952dSDimitry Andric MachineBasicBlock *DefMBB,
2352754fe60SDimitry Andric bool &BreakPHIEdge,
236e580952dSDimitry Andric bool &LocalUse) const {
237f22ef01cSRoman Divacky assert(TargetRegisterInfo::isVirtualRegister(Reg) &&
238f22ef01cSRoman Divacky "Only makes sense for vregs");
2392754fe60SDimitry Andric
240dff0c46cSDimitry Andric // Ignore debug uses because debug info doesn't affect the code.
2412754fe60SDimitry Andric if (MRI->use_nodbg_empty(Reg))
2422754fe60SDimitry Andric return true;
2432754fe60SDimitry Andric
2442754fe60SDimitry Andric // BreakPHIEdge is true if all the uses are in the successor MBB being sunken
2452754fe60SDimitry Andric // into and they are all PHI nodes. In this case, machine-sink must break
2462754fe60SDimitry Andric // the critical edge first. e.g.
2472754fe60SDimitry Andric //
2482cab237bSDimitry Andric // %bb.1: derived from LLVM BB %bb4.preheader
2492cab237bSDimitry Andric // Predecessors according to CFG: %bb.0
2502754fe60SDimitry Andric // ...
2512cab237bSDimitry Andric // %reg16385 = DEC64_32r %reg16437, implicit-def dead %eflags
2522754fe60SDimitry Andric // ...
2532cab237bSDimitry Andric // JE_4 <%bb.37>, implicit %eflags
2542cab237bSDimitry Andric // Successors according to CFG: %bb.37 %bb.2
2552754fe60SDimitry Andric //
2562cab237bSDimitry Andric // %bb.2: derived from LLVM BB %bb.nph
2572cab237bSDimitry Andric // Predecessors according to CFG: %bb.0 %bb.1
2582cab237bSDimitry Andric // %reg16386 = PHI %reg16434, %bb.0, %reg16385, %bb.1
2592754fe60SDimitry Andric BreakPHIEdge = true;
26091bc56edSDimitry Andric for (MachineOperand &MO : MRI->use_nodbg_operands(Reg)) {
26191bc56edSDimitry Andric MachineInstr *UseInst = MO.getParent();
26291bc56edSDimitry Andric unsigned OpNo = &MO - &UseInst->getOperand(0);
2632754fe60SDimitry Andric MachineBasicBlock *UseBlock = UseInst->getParent();
2642754fe60SDimitry Andric if (!(UseBlock == MBB && UseInst->isPHI() &&
26591bc56edSDimitry Andric UseInst->getOperand(OpNo+1).getMBB() == DefMBB)) {
2662754fe60SDimitry Andric BreakPHIEdge = false;
2672754fe60SDimitry Andric break;
2682754fe60SDimitry Andric }
2692754fe60SDimitry Andric }
2702754fe60SDimitry Andric if (BreakPHIEdge)
2712754fe60SDimitry Andric return true;
2722754fe60SDimitry Andric
27391bc56edSDimitry Andric for (MachineOperand &MO : MRI->use_nodbg_operands(Reg)) {
274f22ef01cSRoman Divacky // Determine the block of the use.
27591bc56edSDimitry Andric MachineInstr *UseInst = MO.getParent();
27691bc56edSDimitry Andric unsigned OpNo = &MO - &UseInst->getOperand(0);
277f22ef01cSRoman Divacky MachineBasicBlock *UseBlock = UseInst->getParent();
278f22ef01cSRoman Divacky if (UseInst->isPHI()) {
279f22ef01cSRoman Divacky // PHI nodes use the operand in the predecessor block, not the block with
280f22ef01cSRoman Divacky // the PHI.
28191bc56edSDimitry Andric UseBlock = UseInst->getOperand(OpNo+1).getMBB();
282e580952dSDimitry Andric } else if (UseBlock == DefMBB) {
283e580952dSDimitry Andric LocalUse = true;
284e580952dSDimitry Andric return false;
285f22ef01cSRoman Divacky }
286ffd1746dSEd Schouten
287f22ef01cSRoman Divacky // Check that it dominates.
288f22ef01cSRoman Divacky if (!DT->dominates(MBB, UseBlock))
289f22ef01cSRoman Divacky return false;
290f22ef01cSRoman Divacky }
291ffd1746dSEd Schouten
292f22ef01cSRoman Divacky return true;
293f22ef01cSRoman Divacky }
294f22ef01cSRoman Divacky
runOnMachineFunction(MachineFunction & MF)295f22ef01cSRoman Divacky bool MachineSinking::runOnMachineFunction(MachineFunction &MF) {
2962cab237bSDimitry Andric if (skipFunction(MF.getFunction()))
29791bc56edSDimitry Andric return false;
29891bc56edSDimitry Andric
2994ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << "******** Machine Sinking ********\n");
300f22ef01cSRoman Divacky
30139d628a0SDimitry Andric TII = MF.getSubtarget().getInstrInfo();
30239d628a0SDimitry Andric TRI = MF.getSubtarget().getRegisterInfo();
3032754fe60SDimitry Andric MRI = &MF.getRegInfo();
304f22ef01cSRoman Divacky DT = &getAnalysis<MachineDominatorTree>();
30539d628a0SDimitry Andric PDT = &getAnalysis<MachinePostDominatorTree>();
306f22ef01cSRoman Divacky LI = &getAnalysis<MachineLoopInfo>();
30739d628a0SDimitry Andric MBFI = UseBlockFreqInfo ? &getAnalysis<MachineBlockFrequencyInfo>() : nullptr;
308d88c1a5aSDimitry Andric MBPI = &getAnalysis<MachineBranchProbabilityInfo>();
3097d523365SDimitry Andric AA = &getAnalysis<AAResultsWrapperPass>().getAAResults();
310f22ef01cSRoman Divacky
311f22ef01cSRoman Divacky bool EverMadeChange = false;
312f22ef01cSRoman Divacky
313d88c1a5aSDimitry Andric while (true) {
314f22ef01cSRoman Divacky bool MadeChange = false;
315f22ef01cSRoman Divacky
316f22ef01cSRoman Divacky // Process all basic blocks.
3172754fe60SDimitry Andric CEBCandidates.clear();
31839d628a0SDimitry Andric ToSplit.clear();
3198f0fd8f6SDimitry Andric for (auto &MBB: MF)
3208f0fd8f6SDimitry Andric MadeChange |= ProcessBlock(MBB);
321f22ef01cSRoman Divacky
32239d628a0SDimitry Andric // If we have anything we marked as toSplit, split it now.
32339d628a0SDimitry Andric for (auto &Pair : ToSplit) {
3243ca95b02SDimitry Andric auto NewSucc = Pair.first->SplitCriticalEdge(Pair.second, *this);
32539d628a0SDimitry Andric if (NewSucc != nullptr) {
3264ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** Splitting critical edge: "
3272cab237bSDimitry Andric << printMBBReference(*Pair.first) << " -- "
3282cab237bSDimitry Andric << printMBBReference(*NewSucc) << " -- "
3292cab237bSDimitry Andric << printMBBReference(*Pair.second) << '\n');
33039d628a0SDimitry Andric MadeChange = true;
33139d628a0SDimitry Andric ++NumSplit;
33239d628a0SDimitry Andric } else
3334ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** Not legal to break critical edge\n");
33439d628a0SDimitry Andric }
335f22ef01cSRoman Divacky // If this iteration over the code changed anything, keep iterating.
336f22ef01cSRoman Divacky if (!MadeChange) break;
337f22ef01cSRoman Divacky EverMadeChange = true;
338f22ef01cSRoman Divacky }
339ff0cc061SDimitry Andric
340ff0cc061SDimitry Andric // Now clear any kill flags for recorded registers.
341ff0cc061SDimitry Andric for (auto I : RegsToClearKillFlags)
342ff0cc061SDimitry Andric MRI->clearKillFlags(I);
343ff0cc061SDimitry Andric RegsToClearKillFlags.clear();
344ff0cc061SDimitry Andric
345f22ef01cSRoman Divacky return EverMadeChange;
346f22ef01cSRoman Divacky }
347f22ef01cSRoman Divacky
ProcessBlock(MachineBasicBlock & MBB)348f22ef01cSRoman Divacky bool MachineSinking::ProcessBlock(MachineBasicBlock &MBB) {
349f22ef01cSRoman Divacky // Can't sink anything out of a block that has less than two successors.
350f22ef01cSRoman Divacky if (MBB.succ_size() <= 1 || MBB.empty()) return false;
351f22ef01cSRoman Divacky
352f22ef01cSRoman Divacky // Don't bother sinking code out of unreachable blocks. In addition to being
353ffd1746dSEd Schouten // unprofitable, it can also lead to infinite looping, because in an
354ffd1746dSEd Schouten // unreachable loop there may be nowhere to stop.
355f22ef01cSRoman Divacky if (!DT->isReachableFromEntry(&MBB)) return false;
356f22ef01cSRoman Divacky
357f22ef01cSRoman Divacky bool MadeChange = false;
358f22ef01cSRoman Divacky
3598f0fd8f6SDimitry Andric // Cache all successors, sorted by frequency info and loop depth.
3608f0fd8f6SDimitry Andric AllSuccsCache AllSuccessors;
3618f0fd8f6SDimitry Andric
362f22ef01cSRoman Divacky // Walk the basic block bottom-up. Remember if we saw a store.
363f22ef01cSRoman Divacky MachineBasicBlock::iterator I = MBB.end();
364f22ef01cSRoman Divacky --I;
365f22ef01cSRoman Divacky bool ProcessedBegin, SawStore = false;
366f22ef01cSRoman Divacky do {
3673ca95b02SDimitry Andric MachineInstr &MI = *I; // The instruction to sink.
368f22ef01cSRoman Divacky
369f22ef01cSRoman Divacky // Predecrement I (if it's not begin) so that it isn't invalidated by
370f22ef01cSRoman Divacky // sinking.
371f22ef01cSRoman Divacky ProcessedBegin = I == MBB.begin();
372f22ef01cSRoman Divacky if (!ProcessedBegin)
373f22ef01cSRoman Divacky --I;
374f22ef01cSRoman Divacky
3754ba319b5SDimitry Andric if (MI.isDebugInstr())
376f22ef01cSRoman Divacky continue;
377f22ef01cSRoman Divacky
3783b0f4066SDimitry Andric bool Joined = PerformTrivialForwardCoalescing(MI, &MBB);
3793b0f4066SDimitry Andric if (Joined) {
3803b0f4066SDimitry Andric MadeChange = true;
3812754fe60SDimitry Andric continue;
3823b0f4066SDimitry Andric }
3832754fe60SDimitry Andric
3843ca95b02SDimitry Andric if (SinkInstruction(MI, SawStore, AllSuccessors)) {
3853ca95b02SDimitry Andric ++NumSunk;
3863ca95b02SDimitry Andric MadeChange = true;
3873ca95b02SDimitry Andric }
388f22ef01cSRoman Divacky
389f22ef01cSRoman Divacky // If we just processed the first instruction in the block, we're done.
390f22ef01cSRoman Divacky } while (!ProcessedBegin);
391f22ef01cSRoman Divacky
392f22ef01cSRoman Divacky return MadeChange;
393f22ef01cSRoman Divacky }
394f22ef01cSRoman Divacky
isWorthBreakingCriticalEdge(MachineInstr & MI,MachineBasicBlock * From,MachineBasicBlock * To)3953ca95b02SDimitry Andric bool MachineSinking::isWorthBreakingCriticalEdge(MachineInstr &MI,
3962754fe60SDimitry Andric MachineBasicBlock *From,
3972754fe60SDimitry Andric MachineBasicBlock *To) {
3982754fe60SDimitry Andric // FIXME: Need much better heuristics.
3992754fe60SDimitry Andric
4002754fe60SDimitry Andric // If the pass has already considered breaking this edge (during this pass
4012754fe60SDimitry Andric // through the function), then let's go ahead and break it. This means
4022754fe60SDimitry Andric // sinking multiple "cheap" instructions into the same block.
40339d628a0SDimitry Andric if (!CEBCandidates.insert(std::make_pair(From, To)).second)
4042754fe60SDimitry Andric return true;
4052754fe60SDimitry Andric
4063ca95b02SDimitry Andric if (!MI.isCopy() && !TII->isAsCheapAsAMove(MI))
4072754fe60SDimitry Andric return true;
4082754fe60SDimitry Andric
409d88c1a5aSDimitry Andric if (From->isSuccessor(To) && MBPI->getEdgeProbability(From, To) <=
410d88c1a5aSDimitry Andric BranchProbability(SplitEdgeProbabilityThreshold, 100))
411d88c1a5aSDimitry Andric return true;
412d88c1a5aSDimitry Andric
4132754fe60SDimitry Andric // MI is cheap, we probably don't want to break the critical edge for it.
4142754fe60SDimitry Andric // However, if this would allow some definitions of its source operands
4152754fe60SDimitry Andric // to be sunk then it's probably worth it.
4163ca95b02SDimitry Andric for (unsigned i = 0, e = MI.getNumOperands(); i != e; ++i) {
4173ca95b02SDimitry Andric const MachineOperand &MO = MI.getOperand(i);
418f785676fSDimitry Andric if (!MO.isReg() || !MO.isUse())
4192754fe60SDimitry Andric continue;
420f785676fSDimitry Andric unsigned Reg = MO.getReg();
421f785676fSDimitry Andric if (Reg == 0)
422f785676fSDimitry Andric continue;
423f785676fSDimitry Andric
424f785676fSDimitry Andric // We don't move live definitions of physical registers,
425f785676fSDimitry Andric // so sinking their uses won't enable any opportunities.
426f785676fSDimitry Andric if (TargetRegisterInfo::isPhysicalRegister(Reg))
427f785676fSDimitry Andric continue;
428f785676fSDimitry Andric
429f785676fSDimitry Andric // If this instruction is the only user of a virtual register,
430f785676fSDimitry Andric // check if breaking the edge will enable sinking
431f785676fSDimitry Andric // both this instruction and the defining instruction.
432f785676fSDimitry Andric if (MRI->hasOneNonDBGUse(Reg)) {
433f785676fSDimitry Andric // If the definition resides in same MBB,
434f785676fSDimitry Andric // claim it's likely we can sink these together.
435f785676fSDimitry Andric // If definition resides elsewhere, we aren't
436f785676fSDimitry Andric // blocking it from being sunk so don't break the edge.
437f785676fSDimitry Andric MachineInstr *DefMI = MRI->getVRegDef(Reg);
4383ca95b02SDimitry Andric if (DefMI->getParent() == MI.getParent())
4392754fe60SDimitry Andric return true;
4402754fe60SDimitry Andric }
441f785676fSDimitry Andric }
4422754fe60SDimitry Andric
4432754fe60SDimitry Andric return false;
4442754fe60SDimitry Andric }
4452754fe60SDimitry Andric
PostponeSplitCriticalEdge(MachineInstr & MI,MachineBasicBlock * FromBB,MachineBasicBlock * ToBB,bool BreakPHIEdge)4463ca95b02SDimitry Andric bool MachineSinking::PostponeSplitCriticalEdge(MachineInstr &MI,
4472754fe60SDimitry Andric MachineBasicBlock *FromBB,
4482754fe60SDimitry Andric MachineBasicBlock *ToBB,
4492754fe60SDimitry Andric bool BreakPHIEdge) {
4502754fe60SDimitry Andric if (!isWorthBreakingCriticalEdge(MI, FromBB, ToBB))
45139d628a0SDimitry Andric return false;
452e580952dSDimitry Andric
4532754fe60SDimitry Andric // Avoid breaking back edge. From == To means backedge for single BB loop.
4542754fe60SDimitry Andric if (!SplitEdges || FromBB == ToBB)
45539d628a0SDimitry Andric return false;
4562754fe60SDimitry Andric
4572754fe60SDimitry Andric // Check for backedges of more "complex" loops.
4582754fe60SDimitry Andric if (LI->getLoopFor(FromBB) == LI->getLoopFor(ToBB) &&
4592754fe60SDimitry Andric LI->isLoopHeader(ToBB))
46039d628a0SDimitry Andric return false;
4612754fe60SDimitry Andric
462e580952dSDimitry Andric // It's not always legal to break critical edges and sink the computation
463e580952dSDimitry Andric // to the edge.
464e580952dSDimitry Andric //
4652cab237bSDimitry Andric // %bb.1:
466e580952dSDimitry Andric // v1024
4672cab237bSDimitry Andric // Beq %bb.3
468e580952dSDimitry Andric // <fallthrough>
4692cab237bSDimitry Andric // %bb.2:
470e580952dSDimitry Andric // ... no uses of v1024
471e580952dSDimitry Andric // <fallthrough>
4722cab237bSDimitry Andric // %bb.3:
473e580952dSDimitry Andric // ...
474e580952dSDimitry Andric // = v1024
475e580952dSDimitry Andric //
4762cab237bSDimitry Andric // If %bb.1 -> %bb.3 edge is broken and computation of v1024 is inserted:
477e580952dSDimitry Andric //
4782cab237bSDimitry Andric // %bb.1:
479e580952dSDimitry Andric // ...
4802cab237bSDimitry Andric // Bne %bb.2
4812cab237bSDimitry Andric // %bb.4:
482e580952dSDimitry Andric // v1024 =
4832cab237bSDimitry Andric // B %bb.3
4842cab237bSDimitry Andric // %bb.2:
485e580952dSDimitry Andric // ... no uses of v1024
486e580952dSDimitry Andric // <fallthrough>
4872cab237bSDimitry Andric // %bb.3:
488e580952dSDimitry Andric // ...
489e580952dSDimitry Andric // = v1024
490e580952dSDimitry Andric //
4912cab237bSDimitry Andric // This is incorrect since v1024 is not computed along the %bb.1->%bb.2->%bb.3
492e580952dSDimitry Andric // flow. We need to ensure the new basic block where the computation is
493e580952dSDimitry Andric // sunk to dominates all the uses.
494e580952dSDimitry Andric // It's only legal to break critical edge and sink the computation to the
495e580952dSDimitry Andric // new block if all the predecessors of "To", except for "From", are
496e580952dSDimitry Andric // not dominated by "From". Given SSA property, this means these
497e580952dSDimitry Andric // predecessors are dominated by "To".
4982754fe60SDimitry Andric //
4992754fe60SDimitry Andric // There is no need to do this check if all the uses are PHI nodes. PHI
5002754fe60SDimitry Andric // sources are only defined on the specific predecessor edges.
5012754fe60SDimitry Andric if (!BreakPHIEdge) {
502e580952dSDimitry Andric for (MachineBasicBlock::pred_iterator PI = ToBB->pred_begin(),
503e580952dSDimitry Andric E = ToBB->pred_end(); PI != E; ++PI) {
504e580952dSDimitry Andric if (*PI == FromBB)
505e580952dSDimitry Andric continue;
506e580952dSDimitry Andric if (!DT->dominates(ToBB, *PI))
50739d628a0SDimitry Andric return false;
508e580952dSDimitry Andric }
5092754fe60SDimitry Andric }
510e580952dSDimitry Andric
51139d628a0SDimitry Andric ToSplit.insert(std::make_pair(FromBB, ToBB));
51239d628a0SDimitry Andric
51339d628a0SDimitry Andric return true;
514e580952dSDimitry Andric }
515e580952dSDimitry Andric
516dff0c46cSDimitry Andric /// isProfitableToSinkTo - Return true if it is profitable to sink MI.
isProfitableToSinkTo(unsigned Reg,MachineInstr & MI,MachineBasicBlock * MBB,MachineBasicBlock * SuccToSinkTo,AllSuccsCache & AllSuccessors)5173ca95b02SDimitry Andric bool MachineSinking::isProfitableToSinkTo(unsigned Reg, MachineInstr &MI,
518dff0c46cSDimitry Andric MachineBasicBlock *MBB,
5198f0fd8f6SDimitry Andric MachineBasicBlock *SuccToSinkTo,
5208f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors) {
521dff0c46cSDimitry Andric assert (SuccToSinkTo && "Invalid SinkTo Candidate BB");
522dff0c46cSDimitry Andric
523dff0c46cSDimitry Andric if (MBB == SuccToSinkTo)
524f22ef01cSRoman Divacky return false;
525f22ef01cSRoman Divacky
526dff0c46cSDimitry Andric // It is profitable if SuccToSinkTo does not post dominate current block.
52739d628a0SDimitry Andric if (!PDT->dominates(SuccToSinkTo, MBB))
52839d628a0SDimitry Andric return true;
52939d628a0SDimitry Andric
53039d628a0SDimitry Andric // It is profitable to sink an instruction from a deeper loop to a shallower
53139d628a0SDimitry Andric // loop, even if the latter post-dominates the former (PR21115).
53239d628a0SDimitry Andric if (LI->getLoopDepth(MBB) > LI->getLoopDepth(SuccToSinkTo))
533dff0c46cSDimitry Andric return true;
534dff0c46cSDimitry Andric
535dff0c46cSDimitry Andric // Check if only use in post dominated block is PHI instruction.
536dff0c46cSDimitry Andric bool NonPHIUse = false;
53791bc56edSDimitry Andric for (MachineInstr &UseInst : MRI->use_nodbg_instructions(Reg)) {
53891bc56edSDimitry Andric MachineBasicBlock *UseBlock = UseInst.getParent();
53991bc56edSDimitry Andric if (UseBlock == SuccToSinkTo && !UseInst.isPHI())
540dff0c46cSDimitry Andric NonPHIUse = true;
541dff0c46cSDimitry Andric }
542dff0c46cSDimitry Andric if (!NonPHIUse)
543dff0c46cSDimitry Andric return true;
544dff0c46cSDimitry Andric
545dff0c46cSDimitry Andric // If SuccToSinkTo post dominates then also it may be profitable if MI
546dff0c46cSDimitry Andric // can further profitably sinked into another block in next round.
547dff0c46cSDimitry Andric bool BreakPHIEdge = false;
54839d628a0SDimitry Andric // FIXME - If finding successor is compile time expensive then cache results.
5498f0fd8f6SDimitry Andric if (MachineBasicBlock *MBB2 =
5508f0fd8f6SDimitry Andric FindSuccToSinkTo(MI, SuccToSinkTo, BreakPHIEdge, AllSuccessors))
5518f0fd8f6SDimitry Andric return isProfitableToSinkTo(Reg, MI, SuccToSinkTo, MBB2, AllSuccessors);
552dff0c46cSDimitry Andric
553dff0c46cSDimitry Andric // If SuccToSinkTo is final destination and it is a post dominator of current
554dff0c46cSDimitry Andric // block then it is not profitable to sink MI into SuccToSinkTo block.
555dff0c46cSDimitry Andric return false;
556dff0c46cSDimitry Andric }
557dff0c46cSDimitry Andric
5588f0fd8f6SDimitry Andric /// Get the sorted sequence of successors for this MachineBasicBlock, possibly
5598f0fd8f6SDimitry Andric /// computing it if it was not already cached.
5608f0fd8f6SDimitry Andric SmallVector<MachineBasicBlock *, 4> &
GetAllSortedSuccessors(MachineInstr & MI,MachineBasicBlock * MBB,AllSuccsCache & AllSuccessors) const5613ca95b02SDimitry Andric MachineSinking::GetAllSortedSuccessors(MachineInstr &MI, MachineBasicBlock *MBB,
5628f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors) const {
5638f0fd8f6SDimitry Andric // Do we have the sorted successors in cache ?
5648f0fd8f6SDimitry Andric auto Succs = AllSuccessors.find(MBB);
5658f0fd8f6SDimitry Andric if (Succs != AllSuccessors.end())
5668f0fd8f6SDimitry Andric return Succs->second;
5678f0fd8f6SDimitry Andric
5688f0fd8f6SDimitry Andric SmallVector<MachineBasicBlock *, 4> AllSuccs(MBB->succ_begin(),
5698f0fd8f6SDimitry Andric MBB->succ_end());
5708f0fd8f6SDimitry Andric
5718f0fd8f6SDimitry Andric // Handle cases where sinking can happen but where the sink point isn't a
5728f0fd8f6SDimitry Andric // successor. For example:
5738f0fd8f6SDimitry Andric //
5748f0fd8f6SDimitry Andric // x = computation
5758f0fd8f6SDimitry Andric // if () {} else {}
5768f0fd8f6SDimitry Andric // use x
5778f0fd8f6SDimitry Andric //
5788f0fd8f6SDimitry Andric const std::vector<MachineDomTreeNode *> &Children =
5798f0fd8f6SDimitry Andric DT->getNode(MBB)->getChildren();
5808f0fd8f6SDimitry Andric for (const auto &DTChild : Children)
5818f0fd8f6SDimitry Andric // DomTree children of MBB that have MBB as immediate dominator are added.
5823ca95b02SDimitry Andric if (DTChild->getIDom()->getBlock() == MI.getParent() &&
5838f0fd8f6SDimitry Andric // Skip MBBs already added to the AllSuccs vector above.
5848f0fd8f6SDimitry Andric !MBB->isSuccessor(DTChild->getBlock()))
5858f0fd8f6SDimitry Andric AllSuccs.push_back(DTChild->getBlock());
5868f0fd8f6SDimitry Andric
5878f0fd8f6SDimitry Andric // Sort Successors according to their loop depth or block frequency info.
5888f0fd8f6SDimitry Andric std::stable_sort(
5898f0fd8f6SDimitry Andric AllSuccs.begin(), AllSuccs.end(),
5908f0fd8f6SDimitry Andric [this](const MachineBasicBlock *L, const MachineBasicBlock *R) {
5918f0fd8f6SDimitry Andric uint64_t LHSFreq = MBFI ? MBFI->getBlockFreq(L).getFrequency() : 0;
5928f0fd8f6SDimitry Andric uint64_t RHSFreq = MBFI ? MBFI->getBlockFreq(R).getFrequency() : 0;
5938f0fd8f6SDimitry Andric bool HasBlockFreq = LHSFreq != 0 && RHSFreq != 0;
5948f0fd8f6SDimitry Andric return HasBlockFreq ? LHSFreq < RHSFreq
5958f0fd8f6SDimitry Andric : LI->getLoopDepth(L) < LI->getLoopDepth(R);
5968f0fd8f6SDimitry Andric });
5978f0fd8f6SDimitry Andric
5988f0fd8f6SDimitry Andric auto it = AllSuccessors.insert(std::make_pair(MBB, AllSuccs));
5998f0fd8f6SDimitry Andric
6008f0fd8f6SDimitry Andric return it.first->second;
6018f0fd8f6SDimitry Andric }
6028f0fd8f6SDimitry Andric
603dff0c46cSDimitry Andric /// FindSuccToSinkTo - Find a successor to sink this instruction to.
6043ca95b02SDimitry Andric MachineBasicBlock *
FindSuccToSinkTo(MachineInstr & MI,MachineBasicBlock * MBB,bool & BreakPHIEdge,AllSuccsCache & AllSuccessors)6053ca95b02SDimitry Andric MachineSinking::FindSuccToSinkTo(MachineInstr &MI, MachineBasicBlock *MBB,
6068f0fd8f6SDimitry Andric bool &BreakPHIEdge,
6078f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors) {
608dff0c46cSDimitry Andric assert (MBB && "Invalid MachineBasicBlock!");
609f22ef01cSRoman Divacky
610f22ef01cSRoman Divacky // Loop over all the operands of the specified instruction. If there is
611f22ef01cSRoman Divacky // anything we can't handle, bail out.
612f22ef01cSRoman Divacky
613f22ef01cSRoman Divacky // SuccToSinkTo - This is the successor to sink this instruction to, once we
614f22ef01cSRoman Divacky // decide.
61591bc56edSDimitry Andric MachineBasicBlock *SuccToSinkTo = nullptr;
6163ca95b02SDimitry Andric for (unsigned i = 0, e = MI.getNumOperands(); i != e; ++i) {
6173ca95b02SDimitry Andric const MachineOperand &MO = MI.getOperand(i);
618f22ef01cSRoman Divacky if (!MO.isReg()) continue; // Ignore non-register operands.
619f22ef01cSRoman Divacky
620f22ef01cSRoman Divacky unsigned Reg = MO.getReg();
621f22ef01cSRoman Divacky if (Reg == 0) continue;
622f22ef01cSRoman Divacky
623f22ef01cSRoman Divacky if (TargetRegisterInfo::isPhysicalRegister(Reg)) {
624f22ef01cSRoman Divacky if (MO.isUse()) {
625f22ef01cSRoman Divacky // If the physreg has no defs anywhere, it's just an ambient register
626f22ef01cSRoman Divacky // and we can freely move its uses. Alternatively, if it's allocatable,
627f22ef01cSRoman Divacky // it could get allocated to something with a def during allocation.
628d88c1a5aSDimitry Andric if (!MRI->isConstantPhysReg(Reg))
62991bc56edSDimitry Andric return nullptr;
630f22ef01cSRoman Divacky } else if (!MO.isDead()) {
631f22ef01cSRoman Divacky // A def that isn't dead. We can't move it.
63291bc56edSDimitry Andric return nullptr;
633f22ef01cSRoman Divacky }
634f22ef01cSRoman Divacky } else {
635f22ef01cSRoman Divacky // Virtual register uses are always safe to sink.
636f22ef01cSRoman Divacky if (MO.isUse()) continue;
637f22ef01cSRoman Divacky
638f22ef01cSRoman Divacky // If it's not safe to move defs of the register class, then abort.
6392754fe60SDimitry Andric if (!TII->isSafeToMoveRegClassDefs(MRI->getRegClass(Reg)))
64091bc56edSDimitry Andric return nullptr;
641f22ef01cSRoman Divacky
642f22ef01cSRoman Divacky // Virtual register defs can only be sunk if all their uses are in blocks
643f22ef01cSRoman Divacky // dominated by one of the successors.
644f22ef01cSRoman Divacky if (SuccToSinkTo) {
645f22ef01cSRoman Divacky // If a previous operand picked a block to sink to, then this operand
646f22ef01cSRoman Divacky // must be sinkable to the same block.
647e580952dSDimitry Andric bool LocalUse = false;
648dff0c46cSDimitry Andric if (!AllUsesDominatedByBlock(Reg, SuccToSinkTo, MBB,
6492754fe60SDimitry Andric BreakPHIEdge, LocalUse))
65091bc56edSDimitry Andric return nullptr;
651ffd1746dSEd Schouten
652f22ef01cSRoman Divacky continue;
653f22ef01cSRoman Divacky }
654f22ef01cSRoman Divacky
655f22ef01cSRoman Divacky // Otherwise, we should look at all the successors and decide which one
65639d628a0SDimitry Andric // we should sink to. If we have reliable block frequency information
65739d628a0SDimitry Andric // (frequency != 0) available, give successors with smaller frequencies
65839d628a0SDimitry Andric // higher priority, otherwise prioritize smaller loop depths.
6598f0fd8f6SDimitry Andric for (MachineBasicBlock *SuccBlock :
6608f0fd8f6SDimitry Andric GetAllSortedSuccessors(MI, MBB, AllSuccessors)) {
661e580952dSDimitry Andric bool LocalUse = false;
662dff0c46cSDimitry Andric if (AllUsesDominatedByBlock(Reg, SuccBlock, MBB,
6632754fe60SDimitry Andric BreakPHIEdge, LocalUse)) {
664dff0c46cSDimitry Andric SuccToSinkTo = SuccBlock;
665f22ef01cSRoman Divacky break;
666f22ef01cSRoman Divacky }
667e580952dSDimitry Andric if (LocalUse)
668e580952dSDimitry Andric // Def is used locally, it's never safe to move this def.
66991bc56edSDimitry Andric return nullptr;
670f22ef01cSRoman Divacky }
671f22ef01cSRoman Divacky
672f22ef01cSRoman Divacky // If we couldn't find a block to sink to, ignore this instruction.
67391bc56edSDimitry Andric if (!SuccToSinkTo)
67491bc56edSDimitry Andric return nullptr;
6758f0fd8f6SDimitry Andric if (!isProfitableToSinkTo(Reg, MI, MBB, SuccToSinkTo, AllSuccessors))
67691bc56edSDimitry Andric return nullptr;
677dff0c46cSDimitry Andric }
678dff0c46cSDimitry Andric }
679dff0c46cSDimitry Andric
680dff0c46cSDimitry Andric // It is not possible to sink an instruction into its own block. This can
681dff0c46cSDimitry Andric // happen with loops.
682dff0c46cSDimitry Andric if (MBB == SuccToSinkTo)
68391bc56edSDimitry Andric return nullptr;
684dff0c46cSDimitry Andric
685dff0c46cSDimitry Andric // It's not safe to sink instructions to EH landing pad. Control flow into
686dff0c46cSDimitry Andric // landing pad is implicitly defined.
6877d523365SDimitry Andric if (SuccToSinkTo && SuccToSinkTo->isEHPad())
68891bc56edSDimitry Andric return nullptr;
689dff0c46cSDimitry Andric
690dff0c46cSDimitry Andric return SuccToSinkTo;
691dff0c46cSDimitry Andric }
692dff0c46cSDimitry Andric
6934ba319b5SDimitry Andric /// Return true if MI is likely to be usable as a memory operation by the
6943ca95b02SDimitry Andric /// implicit null check optimization.
6953ca95b02SDimitry Andric ///
6963ca95b02SDimitry Andric /// This is a "best effort" heuristic, and should not be relied upon for
6973ca95b02SDimitry Andric /// correctness. This returning true does not guarantee that the implicit null
6983ca95b02SDimitry Andric /// check optimization is legal over MI, and this returning false does not
6993ca95b02SDimitry Andric /// guarantee MI cannot possibly be used to do a null check.
SinkingPreventsImplicitNullCheck(MachineInstr & MI,const TargetInstrInfo * TII,const TargetRegisterInfo * TRI)7003ca95b02SDimitry Andric static bool SinkingPreventsImplicitNullCheck(MachineInstr &MI,
7013ca95b02SDimitry Andric const TargetInstrInfo *TII,
7023ca95b02SDimitry Andric const TargetRegisterInfo *TRI) {
7032cab237bSDimitry Andric using MachineBranchPredicate = TargetInstrInfo::MachineBranchPredicate;
7043ca95b02SDimitry Andric
7053ca95b02SDimitry Andric auto *MBB = MI.getParent();
7063ca95b02SDimitry Andric if (MBB->pred_size() != 1)
7073ca95b02SDimitry Andric return false;
7083ca95b02SDimitry Andric
7093ca95b02SDimitry Andric auto *PredMBB = *MBB->pred_begin();
7103ca95b02SDimitry Andric auto *PredBB = PredMBB->getBasicBlock();
7113ca95b02SDimitry Andric
7123ca95b02SDimitry Andric // Frontends that don't use implicit null checks have no reason to emit
7133ca95b02SDimitry Andric // branches with make.implicit metadata, and this function should always
7143ca95b02SDimitry Andric // return false for them.
7153ca95b02SDimitry Andric if (!PredBB ||
7163ca95b02SDimitry Andric !PredBB->getTerminator()->getMetadata(LLVMContext::MD_make_implicit))
7173ca95b02SDimitry Andric return false;
7183ca95b02SDimitry Andric
719*b5893f02SDimitry Andric MachineOperand *BaseOp;
7203ca95b02SDimitry Andric int64_t Offset;
721*b5893f02SDimitry Andric if (!TII->getMemOperandWithOffset(MI, BaseOp, Offset, TRI))
722*b5893f02SDimitry Andric return false;
723*b5893f02SDimitry Andric
724*b5893f02SDimitry Andric if (!BaseOp->isReg())
7253ca95b02SDimitry Andric return false;
7263ca95b02SDimitry Andric
7273ca95b02SDimitry Andric if (!(MI.mayLoad() && !MI.isPredicable()))
7283ca95b02SDimitry Andric return false;
7293ca95b02SDimitry Andric
7303ca95b02SDimitry Andric MachineBranchPredicate MBP;
7313ca95b02SDimitry Andric if (TII->analyzeBranchPredicate(*PredMBB, MBP, false))
7323ca95b02SDimitry Andric return false;
7333ca95b02SDimitry Andric
7343ca95b02SDimitry Andric return MBP.LHS.isReg() && MBP.RHS.isImm() && MBP.RHS.getImm() == 0 &&
7353ca95b02SDimitry Andric (MBP.Predicate == MachineBranchPredicate::PRED_NE ||
7363ca95b02SDimitry Andric MBP.Predicate == MachineBranchPredicate::PRED_EQ) &&
737*b5893f02SDimitry Andric MBP.LHS.getReg() == BaseOp->getReg();
7383ca95b02SDimitry Andric }
7393ca95b02SDimitry Andric
740*b5893f02SDimitry Andric /// Sink an instruction and its associated debug instructions. If the debug
741*b5893f02SDimitry Andric /// instructions to be sunk are already known, they can be provided in DbgVals.
performSink(MachineInstr & MI,MachineBasicBlock & SuccToSinkTo,MachineBasicBlock::iterator InsertPos,SmallVectorImpl<MachineInstr * > * DbgVals=nullptr)7424ba319b5SDimitry Andric static void performSink(MachineInstr &MI, MachineBasicBlock &SuccToSinkTo,
743*b5893f02SDimitry Andric MachineBasicBlock::iterator InsertPos,
744*b5893f02SDimitry Andric SmallVectorImpl<MachineInstr *> *DbgVals = nullptr) {
745*b5893f02SDimitry Andric // If debug values are provided use those, otherwise call collectDebugValues.
7464ba319b5SDimitry Andric SmallVector<MachineInstr *, 2> DbgValuesToSink;
747*b5893f02SDimitry Andric if (DbgVals)
748*b5893f02SDimitry Andric DbgValuesToSink.insert(DbgValuesToSink.begin(),
749*b5893f02SDimitry Andric DbgVals->begin(), DbgVals->end());
750*b5893f02SDimitry Andric else
751*b5893f02SDimitry Andric MI.collectDebugValues(DbgValuesToSink);
7524ba319b5SDimitry Andric
7534ba319b5SDimitry Andric // If we cannot find a location to use (merge with), then we erase the debug
7544ba319b5SDimitry Andric // location to prevent debug-info driven tools from potentially reporting
7554ba319b5SDimitry Andric // wrong location information.
7564ba319b5SDimitry Andric if (!SuccToSinkTo.empty() && InsertPos != SuccToSinkTo.end())
7574ba319b5SDimitry Andric MI.setDebugLoc(DILocation::getMergedLocation(MI.getDebugLoc(),
7584ba319b5SDimitry Andric InsertPos->getDebugLoc()));
7594ba319b5SDimitry Andric else
7604ba319b5SDimitry Andric MI.setDebugLoc(DebugLoc());
7614ba319b5SDimitry Andric
7624ba319b5SDimitry Andric // Move the instruction.
7634ba319b5SDimitry Andric MachineBasicBlock *ParentBlock = MI.getParent();
7644ba319b5SDimitry Andric SuccToSinkTo.splice(InsertPos, ParentBlock, MI,
7654ba319b5SDimitry Andric ++MachineBasicBlock::iterator(MI));
7664ba319b5SDimitry Andric
7674ba319b5SDimitry Andric // Move previously adjacent debug value instructions to the insert position.
7684ba319b5SDimitry Andric for (SmallVectorImpl<MachineInstr *>::iterator DBI = DbgValuesToSink.begin(),
7694ba319b5SDimitry Andric DBE = DbgValuesToSink.end();
7704ba319b5SDimitry Andric DBI != DBE; ++DBI) {
7714ba319b5SDimitry Andric MachineInstr *DbgMI = *DBI;
7724ba319b5SDimitry Andric SuccToSinkTo.splice(InsertPos, ParentBlock, DbgMI,
7734ba319b5SDimitry Andric ++MachineBasicBlock::iterator(DbgMI));
7744ba319b5SDimitry Andric }
7754ba319b5SDimitry Andric }
7764ba319b5SDimitry Andric
777dff0c46cSDimitry Andric /// SinkInstruction - Determine whether it is safe to sink the specified machine
778dff0c46cSDimitry Andric /// instruction out of its current block into a successor.
SinkInstruction(MachineInstr & MI,bool & SawStore,AllSuccsCache & AllSuccessors)7793ca95b02SDimitry Andric bool MachineSinking::SinkInstruction(MachineInstr &MI, bool &SawStore,
7808f0fd8f6SDimitry Andric AllSuccsCache &AllSuccessors) {
7813ca95b02SDimitry Andric // Don't sink instructions that the target prefers not to sink.
7823ca95b02SDimitry Andric if (!TII->shouldSink(MI))
783f22ef01cSRoman Divacky return false;
784dff0c46cSDimitry Andric
785dff0c46cSDimitry Andric // Check if it's safe to move the instruction.
7863ca95b02SDimitry Andric if (!MI.isSafeToMove(AA, SawStore))
787dff0c46cSDimitry Andric return false;
788dff0c46cSDimitry Andric
7897d523365SDimitry Andric // Convergent operations may not be made control-dependent on additional
7907d523365SDimitry Andric // values.
7913ca95b02SDimitry Andric if (MI.isConvergent())
7923ca95b02SDimitry Andric return false;
7933ca95b02SDimitry Andric
7943ca95b02SDimitry Andric // Don't break implicit null checks. This is a performance heuristic, and not
7953ca95b02SDimitry Andric // required for correctness.
7963ca95b02SDimitry Andric if (SinkingPreventsImplicitNullCheck(MI, TII, TRI))
79797bc6c73SDimitry Andric return false;
79897bc6c73SDimitry Andric
799dff0c46cSDimitry Andric // FIXME: This should include support for sinking instructions within the
800dff0c46cSDimitry Andric // block they are currently in to shorten the live ranges. We often get
801dff0c46cSDimitry Andric // instructions sunk into the top of a large block, but it would be better to
802dff0c46cSDimitry Andric // also sink them down before their first use in the block. This xform has to
803dff0c46cSDimitry Andric // be careful not to *increase* register pressure though, e.g. sinking
804dff0c46cSDimitry Andric // "x = y + z" down if it kills y and z would increase the live ranges of y
805dff0c46cSDimitry Andric // and z and only shrink the live range of x.
806dff0c46cSDimitry Andric
807dff0c46cSDimitry Andric bool BreakPHIEdge = false;
8083ca95b02SDimitry Andric MachineBasicBlock *ParentBlock = MI.getParent();
8098f0fd8f6SDimitry Andric MachineBasicBlock *SuccToSinkTo =
8108f0fd8f6SDimitry Andric FindSuccToSinkTo(MI, ParentBlock, BreakPHIEdge, AllSuccessors);
811f22ef01cSRoman Divacky
812f22ef01cSRoman Divacky // If there are no outputs, it must have side-effects.
81391bc56edSDimitry Andric if (!SuccToSinkTo)
814f22ef01cSRoman Divacky return false;
815f22ef01cSRoman Divacky
816ffd1746dSEd Schouten // If the instruction to move defines a dead physical register which is live
817ffd1746dSEd Schouten // when leaving the basic block, don't move it because it could turn into a
818ffd1746dSEd Schouten // "zombie" define of that preg. E.g., EFLAGS. (<rdar://problem/8030636>)
8193ca95b02SDimitry Andric for (unsigned I = 0, E = MI.getNumOperands(); I != E; ++I) {
8203ca95b02SDimitry Andric const MachineOperand &MO = MI.getOperand(I);
821ffd1746dSEd Schouten if (!MO.isReg()) continue;
822ffd1746dSEd Schouten unsigned Reg = MO.getReg();
823ffd1746dSEd Schouten if (Reg == 0 || !TargetRegisterInfo::isPhysicalRegister(Reg)) continue;
824ffd1746dSEd Schouten if (SuccToSinkTo->isLiveIn(Reg))
825ffd1746dSEd Schouten return false;
826ffd1746dSEd Schouten }
827ffd1746dSEd Schouten
8284ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << "Sink instr " << MI << "\tinto block " << *SuccToSinkTo);
829f22ef01cSRoman Divacky
830f785676fSDimitry Andric // If the block has multiple predecessors, this is a critical edge.
831f785676fSDimitry Andric // Decide if we can sink along it or need to break the edge.
832f22ef01cSRoman Divacky if (SuccToSinkTo->pred_size() > 1) {
833f22ef01cSRoman Divacky // We cannot sink a load across a critical edge - there may be stores in
834f22ef01cSRoman Divacky // other code paths.
835e580952dSDimitry Andric bool TryBreak = false;
836f22ef01cSRoman Divacky bool store = true;
8373ca95b02SDimitry Andric if (!MI.isSafeToMove(AA, store)) {
8384ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** NOTE: Won't sink load along critical edge.\n");
839e580952dSDimitry Andric TryBreak = true;
840f22ef01cSRoman Divacky }
841f22ef01cSRoman Divacky
842f22ef01cSRoman Divacky // We don't want to sink across a critical edge if we don't dominate the
843f22ef01cSRoman Divacky // successor. We could be introducing calculations to new code paths.
844e580952dSDimitry Andric if (!TryBreak && !DT->dominates(ParentBlock, SuccToSinkTo)) {
8454ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** NOTE: Critical edge found\n");
846e580952dSDimitry Andric TryBreak = true;
847f22ef01cSRoman Divacky }
848f22ef01cSRoman Divacky
849f22ef01cSRoman Divacky // Don't sink instructions into a loop.
850e580952dSDimitry Andric if (!TryBreak && LI->isLoopHeader(SuccToSinkTo)) {
8514ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** NOTE: Loop header found\n");
852e580952dSDimitry Andric TryBreak = true;
853f22ef01cSRoman Divacky }
854f22ef01cSRoman Divacky
855f22ef01cSRoman Divacky // Otherwise we are OK with sinking along a critical edge.
856e580952dSDimitry Andric if (!TryBreak)
8574ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << "Sinking along critical edge.\n");
858e580952dSDimitry Andric else {
85939d628a0SDimitry Andric // Mark this edge as to be split.
86039d628a0SDimitry Andric // If the edge can actually be split, the next iteration of the main loop
86139d628a0SDimitry Andric // will sink MI in the newly created block.
86239d628a0SDimitry Andric bool Status =
86339d628a0SDimitry Andric PostponeSplitCriticalEdge(MI, ParentBlock, SuccToSinkTo, BreakPHIEdge);
86439d628a0SDimitry Andric if (!Status)
8654ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** PUNTING: Not legal or profitable to "
8662754fe60SDimitry Andric "break critical edge\n");
86739d628a0SDimitry Andric // The instruction will not be sunk this time.
868e580952dSDimitry Andric return false;
869e580952dSDimitry Andric }
870f22ef01cSRoman Divacky }
871f22ef01cSRoman Divacky
8722754fe60SDimitry Andric if (BreakPHIEdge) {
8732754fe60SDimitry Andric // BreakPHIEdge is true if all the uses are in the successor MBB being
8742754fe60SDimitry Andric // sunken into and they are all PHI nodes. In this case, machine-sink must
8752754fe60SDimitry Andric // break the critical edge first.
87639d628a0SDimitry Andric bool Status = PostponeSplitCriticalEdge(MI, ParentBlock,
8772754fe60SDimitry Andric SuccToSinkTo, BreakPHIEdge);
87839d628a0SDimitry Andric if (!Status)
8794ba319b5SDimitry Andric LLVM_DEBUG(dbgs() << " *** PUNTING: Not legal or profitable to "
8802754fe60SDimitry Andric "break critical edge\n");
88139d628a0SDimitry Andric // The instruction will not be sunk this time.
8822754fe60SDimitry Andric return false;
8832754fe60SDimitry Andric }
8842754fe60SDimitry Andric
885f22ef01cSRoman Divacky // Determine where to insert into. Skip phi nodes.
886f22ef01cSRoman Divacky MachineBasicBlock::iterator InsertPos = SuccToSinkTo->begin();
887f22ef01cSRoman Divacky while (InsertPos != SuccToSinkTo->end() && InsertPos->isPHI())
888f22ef01cSRoman Divacky ++InsertPos;
889f22ef01cSRoman Divacky
8904ba319b5SDimitry Andric performSink(MI, *SuccToSinkTo, InsertPos);
8916122f3e6SDimitry Andric
892ffd1746dSEd Schouten // Conservatively, clear any kill flags, since it's possible that they are no
893ffd1746dSEd Schouten // longer correct.
894ff0cc061SDimitry Andric // Note that we have to clear the kill flags for any register this instruction
895ff0cc061SDimitry Andric // uses as we may sink over another instruction which currently kills the
896ff0cc061SDimitry Andric // used registers.
8973ca95b02SDimitry Andric for (MachineOperand &MO : MI.operands()) {
898ff0cc061SDimitry Andric if (MO.isReg() && MO.isUse())
899ff0cc061SDimitry Andric RegsToClearKillFlags.set(MO.getReg()); // Remember to clear kill flags.
900ff0cc061SDimitry Andric }
901f22ef01cSRoman Divacky
902f22ef01cSRoman Divacky return true;
903f22ef01cSRoman Divacky }
9044ba319b5SDimitry Andric
9054ba319b5SDimitry Andric //===----------------------------------------------------------------------===//
9064ba319b5SDimitry Andric // This pass is not intended to be a replacement or a complete alternative
9074ba319b5SDimitry Andric // for the pre-ra machine sink pass. It is only designed to sink COPY
9084ba319b5SDimitry Andric // instructions which should be handled after RA.
9094ba319b5SDimitry Andric //
9104ba319b5SDimitry Andric // This pass sinks COPY instructions into a successor block, if the COPY is not
9114ba319b5SDimitry Andric // used in the current block and the COPY is live-in to a single successor
9124ba319b5SDimitry Andric // (i.e., doesn't require the COPY to be duplicated). This avoids executing the
9134ba319b5SDimitry Andric // copy on paths where their results aren't needed. This also exposes
9144ba319b5SDimitry Andric // additional opportunites for dead copy elimination and shrink wrapping.
9154ba319b5SDimitry Andric //
9164ba319b5SDimitry Andric // These copies were either not handled by or are inserted after the MachineSink
9174ba319b5SDimitry Andric // pass. As an example of the former case, the MachineSink pass cannot sink
9184ba319b5SDimitry Andric // COPY instructions with allocatable source registers; for AArch64 these type
9194ba319b5SDimitry Andric // of copy instructions are frequently used to move function parameters (PhyReg)
9204ba319b5SDimitry Andric // into virtual registers in the entry block.
9214ba319b5SDimitry Andric //
9224ba319b5SDimitry Andric // For the machine IR below, this pass will sink %w19 in the entry into its
9234ba319b5SDimitry Andric // successor (%bb.1) because %w19 is only live-in in %bb.1.
9244ba319b5SDimitry Andric // %bb.0:
9254ba319b5SDimitry Andric // %wzr = SUBSWri %w1, 1
9264ba319b5SDimitry Andric // %w19 = COPY %w0
9274ba319b5SDimitry Andric // Bcc 11, %bb.2
9284ba319b5SDimitry Andric // %bb.1:
9294ba319b5SDimitry Andric // Live Ins: %w19
9304ba319b5SDimitry Andric // BL @fun
9314ba319b5SDimitry Andric // %w0 = ADDWrr %w0, %w19
9324ba319b5SDimitry Andric // RET %w0
9334ba319b5SDimitry Andric // %bb.2:
9344ba319b5SDimitry Andric // %w0 = COPY %wzr
9354ba319b5SDimitry Andric // RET %w0
9364ba319b5SDimitry Andric // As we sink %w19 (CSR in AArch64) into %bb.1, the shrink-wrapping pass will be
9374ba319b5SDimitry Andric // able to see %bb.0 as a candidate.
9384ba319b5SDimitry Andric //===----------------------------------------------------------------------===//
9394ba319b5SDimitry Andric namespace {
9404ba319b5SDimitry Andric
9414ba319b5SDimitry Andric class PostRAMachineSinking : public MachineFunctionPass {
9424ba319b5SDimitry Andric public:
9434ba319b5SDimitry Andric bool runOnMachineFunction(MachineFunction &MF) override;
9444ba319b5SDimitry Andric
9454ba319b5SDimitry Andric static char ID;
PostRAMachineSinking()9464ba319b5SDimitry Andric PostRAMachineSinking() : MachineFunctionPass(ID) {}
getPassName() const9474ba319b5SDimitry Andric StringRef getPassName() const override { return "PostRA Machine Sink"; }
9484ba319b5SDimitry Andric
getAnalysisUsage(AnalysisUsage & AU) const9494ba319b5SDimitry Andric void getAnalysisUsage(AnalysisUsage &AU) const override {
9504ba319b5SDimitry Andric AU.setPreservesCFG();
9514ba319b5SDimitry Andric MachineFunctionPass::getAnalysisUsage(AU);
9524ba319b5SDimitry Andric }
9534ba319b5SDimitry Andric
getRequiredProperties() const9544ba319b5SDimitry Andric MachineFunctionProperties getRequiredProperties() const override {
9554ba319b5SDimitry Andric return MachineFunctionProperties().set(
9564ba319b5SDimitry Andric MachineFunctionProperties::Property::NoVRegs);
9574ba319b5SDimitry Andric }
9584ba319b5SDimitry Andric
9594ba319b5SDimitry Andric private:
9604ba319b5SDimitry Andric /// Track which register units have been modified and used.
9614ba319b5SDimitry Andric LiveRegUnits ModifiedRegUnits, UsedRegUnits;
9624ba319b5SDimitry Andric
963*b5893f02SDimitry Andric /// Track DBG_VALUEs of (unmodified) register units.
964*b5893f02SDimitry Andric DenseMap<unsigned, TinyPtrVector<MachineInstr*>> SeenDbgInstrs;
965*b5893f02SDimitry Andric
9664ba319b5SDimitry Andric /// Sink Copy instructions unused in the same block close to their uses in
9674ba319b5SDimitry Andric /// successors.
9684ba319b5SDimitry Andric bool tryToSinkCopy(MachineBasicBlock &BB, MachineFunction &MF,
9694ba319b5SDimitry Andric const TargetRegisterInfo *TRI, const TargetInstrInfo *TII);
9704ba319b5SDimitry Andric };
9714ba319b5SDimitry Andric } // namespace
9724ba319b5SDimitry Andric
9734ba319b5SDimitry Andric char PostRAMachineSinking::ID = 0;
9744ba319b5SDimitry Andric char &llvm::PostRAMachineSinkingID = PostRAMachineSinking::ID;
9754ba319b5SDimitry Andric
9764ba319b5SDimitry Andric INITIALIZE_PASS(PostRAMachineSinking, "postra-machine-sink",
9774ba319b5SDimitry Andric "PostRA Machine Sink", false, false)
9784ba319b5SDimitry Andric
aliasWithRegsInLiveIn(MachineBasicBlock & MBB,unsigned Reg,const TargetRegisterInfo * TRI)9794ba319b5SDimitry Andric static bool aliasWithRegsInLiveIn(MachineBasicBlock &MBB, unsigned Reg,
9804ba319b5SDimitry Andric const TargetRegisterInfo *TRI) {
9814ba319b5SDimitry Andric LiveRegUnits LiveInRegUnits(*TRI);
9824ba319b5SDimitry Andric LiveInRegUnits.addLiveIns(MBB);
9834ba319b5SDimitry Andric return !LiveInRegUnits.available(Reg);
9844ba319b5SDimitry Andric }
9854ba319b5SDimitry Andric
9864ba319b5SDimitry Andric static MachineBasicBlock *
getSingleLiveInSuccBB(MachineBasicBlock & CurBB,const SmallPtrSetImpl<MachineBasicBlock * > & SinkableBBs,unsigned Reg,const TargetRegisterInfo * TRI)9874ba319b5SDimitry Andric getSingleLiveInSuccBB(MachineBasicBlock &CurBB,
9884ba319b5SDimitry Andric const SmallPtrSetImpl<MachineBasicBlock *> &SinkableBBs,
9894ba319b5SDimitry Andric unsigned Reg, const TargetRegisterInfo *TRI) {
9904ba319b5SDimitry Andric // Try to find a single sinkable successor in which Reg is live-in.
9914ba319b5SDimitry Andric MachineBasicBlock *BB = nullptr;
9924ba319b5SDimitry Andric for (auto *SI : SinkableBBs) {
9934ba319b5SDimitry Andric if (aliasWithRegsInLiveIn(*SI, Reg, TRI)) {
9944ba319b5SDimitry Andric // If BB is set here, Reg is live-in to at least two sinkable successors,
9954ba319b5SDimitry Andric // so quit.
9964ba319b5SDimitry Andric if (BB)
9974ba319b5SDimitry Andric return nullptr;
9984ba319b5SDimitry Andric BB = SI;
9994ba319b5SDimitry Andric }
10004ba319b5SDimitry Andric }
10014ba319b5SDimitry Andric // Reg is not live-in to any sinkable successors.
10024ba319b5SDimitry Andric if (!BB)
10034ba319b5SDimitry Andric return nullptr;
10044ba319b5SDimitry Andric
10054ba319b5SDimitry Andric // Check if any register aliased with Reg is live-in in other successors.
10064ba319b5SDimitry Andric for (auto *SI : CurBB.successors()) {
10074ba319b5SDimitry Andric if (!SinkableBBs.count(SI) && aliasWithRegsInLiveIn(*SI, Reg, TRI))
10084ba319b5SDimitry Andric return nullptr;
10094ba319b5SDimitry Andric }
10104ba319b5SDimitry Andric return BB;
10114ba319b5SDimitry Andric }
10124ba319b5SDimitry Andric
10134ba319b5SDimitry Andric static MachineBasicBlock *
getSingleLiveInSuccBB(MachineBasicBlock & CurBB,const SmallPtrSetImpl<MachineBasicBlock * > & SinkableBBs,ArrayRef<unsigned> DefedRegsInCopy,const TargetRegisterInfo * TRI)10144ba319b5SDimitry Andric getSingleLiveInSuccBB(MachineBasicBlock &CurBB,
10154ba319b5SDimitry Andric const SmallPtrSetImpl<MachineBasicBlock *> &SinkableBBs,
10164ba319b5SDimitry Andric ArrayRef<unsigned> DefedRegsInCopy,
10174ba319b5SDimitry Andric const TargetRegisterInfo *TRI) {
10184ba319b5SDimitry Andric MachineBasicBlock *SingleBB = nullptr;
10194ba319b5SDimitry Andric for (auto DefReg : DefedRegsInCopy) {
10204ba319b5SDimitry Andric MachineBasicBlock *BB =
10214ba319b5SDimitry Andric getSingleLiveInSuccBB(CurBB, SinkableBBs, DefReg, TRI);
10224ba319b5SDimitry Andric if (!BB || (SingleBB && SingleBB != BB))
10234ba319b5SDimitry Andric return nullptr;
10244ba319b5SDimitry Andric SingleBB = BB;
10254ba319b5SDimitry Andric }
10264ba319b5SDimitry Andric return SingleBB;
10274ba319b5SDimitry Andric }
10284ba319b5SDimitry Andric
clearKillFlags(MachineInstr * MI,MachineBasicBlock & CurBB,SmallVectorImpl<unsigned> & UsedOpsInCopy,LiveRegUnits & UsedRegUnits,const TargetRegisterInfo * TRI)10294ba319b5SDimitry Andric static void clearKillFlags(MachineInstr *MI, MachineBasicBlock &CurBB,
10304ba319b5SDimitry Andric SmallVectorImpl<unsigned> &UsedOpsInCopy,
10314ba319b5SDimitry Andric LiveRegUnits &UsedRegUnits,
10324ba319b5SDimitry Andric const TargetRegisterInfo *TRI) {
10334ba319b5SDimitry Andric for (auto U : UsedOpsInCopy) {
10344ba319b5SDimitry Andric MachineOperand &MO = MI->getOperand(U);
10354ba319b5SDimitry Andric unsigned SrcReg = MO.getReg();
10364ba319b5SDimitry Andric if (!UsedRegUnits.available(SrcReg)) {
10374ba319b5SDimitry Andric MachineBasicBlock::iterator NI = std::next(MI->getIterator());
10384ba319b5SDimitry Andric for (MachineInstr &UI : make_range(NI, CurBB.end())) {
10394ba319b5SDimitry Andric if (UI.killsRegister(SrcReg, TRI)) {
10404ba319b5SDimitry Andric UI.clearRegisterKills(SrcReg, TRI);
10414ba319b5SDimitry Andric MO.setIsKill(true);
10424ba319b5SDimitry Andric break;
10434ba319b5SDimitry Andric }
10444ba319b5SDimitry Andric }
10454ba319b5SDimitry Andric }
10464ba319b5SDimitry Andric }
10474ba319b5SDimitry Andric }
10484ba319b5SDimitry Andric
updateLiveIn(MachineInstr * MI,MachineBasicBlock * SuccBB,SmallVectorImpl<unsigned> & UsedOpsInCopy,SmallVectorImpl<unsigned> & DefedRegsInCopy)10494ba319b5SDimitry Andric static void updateLiveIn(MachineInstr *MI, MachineBasicBlock *SuccBB,
10504ba319b5SDimitry Andric SmallVectorImpl<unsigned> &UsedOpsInCopy,
10514ba319b5SDimitry Andric SmallVectorImpl<unsigned> &DefedRegsInCopy) {
1052*b5893f02SDimitry Andric MachineFunction &MF = *SuccBB->getParent();
1053*b5893f02SDimitry Andric const TargetRegisterInfo *TRI = MF.getSubtarget().getRegisterInfo();
1054*b5893f02SDimitry Andric for (unsigned DefReg : DefedRegsInCopy)
1055*b5893f02SDimitry Andric for (MCSubRegIterator S(DefReg, TRI, true); S.isValid(); ++S)
1056*b5893f02SDimitry Andric SuccBB->removeLiveIn(*S);
10574ba319b5SDimitry Andric for (auto U : UsedOpsInCopy) {
10584ba319b5SDimitry Andric unsigned Reg = MI->getOperand(U).getReg();
10594ba319b5SDimitry Andric if (!SuccBB->isLiveIn(Reg))
10604ba319b5SDimitry Andric SuccBB->addLiveIn(Reg);
10614ba319b5SDimitry Andric }
10624ba319b5SDimitry Andric }
10634ba319b5SDimitry Andric
hasRegisterDependency(MachineInstr * MI,SmallVectorImpl<unsigned> & UsedOpsInCopy,SmallVectorImpl<unsigned> & DefedRegsInCopy,LiveRegUnits & ModifiedRegUnits,LiveRegUnits & UsedRegUnits)10644ba319b5SDimitry Andric static bool hasRegisterDependency(MachineInstr *MI,
10654ba319b5SDimitry Andric SmallVectorImpl<unsigned> &UsedOpsInCopy,
10664ba319b5SDimitry Andric SmallVectorImpl<unsigned> &DefedRegsInCopy,
10674ba319b5SDimitry Andric LiveRegUnits &ModifiedRegUnits,
10684ba319b5SDimitry Andric LiveRegUnits &UsedRegUnits) {
10694ba319b5SDimitry Andric bool HasRegDependency = false;
10704ba319b5SDimitry Andric for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) {
10714ba319b5SDimitry Andric MachineOperand &MO = MI->getOperand(i);
10724ba319b5SDimitry Andric if (!MO.isReg())
10734ba319b5SDimitry Andric continue;
10744ba319b5SDimitry Andric unsigned Reg = MO.getReg();
10754ba319b5SDimitry Andric if (!Reg)
10764ba319b5SDimitry Andric continue;
10774ba319b5SDimitry Andric if (MO.isDef()) {
10784ba319b5SDimitry Andric if (!ModifiedRegUnits.available(Reg) || !UsedRegUnits.available(Reg)) {
10794ba319b5SDimitry Andric HasRegDependency = true;
10804ba319b5SDimitry Andric break;
10814ba319b5SDimitry Andric }
10824ba319b5SDimitry Andric DefedRegsInCopy.push_back(Reg);
10834ba319b5SDimitry Andric
10844ba319b5SDimitry Andric // FIXME: instead of isUse(), readsReg() would be a better fix here,
10854ba319b5SDimitry Andric // For example, we can ignore modifications in reg with undef. However,
10864ba319b5SDimitry Andric // it's not perfectly clear if skipping the internal read is safe in all
10874ba319b5SDimitry Andric // other targets.
10884ba319b5SDimitry Andric } else if (MO.isUse()) {
10894ba319b5SDimitry Andric if (!ModifiedRegUnits.available(Reg)) {
10904ba319b5SDimitry Andric HasRegDependency = true;
10914ba319b5SDimitry Andric break;
10924ba319b5SDimitry Andric }
10934ba319b5SDimitry Andric UsedOpsInCopy.push_back(i);
10944ba319b5SDimitry Andric }
10954ba319b5SDimitry Andric }
10964ba319b5SDimitry Andric return HasRegDependency;
10974ba319b5SDimitry Andric }
10984ba319b5SDimitry Andric
tryToSinkCopy(MachineBasicBlock & CurBB,MachineFunction & MF,const TargetRegisterInfo * TRI,const TargetInstrInfo * TII)10994ba319b5SDimitry Andric bool PostRAMachineSinking::tryToSinkCopy(MachineBasicBlock &CurBB,
11004ba319b5SDimitry Andric MachineFunction &MF,
11014ba319b5SDimitry Andric const TargetRegisterInfo *TRI,
11024ba319b5SDimitry Andric const TargetInstrInfo *TII) {
11034ba319b5SDimitry Andric SmallPtrSet<MachineBasicBlock *, 2> SinkableBBs;
11044ba319b5SDimitry Andric // FIXME: For now, we sink only to a successor which has a single predecessor
11054ba319b5SDimitry Andric // so that we can directly sink COPY instructions to the successor without
11064ba319b5SDimitry Andric // adding any new block or branch instruction.
11074ba319b5SDimitry Andric for (MachineBasicBlock *SI : CurBB.successors())
11084ba319b5SDimitry Andric if (!SI->livein_empty() && SI->pred_size() == 1)
11094ba319b5SDimitry Andric SinkableBBs.insert(SI);
11104ba319b5SDimitry Andric
11114ba319b5SDimitry Andric if (SinkableBBs.empty())
11124ba319b5SDimitry Andric return false;
11134ba319b5SDimitry Andric
11144ba319b5SDimitry Andric bool Changed = false;
11154ba319b5SDimitry Andric
11164ba319b5SDimitry Andric // Track which registers have been modified and used between the end of the
11174ba319b5SDimitry Andric // block and the current instruction.
11184ba319b5SDimitry Andric ModifiedRegUnits.clear();
11194ba319b5SDimitry Andric UsedRegUnits.clear();
1120*b5893f02SDimitry Andric SeenDbgInstrs.clear();
11214ba319b5SDimitry Andric
11224ba319b5SDimitry Andric for (auto I = CurBB.rbegin(), E = CurBB.rend(); I != E;) {
11234ba319b5SDimitry Andric MachineInstr *MI = &*I;
11244ba319b5SDimitry Andric ++I;
11254ba319b5SDimitry Andric
1126*b5893f02SDimitry Andric // Track the operand index for use in Copy.
1127*b5893f02SDimitry Andric SmallVector<unsigned, 2> UsedOpsInCopy;
1128*b5893f02SDimitry Andric // Track the register number defed in Copy.
1129*b5893f02SDimitry Andric SmallVector<unsigned, 2> DefedRegsInCopy;
1130*b5893f02SDimitry Andric
1131*b5893f02SDimitry Andric // We must sink this DBG_VALUE if its operand is sunk. To avoid searching
1132*b5893f02SDimitry Andric // for DBG_VALUEs later, record them when they're encountered.
1133*b5893f02SDimitry Andric if (MI->isDebugValue()) {
1134*b5893f02SDimitry Andric auto &MO = MI->getOperand(0);
1135*b5893f02SDimitry Andric if (MO.isReg() && TRI->isPhysicalRegister(MO.getReg())) {
1136*b5893f02SDimitry Andric // Bail if we can already tell the sink would be rejected, rather
1137*b5893f02SDimitry Andric // than needlessly accumulating lots of DBG_VALUEs.
1138*b5893f02SDimitry Andric if (hasRegisterDependency(MI, UsedOpsInCopy, DefedRegsInCopy,
1139*b5893f02SDimitry Andric ModifiedRegUnits, UsedRegUnits))
1140*b5893f02SDimitry Andric continue;
1141*b5893f02SDimitry Andric
1142*b5893f02SDimitry Andric // Record debug use of this register.
1143*b5893f02SDimitry Andric SeenDbgInstrs[MO.getReg()].push_back(MI);
1144*b5893f02SDimitry Andric }
1145*b5893f02SDimitry Andric continue;
1146*b5893f02SDimitry Andric }
1147*b5893f02SDimitry Andric
11484ba319b5SDimitry Andric if (MI->isDebugInstr())
11494ba319b5SDimitry Andric continue;
11504ba319b5SDimitry Andric
11514ba319b5SDimitry Andric // Do not move any instruction across function call.
11524ba319b5SDimitry Andric if (MI->isCall())
11534ba319b5SDimitry Andric return false;
11544ba319b5SDimitry Andric
11554ba319b5SDimitry Andric if (!MI->isCopy() || !MI->getOperand(0).isRenamable()) {
11564ba319b5SDimitry Andric LiveRegUnits::accumulateUsedDefed(*MI, ModifiedRegUnits, UsedRegUnits,
11574ba319b5SDimitry Andric TRI);
11584ba319b5SDimitry Andric continue;
11594ba319b5SDimitry Andric }
11604ba319b5SDimitry Andric
11614ba319b5SDimitry Andric // Don't sink the COPY if it would violate a register dependency.
11624ba319b5SDimitry Andric if (hasRegisterDependency(MI, UsedOpsInCopy, DefedRegsInCopy,
11634ba319b5SDimitry Andric ModifiedRegUnits, UsedRegUnits)) {
11644ba319b5SDimitry Andric LiveRegUnits::accumulateUsedDefed(*MI, ModifiedRegUnits, UsedRegUnits,
11654ba319b5SDimitry Andric TRI);
11664ba319b5SDimitry Andric continue;
11674ba319b5SDimitry Andric }
11684ba319b5SDimitry Andric assert((!UsedOpsInCopy.empty() && !DefedRegsInCopy.empty()) &&
11694ba319b5SDimitry Andric "Unexpect SrcReg or DefReg");
11704ba319b5SDimitry Andric MachineBasicBlock *SuccBB =
11714ba319b5SDimitry Andric getSingleLiveInSuccBB(CurBB, SinkableBBs, DefedRegsInCopy, TRI);
11724ba319b5SDimitry Andric // Don't sink if we cannot find a single sinkable successor in which Reg
11734ba319b5SDimitry Andric // is live-in.
11744ba319b5SDimitry Andric if (!SuccBB) {
11754ba319b5SDimitry Andric LiveRegUnits::accumulateUsedDefed(*MI, ModifiedRegUnits, UsedRegUnits,
11764ba319b5SDimitry Andric TRI);
11774ba319b5SDimitry Andric continue;
11784ba319b5SDimitry Andric }
11794ba319b5SDimitry Andric assert((SuccBB->pred_size() == 1 && *SuccBB->pred_begin() == &CurBB) &&
11804ba319b5SDimitry Andric "Unexpected predecessor");
11814ba319b5SDimitry Andric
1182*b5893f02SDimitry Andric // Collect DBG_VALUEs that must sink with this copy.
1183*b5893f02SDimitry Andric SmallVector<MachineInstr *, 4> DbgValsToSink;
1184*b5893f02SDimitry Andric for (auto &MO : MI->operands()) {
1185*b5893f02SDimitry Andric if (!MO.isReg() || !MO.isDef())
1186*b5893f02SDimitry Andric continue;
1187*b5893f02SDimitry Andric unsigned reg = MO.getReg();
1188*b5893f02SDimitry Andric for (auto *MI : SeenDbgInstrs.lookup(reg))
1189*b5893f02SDimitry Andric DbgValsToSink.push_back(MI);
1190*b5893f02SDimitry Andric }
1191*b5893f02SDimitry Andric
11924ba319b5SDimitry Andric // Clear the kill flag if SrcReg is killed between MI and the end of the
11934ba319b5SDimitry Andric // block.
11944ba319b5SDimitry Andric clearKillFlags(MI, CurBB, UsedOpsInCopy, UsedRegUnits, TRI);
11954ba319b5SDimitry Andric MachineBasicBlock::iterator InsertPos = SuccBB->getFirstNonPHI();
1196*b5893f02SDimitry Andric performSink(*MI, *SuccBB, InsertPos, &DbgValsToSink);
11974ba319b5SDimitry Andric updateLiveIn(MI, SuccBB, UsedOpsInCopy, DefedRegsInCopy);
11984ba319b5SDimitry Andric
11994ba319b5SDimitry Andric Changed = true;
12004ba319b5SDimitry Andric ++NumPostRACopySink;
12014ba319b5SDimitry Andric }
12024ba319b5SDimitry Andric return Changed;
12034ba319b5SDimitry Andric }
12044ba319b5SDimitry Andric
runOnMachineFunction(MachineFunction & MF)12054ba319b5SDimitry Andric bool PostRAMachineSinking::runOnMachineFunction(MachineFunction &MF) {
12064ba319b5SDimitry Andric bool Changed = false;
12074ba319b5SDimitry Andric const TargetRegisterInfo *TRI = MF.getSubtarget().getRegisterInfo();
12084ba319b5SDimitry Andric const TargetInstrInfo *TII = MF.getSubtarget().getInstrInfo();
12094ba319b5SDimitry Andric
12104ba319b5SDimitry Andric ModifiedRegUnits.init(*TRI);
12114ba319b5SDimitry Andric UsedRegUnits.init(*TRI);
12124ba319b5SDimitry Andric for (auto &BB : MF)
12134ba319b5SDimitry Andric Changed |= tryToSinkCopy(BB, MF, TRI, TII);
12144ba319b5SDimitry Andric
12154ba319b5SDimitry Andric return Changed;
12164ba319b5SDimitry Andric }
1217