Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 1 | //===- PhiElimination.cpp - Eliminate PHI nodes by inserting copies -------===// |
Misha Brukman | 835702a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 2 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
Chris Lattner | f3ebc3f | 2007-12-29 20:36:04 +0000 | [diff] [blame] | 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
Misha Brukman | 835702a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 7 | // |
John Criswell | 482202a | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 9 | // |
| 10 | // This pass eliminates machine instruction PHI nodes by inserting copy |
| 11 | // instructions. This destroys SSA information, but is the desired input for |
| 12 | // some register allocators. |
| 13 | // |
| 14 | //===----------------------------------------------------------------------===// |
| 15 | |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 16 | #include "PHIEliminationUtils.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 17 | #include "llvm/ADT/DenseMap.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 18 | #include "llvm/ADT/SmallPtrSet.h" |
| 19 | #include "llvm/ADT/Statistic.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 20 | #include "llvm/Analysis/LoopInfo.h" |
| 21 | #include "llvm/CodeGen/LiveInterval.h" |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/LiveIntervalAnalysis.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 23 | #include "llvm/CodeGen/LiveVariables.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 24 | #include "llvm/CodeGen/MachineBasicBlock.h" |
Jakob Stoklund Olesen | 15ca009 | 2009-11-14 00:38:06 +0000 | [diff] [blame] | 25 | #include "llvm/CodeGen/MachineDominators.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 26 | #include "llvm/CodeGen/MachineFunction.h" |
| 27 | #include "llvm/CodeGen/MachineFunctionPass.h" |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 28 | #include "llvm/CodeGen/MachineInstr.h" |
Evan Cheng | 3328186 | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 29 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
Evan Cheng | f259efd | 2010-08-17 01:20:36 +0000 | [diff] [blame] | 30 | #include "llvm/CodeGen/MachineLoopInfo.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 31 | #include "llvm/CodeGen/MachineOperand.h" |
Chris Lattner | a10fff5 | 2007-12-31 04:13:23 +0000 | [diff] [blame] | 32 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 33 | #include "llvm/CodeGen/SlotIndexes.h" |
| 34 | #include "llvm/Pass.h" |
Cameron Zwarich | 7930407 | 2011-03-10 05:59:17 +0000 | [diff] [blame] | 35 | #include "llvm/Support/CommandLine.h" |
Jakob Stoklund Olesen | 4453dc9 | 2009-11-10 22:01:05 +0000 | [diff] [blame] | 36 | #include "llvm/Support/Debug.h" |
Benjamin Kramer | 799003b | 2015-03-23 19:32:43 +0000 | [diff] [blame] | 37 | #include "llvm/Support/raw_ostream.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 38 | #include "llvm/Target/TargetInstrInfo.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 39 | #include "llvm/Target/TargetOpcodes.h" |
| 40 | #include "llvm/Target/TargetRegisterInfo.h" |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 41 | #include "llvm/Target/TargetSubtargetInfo.h" |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 42 | #include <cassert> |
| 43 | #include <iterator> |
| 44 | #include <utility> |
| 45 | |
Chris Lattner | 43df6c2 | 2004-02-23 18:38:20 +0000 | [diff] [blame] | 46 | using namespace llvm; |
Brian Gaeke | 960707c | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 47 | |
Davide Italiano | 1e8c59a | 2017-05-10 23:13:26 +0000 | [diff] [blame] | 48 | #define DEBUG_TYPE "phi-node-elimination" |
Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 49 | |
Cameron Zwarich | 7930407 | 2011-03-10 05:59:17 +0000 | [diff] [blame] | 50 | static cl::opt<bool> |
| 51 | DisableEdgeSplitting("disable-phi-elim-edge-splitting", cl::init(false), |
| 52 | cl::Hidden, cl::desc("Disable critical edge splitting " |
| 53 | "during PHI elimination")); |
| 54 | |
Cameron Zwarich | 15eb925 | 2013-02-12 03:49:25 +0000 | [diff] [blame] | 55 | static cl::opt<bool> |
| 56 | SplitAllCriticalEdges("phi-elim-split-all-critical-edges", cl::init(false), |
| 57 | cl::Hidden, cl::desc("Split all critical edges during " |
| 58 | "PHI elimination")); |
| 59 | |
Daniel Jasper | 8f239f8 | 2015-03-03 10:23:11 +0000 | [diff] [blame] | 60 | static cl::opt<bool> NoPhiElimLiveOutEarlyExit( |
| 61 | "no-phi-elim-live-out-early-exit", cl::init(false), cl::Hidden, |
| 62 | cl::desc("Do not use an early exit if isLiveOutPastPHIs returns true.")); |
| 63 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 64 | namespace { |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 65 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 66 | class PHIElimination : public MachineFunctionPass { |
| 67 | MachineRegisterInfo *MRI; // Machine register information |
Cameron Zwarich | e096673 | 2013-02-10 06:42:30 +0000 | [diff] [blame] | 68 | LiveVariables *LV; |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 69 | LiveIntervals *LIS; |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 70 | |
| 71 | public: |
| 72 | static char ID; // Pass identification, replacement for typeid |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 73 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 74 | PHIElimination() : MachineFunctionPass(ID) { |
| 75 | initializePHIEliminationPass(*PassRegistry::getPassRegistry()); |
| 76 | } |
| 77 | |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 78 | bool runOnMachineFunction(MachineFunction &Fn) override; |
| 79 | void getAnalysisUsage(AnalysisUsage &AU) const override; |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 80 | |
| 81 | private: |
| 82 | /// EliminatePHINodes - Eliminate phi nodes by inserting copy instructions |
| 83 | /// in predecessor basic blocks. |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 84 | bool EliminatePHINodes(MachineFunction &MF, MachineBasicBlock &MBB); |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 85 | |
Cameron Zwarich | a158d39 | 2013-02-10 06:42:32 +0000 | [diff] [blame] | 86 | void LowerPHINode(MachineBasicBlock &MBB, |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 87 | MachineBasicBlock::iterator LastPHIIt); |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 88 | |
| 89 | /// analyzePHINodes - Gather information about the PHI nodes in |
| 90 | /// here. In particular, we want to map the number of uses of a virtual |
| 91 | /// register which is used in a PHI node. We map that to the BB the |
| 92 | /// vreg is coming from. This is used later to determine when the vreg |
| 93 | /// is killed in the BB. |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 94 | void analyzePHINodes(const MachineFunction& Fn); |
| 95 | |
| 96 | /// Split critical edges where necessary for good coalescer performance. |
| 97 | bool SplitPHIEdges(MachineFunction &MF, MachineBasicBlock &MBB, |
Cameron Zwarich | e096673 | 2013-02-10 06:42:30 +0000 | [diff] [blame] | 98 | MachineLoopInfo *MLI); |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 99 | |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 100 | // These functions are temporary abstractions around LiveVariables and |
| 101 | // LiveIntervals, so they can go away when LiveVariables does. |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 102 | bool isLiveIn(unsigned Reg, const MachineBasicBlock *MBB); |
| 103 | bool isLiveOutPastPHIs(unsigned Reg, const MachineBasicBlock *MBB); |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 104 | |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 105 | using BBVRegPair = std::pair<unsigned, unsigned>; |
| 106 | using VRegPHIUse = DenseMap<BBVRegPair, unsigned>; |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 107 | |
| 108 | VRegPHIUse VRegPHIUseCount; |
| 109 | |
| 110 | // Defs of PHI sources which are implicit_def. |
| 111 | SmallPtrSet<MachineInstr*, 4> ImpDefs; |
| 112 | |
| 113 | // Map reusable lowered PHI node -> incoming join register. |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 114 | using LoweredPHIMap = |
| 115 | DenseMap<MachineInstr*, unsigned, MachineInstrExpressionTrait>; |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 116 | LoweredPHIMap LoweredPHIs; |
| 117 | }; |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 118 | |
| 119 | } // end anonymous namespace |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 120 | |
Cameron Zwarich | a158d39 | 2013-02-10 06:42:32 +0000 | [diff] [blame] | 121 | STATISTIC(NumLowered, "Number of phis lowered"); |
Cameron Zwarich | 8790396 | 2011-02-14 02:09:11 +0000 | [diff] [blame] | 122 | STATISTIC(NumCriticalEdgesSplit, "Number of critical edges split"); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 123 | STATISTIC(NumReused, "Number of reused lowered phis"); |
Jakob Stoklund Olesen | 4453dc9 | 2009-11-10 22:01:05 +0000 | [diff] [blame] | 124 | |
Lang Hames | aa03775 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 125 | char PHIElimination::ID = 0; |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 126 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 127 | char& llvm::PHIEliminationID = PHIElimination::ID; |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 128 | |
Matthias Braun | 1527baa | 2017-05-25 21:26:32 +0000 | [diff] [blame] | 129 | INITIALIZE_PASS_BEGIN(PHIElimination, DEBUG_TYPE, |
Andrew Trick | d3f8fe8 | 2012-02-10 04:10:36 +0000 | [diff] [blame] | 130 | "Eliminate PHI nodes for register allocation", |
| 131 | false, false) |
| 132 | INITIALIZE_PASS_DEPENDENCY(LiveVariables) |
Matthias Braun | 1527baa | 2017-05-25 21:26:32 +0000 | [diff] [blame] | 133 | INITIALIZE_PASS_END(PHIElimination, DEBUG_TYPE, |
Andrew Trick | d3f8fe8 | 2012-02-10 04:10:36 +0000 | [diff] [blame] | 134 | "Eliminate PHI nodes for register allocation", false, false) |
| 135 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 136 | void PHIElimination::getAnalysisUsage(AnalysisUsage &AU) const { |
Matthias Braun | f84547c | 2016-04-28 23:42:51 +0000 | [diff] [blame] | 137 | AU.addUsedIfAvailable<LiveVariables>(); |
Dan Gohman | 0402315 | 2009-07-31 23:37:33 +0000 | [diff] [blame] | 138 | AU.addPreserved<LiveVariables>(); |
Cameron Zwarich | 37ca2e8 | 2013-02-20 06:46:28 +0000 | [diff] [blame] | 139 | AU.addPreserved<SlotIndexes>(); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 140 | AU.addPreserved<LiveIntervals>(); |
Jakob Stoklund Olesen | 15ca009 | 2009-11-14 00:38:06 +0000 | [diff] [blame] | 141 | AU.addPreserved<MachineDominatorTree>(); |
Evan Cheng | 16bfe5b | 2010-08-17 21:00:37 +0000 | [diff] [blame] | 142 | AU.addPreserved<MachineLoopInfo>(); |
Dan Gohman | 0402315 | 2009-07-31 23:37:33 +0000 | [diff] [blame] | 143 | MachineFunctionPass::getAnalysisUsage(AU); |
| 144 | } |
Lang Hames | aa03775 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 145 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 146 | bool PHIElimination::runOnMachineFunction(MachineFunction &MF) { |
Evan Cheng | a5c0cc3 | 2010-05-04 17:12:26 +0000 | [diff] [blame] | 147 | MRI = &MF.getRegInfo(); |
Cameron Zwarich | e096673 | 2013-02-10 06:42:30 +0000 | [diff] [blame] | 148 | LV = getAnalysisIfAvailable<LiveVariables>(); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 149 | LIS = getAnalysisIfAvailable<LiveIntervals>(); |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 150 | |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 151 | bool Changed = false; |
| 152 | |
Jakob Stoklund Olesen | 9760f04 | 2011-07-29 22:51:22 +0000 | [diff] [blame] | 153 | // This pass takes the function out of SSA form. |
| 154 | MRI->leaveSSA(); |
| 155 | |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 156 | // Split critical edges to help the coalescer. This does not yet support |
| 157 | // updating LiveIntervals, so we disable it. |
Cameron Zwarich | b47fb38 | 2013-02-11 09:24:47 +0000 | [diff] [blame] | 158 | if (!DisableEdgeSplitting && (LV || LIS)) { |
Cameron Zwarich | e096673 | 2013-02-10 06:42:30 +0000 | [diff] [blame] | 159 | MachineLoopInfo *MLI = getAnalysisIfAvailable<MachineLoopInfo>(); |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 160 | for (auto &MBB : MF) |
| 161 | Changed |= SplitPHIEdges(MF, MBB, MLI); |
Evan Cheng | 16bfe5b | 2010-08-17 21:00:37 +0000 | [diff] [blame] | 162 | } |
Jakob Stoklund Olesen | 4f7fd3b | 2009-11-11 19:31:31 +0000 | [diff] [blame] | 163 | |
| 164 | // Populate VRegPHIUseCount |
Evan Cheng | a5c0cc3 | 2010-05-04 17:12:26 +0000 | [diff] [blame] | 165 | analyzePHINodes(MF); |
Jakob Stoklund Olesen | 4f7fd3b | 2009-11-11 19:31:31 +0000 | [diff] [blame] | 166 | |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 167 | // Eliminate PHI instructions by inserting copies into predecessor blocks. |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 168 | for (auto &MBB : MF) |
| 169 | Changed |= EliminatePHINodes(MF, MBB); |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 170 | |
| 171 | // Remove dead IMPLICIT_DEF instructions. |
Craig Topper | 4627679 | 2014-08-24 23:23:06 +0000 | [diff] [blame] | 172 | for (MachineInstr *DefMI : ImpDefs) { |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 173 | unsigned DefReg = DefMI->getOperand(0).getReg(); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 174 | if (MRI->use_nodbg_empty(DefReg)) { |
| 175 | if (LIS) |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 176 | LIS->RemoveMachineInstrFromMaps(*DefMI); |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 177 | DefMI->eraseFromParent(); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 178 | } |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 179 | } |
| 180 | |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 181 | // Clean up the lowered PHI instructions. |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 182 | for (auto &I : LoweredPHIs) { |
Cameron Zwarich | 4ee9aef | 2013-02-12 05:48:56 +0000 | [diff] [blame] | 183 | if (LIS) |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 184 | LIS->RemoveMachineInstrFromMaps(*I.first); |
| 185 | MF.DeleteMachineInstr(I.first); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 186 | } |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 187 | |
Bill Wendling | 819c356 | 2009-12-17 23:42:32 +0000 | [diff] [blame] | 188 | LoweredPHIs.clear(); |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 189 | ImpDefs.clear(); |
| 190 | VRegPHIUseCount.clear(); |
Evan Cheng | a5c0cc3 | 2010-05-04 17:12:26 +0000 | [diff] [blame] | 191 | |
Matthias Braun | 90799ce | 2016-08-23 21:19:49 +0000 | [diff] [blame] | 192 | MF.getProperties().set(MachineFunctionProperties::Property::NoPHIs); |
| 193 | |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 194 | return Changed; |
| 195 | } |
| 196 | |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 197 | /// EliminatePHINodes - Eliminate phi nodes by inserting copy instructions in |
| 198 | /// predecessor basic blocks. |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 199 | bool PHIElimination::EliminatePHINodes(MachineFunction &MF, |
Lang Hames | aa03775 | 2009-07-21 23:47:33 +0000 | [diff] [blame] | 200 | MachineBasicBlock &MBB) { |
Chris Lattner | b06015a | 2010-02-09 19:54:29 +0000 | [diff] [blame] | 201 | if (MBB.empty() || !MBB.front().isPHI()) |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 202 | return false; // Quick exit for basic blocks without PHIs. |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 203 | |
Chris Lattner | a2f7b9b | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 204 | // Get an iterator to the first instruction after the last PHI node (this may |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 205 | // also be the end of the basic block). |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 206 | MachineBasicBlock::iterator LastPHIIt = |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 207 | std::prev(MBB.SkipPHIsAndLabels(MBB.begin())); |
Chris Lattner | a2f7b9b | 2004-05-10 18:47:18 +0000 | [diff] [blame] | 208 | |
Chris Lattner | b06015a | 2010-02-09 19:54:29 +0000 | [diff] [blame] | 209 | while (MBB.front().isPHI()) |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 210 | LowerPHINode(MBB, LastPHIIt); |
Bill Wendling | 5d40982 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 211 | |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 212 | return true; |
| 213 | } |
Misha Brukman | 835702a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 214 | |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 215 | /// isImplicitlyDefined - Return true if all defs of VirtReg are implicit-defs. |
| 216 | /// This includes registers with no defs. |
| 217 | static bool isImplicitlyDefined(unsigned VirtReg, |
| 218 | const MachineRegisterInfo *MRI) { |
Owen Anderson | b36376e | 2014-03-17 19:36:09 +0000 | [diff] [blame] | 219 | for (MachineInstr &DI : MRI->def_instructions(VirtReg)) |
| 220 | if (!DI.isImplicitDef()) |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 221 | return false; |
| 222 | return true; |
| 223 | } |
| 224 | |
Evan Cheng | 18e46d4 | 2008-06-19 01:21:26 +0000 | [diff] [blame] | 225 | /// isSourceDefinedByImplicitDef - Return true if all sources of the phi node |
| 226 | /// are implicit_def's. |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 227 | static bool isSourceDefinedByImplicitDef(const MachineInstr *MPhi, |
Evan Cheng | 18e46d4 | 2008-06-19 01:21:26 +0000 | [diff] [blame] | 228 | const MachineRegisterInfo *MRI) { |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 229 | for (unsigned i = 1; i != MPhi->getNumOperands(); i += 2) |
| 230 | if (!isImplicitlyDefined(MPhi->getOperand(i).getReg(), MRI)) |
Evan Cheng | bec201f | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 231 | return false; |
Evan Cheng | bec201f | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 232 | return true; |
Evan Cheng | 3328186 | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 233 | } |
| 234 | |
Eugene Zelenko | 32a4056 | 2017-09-11 23:00:48 +0000 | [diff] [blame] | 235 | /// LowerPHINode - Lower the PHI node at the top of the specified block. |
Cameron Zwarich | a158d39 | 2013-02-10 06:42:32 +0000 | [diff] [blame] | 236 | void PHIElimination::LowerPHINode(MachineBasicBlock &MBB, |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 237 | MachineBasicBlock::iterator LastPHIIt) { |
Cameron Zwarich | a158d39 | 2013-02-10 06:42:32 +0000 | [diff] [blame] | 238 | ++NumLowered; |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 239 | |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 240 | MachineBasicBlock::iterator AfterPHIsIt = std::next(LastPHIIt); |
Cameron Zwarich | 867bfcd | 2013-07-01 19:42:46 +0000 | [diff] [blame] | 241 | |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 242 | // Unlink the PHI node from the basic block, but don't delete the PHI yet. |
Duncan P. N. Exon Smith | 1df1d1d | 2016-07-01 01:27:19 +0000 | [diff] [blame] | 243 | MachineInstr *MPhi = MBB.remove(&*MBB.begin()); |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 244 | |
Evan Cheng | 3328186 | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 245 | unsigned NumSrcs = (MPhi->getNumOperands() - 1) / 2; |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 246 | unsigned DestReg = MPhi->getOperand(0).getReg(); |
Jakob Stoklund Olesen | 952a621 | 2010-08-18 16:09:47 +0000 | [diff] [blame] | 247 | assert(MPhi->getOperand(0).getSubReg() == 0 && "Can't handle sub-reg PHIs"); |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 248 | bool isDead = MPhi->getOperand(0).isDead(); |
Misha Brukman | 835702a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 249 | |
Bill Wendling | 5d40982 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 250 | // Create a new register for the incoming PHI arguments. |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 251 | MachineFunction &MF = *MBB.getParent(); |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 252 | unsigned IncomingReg = 0; |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 253 | bool reusedIncoming = false; // Is IncomingReg reused from an earlier PHI? |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 254 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 255 | // Insert a register to register copy at the top of the current block (but |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 256 | // after any remaining phi nodes) which copies the new incoming register |
| 257 | // into the phi node destination. |
Eric Christopher | fc6de42 | 2014-08-05 02:39:49 +0000 | [diff] [blame] | 258 | const TargetInstrInfo *TII = MF.getSubtarget().getInstrInfo(); |
Evan Cheng | bec201f | 2008-05-10 00:17:50 +0000 | [diff] [blame] | 259 | if (isSourceDefinedByImplicitDef(MPhi, MRI)) |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 260 | // If all sources of a PHI node are implicit_def, just emit an |
| 261 | // implicit_def instead of a copy. |
Bill Wendling | 67cd395 | 2009-02-03 02:29:34 +0000 | [diff] [blame] | 262 | BuildMI(MBB, AfterPHIsIt, MPhi->getDebugLoc(), |
Chris Lattner | b06015a | 2010-02-09 19:54:29 +0000 | [diff] [blame] | 263 | TII->get(TargetOpcode::IMPLICIT_DEF), DestReg); |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 264 | else { |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 265 | // Can we reuse an earlier PHI node? This only happens for critical edges, |
| 266 | // typically those created by tail duplication. |
| 267 | unsigned &entry = LoweredPHIs[MPhi]; |
| 268 | if (entry) { |
| 269 | // An identical PHI node was already lowered. Reuse the incoming register. |
| 270 | IncomingReg = entry; |
| 271 | reusedIncoming = true; |
| 272 | ++NumReused; |
Jakob Stoklund Olesen | 1331a15 | 2011-01-09 03:05:53 +0000 | [diff] [blame] | 273 | DEBUG(dbgs() << "Reusing " << PrintReg(IncomingReg) << " for " << *MPhi); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 274 | } else { |
Jakob Stoklund Olesen | e50d30d | 2010-07-10 19:08:25 +0000 | [diff] [blame] | 275 | const TargetRegisterClass *RC = MF.getRegInfo().getRegClass(DestReg); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 276 | entry = IncomingReg = MF.getRegInfo().createVirtualRegister(RC); |
| 277 | } |
Jakob Stoklund Olesen | e50d30d | 2010-07-10 19:08:25 +0000 | [diff] [blame] | 278 | BuildMI(MBB, AfterPHIsIt, MPhi->getDebugLoc(), |
| 279 | TII->get(TargetOpcode::COPY), DestReg) |
| 280 | .addReg(IncomingReg); |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 281 | } |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 282 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 283 | // Update live variable information if there is any. |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 284 | if (LV) { |
Duncan P. N. Exon Smith | 1df1d1d | 2016-07-01 01:27:19 +0000 | [diff] [blame] | 285 | MachineInstr &PHICopy = *std::prev(AfterPHIsIt); |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 286 | |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 287 | if (IncomingReg) { |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 288 | LiveVariables::VarInfo &VI = LV->getVarInfo(IncomingReg); |
| 289 | |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 290 | // Increment use count of the newly created virtual register. |
Jakob Stoklund Olesen | 38b76e2 | 2010-02-23 22:43:58 +0000 | [diff] [blame] | 291 | LV->setPHIJoin(IncomingReg); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 292 | |
| 293 | // When we are reusing the incoming register, it may already have been |
| 294 | // killed in this block. The old kill will also have been inserted at |
| 295 | // AfterPHIsIt, so it appears before the current PHICopy. |
| 296 | if (reusedIncoming) |
| 297 | if (MachineInstr *OldKill = VI.findKill(&MBB)) { |
David Greene | 2555292 | 2010-01-05 01:24:24 +0000 | [diff] [blame] | 298 | DEBUG(dbgs() << "Remove old kill from " << *OldKill); |
Duncan P. N. Exon Smith | d26fdc8 | 2016-07-01 01:51:32 +0000 | [diff] [blame] | 299 | LV->removeVirtualRegisterKilled(IncomingReg, *OldKill); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 300 | DEBUG(MBB.dump()); |
| 301 | } |
Evan Cheng | a5a0c7c | 2007-04-18 00:36:11 +0000 | [diff] [blame] | 302 | |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 303 | // Add information to LiveVariables to know that the incoming value is |
| 304 | // killed. Note that because the value is defined in several places (once |
| 305 | // each for each incoming block), the "def" block and instruction fields |
| 306 | // for the VarInfo is not filled in. |
Duncan P. N. Exon Smith | d26fdc8 | 2016-07-01 01:51:32 +0000 | [diff] [blame] | 307 | LV->addVirtualRegisterKilled(IncomingReg, PHICopy); |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 308 | } |
Misha Brukman | 835702a | 2005-04-21 22:36:52 +0000 | [diff] [blame] | 309 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 310 | // Since we are going to be deleting the PHI node, if it is the last use of |
| 311 | // any registers, or if the value itself is dead, we need to move this |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 312 | // information over to the new copy we just inserted. |
Duncan P. N. Exon Smith | d26fdc8 | 2016-07-01 01:51:32 +0000 | [diff] [blame] | 313 | LV->removeVirtualRegistersKilled(*MPhi); |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 314 | |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 315 | // If the result is dead, update LV. |
Evan Cheng | 7d98a48 | 2008-07-03 09:09:37 +0000 | [diff] [blame] | 316 | if (isDead) { |
Duncan P. N. Exon Smith | d26fdc8 | 2016-07-01 01:51:32 +0000 | [diff] [blame] | 317 | LV->addVirtualRegisterDead(DestReg, PHICopy); |
| 318 | LV->removeVirtualRegisterDead(DestReg, *MPhi); |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 319 | } |
| 320 | } |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 321 | |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 322 | // Update LiveIntervals for the new copy or implicit def. |
| 323 | if (LIS) { |
Duncan P. N. Exon Smith | 1df1d1d | 2016-07-01 01:27:19 +0000 | [diff] [blame] | 324 | SlotIndex DestCopyIndex = |
| 325 | LIS->InsertMachineInstrInMaps(*std::prev(AfterPHIsIt)); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 326 | |
| 327 | SlotIndex MBBStartIndex = LIS->getMBBStartIdx(&MBB); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 328 | if (IncomingReg) { |
| 329 | // Add the region from the beginning of MBB to the copy instruction to |
| 330 | // IncomingReg's live interval. |
Mark Lacey | 9d8103d | 2013-08-14 23:50:16 +0000 | [diff] [blame] | 331 | LiveInterval &IncomingLI = LIS->createEmptyInterval(IncomingReg); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 332 | VNInfo *IncomingVNI = IncomingLI.getVNInfoAt(MBBStartIndex); |
| 333 | if (!IncomingVNI) |
| 334 | IncomingVNI = IncomingLI.getNextValue(MBBStartIndex, |
| 335 | LIS->getVNInfoAllocator()); |
Matthias Braun | 13ddb7c | 2013-10-10 21:28:43 +0000 | [diff] [blame] | 336 | IncomingLI.addSegment(LiveInterval::Segment(MBBStartIndex, |
| 337 | DestCopyIndex.getRegSlot(), |
| 338 | IncomingVNI)); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 339 | } |
| 340 | |
Cameron Zwarich | d113292 | 2013-02-21 08:51:55 +0000 | [diff] [blame] | 341 | LiveInterval &DestLI = LIS->getInterval(DestReg); |
Cameron Zwarich | 3ab4c4b | 2013-02-21 08:51:58 +0000 | [diff] [blame] | 342 | assert(DestLI.begin() != DestLI.end() && |
| 343 | "PHIs should have nonempty LiveIntervals."); |
| 344 | if (DestLI.endIndex().isDead()) { |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 345 | // A dead PHI's live range begins and ends at the start of the MBB, but |
| 346 | // the lowered copy, which will still be dead, needs to begin and end at |
| 347 | // the copy instruction. |
| 348 | VNInfo *OrigDestVNI = DestLI.getVNInfoAt(MBBStartIndex); |
| 349 | assert(OrigDestVNI && "PHI destination should be live at block entry."); |
Matthias Braun | 13ddb7c | 2013-10-10 21:28:43 +0000 | [diff] [blame] | 350 | DestLI.removeSegment(MBBStartIndex, MBBStartIndex.getDeadSlot()); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 351 | DestLI.createDeadDef(DestCopyIndex.getRegSlot(), |
| 352 | LIS->getVNInfoAllocator()); |
| 353 | DestLI.removeValNo(OrigDestVNI); |
| 354 | } else { |
| 355 | // Otherwise, remove the region from the beginning of MBB to the copy |
| 356 | // instruction from DestReg's live interval. |
Matthias Braun | 13ddb7c | 2013-10-10 21:28:43 +0000 | [diff] [blame] | 357 | DestLI.removeSegment(MBBStartIndex, DestCopyIndex.getRegSlot()); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 358 | VNInfo *DestVNI = DestLI.getVNInfoAt(DestCopyIndex.getRegSlot()); |
| 359 | assert(DestVNI && "PHI destination should be live at its definition."); |
| 360 | DestVNI->def = DestCopyIndex.getRegSlot(); |
| 361 | } |
| 362 | } |
| 363 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 364 | // Adjust the VRegPHIUseCount map to account for the removal of this PHI node. |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 365 | for (unsigned i = 1; i != MPhi->getNumOperands(); i += 2) |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 366 | --VRegPHIUseCount[BBVRegPair(MPhi->getOperand(i+1).getMBB()->getNumber(), |
Chris Lattner | a5bb370 | 2007-12-30 23:10:15 +0000 | [diff] [blame] | 367 | MPhi->getOperand(i).getReg())]; |
Chris Lattner | 51ae817 | 2003-05-12 14:28:28 +0000 | [diff] [blame] | 368 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 369 | // Now loop over all of the incoming arguments, changing them to copy into the |
| 370 | // IncomingReg register in the corresponding predecessor basic block. |
Evan Cheng | aacf4f1 | 2008-04-03 16:38:20 +0000 | [diff] [blame] | 371 | SmallPtrSet<MachineBasicBlock*, 8> MBBsInsertedInto; |
Evan Cheng | 3328186 | 2008-04-11 17:54:45 +0000 | [diff] [blame] | 372 | for (int i = NumSrcs - 1; i >= 0; --i) { |
| 373 | unsigned SrcReg = MPhi->getOperand(i*2+1).getReg(); |
Jakob Stoklund Olesen | 952a621 | 2010-08-18 16:09:47 +0000 | [diff] [blame] | 374 | unsigned SrcSubReg = MPhi->getOperand(i*2+1).getSubReg(); |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 375 | bool SrcUndef = MPhi->getOperand(i*2+1).isUndef() || |
| 376 | isImplicitlyDefined(SrcReg, MRI); |
Dan Gohman | 3a4be0f | 2008-02-10 18:45:23 +0000 | [diff] [blame] | 377 | assert(TargetRegisterInfo::isVirtualRegister(SrcReg) && |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 378 | "Machine PHI Operands must all be virtual registers!"); |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 379 | |
Lang Hames | a77a3c3 | 2009-07-23 04:34:03 +0000 | [diff] [blame] | 380 | // Get the MachineBasicBlock equivalent of the BasicBlock that is the source |
| 381 | // path the PHI. |
| 382 | MachineBasicBlock &opBlock = *MPhi->getOperand(i*2+2).getMBB(); |
| 383 | |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 384 | // Check to make sure we haven't already emitted the copy for this block. |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 385 | // This can happen because PHI nodes may have multiple entries for the same |
| 386 | // basic block. |
David Blaikie | 70573dc | 2014-11-19 07:49:26 +0000 | [diff] [blame] | 387 | if (!MBBsInsertedInto.insert(&opBlock).second) |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 388 | continue; // If the copy has already been emitted, we're done. |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 389 | |
Bill Wendling | 6b8bd51 | 2008-05-12 22:15:05 +0000 | [diff] [blame] | 390 | // Find a safe location to insert the copy, this may be the first terminator |
| 391 | // in the block (or end()). |
Jakob Stoklund Olesen | ad205d6 | 2009-11-13 21:56:15 +0000 | [diff] [blame] | 392 | MachineBasicBlock::iterator InsertPos = |
Cameron Zwarich | da592a9e | 2010-12-05 19:51:05 +0000 | [diff] [blame] | 393 | findPHICopyInsertPoint(&opBlock, &MBB, SrcReg); |
Evan Cheng | 94419d6 | 2009-03-13 22:59:14 +0000 | [diff] [blame] | 394 | |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 395 | // Insert the copy. |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 396 | MachineInstr *NewSrcInstr = nullptr; |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 397 | if (!reusedIncoming && IncomingReg) { |
| 398 | if (SrcUndef) { |
| 399 | // The source register is undefined, so there is no need for a real |
| 400 | // COPY, but we still need to ensure joint dominance by defs. |
| 401 | // Insert an IMPLICIT_DEF instruction. |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 402 | NewSrcInstr = BuildMI(opBlock, InsertPos, MPhi->getDebugLoc(), |
| 403 | TII->get(TargetOpcode::IMPLICIT_DEF), |
| 404 | IncomingReg); |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 405 | |
| 406 | // Clean up the old implicit-def, if there even was one. |
| 407 | if (MachineInstr *DefMI = MRI->getVRegDef(SrcReg)) |
| 408 | if (DefMI->isImplicitDef()) |
| 409 | ImpDefs.insert(DefMI); |
| 410 | } else { |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 411 | NewSrcInstr = BuildMI(opBlock, InsertPos, MPhi->getDebugLoc(), |
| 412 | TII->get(TargetOpcode::COPY), IncomingReg) |
| 413 | .addReg(SrcReg, 0, SrcSubReg); |
Jakob Stoklund Olesen | 70ed924 | 2012-06-25 03:36:12 +0000 | [diff] [blame] | 414 | } |
| 415 | } |
Chris Lattner | 5f096e2 | 2005-10-03 04:47:08 +0000 | [diff] [blame] | 416 | |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 417 | // We only need to update the LiveVariables kill of SrcReg if this was the |
| 418 | // last PHI use of SrcReg to be lowered on this CFG edge and it is not live |
| 419 | // out of the predecessor. We can also ignore undef sources. |
| 420 | if (LV && !SrcUndef && |
| 421 | !VRegPHIUseCount[BBVRegPair(opBlock.getNumber(), SrcReg)] && |
| 422 | !LV->isLiveOut(SrcReg, opBlock)) { |
| 423 | // We want to be able to insert a kill of the register if this PHI (aka, |
| 424 | // the copy we just inserted) is the last use of the source value. Live |
| 425 | // variable analysis conservatively handles this by saying that the value |
| 426 | // is live until the end of the block the PHI entry lives in. If the value |
| 427 | // really is dead at the PHI copy, there will be no successor blocks which |
| 428 | // have the value live-in. |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 429 | |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 430 | // Okay, if we now know that the value is not live out of the block, we |
| 431 | // can add a kill marker in this block saying that it kills the incoming |
| 432 | // value! |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 433 | |
Chris Lattner | 227e936 | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 434 | // In our final twist, we have to decide which instruction kills the |
Jakob Stoklund Olesen | 2d827d6 | 2012-07-04 19:52:05 +0000 | [diff] [blame] | 435 | // register. In most cases this is the copy, however, terminator |
| 436 | // instructions at the end of the block may also use the value. In this |
| 437 | // case, we should mark the last such terminator as being the killing |
| 438 | // block, not the copy. |
| 439 | MachineBasicBlock::iterator KillInst = opBlock.end(); |
| 440 | MachineBasicBlock::iterator FirstTerm = opBlock.getFirstTerminator(); |
| 441 | for (MachineBasicBlock::iterator Term = FirstTerm; |
| 442 | Term != opBlock.end(); ++Term) { |
| 443 | if (Term->readsRegister(SrcReg)) |
| 444 | KillInst = Term; |
| 445 | } |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 446 | |
Jakob Stoklund Olesen | 2d827d6 | 2012-07-04 19:52:05 +0000 | [diff] [blame] | 447 | if (KillInst == opBlock.end()) { |
| 448 | // No terminator uses the register. |
| 449 | |
| 450 | if (reusedIncoming || !IncomingReg) { |
| 451 | // We may have to rewind a bit if we didn't insert a copy this time. |
| 452 | KillInst = FirstTerm; |
| 453 | while (KillInst != opBlock.begin()) { |
| 454 | --KillInst; |
| 455 | if (KillInst->isDebugValue()) |
| 456 | continue; |
| 457 | if (KillInst->readsRegister(SrcReg)) |
| 458 | break; |
| 459 | } |
| 460 | } else { |
| 461 | // We just inserted this copy. |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 462 | KillInst = std::prev(InsertPos); |
Chris Lattner | 227e936 | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 463 | } |
Chris Lattner | 227e936 | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 464 | } |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 465 | assert(KillInst->readsRegister(SrcReg) && "Cannot find kill instruction"); |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 466 | |
Chris Lattner | 227e936 | 2006-01-04 07:12:21 +0000 | [diff] [blame] | 467 | // Finally, mark it killed. |
Duncan P. N. Exon Smith | d26fdc8 | 2016-07-01 01:51:32 +0000 | [diff] [blame] | 468 | LV->addVirtualRegisterKilled(SrcReg, *KillInst); |
Chris Lattner | 57b21f9 | 2005-10-03 07:22:07 +0000 | [diff] [blame] | 469 | |
| 470 | // This vreg no longer lives all of the way through opBlock. |
| 471 | unsigned opBlockNum = opBlock.getNumber(); |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 472 | LV->getVarInfo(SrcReg).AliveBlocks.reset(opBlockNum); |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 473 | } |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 474 | |
| 475 | if (LIS) { |
| 476 | if (NewSrcInstr) { |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 477 | LIS->InsertMachineInstrInMaps(*NewSrcInstr); |
| 478 | LIS->addSegmentToEndOfBlock(IncomingReg, *NewSrcInstr); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 479 | } |
| 480 | |
| 481 | if (!SrcUndef && |
| 482 | !VRegPHIUseCount[BBVRegPair(opBlock.getNumber(), SrcReg)]) { |
| 483 | LiveInterval &SrcLI = LIS->getInterval(SrcReg); |
| 484 | |
| 485 | bool isLiveOut = false; |
| 486 | for (MachineBasicBlock::succ_iterator SI = opBlock.succ_begin(), |
| 487 | SE = opBlock.succ_end(); SI != SE; ++SI) { |
Cameron Zwarich | 7c85c94 | 2013-02-12 05:48:58 +0000 | [diff] [blame] | 488 | SlotIndex startIdx = LIS->getMBBStartIdx(*SI); |
| 489 | VNInfo *VNI = SrcLI.getVNInfoAt(startIdx); |
| 490 | |
| 491 | // Definitions by other PHIs are not truly live-in for our purposes. |
| 492 | if (VNI && VNI->def != startIdx) { |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 493 | isLiveOut = true; |
| 494 | break; |
| 495 | } |
| 496 | } |
| 497 | |
| 498 | if (!isLiveOut) { |
| 499 | MachineBasicBlock::iterator KillInst = opBlock.end(); |
| 500 | MachineBasicBlock::iterator FirstTerm = opBlock.getFirstTerminator(); |
| 501 | for (MachineBasicBlock::iterator Term = FirstTerm; |
| 502 | Term != opBlock.end(); ++Term) { |
| 503 | if (Term->readsRegister(SrcReg)) |
| 504 | KillInst = Term; |
| 505 | } |
| 506 | |
| 507 | if (KillInst == opBlock.end()) { |
| 508 | // No terminator uses the register. |
| 509 | |
| 510 | if (reusedIncoming || !IncomingReg) { |
| 511 | // We may have to rewind a bit if we didn't just insert a copy. |
| 512 | KillInst = FirstTerm; |
| 513 | while (KillInst != opBlock.begin()) { |
| 514 | --KillInst; |
| 515 | if (KillInst->isDebugValue()) |
| 516 | continue; |
| 517 | if (KillInst->readsRegister(SrcReg)) |
| 518 | break; |
| 519 | } |
| 520 | } else { |
| 521 | // We just inserted this copy. |
Benjamin Kramer | b6d0bd4 | 2014-03-02 12:27:27 +0000 | [diff] [blame] | 522 | KillInst = std::prev(InsertPos); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 523 | } |
| 524 | } |
| 525 | assert(KillInst->readsRegister(SrcReg) && |
| 526 | "Cannot find kill instruction"); |
| 527 | |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 528 | SlotIndex LastUseIndex = LIS->getInstructionIndex(*KillInst); |
Matthias Braun | 13ddb7c | 2013-10-10 21:28:43 +0000 | [diff] [blame] | 529 | SrcLI.removeSegment(LastUseIndex.getRegSlot(), |
| 530 | LIS->getMBBEndIdx(&opBlock)); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 531 | } |
| 532 | } |
| 533 | } |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 534 | } |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 535 | |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 536 | // Really delete the PHI instruction now, if it is not in the LoweredPHIs map. |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 537 | if (reusedIncoming || !IncomingReg) { |
| 538 | if (LIS) |
Duncan P. N. Exon Smith | 3ac9cc6 | 2016-02-27 06:40:41 +0000 | [diff] [blame] | 539 | LIS->RemoveMachineInstrFromMaps(*MPhi); |
Jakob Stoklund Olesen | ec20a88 | 2009-12-16 18:55:53 +0000 | [diff] [blame] | 540 | MF.DeleteMachineInstr(MPhi); |
Cameron Zwarich | 16b64cb | 2013-02-10 06:42:36 +0000 | [diff] [blame] | 541 | } |
Chris Lattner | cab0b44 | 2003-01-13 20:01:16 +0000 | [diff] [blame] | 542 | } |
Bill Wendling | 5d40982 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 543 | |
| 544 | /// analyzePHINodes - Gather information about the PHI nodes in here. In |
| 545 | /// particular, we want to map the number of uses of a virtual register which is |
| 546 | /// used in a PHI node. We map that to the BB the vreg is coming from. This is |
| 547 | /// used later to determine when the vreg is killed in the BB. |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 548 | void PHIElimination::analyzePHINodes(const MachineFunction& MF) { |
Alexey Samsonov | 41b977d | 2014-04-30 18:29:51 +0000 | [diff] [blame] | 549 | for (const auto &MBB : MF) |
Alexey Samsonov | f74bde6 | 2014-04-30 22:17:38 +0000 | [diff] [blame] | 550 | for (const auto &BBI : MBB) { |
| 551 | if (!BBI.isPHI()) |
| 552 | break; |
| 553 | for (unsigned i = 1, e = BBI.getNumOperands(); i != e; i += 2) |
| 554 | ++VRegPHIUseCount[BBVRegPair(BBI.getOperand(i+1).getMBB()->getNumber(), |
| 555 | BBI.getOperand(i).getReg())]; |
| 556 | } |
Bill Wendling | 5d40982 | 2006-09-28 07:10:24 +0000 | [diff] [blame] | 557 | } |
Jakob Stoklund Olesen | 19f235e | 2009-11-10 22:00:56 +0000 | [diff] [blame] | 558 | |
Cameron Zwarich | a3fb8cb | 2010-12-05 21:39:42 +0000 | [diff] [blame] | 559 | bool PHIElimination::SplitPHIEdges(MachineFunction &MF, |
Cameron Zwarich | ecd4492 | 2011-02-17 06:13:46 +0000 | [diff] [blame] | 560 | MachineBasicBlock &MBB, |
Cameron Zwarich | ecd4492 | 2011-02-17 06:13:46 +0000 | [diff] [blame] | 561 | MachineLoopInfo *MLI) { |
Reid Kleckner | 0e28823 | 2015-08-27 23:27:47 +0000 | [diff] [blame] | 562 | if (MBB.empty() || !MBB.front().isPHI() || MBB.isEHPad()) |
Jakob Stoklund Olesen | 4f7fd3b | 2009-11-11 19:31:31 +0000 | [diff] [blame] | 563 | return false; // Quick exit for basic blocks without PHIs. |
Jakob Stoklund Olesen | 736888f | 2009-11-18 18:01:35 +0000 | [diff] [blame] | 564 | |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 565 | const MachineLoop *CurLoop = MLI ? MLI->getLoopFor(&MBB) : nullptr; |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 566 | bool IsLoopHeader = CurLoop && &MBB == CurLoop->getHeader(); |
| 567 | |
Evan Cheng | f259efd | 2010-08-17 01:20:36 +0000 | [diff] [blame] | 568 | bool Changed = false; |
Evan Cheng | 2a81dd4 | 2011-12-06 22:12:01 +0000 | [diff] [blame] | 569 | for (MachineBasicBlock::iterator BBI = MBB.begin(), BBE = MBB.end(); |
Chris Lattner | b06015a | 2010-02-09 19:54:29 +0000 | [diff] [blame] | 570 | BBI != BBE && BBI->isPHI(); ++BBI) { |
Jakob Stoklund Olesen | 4453dc9 | 2009-11-10 22:01:05 +0000 | [diff] [blame] | 571 | for (unsigned i = 1, e = BBI->getNumOperands(); i != e; i += 2) { |
| 572 | unsigned Reg = BBI->getOperand(i).getReg(); |
| 573 | MachineBasicBlock *PreMBB = BBI->getOperand(i+1).getMBB(); |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 574 | // Is there a critical edge from PreMBB to MBB? |
| 575 | if (PreMBB->succ_size() == 1) |
| 576 | continue; |
| 577 | |
Evan Cheng | 647c559 | 2010-08-17 17:43:50 +0000 | [diff] [blame] | 578 | // Avoid splitting backedges of loops. It would introduce small |
| 579 | // out-of-line blocks into the loop which is very bad for code placement. |
Cameron Zwarich | 15eb925 | 2013-02-12 03:49:25 +0000 | [diff] [blame] | 580 | if (PreMBB == &MBB && !SplitAllCriticalEdges) |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 581 | continue; |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 582 | const MachineLoop *PreLoop = MLI ? MLI->getLoopFor(PreMBB) : nullptr; |
Cameron Zwarich | 15eb925 | 2013-02-12 03:49:25 +0000 | [diff] [blame] | 583 | if (IsLoopHeader && PreLoop == CurLoop && !SplitAllCriticalEdges) |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 584 | continue; |
| 585 | |
| 586 | // LV doesn't consider a phi use live-out, so isLiveOut only returns true |
| 587 | // when the source register is live-out for some other reason than a phi |
| 588 | // use. That means the copy we will insert in PreMBB won't be a kill, and |
| 589 | // there is a risk it may not be coalesced away. |
| 590 | // |
| 591 | // If the copy would be a kill, there is no need to split the edge. |
Daniel Jasper | 8f239f8 | 2015-03-03 10:23:11 +0000 | [diff] [blame] | 592 | bool ShouldSplit = isLiveOutPastPHIs(Reg, PreMBB); |
| 593 | if (!ShouldSplit && !NoPhiElimLiveOutEarlyExit) |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 594 | continue; |
Daniel Jasper | 8f239f8 | 2015-03-03 10:23:11 +0000 | [diff] [blame] | 595 | if (ShouldSplit) { |
| 596 | DEBUG(dbgs() << PrintReg(Reg) << " live-out before critical edge BB#" |
| 597 | << PreMBB->getNumber() << " -> BB#" << MBB.getNumber() |
| 598 | << ": " << *BBI); |
| 599 | } |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 600 | |
| 601 | // If Reg is not live-in to MBB, it means it must be live-in to some |
| 602 | // other PreMBB successor, and we can avoid the interference by splitting |
| 603 | // the edge. |
| 604 | // |
| 605 | // If Reg *is* live-in to MBB, the interference is inevitable and a copy |
| 606 | // is likely to be left after coalescing. If we are looking at a loop |
| 607 | // exiting edge, split it so we won't insert code in the loop, otherwise |
| 608 | // don't bother. |
Daniel Jasper | 8f239f8 | 2015-03-03 10:23:11 +0000 | [diff] [blame] | 609 | ShouldSplit = ShouldSplit && !isLiveIn(Reg, &MBB); |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 610 | |
| 611 | // Check for a loop exiting edge. |
| 612 | if (!ShouldSplit && CurLoop != PreLoop) { |
| 613 | DEBUG({ |
| 614 | dbgs() << "Split wouldn't help, maybe avoid loop copies?\n"; |
| 615 | if (PreLoop) dbgs() << "PreLoop: " << *PreLoop; |
| 616 | if (CurLoop) dbgs() << "CurLoop: " << *CurLoop; |
| 617 | }); |
| 618 | // This edge could be entering a loop, exiting a loop, or it could be |
| 619 | // both: Jumping directly form one loop to the header of a sibling |
| 620 | // loop. |
| 621 | // Split unless this edge is entering CurLoop from an outer loop. |
| 622 | ShouldSplit = PreLoop && !PreLoop->contains(CurLoop); |
Evan Cheng | 647c559 | 2010-08-17 17:43:50 +0000 | [diff] [blame] | 623 | } |
Daniel Jasper | 8f239f8 | 2015-03-03 10:23:11 +0000 | [diff] [blame] | 624 | if (!ShouldSplit && !SplitAllCriticalEdges) |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 625 | continue; |
Quentin Colombet | 23341a8 | 2016-04-21 21:01:13 +0000 | [diff] [blame] | 626 | if (!PreMBB->SplitCriticalEdge(&MBB, *this)) { |
Matt Arsenault | d850a06 | 2014-01-22 02:38:23 +0000 | [diff] [blame] | 627 | DEBUG(dbgs() << "Failed to split critical edge.\n"); |
Jakob Stoklund Olesen | f62c07f | 2012-07-20 20:49:53 +0000 | [diff] [blame] | 628 | continue; |
| 629 | } |
| 630 | Changed = true; |
| 631 | ++NumCriticalEdgesSplit; |
Jakob Stoklund Olesen | 4453dc9 | 2009-11-10 22:01:05 +0000 | [diff] [blame] | 632 | } |
| 633 | } |
Cameron Zwarich | 0b0cc4d | 2011-02-17 06:13:43 +0000 | [diff] [blame] | 634 | return Changed; |
Jakob Stoklund Olesen | 4453dc9 | 2009-11-10 22:01:05 +0000 | [diff] [blame] | 635 | } |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 636 | |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 637 | bool PHIElimination::isLiveIn(unsigned Reg, const MachineBasicBlock *MBB) { |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 638 | assert((LV || LIS) && |
| 639 | "isLiveIn() requires either LiveVariables or LiveIntervals"); |
| 640 | if (LIS) |
| 641 | return LIS->isLiveInToMBB(LIS->getInterval(Reg), MBB); |
| 642 | else |
| 643 | return LV->isLiveIn(Reg, *MBB); |
| 644 | } |
| 645 | |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 646 | bool PHIElimination::isLiveOutPastPHIs(unsigned Reg, |
| 647 | const MachineBasicBlock *MBB) { |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 648 | assert((LV || LIS) && |
| 649 | "isLiveOutPastPHIs() requires either LiveVariables or LiveIntervals"); |
| 650 | // LiveVariables considers uses in PHIs to be in the predecessor basic block, |
| 651 | // so that a register used only in a PHI is not live out of the block. In |
| 652 | // contrast, LiveIntervals considers uses in PHIs to be on the edge rather than |
| 653 | // in the predecessor basic block, so that a register used only in a PHI is live |
| 654 | // out of the block. |
| 655 | if (LIS) { |
| 656 | const LiveInterval &LI = LIS->getInterval(Reg); |
Arnaud A. de Grandmaison | 2e8ffa3 | 2015-06-11 07:45:05 +0000 | [diff] [blame] | 657 | for (const MachineBasicBlock *SI : MBB->successors()) |
| 658 | if (LI.liveAt(LIS->getMBBStartIdx(SI))) |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 659 | return true; |
Cameron Zwarich | bb9ad31 | 2013-02-10 23:29:49 +0000 | [diff] [blame] | 660 | return false; |
| 661 | } else { |
| 662 | return LV->isLiveOut(Reg, *MBB); |
| 663 | } |
| 664 | } |