Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1 | //===-- PeepholeOptimizer.cpp - Peephole Optimizations --------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | // |
| 10 | // Perform peephole optimizations on the machine code: |
| 11 | // |
| 12 | // - Optimize Extensions |
| 13 | // |
| 14 | // Optimization of sign / zero extension instructions. It may be extended to |
| 15 | // handle other instructions with similar properties. |
| 16 | // |
| 17 | // On some targets, some instructions, e.g. X86 sign / zero extension, may |
| 18 | // leave the source value in the lower part of the result. This optimization |
| 19 | // will replace some uses of the pre-extension value with uses of the |
| 20 | // sub-register of the results. |
| 21 | // |
| 22 | // - Optimize Comparisons |
| 23 | // |
| 24 | // Optimization of comparison instructions. For instance, in this code: |
| 25 | // |
| 26 | // sub r1, 1 |
| 27 | // cmp r1, 0 |
| 28 | // bz L1 |
| 29 | // |
| 30 | // If the "sub" instruction all ready sets (or could be modified to set) the |
| 31 | // same flag that the "cmp" instruction sets and that "bz" uses, then we can |
| 32 | // eliminate the "cmp" instruction. |
Evan Cheng | e4b8ac9 | 2011-03-15 05:13:13 +0000 | [diff] [blame] | 33 | // |
Manman Ren | dc8ad00 | 2012-05-11 01:30:47 +0000 | [diff] [blame] | 34 | // Another instance, in this code: |
| 35 | // |
| 36 | // sub r1, r3 | sub r1, imm |
| 37 | // cmp r3, r1 or cmp r1, r3 | cmp r1, imm |
| 38 | // bge L1 |
| 39 | // |
| 40 | // If the branch instruction can use flag from "sub", then we can replace |
| 41 | // "sub" with "subs" and eliminate the "cmp" instruction. |
| 42 | // |
Joel Jones | 24e440d | 2012-12-11 16:10:25 +0000 | [diff] [blame] | 43 | // - Optimize Loads: |
| 44 | // |
| 45 | // Loads that can be folded into a later instruction. A load is foldable |
| 46 | // if it loads to virtual registers and the virtual register defined has |
| 47 | // a single use. |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 48 | // |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 49 | // - Optimize Copies and Bitcast (more generally, target specific copies): |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 50 | // |
| 51 | // Rewrite copies and bitcasts to avoid cross register bank copies |
| 52 | // when possible. |
| 53 | // E.g., Consider the following example, where capital and lower |
| 54 | // letters denote different register file: |
| 55 | // b = copy A <-- cross-bank copy |
| 56 | // C = copy b <-- cross-bank copy |
| 57 | // => |
| 58 | // b = copy A <-- cross-bank copy |
| 59 | // C = copy A <-- same-bank copy |
| 60 | // |
| 61 | // E.g., for bitcast: |
| 62 | // b = bitcast A <-- cross-bank copy |
| 63 | // C = bitcast b <-- cross-bank copy |
| 64 | // => |
| 65 | // b = bitcast A <-- cross-bank copy |
| 66 | // C = copy A <-- same-bank copy |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 67 | //===----------------------------------------------------------------------===// |
| 68 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 69 | #include "llvm/CodeGen/Passes.h" |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 70 | #include "llvm/ADT/DenseMap.h" |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 71 | #include "llvm/ADT/SmallPtrSet.h" |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 72 | #include "llvm/ADT/SmallSet.h" |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 73 | #include "llvm/ADT/Statistic.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 74 | #include "llvm/CodeGen/MachineDominators.h" |
| 75 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| 76 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
| 77 | #include "llvm/Support/CommandLine.h" |
Craig Topper | 588ceec | 2012-12-17 03:56:00 +0000 | [diff] [blame] | 78 | #include "llvm/Support/Debug.h" |
Benjamin Kramer | 799003b | 2015-03-23 19:32:43 +0000 | [diff] [blame] | 79 | #include "llvm/Support/raw_ostream.h" |
Chandler Carruth | ed0881b | 2012-12-03 16:50:05 +0000 | [diff] [blame] | 80 | #include "llvm/Target/TargetInstrInfo.h" |
| 81 | #include "llvm/Target/TargetRegisterInfo.h" |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 82 | #include "llvm/Target/TargetSubtargetInfo.h" |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 83 | #include <utility> |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 84 | using namespace llvm; |
| 85 | |
Chandler Carruth | 1b9dde0 | 2014-04-22 02:02:50 +0000 | [diff] [blame] | 86 | #define DEBUG_TYPE "peephole-opt" |
| 87 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 88 | // Optimize Extensions |
| 89 | static cl::opt<bool> |
| 90 | Aggressive("aggressive-ext-opt", cl::Hidden, |
| 91 | cl::desc("Aggressive extension optimization")); |
| 92 | |
Bill Wendling | c6627ee | 2010-11-01 20:41:43 +0000 | [diff] [blame] | 93 | static cl::opt<bool> |
| 94 | DisablePeephole("disable-peephole", cl::Hidden, cl::init(false), |
| 95 | cl::desc("Disable the peephole optimizer")); |
| 96 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 97 | static cl::opt<bool> |
Quentin Colombet | 6674b09 | 2014-08-21 22:23:52 +0000 | [diff] [blame] | 98 | DisableAdvCopyOpt("disable-adv-copy-opt", cl::Hidden, cl::init(false), |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 99 | cl::desc("Disable advanced copy optimization")); |
| 100 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 101 | // Limit the number of PHI instructions to process |
| 102 | // in PeepholeOptimizer::getNextSource. |
| 103 | static cl::opt<unsigned> RewritePHILimit( |
| 104 | "rewrite-phi-limit", cl::Hidden, cl::init(10), |
| 105 | cl::desc("Limit the length of PHI chains to lookup")); |
| 106 | |
Bill Wendling | 6628431 | 2010-08-27 20:39:09 +0000 | [diff] [blame] | 107 | STATISTIC(NumReuse, "Number of extension results reused"); |
Evan Cheng | e4b8ac9 | 2011-03-15 05:13:13 +0000 | [diff] [blame] | 108 | STATISTIC(NumCmps, "Number of compares eliminated"); |
Lang Hames | 31bb57b | 2012-02-25 00:46:38 +0000 | [diff] [blame] | 109 | STATISTIC(NumImmFold, "Number of move immediate folded"); |
Manman Ren | 5759d01 | 2012-08-02 00:56:42 +0000 | [diff] [blame] | 110 | STATISTIC(NumLoadFold, "Number of loads folded"); |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 111 | STATISTIC(NumSelects, "Number of selects optimized"); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 112 | STATISTIC(NumUncoalescableCopies, "Number of uncoalescable copies optimized"); |
| 113 | STATISTIC(NumRewrittenCopies, "Number of copies rewritten"); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 114 | |
| 115 | namespace { |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 116 | class ValueTrackerResult; |
| 117 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 118 | class PeepholeOptimizer : public MachineFunctionPass { |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 119 | const TargetInstrInfo *TII; |
Eric Christopher | 92b4bcb | 2014-10-14 07:17:20 +0000 | [diff] [blame] | 120 | const TargetRegisterInfo *TRI; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 121 | MachineRegisterInfo *MRI; |
| 122 | MachineDominatorTree *DT; // Machine dominator tree |
| 123 | |
| 124 | public: |
| 125 | static char ID; // Pass identification |
Owen Anderson | 6c18d1a | 2010-10-19 17:21:58 +0000 | [diff] [blame] | 126 | PeepholeOptimizer() : MachineFunctionPass(ID) { |
| 127 | initializePeepholeOptimizerPass(*PassRegistry::getPassRegistry()); |
| 128 | } |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 129 | |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 130 | bool runOnMachineFunction(MachineFunction &MF) override; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 131 | |
Craig Topper | 4584cd5 | 2014-03-07 09:26:03 +0000 | [diff] [blame] | 132 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 133 | AU.setPreservesCFG(); |
| 134 | MachineFunctionPass::getAnalysisUsage(AU); |
| 135 | if (Aggressive) { |
| 136 | AU.addRequired<MachineDominatorTree>(); |
| 137 | AU.addPreserved<MachineDominatorTree>(); |
| 138 | } |
| 139 | } |
| 140 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 141 | /// \brief Track Def -> Use info used for rewriting copies. |
| 142 | typedef SmallDenseMap<TargetInstrInfo::RegSubRegPair, ValueTrackerResult> |
| 143 | RewriteMapTy; |
| 144 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 145 | private: |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 146 | bool optimizeCmpInstr(MachineInstr *MI, MachineBasicBlock *MBB); |
| 147 | bool optimizeExtInstr(MachineInstr *MI, MachineBasicBlock *MBB, |
Hans Wennborg | 97a59ae | 2014-08-11 13:52:46 +0000 | [diff] [blame] | 148 | SmallPtrSetImpl<MachineInstr*> &LocalMIs); |
Mehdi Amini | 22e5974 | 2015-01-13 07:07:13 +0000 | [diff] [blame] | 149 | bool optimizeSelect(MachineInstr *MI, |
| 150 | SmallPtrSetImpl<MachineInstr *> &LocalMIs); |
Gerolf Hoflehner | a4c96d0 | 2014-10-14 23:07:53 +0000 | [diff] [blame] | 151 | bool optimizeCondBranch(MachineInstr *MI); |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 152 | bool optimizeCopyOrBitcast(MachineInstr *MI); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 153 | bool optimizeCoalescableCopy(MachineInstr *MI); |
| 154 | bool optimizeUncoalescableCopy(MachineInstr *MI, |
| 155 | SmallPtrSetImpl<MachineInstr *> &LocalMIs); |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 156 | bool findNextSource(unsigned Reg, unsigned SubReg, |
| 157 | RewriteMapTy &RewriteMap); |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 158 | bool isMoveImmediate(MachineInstr *MI, |
| 159 | SmallSet<unsigned, 4> &ImmDefRegs, |
| 160 | DenseMap<unsigned, MachineInstr*> &ImmDefMIs); |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 161 | bool foldImmediate(MachineInstr *MI, MachineBasicBlock *MBB, |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 162 | SmallSet<unsigned, 4> &ImmDefRegs, |
| 163 | DenseMap<unsigned, MachineInstr*> &ImmDefMIs); |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 164 | bool isLoadFoldable(MachineInstr *MI, |
| 165 | SmallSet<unsigned, 16> &FoldAsLoadDefCandidates); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 166 | |
| 167 | /// \brief Check whether \p MI is understood by the register coalescer |
| 168 | /// but may require some rewriting. |
| 169 | bool isCoalescableCopy(const MachineInstr &MI) { |
| 170 | // SubregToRegs are not interesting, because they are already register |
| 171 | // coalescer friendly. |
| 172 | return MI.isCopy() || (!DisableAdvCopyOpt && |
| 173 | (MI.isRegSequence() || MI.isInsertSubreg() || |
| 174 | MI.isExtractSubreg())); |
| 175 | } |
| 176 | |
| 177 | /// \brief Check whether \p MI is a copy like instruction that is |
| 178 | /// not recognized by the register coalescer. |
| 179 | bool isUncoalescableCopy(const MachineInstr &MI) { |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 180 | return MI.isBitcast() || |
| 181 | (!DisableAdvCopyOpt && |
| 182 | (MI.isRegSequenceLike() || MI.isInsertSubregLike() || |
| 183 | MI.isExtractSubregLike())); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 184 | } |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 185 | }; |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 186 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 187 | /// \brief Helper class to hold a reply for ValueTracker queries. Contains the |
| 188 | /// returned sources for a given search and the instructions where the sources |
| 189 | /// were tracked from. |
| 190 | class ValueTrackerResult { |
| 191 | private: |
| 192 | /// Track all sources found by one ValueTracker query. |
| 193 | SmallVector<TargetInstrInfo::RegSubRegPair, 2> RegSrcs; |
| 194 | |
| 195 | /// Instruction using the sources in 'RegSrcs'. |
| 196 | const MachineInstr *Inst; |
| 197 | |
| 198 | public: |
| 199 | ValueTrackerResult() : Inst(nullptr) {} |
| 200 | ValueTrackerResult(unsigned Reg, unsigned SubReg) : Inst(nullptr) { |
| 201 | addSource(Reg, SubReg); |
| 202 | } |
| 203 | |
| 204 | bool isValid() const { return getNumSources() > 0; } |
| 205 | |
| 206 | void setInst(const MachineInstr *I) { Inst = I; } |
| 207 | const MachineInstr *getInst() const { return Inst; } |
| 208 | |
| 209 | void clear() { |
| 210 | RegSrcs.clear(); |
| 211 | Inst = nullptr; |
| 212 | } |
| 213 | |
| 214 | void addSource(unsigned SrcReg, unsigned SrcSubReg) { |
| 215 | RegSrcs.push_back(TargetInstrInfo::RegSubRegPair(SrcReg, SrcSubReg)); |
| 216 | } |
| 217 | |
| 218 | void setSource(int Idx, unsigned SrcReg, unsigned SrcSubReg) { |
| 219 | assert(Idx < getNumSources() && "Reg pair source out of index"); |
| 220 | RegSrcs[Idx] = TargetInstrInfo::RegSubRegPair(SrcReg, SrcSubReg); |
| 221 | } |
| 222 | |
| 223 | int getNumSources() const { return RegSrcs.size(); } |
| 224 | |
| 225 | unsigned getSrcReg(int Idx) const { |
| 226 | assert(Idx < getNumSources() && "Reg source out of index"); |
| 227 | return RegSrcs[Idx].Reg; |
| 228 | } |
| 229 | |
| 230 | unsigned getSrcSubReg(int Idx) const { |
| 231 | assert(Idx < getNumSources() && "SubReg source out of index"); |
| 232 | return RegSrcs[Idx].SubReg; |
| 233 | } |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 234 | |
| 235 | bool operator==(const ValueTrackerResult &Other) { |
| 236 | if (Other.getInst() != getInst()) |
| 237 | return false; |
| 238 | |
| 239 | if (Other.getNumSources() != getNumSources()) |
| 240 | return false; |
| 241 | |
| 242 | for (int i = 0, e = Other.getNumSources(); i != e; ++i) |
| 243 | if (Other.getSrcReg(i) != getSrcReg(i) || |
| 244 | Other.getSrcSubReg(i) != getSrcSubReg(i)) |
| 245 | return false; |
| 246 | return true; |
| 247 | } |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 248 | }; |
| 249 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 250 | /// \brief Helper class to track the possible sources of a value defined by |
| 251 | /// a (chain of) copy related instructions. |
| 252 | /// Given a definition (instruction and definition index), this class |
| 253 | /// follows the use-def chain to find successive suitable sources. |
| 254 | /// The given source can be used to rewrite the definition into |
| 255 | /// def = COPY src. |
| 256 | /// |
| 257 | /// For instance, let us consider the following snippet: |
| 258 | /// v0 = |
| 259 | /// v2 = INSERT_SUBREG v1, v0, sub0 |
| 260 | /// def = COPY v2.sub0 |
| 261 | /// |
| 262 | /// Using a ValueTracker for def = COPY v2.sub0 will give the following |
| 263 | /// suitable sources: |
| 264 | /// v2.sub0 and v0. |
| 265 | /// Then, def can be rewritten into def = COPY v0. |
| 266 | class ValueTracker { |
| 267 | private: |
| 268 | /// The current point into the use-def chain. |
| 269 | const MachineInstr *Def; |
| 270 | /// The index of the definition in Def. |
| 271 | unsigned DefIdx; |
| 272 | /// The sub register index of the definition. |
| 273 | unsigned DefSubReg; |
| 274 | /// The register where the value can be found. |
| 275 | unsigned Reg; |
| 276 | /// Specifiy whether or not the value tracking looks through |
| 277 | /// complex instructions. When this is false, the value tracker |
| 278 | /// bails on everything that is not a copy or a bitcast. |
| 279 | /// |
| 280 | /// Note: This could have been implemented as a specialized version of |
| 281 | /// the ValueTracker class but that would have complicated the code of |
| 282 | /// the users of this class. |
| 283 | bool UseAdvancedTracking; |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 284 | /// MachineRegisterInfo used to perform tracking. |
| 285 | const MachineRegisterInfo &MRI; |
| 286 | /// Optional TargetInstrInfo used to perform some complex |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 287 | /// tracking. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 288 | const TargetInstrInfo *TII; |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 289 | |
| 290 | /// \brief Dispatcher to the right underlying implementation of |
| 291 | /// getNextSource. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 292 | ValueTrackerResult getNextSourceImpl(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 293 | /// \brief Specialized version of getNextSource for Copy instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 294 | ValueTrackerResult getNextSourceFromCopy(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 295 | /// \brief Specialized version of getNextSource for Bitcast instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 296 | ValueTrackerResult getNextSourceFromBitcast(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 297 | /// \brief Specialized version of getNextSource for RegSequence |
| 298 | /// instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 299 | ValueTrackerResult getNextSourceFromRegSequence(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 300 | /// \brief Specialized version of getNextSource for InsertSubreg |
| 301 | /// instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 302 | ValueTrackerResult getNextSourceFromInsertSubreg(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 303 | /// \brief Specialized version of getNextSource for ExtractSubreg |
| 304 | /// instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 305 | ValueTrackerResult getNextSourceFromExtractSubreg(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 306 | /// \brief Specialized version of getNextSource for SubregToReg |
| 307 | /// instructions. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 308 | ValueTrackerResult getNextSourceFromSubregToReg(); |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 309 | /// \brief Specialized version of getNextSource for PHI instructions. |
| 310 | ValueTrackerResult getNextSourceFromPHI(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 311 | |
| 312 | public: |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 313 | /// \brief Create a ValueTracker instance for the value defined by \p Reg. |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 314 | /// \p DefSubReg represents the sub register index the value tracker will |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 315 | /// track. It does not need to match the sub register index used in the |
| 316 | /// definition of \p Reg. |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 317 | /// \p UseAdvancedTracking specifies whether or not the value tracker looks |
| 318 | /// through complex instructions. By default (false), it handles only copy |
| 319 | /// and bitcast instructions. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 320 | /// If \p Reg is a physical register, a value tracker constructed with |
| 321 | /// this constructor will not find any alternative source. |
| 322 | /// Indeed, when \p Reg is a physical register that constructor does not |
| 323 | /// know which definition of \p Reg it should track. |
| 324 | /// Use the next constructor to track a physical register. |
| 325 | ValueTracker(unsigned Reg, unsigned DefSubReg, |
| 326 | const MachineRegisterInfo &MRI, |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 327 | bool UseAdvancedTracking = false, |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 328 | const TargetInstrInfo *TII = nullptr) |
| 329 | : Def(nullptr), DefIdx(0), DefSubReg(DefSubReg), Reg(Reg), |
| 330 | UseAdvancedTracking(UseAdvancedTracking), MRI(MRI), TII(TII) { |
| 331 | if (!TargetRegisterInfo::isPhysicalRegister(Reg)) { |
| 332 | Def = MRI.getVRegDef(Reg); |
| 333 | DefIdx = MRI.def_begin(Reg).getOperandNo(); |
| 334 | } |
| 335 | } |
| 336 | |
| 337 | /// \brief Create a ValueTracker instance for the value defined by |
| 338 | /// the pair \p MI, \p DefIdx. |
| 339 | /// Unlike the other constructor, the value tracker produced by this one |
| 340 | /// may be able to find a new source when the definition is a physical |
| 341 | /// register. |
| 342 | /// This could be useful to rewrite target specific instructions into |
| 343 | /// generic copy instructions. |
| 344 | ValueTracker(const MachineInstr &MI, unsigned DefIdx, unsigned DefSubReg, |
| 345 | const MachineRegisterInfo &MRI, |
| 346 | bool UseAdvancedTracking = false, |
| 347 | const TargetInstrInfo *TII = nullptr) |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 348 | : Def(&MI), DefIdx(DefIdx), DefSubReg(DefSubReg), |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 349 | UseAdvancedTracking(UseAdvancedTracking), MRI(MRI), TII(TII) { |
| 350 | assert(DefIdx < Def->getDesc().getNumDefs() && |
| 351 | Def->getOperand(DefIdx).isReg() && "Invalid definition"); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 352 | Reg = Def->getOperand(DefIdx).getReg(); |
| 353 | } |
| 354 | |
| 355 | /// \brief Following the use-def chain, get the next available source |
| 356 | /// for the tracked value. |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 357 | /// \return A ValueTrackerResult containing a set of registers |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 358 | /// and sub registers with tracked values. A ValueTrackerResult with |
| 359 | /// an empty set of registers means no source was found. |
| 360 | ValueTrackerResult getNextSource(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 361 | |
| 362 | /// \brief Get the last register where the initial value can be found. |
| 363 | /// Initially this is the register of the definition. |
| 364 | /// Then, after each successful call to getNextSource, this is the |
| 365 | /// register of the last source. |
| 366 | unsigned getReg() const { return Reg; } |
| 367 | }; |
Alexander Kornienko | f00654e | 2015-06-23 09:49:53 +0000 | [diff] [blame] | 368 | } |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 369 | |
| 370 | char PeepholeOptimizer::ID = 0; |
Andrew Trick | 1fa5bcb | 2012-02-08 21:23:13 +0000 | [diff] [blame] | 371 | char &llvm::PeepholeOptimizerID = PeepholeOptimizer::ID; |
Owen Anderson | 8ac477f | 2010-10-12 19:48:12 +0000 | [diff] [blame] | 372 | INITIALIZE_PASS_BEGIN(PeepholeOptimizer, "peephole-opts", |
| 373 | "Peephole Optimizations", false, false) |
| 374 | INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree) |
| 375 | INITIALIZE_PASS_END(PeepholeOptimizer, "peephole-opts", |
Owen Anderson | df7a4f2 | 2010-10-07 22:25:06 +0000 | [diff] [blame] | 376 | "Peephole Optimizations", false, false) |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 377 | |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 378 | /// optimizeExtInstr - If instruction is a copy-like instruction, i.e. it reads |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 379 | /// a single register and writes a single register and it does not modify the |
| 380 | /// source, and if the source value is preserved as a sub-register of the |
| 381 | /// result, then replace all reachable uses of the source with the subreg of the |
| 382 | /// result. |
Andrew Trick | 9e76199 | 2012-02-08 21:22:43 +0000 | [diff] [blame] | 383 | /// |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 384 | /// Do not generate an EXTRACT that is used only in a debug use, as this changes |
| 385 | /// the code. Since this code does not currently share EXTRACTs, just ignore all |
| 386 | /// debug uses. |
| 387 | bool PeepholeOptimizer:: |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 388 | optimizeExtInstr(MachineInstr *MI, MachineBasicBlock *MBB, |
Hans Wennborg | 97a59ae | 2014-08-11 13:52:46 +0000 | [diff] [blame] | 389 | SmallPtrSetImpl<MachineInstr*> &LocalMIs) { |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 390 | unsigned SrcReg, DstReg, SubIdx; |
| 391 | if (!TII->isCoalescableExtInstr(*MI, SrcReg, DstReg, SubIdx)) |
| 392 | return false; |
Andrew Trick | 9e76199 | 2012-02-08 21:22:43 +0000 | [diff] [blame] | 393 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 394 | if (TargetRegisterInfo::isPhysicalRegister(DstReg) || |
| 395 | TargetRegisterInfo::isPhysicalRegister(SrcReg)) |
| 396 | return false; |
| 397 | |
Jakob Stoklund Olesen | 8eb9905 | 2012-06-19 21:10:18 +0000 | [diff] [blame] | 398 | if (MRI->hasOneNonDBGUse(SrcReg)) |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 399 | // No other uses. |
| 400 | return false; |
| 401 | |
Jakob Stoklund Olesen | 2f06a65 | 2012-05-20 18:42:55 +0000 | [diff] [blame] | 402 | // Ensure DstReg can get a register class that actually supports |
| 403 | // sub-registers. Don't change the class until we commit. |
| 404 | const TargetRegisterClass *DstRC = MRI->getRegClass(DstReg); |
Eric Christopher | 92b4bcb | 2014-10-14 07:17:20 +0000 | [diff] [blame] | 405 | DstRC = TRI->getSubClassWithSubReg(DstRC, SubIdx); |
Jakob Stoklund Olesen | 2f06a65 | 2012-05-20 18:42:55 +0000 | [diff] [blame] | 406 | if (!DstRC) |
| 407 | return false; |
| 408 | |
Jakob Stoklund Olesen | 0f855e4 | 2012-06-19 21:14:34 +0000 | [diff] [blame] | 409 | // The ext instr may be operating on a sub-register of SrcReg as well. |
| 410 | // PPC::EXTSW is a 32 -> 64-bit sign extension, but it reads a 64-bit |
| 411 | // register. |
| 412 | // If UseSrcSubIdx is Set, SubIdx also applies to SrcReg, and only uses of |
| 413 | // SrcReg:SubIdx should be replaced. |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 414 | bool UseSrcSubIdx = |
Eric Christopher | 92b4bcb | 2014-10-14 07:17:20 +0000 | [diff] [blame] | 415 | TRI->getSubClassWithSubReg(MRI->getRegClass(SrcReg), SubIdx) != nullptr; |
Jakob Stoklund Olesen | 0f855e4 | 2012-06-19 21:14:34 +0000 | [diff] [blame] | 416 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 417 | // The source has other uses. See if we can replace the other uses with use of |
| 418 | // the result of the extension. |
| 419 | SmallPtrSet<MachineBasicBlock*, 4> ReachedBBs; |
Owen Anderson | b36376e | 2014-03-17 19:36:09 +0000 | [diff] [blame] | 420 | for (MachineInstr &UI : MRI->use_nodbg_instructions(DstReg)) |
| 421 | ReachedBBs.insert(UI.getParent()); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 422 | |
| 423 | // Uses that are in the same BB of uses of the result of the instruction. |
| 424 | SmallVector<MachineOperand*, 8> Uses; |
| 425 | |
| 426 | // Uses that the result of the instruction can reach. |
| 427 | SmallVector<MachineOperand*, 8> ExtendedUses; |
| 428 | |
| 429 | bool ExtendLife = true; |
Owen Anderson | b36376e | 2014-03-17 19:36:09 +0000 | [diff] [blame] | 430 | for (MachineOperand &UseMO : MRI->use_nodbg_operands(SrcReg)) { |
Owen Anderson | 16c6bf4 | 2014-03-13 23:12:04 +0000 | [diff] [blame] | 431 | MachineInstr *UseMI = UseMO.getParent(); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 432 | if (UseMI == MI) |
| 433 | continue; |
| 434 | |
| 435 | if (UseMI->isPHI()) { |
| 436 | ExtendLife = false; |
| 437 | continue; |
| 438 | } |
| 439 | |
Jakob Stoklund Olesen | 0f855e4 | 2012-06-19 21:14:34 +0000 | [diff] [blame] | 440 | // Only accept uses of SrcReg:SubIdx. |
| 441 | if (UseSrcSubIdx && UseMO.getSubReg() != SubIdx) |
| 442 | continue; |
| 443 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 444 | // It's an error to translate this: |
| 445 | // |
| 446 | // %reg1025 = <sext> %reg1024 |
| 447 | // ... |
| 448 | // %reg1026 = SUBREG_TO_REG 0, %reg1024, 4 |
| 449 | // |
| 450 | // into this: |
| 451 | // |
| 452 | // %reg1025 = <sext> %reg1024 |
| 453 | // ... |
| 454 | // %reg1027 = COPY %reg1025:4 |
| 455 | // %reg1026 = SUBREG_TO_REG 0, %reg1027, 4 |
| 456 | // |
| 457 | // The problem here is that SUBREG_TO_REG is there to assert that an |
| 458 | // implicit zext occurs. It doesn't insert a zext instruction. If we allow |
| 459 | // the COPY here, it will give us the value after the <sext>, not the |
| 460 | // original value of %reg1024 before <sext>. |
| 461 | if (UseMI->getOpcode() == TargetOpcode::SUBREG_TO_REG) |
| 462 | continue; |
| 463 | |
| 464 | MachineBasicBlock *UseMBB = UseMI->getParent(); |
| 465 | if (UseMBB == MBB) { |
| 466 | // Local uses that come after the extension. |
| 467 | if (!LocalMIs.count(UseMI)) |
| 468 | Uses.push_back(&UseMO); |
| 469 | } else if (ReachedBBs.count(UseMBB)) { |
| 470 | // Non-local uses where the result of the extension is used. Always |
| 471 | // replace these unless it's a PHI. |
| 472 | Uses.push_back(&UseMO); |
| 473 | } else if (Aggressive && DT->dominates(MBB, UseMBB)) { |
| 474 | // We may want to extend the live range of the extension result in order |
| 475 | // to replace these uses. |
| 476 | ExtendedUses.push_back(&UseMO); |
| 477 | } else { |
| 478 | // Both will be live out of the def MBB anyway. Don't extend live range of |
| 479 | // the extension result. |
| 480 | ExtendLife = false; |
| 481 | break; |
| 482 | } |
| 483 | } |
| 484 | |
| 485 | if (ExtendLife && !ExtendedUses.empty()) |
| 486 | // Extend the liveness of the extension result. |
Benjamin Kramer | 4f6ac16 | 2015-02-28 10:11:12 +0000 | [diff] [blame] | 487 | Uses.append(ExtendedUses.begin(), ExtendedUses.end()); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 488 | |
| 489 | // Now replace all uses. |
| 490 | bool Changed = false; |
| 491 | if (!Uses.empty()) { |
| 492 | SmallPtrSet<MachineBasicBlock*, 4> PHIBBs; |
| 493 | |
| 494 | // Look for PHI uses of the extended result, we don't want to extend the |
| 495 | // liveness of a PHI input. It breaks all kinds of assumptions down |
| 496 | // stream. A PHI use is expected to be the kill of its source values. |
Owen Anderson | b36376e | 2014-03-17 19:36:09 +0000 | [diff] [blame] | 497 | for (MachineInstr &UI : MRI->use_nodbg_instructions(DstReg)) |
| 498 | if (UI.isPHI()) |
| 499 | PHIBBs.insert(UI.getParent()); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 500 | |
| 501 | const TargetRegisterClass *RC = MRI->getRegClass(SrcReg); |
| 502 | for (unsigned i = 0, e = Uses.size(); i != e; ++i) { |
| 503 | MachineOperand *UseMO = Uses[i]; |
| 504 | MachineInstr *UseMI = UseMO->getParent(); |
| 505 | MachineBasicBlock *UseMBB = UseMI->getParent(); |
| 506 | if (PHIBBs.count(UseMBB)) |
| 507 | continue; |
| 508 | |
Lang Hames | d5862ce | 2012-02-25 02:01:00 +0000 | [diff] [blame] | 509 | // About to add uses of DstReg, clear DstReg's kill flags. |
Jakob Stoklund Olesen | 2f06a65 | 2012-05-20 18:42:55 +0000 | [diff] [blame] | 510 | if (!Changed) { |
Lang Hames | d5862ce | 2012-02-25 02:01:00 +0000 | [diff] [blame] | 511 | MRI->clearKillFlags(DstReg); |
Jakob Stoklund Olesen | 2f06a65 | 2012-05-20 18:42:55 +0000 | [diff] [blame] | 512 | MRI->constrainRegClass(DstReg, DstRC); |
| 513 | } |
Lang Hames | d5862ce | 2012-02-25 02:01:00 +0000 | [diff] [blame] | 514 | |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 515 | unsigned NewVR = MRI->createVirtualRegister(RC); |
Jakob Stoklund Olesen | 0f855e4 | 2012-06-19 21:14:34 +0000 | [diff] [blame] | 516 | MachineInstr *Copy = BuildMI(*UseMBB, UseMI, UseMI->getDebugLoc(), |
| 517 | TII->get(TargetOpcode::COPY), NewVR) |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 518 | .addReg(DstReg, 0, SubIdx); |
Jakob Stoklund Olesen | 0f855e4 | 2012-06-19 21:14:34 +0000 | [diff] [blame] | 519 | // SubIdx applies to both SrcReg and DstReg when UseSrcSubIdx is set. |
| 520 | if (UseSrcSubIdx) { |
| 521 | Copy->getOperand(0).setSubReg(SubIdx); |
| 522 | Copy->getOperand(0).setIsUndef(); |
| 523 | } |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 524 | UseMO->setReg(NewVR); |
| 525 | ++NumReuse; |
| 526 | Changed = true; |
| 527 | } |
| 528 | } |
| 529 | |
| 530 | return Changed; |
| 531 | } |
| 532 | |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 533 | /// optimizeCmpInstr - If the instruction is a compare and the previous |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 534 | /// instruction it's comparing against all ready sets (or could be modified to |
| 535 | /// set) the same flag as the compare, then we can remove the comparison and use |
| 536 | /// the flag from the previous instruction. |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 537 | bool PeepholeOptimizer::optimizeCmpInstr(MachineInstr *MI, |
Evan Cheng | e4b8ac9 | 2011-03-15 05:13:13 +0000 | [diff] [blame] | 538 | MachineBasicBlock *MBB) { |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 539 | // If this instruction is a comparison against zero and isn't comparing a |
| 540 | // physical register, we can try to optimize it. |
Manman Ren | 6fa76dc | 2012-06-29 21:33:59 +0000 | [diff] [blame] | 541 | unsigned SrcReg, SrcReg2; |
Gabor Greif | adbbb93 | 2010-09-21 12:01:15 +0000 | [diff] [blame] | 542 | int CmpMask, CmpValue; |
Manman Ren | 6fa76dc | 2012-06-29 21:33:59 +0000 | [diff] [blame] | 543 | if (!TII->analyzeCompare(MI, SrcReg, SrcReg2, CmpMask, CmpValue) || |
| 544 | TargetRegisterInfo::isPhysicalRegister(SrcReg) || |
| 545 | (SrcReg2 != 0 && TargetRegisterInfo::isPhysicalRegister(SrcReg2))) |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 546 | return false; |
| 547 | |
Bill Wendling | 27dddd1 | 2010-09-11 00:13:50 +0000 | [diff] [blame] | 548 | // Attempt to optimize the comparison instruction. |
Manman Ren | 6fa76dc | 2012-06-29 21:33:59 +0000 | [diff] [blame] | 549 | if (TII->optimizeCompareInstr(MI, SrcReg, SrcReg2, CmpMask, CmpValue, MRI)) { |
Evan Cheng | e4b8ac9 | 2011-03-15 05:13:13 +0000 | [diff] [blame] | 550 | ++NumCmps; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 551 | return true; |
| 552 | } |
| 553 | |
| 554 | return false; |
| 555 | } |
| 556 | |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 557 | /// Optimize a select instruction. |
Mehdi Amini | 22e5974 | 2015-01-13 07:07:13 +0000 | [diff] [blame] | 558 | bool PeepholeOptimizer::optimizeSelect(MachineInstr *MI, |
| 559 | SmallPtrSetImpl<MachineInstr *> &LocalMIs) { |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 560 | unsigned TrueOp = 0; |
| 561 | unsigned FalseOp = 0; |
| 562 | bool Optimizable = false; |
| 563 | SmallVector<MachineOperand, 4> Cond; |
| 564 | if (TII->analyzeSelect(MI, Cond, TrueOp, FalseOp, Optimizable)) |
| 565 | return false; |
| 566 | if (!Optimizable) |
| 567 | return false; |
Mehdi Amini | 22e5974 | 2015-01-13 07:07:13 +0000 | [diff] [blame] | 568 | if (!TII->optimizeSelect(MI, LocalMIs)) |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 569 | return false; |
| 570 | MI->eraseFromParent(); |
| 571 | ++NumSelects; |
| 572 | return true; |
| 573 | } |
| 574 | |
Gerolf Hoflehner | a4c96d0 | 2014-10-14 23:07:53 +0000 | [diff] [blame] | 575 | /// \brief Check if a simpler conditional branch can be |
| 576 | // generated |
| 577 | bool PeepholeOptimizer::optimizeCondBranch(MachineInstr *MI) { |
| 578 | return TII->optimizeCondBranch(MI); |
| 579 | } |
| 580 | |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 581 | /// \brief Check if the registers defined by the pair (RegisterClass, SubReg) |
| 582 | /// share the same register file. |
| 583 | static bool shareSameRegisterFile(const TargetRegisterInfo &TRI, |
| 584 | const TargetRegisterClass *DefRC, |
| 585 | unsigned DefSubReg, |
| 586 | const TargetRegisterClass *SrcRC, |
| 587 | unsigned SrcSubReg) { |
| 588 | // Same register class. |
| 589 | if (DefRC == SrcRC) |
| 590 | return true; |
| 591 | |
| 592 | // Both operands are sub registers. Check if they share a register class. |
| 593 | unsigned SrcIdx, DefIdx; |
| 594 | if (SrcSubReg && DefSubReg) |
| 595 | return TRI.getCommonSuperRegClass(SrcRC, SrcSubReg, DefRC, DefSubReg, |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 596 | SrcIdx, DefIdx) != nullptr; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 597 | // At most one of the register is a sub register, make it Src to avoid |
| 598 | // duplicating the test. |
| 599 | if (!SrcSubReg) { |
| 600 | std::swap(DefSubReg, SrcSubReg); |
| 601 | std::swap(DefRC, SrcRC); |
| 602 | } |
| 603 | |
| 604 | // One of the register is a sub register, check if we can get a superclass. |
| 605 | if (SrcSubReg) |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 606 | return TRI.getMatchingSuperRegClass(SrcRC, DefRC, SrcSubReg) != nullptr; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 607 | // Plain copy. |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 608 | return TRI.getCommonSubClass(DefRC, SrcRC) != nullptr; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 609 | } |
| 610 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 611 | /// \brief Try to find the next source that share the same register file |
| 612 | /// for the value defined by \p Reg and \p SubReg. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 613 | /// When true is returned, the \p RewriteMap can be used by the client to |
| 614 | /// retrieve all Def -> Use along the way up to the next source. Any found |
| 615 | /// Use that is not itself a key for another entry, is the next source to |
| 616 | /// use. During the search for the next source, multiple sources can be found |
| 617 | /// given multiple incoming sources of a PHI instruction. In this case, we |
| 618 | /// look in each PHI source for the next source; all found next sources must |
| 619 | /// share the same register file as \p Reg and \p SubReg. The client should |
| 620 | /// then be capable to rewrite all intermediate PHIs to get the next source. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 621 | /// \return False if no alternative sources are available. True otherwise. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 622 | bool PeepholeOptimizer::findNextSource(unsigned Reg, unsigned SubReg, |
| 623 | RewriteMapTy &RewriteMap) { |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 624 | // Do not try to find a new source for a physical register. |
| 625 | // So far we do not have any motivating example for doing that. |
| 626 | // Thus, instead of maintaining untested code, we will revisit that if |
| 627 | // that changes at some point. |
| 628 | if (TargetRegisterInfo::isPhysicalRegister(Reg)) |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 629 | return false; |
Bruno Cardoso Lopes | 38c0250 | 2015-07-29 17:46:47 +0000 | [diff] [blame] | 630 | const TargetRegisterClass *DefRC = MRI->getRegClass(Reg); |
Bruno Cardoso Lopes | 38c0250 | 2015-07-29 17:46:47 +0000 | [diff] [blame] | 631 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 632 | SmallVector<TargetInstrInfo::RegSubRegPair, 4> SrcToLook; |
| 633 | TargetInstrInfo::RegSubRegPair CurSrcPair(Reg, SubReg); |
| 634 | SrcToLook.push_back(CurSrcPair); |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 635 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 636 | unsigned PHICount = 0; |
| 637 | while (!SrcToLook.empty() && PHICount < RewritePHILimit) { |
| 638 | TargetInstrInfo::RegSubRegPair Pair = SrcToLook.pop_back_val(); |
| 639 | // As explained above, do not handle physical registers |
| 640 | if (TargetRegisterInfo::isPhysicalRegister(Pair.Reg)) |
| 641 | return false; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 642 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 643 | CurSrcPair = Pair; |
| 644 | ValueTracker ValTracker(CurSrcPair.Reg, CurSrcPair.SubReg, *MRI, |
| 645 | !DisableAdvCopyOpt, TII); |
| 646 | ValueTrackerResult Res; |
| 647 | bool ShouldRewrite = false; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 648 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 649 | do { |
| 650 | // Follow the chain of copies until we reach the top of the use-def chain |
| 651 | // or find a more suitable source. |
| 652 | Res = ValTracker.getNextSource(); |
| 653 | if (!Res.isValid()) |
| 654 | break; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 655 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 656 | // Insert the Def -> Use entry for the recently found source. |
| 657 | ValueTrackerResult CurSrcRes = RewriteMap.lookup(CurSrcPair); |
| 658 | if (CurSrcRes.isValid()) { |
| 659 | assert(CurSrcRes == Res && "ValueTrackerResult found must match"); |
| 660 | // An existent entry with multiple sources is a PHI cycle we must avoid. |
| 661 | // Otherwise it's an entry with a valid next source we already found. |
| 662 | if (CurSrcRes.getNumSources() > 1) { |
| 663 | DEBUG(dbgs() << "findNextSource: found PHI cycle, aborting...\n"); |
| 664 | return false; |
| 665 | } |
| 666 | break; |
| 667 | } |
| 668 | RewriteMap.insert(std::make_pair(CurSrcPair, Res)); |
| 669 | |
| 670 | // ValueTrackerResult usually have one source unless it's the result from |
| 671 | // a PHI instruction. Add the found PHI edges to be looked up further. |
| 672 | unsigned NumSrcs = Res.getNumSources(); |
| 673 | if (NumSrcs > 1) { |
| 674 | PHICount++; |
| 675 | for (unsigned i = 0; i < NumSrcs; ++i) |
| 676 | SrcToLook.push_back(TargetInstrInfo::RegSubRegPair( |
| 677 | Res.getSrcReg(i), Res.getSrcSubReg(i))); |
| 678 | break; |
| 679 | } |
| 680 | |
| 681 | CurSrcPair.Reg = Res.getSrcReg(0); |
| 682 | CurSrcPair.SubReg = Res.getSrcSubReg(0); |
| 683 | // Do not extend the live-ranges of physical registers as they add |
| 684 | // constraints to the register allocator. Moreover, if we want to extend |
| 685 | // the live-range of a physical register, unlike SSA virtual register, |
| 686 | // we will have to check that they aren't redefine before the related use. |
| 687 | if (TargetRegisterInfo::isPhysicalRegister(CurSrcPair.Reg)) |
| 688 | return false; |
| 689 | |
| 690 | const TargetRegisterClass *SrcRC = MRI->getRegClass(CurSrcPair.Reg); |
| 691 | |
| 692 | // If this source does not incur a cross register bank copy, use it. |
| 693 | ShouldRewrite = shareSameRegisterFile(*TRI, DefRC, SubReg, SrcRC, |
| 694 | CurSrcPair.SubReg); |
| 695 | } while (!ShouldRewrite); |
| 696 | |
| 697 | // Continue looking for new sources... |
| 698 | if (Res.isValid()) |
| 699 | continue; |
| 700 | |
| 701 | // Do not continue searching for a new source if the there's at least |
| 702 | // one use-def which cannot be rewritten. |
| 703 | if (!ShouldRewrite) |
| 704 | return false; |
| 705 | } |
| 706 | |
| 707 | if (PHICount >= RewritePHILimit) { |
| 708 | DEBUG(dbgs() << "findNextSource: PHI limit reached\n"); |
| 709 | return false; |
| 710 | } |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 711 | |
| 712 | // If we did not find a more suitable source, there is nothing to optimize. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 713 | if (CurSrcPair.Reg == Reg) |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 714 | return false; |
| 715 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 716 | return true; |
| 717 | } |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 718 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 719 | /// \brief Insert a PHI instruction with incoming edges \p SrcRegs that are |
| 720 | /// guaranteed to have the same register class. This is necessary whenever we |
| 721 | /// successfully traverse a PHI instruction and find suitable sources coming |
| 722 | /// from its edges. By inserting a new PHI, we provide a rewritten PHI def |
| 723 | /// suitable to be used in a new COPY instruction. |
| 724 | MachineInstr * |
| 725 | insertPHI(MachineRegisterInfo *MRI, const TargetInstrInfo *TII, |
| 726 | const SmallVectorImpl<TargetInstrInfo::RegSubRegPair> &SrcRegs, |
| 727 | MachineInstr *OrigPHI) { |
| 728 | assert(!SrcRegs.empty() && "No sources to create a PHI instruction?"); |
| 729 | |
| 730 | const TargetRegisterClass *NewRC = MRI->getRegClass(SrcRegs[0].Reg); |
| 731 | unsigned NewVR = MRI->createVirtualRegister(NewRC); |
| 732 | MachineBasicBlock *MBB = OrigPHI->getParent(); |
| 733 | MachineInstrBuilder MIB = BuildMI(*MBB, OrigPHI, OrigPHI->getDebugLoc(), |
| 734 | TII->get(TargetOpcode::PHI), NewVR); |
| 735 | |
| 736 | unsigned MBBOpIdx = 2; |
| 737 | for (auto RegPair : SrcRegs) { |
| 738 | MIB.addReg(RegPair.Reg, 0, RegPair.SubReg); |
| 739 | MIB.addMBB(OrigPHI->getOperand(MBBOpIdx).getMBB()); |
| 740 | // Since we're extended the lifetime of RegPair.Reg, clear the |
| 741 | // kill flags to account for that and make RegPair.Reg reaches |
| 742 | // the new PHI. |
| 743 | MRI->clearKillFlags(RegPair.Reg); |
| 744 | MBBOpIdx += 2; |
| 745 | } |
| 746 | |
| 747 | return MIB; |
| 748 | } |
| 749 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 750 | namespace { |
| 751 | /// \brief Helper class to rewrite the arguments of a copy-like instruction. |
| 752 | class CopyRewriter { |
| 753 | protected: |
| 754 | /// The copy-like instruction. |
| 755 | MachineInstr &CopyLike; |
| 756 | /// The index of the source being rewritten. |
| 757 | unsigned CurrentSrcIdx; |
| 758 | |
| 759 | public: |
| 760 | CopyRewriter(MachineInstr &MI) : CopyLike(MI), CurrentSrcIdx(0) {} |
| 761 | |
| 762 | virtual ~CopyRewriter() {} |
| 763 | |
| 764 | /// \brief Get the next rewritable source (SrcReg, SrcSubReg) and |
| 765 | /// the related value that it affects (TrackReg, TrackSubReg). |
| 766 | /// A source is considered rewritable if its register class and the |
| 767 | /// register class of the related TrackReg may not be register |
| 768 | /// coalescer friendly. In other words, given a copy-like instruction |
| 769 | /// not all the arguments may be returned at rewritable source, since |
| 770 | /// some arguments are none to be register coalescer friendly. |
| 771 | /// |
| 772 | /// Each call of this method moves the current source to the next |
| 773 | /// rewritable source. |
| 774 | /// For instance, let CopyLike be the instruction to rewrite. |
| 775 | /// CopyLike has one definition and one source: |
| 776 | /// dst.dstSubIdx = CopyLike src.srcSubIdx. |
| 777 | /// |
| 778 | /// The first call will give the first rewritable source, i.e., |
| 779 | /// the only source this instruction has: |
| 780 | /// (SrcReg, SrcSubReg) = (src, srcSubIdx). |
| 781 | /// This source defines the whole definition, i.e., |
| 782 | /// (TrackReg, TrackSubReg) = (dst, dstSubIdx). |
| 783 | /// |
| 784 | /// The second and subsequent calls will return false, has there is only one |
| 785 | /// rewritable source. |
| 786 | /// |
| 787 | /// \return True if a rewritable source has been found, false otherwise. |
| 788 | /// The output arguments are valid if and only if true is returned. |
| 789 | virtual bool getNextRewritableSource(unsigned &SrcReg, unsigned &SrcSubReg, |
| 790 | unsigned &TrackReg, |
| 791 | unsigned &TrackSubReg) { |
| 792 | // If CurrentSrcIdx == 1, this means this function has already been |
| 793 | // called once. CopyLike has one defintiion and one argument, thus, |
| 794 | // there is nothing else to rewrite. |
| 795 | if (!CopyLike.isCopy() || CurrentSrcIdx == 1) |
| 796 | return false; |
| 797 | // This is the first call to getNextRewritableSource. |
| 798 | // Move the CurrentSrcIdx to remember that we made that call. |
| 799 | CurrentSrcIdx = 1; |
| 800 | // The rewritable source is the argument. |
| 801 | const MachineOperand &MOSrc = CopyLike.getOperand(1); |
| 802 | SrcReg = MOSrc.getReg(); |
| 803 | SrcSubReg = MOSrc.getSubReg(); |
| 804 | // What we track are the alternative sources of the definition. |
| 805 | const MachineOperand &MODef = CopyLike.getOperand(0); |
| 806 | TrackReg = MODef.getReg(); |
| 807 | TrackSubReg = MODef.getSubReg(); |
| 808 | return true; |
| 809 | } |
| 810 | |
| 811 | /// \brief Rewrite the current source with \p NewReg and \p NewSubReg |
| 812 | /// if possible. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 813 | /// \return True if the rewriting was possible, false otherwise. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 814 | virtual bool RewriteCurrentSource(unsigned NewReg, unsigned NewSubReg) { |
| 815 | if (!CopyLike.isCopy() || CurrentSrcIdx != 1) |
| 816 | return false; |
| 817 | MachineOperand &MOSrc = CopyLike.getOperand(CurrentSrcIdx); |
| 818 | MOSrc.setReg(NewReg); |
| 819 | MOSrc.setSubReg(NewSubReg); |
| 820 | return true; |
| 821 | } |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 822 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 823 | /// \brief Given a \p Def.Reg and Def.SubReg pair, use \p RewriteMap to find |
| 824 | /// the new source to use for rewrite. If \p HandleMultipleSources is true and |
| 825 | /// multiple sources for a given \p Def are found along the way, we found a |
| 826 | /// PHI instructions that needs to be rewritten. |
| 827 | /// TODO: HandleMultipleSources should be removed once we test PHI handling |
| 828 | /// with coalescable copies. |
| 829 | TargetInstrInfo::RegSubRegPair |
| 830 | getNewSource(MachineRegisterInfo *MRI, const TargetInstrInfo *TII, |
| 831 | TargetInstrInfo::RegSubRegPair Def, |
| 832 | PeepholeOptimizer::RewriteMapTy &RewriteMap, |
| 833 | bool HandleMultipleSources = true) { |
| 834 | |
| 835 | TargetInstrInfo::RegSubRegPair LookupSrc(Def.Reg, Def.SubReg); |
| 836 | do { |
| 837 | ValueTrackerResult Res = RewriteMap.lookup(LookupSrc); |
| 838 | // If there are no entries on the map, LookupSrc is the new source. |
| 839 | if (!Res.isValid()) |
| 840 | return LookupSrc; |
| 841 | |
| 842 | // There's only one source for this definition, keep searching... |
| 843 | unsigned NumSrcs = Res.getNumSources(); |
| 844 | if (NumSrcs == 1) { |
| 845 | LookupSrc.Reg = Res.getSrcReg(0); |
| 846 | LookupSrc.SubReg = Res.getSrcSubReg(0); |
| 847 | continue; |
| 848 | } |
| 849 | |
| 850 | // TODO: remove once multiple srcs w/ coaslescable copies are supported. |
| 851 | if (!HandleMultipleSources) |
| 852 | break; |
| 853 | |
| 854 | // Multiple sources, recurse into each source to find a new source |
| 855 | // for it. Then, rewrite the PHI accordingly to its new edges. |
| 856 | SmallVector<TargetInstrInfo::RegSubRegPair, 4> NewPHISrcs; |
| 857 | for (unsigned i = 0; i < NumSrcs; ++i) { |
| 858 | TargetInstrInfo::RegSubRegPair PHISrc(Res.getSrcReg(i), |
| 859 | Res.getSrcSubReg(i)); |
| 860 | NewPHISrcs.push_back( |
| 861 | getNewSource(MRI, TII, PHISrc, RewriteMap, HandleMultipleSources)); |
| 862 | } |
| 863 | |
| 864 | // Build the new PHI node and return its def register as the new source. |
| 865 | MachineInstr *OrigPHI = const_cast<MachineInstr *>(Res.getInst()); |
| 866 | MachineInstr *NewPHI = insertPHI(MRI, TII, NewPHISrcs, OrigPHI); |
| 867 | DEBUG(dbgs() << "-- getNewSource\n"); |
| 868 | DEBUG(dbgs() << " Replacing: " << *OrigPHI); |
| 869 | DEBUG(dbgs() << " With: " << *NewPHI); |
| 870 | const MachineOperand &MODef = NewPHI->getOperand(0); |
| 871 | return TargetInstrInfo::RegSubRegPair(MODef.getReg(), MODef.getSubReg()); |
| 872 | |
| 873 | } while (1); |
| 874 | |
| 875 | return TargetInstrInfo::RegSubRegPair(0, 0); |
| 876 | } |
| 877 | |
| 878 | /// \brief Rewrite the source found through \p Def, by using the \p RewriteMap |
| 879 | /// and create a new COPY instruction. More info about RewriteMap in |
| 880 | /// PeepholeOptimizer::findNextSource. Right now this is only used to handle |
| 881 | /// Uncoalescable copies, since they are copy like instructions that aren't |
| 882 | /// recognized by the register allocator. |
| 883 | virtual MachineInstr * |
| 884 | RewriteSource(TargetInstrInfo::RegSubRegPair Def, |
| 885 | PeepholeOptimizer::RewriteMapTy &RewriteMap) { |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 886 | return nullptr; |
| 887 | } |
| 888 | }; |
| 889 | |
| 890 | /// \brief Helper class to rewrite uncoalescable copy like instructions |
| 891 | /// into new COPY (coalescable friendly) instructions. |
| 892 | class UncoalescableRewriter : public CopyRewriter { |
| 893 | protected: |
| 894 | const TargetInstrInfo &TII; |
| 895 | MachineRegisterInfo &MRI; |
| 896 | /// The number of defs in the bitcast |
| 897 | unsigned NumDefs; |
| 898 | |
| 899 | public: |
| 900 | UncoalescableRewriter(MachineInstr &MI, const TargetInstrInfo &TII, |
| 901 | MachineRegisterInfo &MRI) |
| 902 | : CopyRewriter(MI), TII(TII), MRI(MRI) { |
| 903 | NumDefs = MI.getDesc().getNumDefs(); |
| 904 | } |
| 905 | |
| 906 | /// \brief Get the next rewritable def source (TrackReg, TrackSubReg) |
| 907 | /// All such sources need to be considered rewritable in order to |
| 908 | /// rewrite a uncoalescable copy-like instruction. This method return |
| 909 | /// each definition that must be checked if rewritable. |
| 910 | /// |
| 911 | bool getNextRewritableSource(unsigned &SrcReg, unsigned &SrcSubReg, |
| 912 | unsigned &TrackReg, |
| 913 | unsigned &TrackSubReg) override { |
| 914 | // Find the next non-dead definition and continue from there. |
| 915 | if (CurrentSrcIdx == NumDefs) |
| 916 | return false; |
| 917 | |
| 918 | while (CopyLike.getOperand(CurrentSrcIdx).isDead()) { |
| 919 | ++CurrentSrcIdx; |
| 920 | if (CurrentSrcIdx == NumDefs) |
| 921 | return false; |
| 922 | } |
| 923 | |
| 924 | // What we track are the alternative sources of the definition. |
| 925 | const MachineOperand &MODef = CopyLike.getOperand(CurrentSrcIdx); |
| 926 | TrackReg = MODef.getReg(); |
| 927 | TrackSubReg = MODef.getSubReg(); |
| 928 | |
| 929 | CurrentSrcIdx++; |
| 930 | return true; |
| 931 | } |
| 932 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 933 | /// \brief Rewrite the source found through \p Def, by using the \p RewriteMap |
| 934 | /// and create a new COPY instruction. More info about RewriteMap in |
| 935 | /// PeepholeOptimizer::findNextSource. Right now this is only used to handle |
| 936 | /// Uncoalescable copies, since they are copy like instructions that aren't |
| 937 | /// recognized by the register allocator. |
| 938 | MachineInstr * |
| 939 | RewriteSource(TargetInstrInfo::RegSubRegPair Def, |
| 940 | PeepholeOptimizer::RewriteMapTy &RewriteMap) override { |
| 941 | assert(!TargetRegisterInfo::isPhysicalRegister(Def.Reg) && |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 942 | "We do not rewrite physical registers"); |
| 943 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 944 | // Find the new source to use in the COPY rewrite. |
| 945 | TargetInstrInfo::RegSubRegPair NewSrc = |
| 946 | getNewSource(&MRI, &TII, Def, RewriteMap); |
| 947 | |
| 948 | // Insert the COPY. |
| 949 | const TargetRegisterClass *DefRC = MRI.getRegClass(Def.Reg); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 950 | unsigned NewVR = MRI.createVirtualRegister(DefRC); |
| 951 | |
| 952 | MachineInstr *NewCopy = |
| 953 | BuildMI(*CopyLike.getParent(), &CopyLike, CopyLike.getDebugLoc(), |
| 954 | TII.get(TargetOpcode::COPY), NewVR) |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 955 | .addReg(NewSrc.Reg, 0, NewSrc.SubReg); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 956 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 957 | NewCopy->getOperand(0).setSubReg(Def.SubReg); |
| 958 | if (Def.SubReg) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 959 | NewCopy->getOperand(0).setIsUndef(); |
| 960 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 961 | DEBUG(dbgs() << "-- RewriteSource\n"); |
| 962 | DEBUG(dbgs() << " Replacing: " << CopyLike); |
| 963 | DEBUG(dbgs() << " With: " << *NewCopy); |
| 964 | MRI.replaceRegWith(Def.Reg, NewVR); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 965 | MRI.clearKillFlags(NewVR); |
| 966 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 967 | // We extended the lifetime of NewSrc.Reg, clear the kill flags to |
| 968 | // account for that. |
| 969 | MRI.clearKillFlags(NewSrc.Reg); |
| 970 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 971 | return NewCopy; |
| 972 | } |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 973 | }; |
| 974 | |
| 975 | /// \brief Specialized rewriter for INSERT_SUBREG instruction. |
| 976 | class InsertSubregRewriter : public CopyRewriter { |
| 977 | public: |
| 978 | InsertSubregRewriter(MachineInstr &MI) : CopyRewriter(MI) { |
| 979 | assert(MI.isInsertSubreg() && "Invalid instruction"); |
| 980 | } |
| 981 | |
| 982 | /// \brief See CopyRewriter::getNextRewritableSource. |
| 983 | /// Here CopyLike has the following form: |
| 984 | /// dst = INSERT_SUBREG Src1, Src2.src2SubIdx, subIdx. |
| 985 | /// Src1 has the same register class has dst, hence, there is |
| 986 | /// nothing to rewrite. |
| 987 | /// Src2.src2SubIdx, may not be register coalescer friendly. |
| 988 | /// Therefore, the first call to this method returns: |
| 989 | /// (SrcReg, SrcSubReg) = (Src2, src2SubIdx). |
| 990 | /// (TrackReg, TrackSubReg) = (dst, subIdx). |
| 991 | /// |
| 992 | /// Subsequence calls will return false. |
| 993 | bool getNextRewritableSource(unsigned &SrcReg, unsigned &SrcSubReg, |
| 994 | unsigned &TrackReg, |
| 995 | unsigned &TrackSubReg) override { |
| 996 | // If we already get the only source we can rewrite, return false. |
| 997 | if (CurrentSrcIdx == 2) |
| 998 | return false; |
| 999 | // We are looking at v2 = INSERT_SUBREG v0, v1, sub0. |
| 1000 | CurrentSrcIdx = 2; |
| 1001 | const MachineOperand &MOInsertedReg = CopyLike.getOperand(2); |
| 1002 | SrcReg = MOInsertedReg.getReg(); |
| 1003 | SrcSubReg = MOInsertedReg.getSubReg(); |
| 1004 | const MachineOperand &MODef = CopyLike.getOperand(0); |
| 1005 | |
| 1006 | // We want to track something that is compatible with the |
| 1007 | // partial definition. |
| 1008 | TrackReg = MODef.getReg(); |
| 1009 | if (MODef.getSubReg()) |
| 1010 | // Bails if we have to compose sub-register indices. |
| 1011 | return false; |
| 1012 | TrackSubReg = (unsigned)CopyLike.getOperand(3).getImm(); |
| 1013 | return true; |
| 1014 | } |
| 1015 | bool RewriteCurrentSource(unsigned NewReg, unsigned NewSubReg) override { |
| 1016 | if (CurrentSrcIdx != 2) |
| 1017 | return false; |
| 1018 | // We are rewriting the inserted reg. |
| 1019 | MachineOperand &MO = CopyLike.getOperand(CurrentSrcIdx); |
| 1020 | MO.setReg(NewReg); |
| 1021 | MO.setSubReg(NewSubReg); |
| 1022 | return true; |
| 1023 | } |
| 1024 | }; |
| 1025 | |
| 1026 | /// \brief Specialized rewriter for EXTRACT_SUBREG instruction. |
| 1027 | class ExtractSubregRewriter : public CopyRewriter { |
| 1028 | const TargetInstrInfo &TII; |
| 1029 | |
| 1030 | public: |
| 1031 | ExtractSubregRewriter(MachineInstr &MI, const TargetInstrInfo &TII) |
| 1032 | : CopyRewriter(MI), TII(TII) { |
| 1033 | assert(MI.isExtractSubreg() && "Invalid instruction"); |
| 1034 | } |
| 1035 | |
| 1036 | /// \brief See CopyRewriter::getNextRewritableSource. |
| 1037 | /// Here CopyLike has the following form: |
| 1038 | /// dst.dstSubIdx = EXTRACT_SUBREG Src, subIdx. |
| 1039 | /// There is only one rewritable source: Src.subIdx, |
| 1040 | /// which defines dst.dstSubIdx. |
| 1041 | bool getNextRewritableSource(unsigned &SrcReg, unsigned &SrcSubReg, |
| 1042 | unsigned &TrackReg, |
| 1043 | unsigned &TrackSubReg) override { |
| 1044 | // If we already get the only source we can rewrite, return false. |
| 1045 | if (CurrentSrcIdx == 1) |
| 1046 | return false; |
| 1047 | // We are looking at v1 = EXTRACT_SUBREG v0, sub0. |
| 1048 | CurrentSrcIdx = 1; |
| 1049 | const MachineOperand &MOExtractedReg = CopyLike.getOperand(1); |
| 1050 | SrcReg = MOExtractedReg.getReg(); |
| 1051 | // If we have to compose sub-register indices, bails out. |
| 1052 | if (MOExtractedReg.getSubReg()) |
| 1053 | return false; |
| 1054 | |
| 1055 | SrcSubReg = CopyLike.getOperand(2).getImm(); |
| 1056 | |
| 1057 | // We want to track something that is compatible with the definition. |
| 1058 | const MachineOperand &MODef = CopyLike.getOperand(0); |
| 1059 | TrackReg = MODef.getReg(); |
| 1060 | TrackSubReg = MODef.getSubReg(); |
| 1061 | return true; |
| 1062 | } |
| 1063 | |
| 1064 | bool RewriteCurrentSource(unsigned NewReg, unsigned NewSubReg) override { |
| 1065 | // The only source we can rewrite is the input register. |
| 1066 | if (CurrentSrcIdx != 1) |
| 1067 | return false; |
| 1068 | |
| 1069 | CopyLike.getOperand(CurrentSrcIdx).setReg(NewReg); |
| 1070 | |
| 1071 | // If we find a source that does not require to extract something, |
| 1072 | // rewrite the operation with a copy. |
| 1073 | if (!NewSubReg) { |
| 1074 | // Move the current index to an invalid position. |
| 1075 | // We do not want another call to this method to be able |
| 1076 | // to do any change. |
| 1077 | CurrentSrcIdx = -1; |
| 1078 | // Rewrite the operation as a COPY. |
| 1079 | // Get rid of the sub-register index. |
| 1080 | CopyLike.RemoveOperand(2); |
| 1081 | // Morph the operation into a COPY. |
| 1082 | CopyLike.setDesc(TII.get(TargetOpcode::COPY)); |
| 1083 | return true; |
| 1084 | } |
| 1085 | CopyLike.getOperand(CurrentSrcIdx + 1).setImm(NewSubReg); |
| 1086 | return true; |
| 1087 | } |
| 1088 | }; |
| 1089 | |
| 1090 | /// \brief Specialized rewriter for REG_SEQUENCE instruction. |
| 1091 | class RegSequenceRewriter : public CopyRewriter { |
| 1092 | public: |
| 1093 | RegSequenceRewriter(MachineInstr &MI) : CopyRewriter(MI) { |
| 1094 | assert(MI.isRegSequence() && "Invalid instruction"); |
| 1095 | } |
| 1096 | |
| 1097 | /// \brief See CopyRewriter::getNextRewritableSource. |
| 1098 | /// Here CopyLike has the following form: |
| 1099 | /// dst = REG_SEQUENCE Src1.src1SubIdx, subIdx1, Src2.src2SubIdx, subIdx2. |
| 1100 | /// Each call will return a different source, walking all the available |
| 1101 | /// source. |
| 1102 | /// |
| 1103 | /// The first call returns: |
| 1104 | /// (SrcReg, SrcSubReg) = (Src1, src1SubIdx). |
| 1105 | /// (TrackReg, TrackSubReg) = (dst, subIdx1). |
| 1106 | /// |
| 1107 | /// The second call returns: |
| 1108 | /// (SrcReg, SrcSubReg) = (Src2, src2SubIdx). |
| 1109 | /// (TrackReg, TrackSubReg) = (dst, subIdx2). |
| 1110 | /// |
| 1111 | /// And so on, until all the sources have been traversed, then |
| 1112 | /// it returns false. |
| 1113 | bool getNextRewritableSource(unsigned &SrcReg, unsigned &SrcSubReg, |
| 1114 | unsigned &TrackReg, |
| 1115 | unsigned &TrackSubReg) override { |
| 1116 | // We are looking at v0 = REG_SEQUENCE v1, sub1, v2, sub2, etc. |
| 1117 | |
| 1118 | // If this is the first call, move to the first argument. |
| 1119 | if (CurrentSrcIdx == 0) { |
| 1120 | CurrentSrcIdx = 1; |
| 1121 | } else { |
| 1122 | // Otherwise, move to the next argument and check that it is valid. |
| 1123 | CurrentSrcIdx += 2; |
| 1124 | if (CurrentSrcIdx >= CopyLike.getNumOperands()) |
| 1125 | return false; |
| 1126 | } |
| 1127 | const MachineOperand &MOInsertedReg = CopyLike.getOperand(CurrentSrcIdx); |
| 1128 | SrcReg = MOInsertedReg.getReg(); |
| 1129 | // If we have to compose sub-register indices, bails out. |
| 1130 | if ((SrcSubReg = MOInsertedReg.getSubReg())) |
| 1131 | return false; |
| 1132 | |
| 1133 | // We want to track something that is compatible with the related |
| 1134 | // partial definition. |
| 1135 | TrackSubReg = CopyLike.getOperand(CurrentSrcIdx + 1).getImm(); |
| 1136 | |
| 1137 | const MachineOperand &MODef = CopyLike.getOperand(0); |
| 1138 | TrackReg = MODef.getReg(); |
| 1139 | // If we have to compose sub-registers, bails. |
| 1140 | return MODef.getSubReg() == 0; |
| 1141 | } |
| 1142 | |
| 1143 | bool RewriteCurrentSource(unsigned NewReg, unsigned NewSubReg) override { |
| 1144 | // We cannot rewrite out of bound operands. |
| 1145 | // Moreover, rewritable sources are at odd positions. |
| 1146 | if ((CurrentSrcIdx & 1) != 1 || CurrentSrcIdx > CopyLike.getNumOperands()) |
| 1147 | return false; |
| 1148 | |
| 1149 | MachineOperand &MO = CopyLike.getOperand(CurrentSrcIdx); |
| 1150 | MO.setReg(NewReg); |
| 1151 | MO.setSubReg(NewSubReg); |
| 1152 | return true; |
| 1153 | } |
| 1154 | }; |
| 1155 | } // End namespace. |
| 1156 | |
| 1157 | /// \brief Get the appropriated CopyRewriter for \p MI. |
| 1158 | /// \return A pointer to a dynamically allocated CopyRewriter or nullptr |
| 1159 | /// if no rewriter works for \p MI. |
| 1160 | static CopyRewriter *getCopyRewriter(MachineInstr &MI, |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1161 | const TargetInstrInfo &TII, |
| 1162 | MachineRegisterInfo &MRI) { |
| 1163 | // Handle uncoalescable copy-like instructions. |
| 1164 | if (MI.isBitcast() || (MI.isRegSequenceLike() || MI.isInsertSubregLike() || |
| 1165 | MI.isExtractSubregLike())) |
| 1166 | return new UncoalescableRewriter(MI, TII, MRI); |
| 1167 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1168 | switch (MI.getOpcode()) { |
| 1169 | default: |
| 1170 | return nullptr; |
| 1171 | case TargetOpcode::COPY: |
| 1172 | return new CopyRewriter(MI); |
| 1173 | case TargetOpcode::INSERT_SUBREG: |
| 1174 | return new InsertSubregRewriter(MI); |
| 1175 | case TargetOpcode::EXTRACT_SUBREG: |
| 1176 | return new ExtractSubregRewriter(MI, TII); |
| 1177 | case TargetOpcode::REG_SEQUENCE: |
| 1178 | return new RegSequenceRewriter(MI); |
| 1179 | } |
| 1180 | llvm_unreachable(nullptr); |
| 1181 | } |
| 1182 | |
| 1183 | /// \brief Optimize generic copy instructions to avoid cross |
| 1184 | /// register bank copy. The optimization looks through a chain of |
| 1185 | /// copies and tries to find a source that has a compatible register |
| 1186 | /// class. |
| 1187 | /// Two register classes are considered to be compatible if they share |
| 1188 | /// the same register bank. |
| 1189 | /// New copies issued by this optimization are register allocator |
| 1190 | /// friendly. This optimization does not remove any copy as it may |
| 1191 | /// overconstraint the register allocator, but replaces some operands |
| 1192 | /// when possible. |
| 1193 | /// \pre isCoalescableCopy(*MI) is true. |
| 1194 | /// \return True, when \p MI has been rewritten. False otherwise. |
| 1195 | bool PeepholeOptimizer::optimizeCoalescableCopy(MachineInstr *MI) { |
| 1196 | assert(MI && isCoalescableCopy(*MI) && "Invalid argument"); |
| 1197 | assert(MI->getDesc().getNumDefs() == 1 && |
| 1198 | "Coalescer can understand multiple defs?!"); |
| 1199 | const MachineOperand &MODef = MI->getOperand(0); |
| 1200 | // Do not rewrite physical definitions. |
| 1201 | if (TargetRegisterInfo::isPhysicalRegister(MODef.getReg())) |
| 1202 | return false; |
| 1203 | |
| 1204 | bool Changed = false; |
| 1205 | // Get the right rewriter for the current copy. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1206 | std::unique_ptr<CopyRewriter> CpyRewriter(getCopyRewriter(*MI, *TII, *MRI)); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1207 | // If none exists, bails out. |
| 1208 | if (!CpyRewriter) |
| 1209 | return false; |
| 1210 | // Rewrite each rewritable source. |
| 1211 | unsigned SrcReg, SrcSubReg, TrackReg, TrackSubReg; |
| 1212 | while (CpyRewriter->getNextRewritableSource(SrcReg, SrcSubReg, TrackReg, |
| 1213 | TrackSubReg)) { |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1214 | // Keep track of PHI nodes and its incoming edges when looking for sources. |
| 1215 | RewriteMapTy RewriteMap; |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1216 | // Try to find a more suitable source. If we failed to do so, or get the |
| 1217 | // actual source, move to the next source. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1218 | if (!findNextSource(TrackReg, TrackSubReg, RewriteMap)) |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1219 | continue; |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1220 | |
| 1221 | // Get the new source to rewrite. TODO: Only enable handling of multiple |
| 1222 | // sources (PHIs) once we have a motivating example and testcases for it. |
| 1223 | TargetInstrInfo::RegSubRegPair TrackPair(TrackReg, TrackSubReg); |
| 1224 | TargetInstrInfo::RegSubRegPair NewSrc = CpyRewriter->getNewSource( |
| 1225 | MRI, TII, TrackPair, RewriteMap, false /* multiple sources */); |
| 1226 | if (SrcReg == NewSrc.Reg || NewSrc.Reg == 0) |
| 1227 | continue; |
| 1228 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1229 | // Rewrite source. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1230 | if (CpyRewriter->RewriteCurrentSource(NewSrc.Reg, NewSrc.SubReg)) { |
Quentin Colombet | 6b36337 | 2014-08-21 21:34:06 +0000 | [diff] [blame] | 1231 | // We may have extended the live-range of NewSrc, account for that. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1232 | MRI->clearKillFlags(NewSrc.Reg); |
Quentin Colombet | 6b36337 | 2014-08-21 21:34:06 +0000 | [diff] [blame] | 1233 | Changed = true; |
| 1234 | } |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1235 | } |
| 1236 | // TODO: We could have a clean-up method to tidy the instruction. |
| 1237 | // E.g., v0 = INSERT_SUBREG v1, v1.sub0, sub0 |
| 1238 | // => v0 = COPY v1 |
| 1239 | // Currently we haven't seen motivating example for that and we |
| 1240 | // want to avoid untested code. |
David Blaikie | dc3f01e | 2015-03-09 01:57:13 +0000 | [diff] [blame] | 1241 | NumRewrittenCopies += Changed; |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1242 | return Changed; |
| 1243 | } |
| 1244 | |
| 1245 | /// \brief Optimize copy-like instructions to create |
| 1246 | /// register coalescer friendly instruction. |
| 1247 | /// The optimization tries to kill-off the \p MI by looking |
| 1248 | /// through a chain of copies to find a source that has a compatible |
| 1249 | /// register class. |
| 1250 | /// If such a source is found, it replace \p MI by a generic COPY |
| 1251 | /// operation. |
| 1252 | /// \pre isUncoalescableCopy(*MI) is true. |
| 1253 | /// \return True, when \p MI has been optimized. In that case, \p MI has |
| 1254 | /// been removed from its parent. |
| 1255 | /// All COPY instructions created, are inserted in \p LocalMIs. |
| 1256 | bool PeepholeOptimizer::optimizeUncoalescableCopy( |
| 1257 | MachineInstr *MI, SmallPtrSetImpl<MachineInstr *> &LocalMIs) { |
| 1258 | assert(MI && isUncoalescableCopy(*MI) && "Invalid argument"); |
| 1259 | |
| 1260 | // Check if we can rewrite all the values defined by this instruction. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1261 | SmallVector<TargetInstrInfo::RegSubRegPair, 4> RewritePairs; |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1262 | // Get the right rewriter for the current copy. |
| 1263 | std::unique_ptr<CopyRewriter> CpyRewriter(getCopyRewriter(*MI, *TII, *MRI)); |
| 1264 | // If none exists, bails out. |
| 1265 | if (!CpyRewriter) |
| 1266 | return false; |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1267 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1268 | // Rewrite each rewritable source by generating new COPYs. This works |
| 1269 | // differently from optimizeCoalescableCopy since it first makes sure that all |
| 1270 | // definitions can be rewritten. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1271 | RewriteMapTy RewriteMap; |
| 1272 | unsigned Reg, SubReg, CopyDefReg, CopyDefSubReg; |
| 1273 | while (CpyRewriter->getNextRewritableSource(Reg, SubReg, CopyDefReg, |
| 1274 | CopyDefSubReg)) { |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1275 | // If a physical register is here, this is probably for a good reason. |
| 1276 | // Do not rewrite that. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1277 | if (TargetRegisterInfo::isPhysicalRegister(CopyDefReg)) |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1278 | return false; |
| 1279 | |
| 1280 | // If we do not know how to rewrite this definition, there is no point |
| 1281 | // in trying to kill this instruction. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1282 | TargetInstrInfo::RegSubRegPair Def(CopyDefReg, CopyDefSubReg); |
| 1283 | if (!findNextSource(Def.Reg, Def.SubReg, RewriteMap)) |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1284 | return false; |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1285 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1286 | RewritePairs.push_back(Def); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1287 | } |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1288 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1289 | // The change is possible for all defs, do it. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1290 | for (const auto &Def : RewritePairs) { |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1291 | // Rewrite the "copy" in a way the register coalescer understands. |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1292 | MachineInstr *NewCopy = CpyRewriter->RewriteSource(Def, RewriteMap); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1293 | assert(NewCopy && "Should be able to always generate a new copy"); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1294 | LocalMIs.insert(NewCopy); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1295 | } |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1296 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1297 | // MI is now dead. |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 1298 | MI->eraseFromParent(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1299 | ++NumUncoalescableCopies; |
Quentin Colombet | cf71c63 | 2013-09-13 18:26:31 +0000 | [diff] [blame] | 1300 | return true; |
| 1301 | } |
| 1302 | |
Manman Ren | 5759d01 | 2012-08-02 00:56:42 +0000 | [diff] [blame] | 1303 | /// isLoadFoldable - Check whether MI is a candidate for folding into a later |
| 1304 | /// instruction. We only fold loads to virtual registers and the virtual |
| 1305 | /// register defined has a single use. |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1306 | bool PeepholeOptimizer::isLoadFoldable( |
| 1307 | MachineInstr *MI, |
| 1308 | SmallSet<unsigned, 16> &FoldAsLoadDefCandidates) { |
Manman Ren | ba8122c | 2012-08-02 19:37:32 +0000 | [diff] [blame] | 1309 | if (!MI->canFoldAsLoad() || !MI->mayLoad()) |
| 1310 | return false; |
| 1311 | const MCInstrDesc &MCID = MI->getDesc(); |
| 1312 | if (MCID.getNumDefs() != 1) |
| 1313 | return false; |
| 1314 | |
| 1315 | unsigned Reg = MI->getOperand(0).getReg(); |
Ekaterina Romanova | 8d62008 | 2014-03-13 18:47:12 +0000 | [diff] [blame] | 1316 | // To reduce compilation time, we check MRI->hasOneNonDBGUse when inserting |
Manman Ren | ba8122c | 2012-08-02 19:37:32 +0000 | [diff] [blame] | 1317 | // loads. It should be checked when processing uses of the load, since |
| 1318 | // uses can be removed during peephole. |
| 1319 | if (!MI->getOperand(0).getSubReg() && |
| 1320 | TargetRegisterInfo::isVirtualRegister(Reg) && |
Ekaterina Romanova | 8d62008 | 2014-03-13 18:47:12 +0000 | [diff] [blame] | 1321 | MRI->hasOneNonDBGUse(Reg)) { |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1322 | FoldAsLoadDefCandidates.insert(Reg); |
Manman Ren | ba8122c | 2012-08-02 19:37:32 +0000 | [diff] [blame] | 1323 | return true; |
Manman Ren | 5759d01 | 2012-08-02 00:56:42 +0000 | [diff] [blame] | 1324 | } |
| 1325 | return false; |
| 1326 | } |
| 1327 | |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1328 | bool PeepholeOptimizer::isMoveImmediate(MachineInstr *MI, |
| 1329 | SmallSet<unsigned, 4> &ImmDefRegs, |
| 1330 | DenseMap<unsigned, MachineInstr*> &ImmDefMIs) { |
Evan Cheng | 6cc775f | 2011-06-28 19:10:37 +0000 | [diff] [blame] | 1331 | const MCInstrDesc &MCID = MI->getDesc(); |
Evan Cheng | 7f8e563 | 2011-12-07 07:15:52 +0000 | [diff] [blame] | 1332 | if (!MI->isMoveImmediate()) |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1333 | return false; |
Evan Cheng | 6cc775f | 2011-06-28 19:10:37 +0000 | [diff] [blame] | 1334 | if (MCID.getNumDefs() != 1) |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1335 | return false; |
| 1336 | unsigned Reg = MI->getOperand(0).getReg(); |
| 1337 | if (TargetRegisterInfo::isVirtualRegister(Reg)) { |
| 1338 | ImmDefMIs.insert(std::make_pair(Reg, MI)); |
| 1339 | ImmDefRegs.insert(Reg); |
| 1340 | return true; |
| 1341 | } |
Andrew Trick | 9e76199 | 2012-02-08 21:22:43 +0000 | [diff] [blame] | 1342 | |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1343 | return false; |
| 1344 | } |
| 1345 | |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 1346 | /// foldImmediate - Try folding register operands that are defined by move |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1347 | /// immediate instructions, i.e. a trivial constant folding optimization, if |
| 1348 | /// and only if the def and use are in the same BB. |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 1349 | bool PeepholeOptimizer::foldImmediate(MachineInstr *MI, MachineBasicBlock *MBB, |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1350 | SmallSet<unsigned, 4> &ImmDefRegs, |
| 1351 | DenseMap<unsigned, MachineInstr*> &ImmDefMIs) { |
| 1352 | for (unsigned i = 0, e = MI->getDesc().getNumOperands(); i != e; ++i) { |
| 1353 | MachineOperand &MO = MI->getOperand(i); |
| 1354 | if (!MO.isReg() || MO.isDef()) |
| 1355 | continue; |
| 1356 | unsigned Reg = MO.getReg(); |
Jakob Stoklund Olesen | 2fb5b31 | 2011-01-10 02:58:51 +0000 | [diff] [blame] | 1357 | if (!TargetRegisterInfo::isVirtualRegister(Reg)) |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1358 | continue; |
| 1359 | if (ImmDefRegs.count(Reg) == 0) |
| 1360 | continue; |
| 1361 | DenseMap<unsigned, MachineInstr*>::iterator II = ImmDefMIs.find(Reg); |
| 1362 | assert(II != ImmDefMIs.end()); |
| 1363 | if (TII->FoldImmediate(MI, II->second, Reg, MRI)) { |
| 1364 | ++NumImmFold; |
| 1365 | return true; |
| 1366 | } |
| 1367 | } |
| 1368 | return false; |
| 1369 | } |
| 1370 | |
Eric Christopher | 2181fb2 | 2014-10-15 21:06:25 +0000 | [diff] [blame] | 1371 | bool PeepholeOptimizer::runOnMachineFunction(MachineFunction &MF) { |
| 1372 | if (skipOptnoneFunction(*MF.getFunction())) |
Paul Robinson | 7c99ec5 | 2014-03-31 17:43:35 +0000 | [diff] [blame] | 1373 | return false; |
| 1374 | |
Craig Topper | 588ceec | 2012-12-17 03:56:00 +0000 | [diff] [blame] | 1375 | DEBUG(dbgs() << "********** PEEPHOLE OPTIMIZER **********\n"); |
Eric Christopher | 2181fb2 | 2014-10-15 21:06:25 +0000 | [diff] [blame] | 1376 | DEBUG(dbgs() << "********** Function: " << MF.getName() << '\n'); |
Craig Topper | 588ceec | 2012-12-17 03:56:00 +0000 | [diff] [blame] | 1377 | |
Evan Cheng | 2ce016c | 2010-11-15 21:20:45 +0000 | [diff] [blame] | 1378 | if (DisablePeephole) |
| 1379 | return false; |
Andrew Trick | 9e76199 | 2012-02-08 21:22:43 +0000 | [diff] [blame] | 1380 | |
Eric Christopher | 2181fb2 | 2014-10-15 21:06:25 +0000 | [diff] [blame] | 1381 | TII = MF.getSubtarget().getInstrInfo(); |
| 1382 | TRI = MF.getSubtarget().getRegisterInfo(); |
| 1383 | MRI = &MF.getRegInfo(); |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 1384 | DT = Aggressive ? &getAnalysis<MachineDominatorTree>() : nullptr; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1385 | |
| 1386 | bool Changed = false; |
| 1387 | |
Eric Christopher | 2181fb2 | 2014-10-15 21:06:25 +0000 | [diff] [blame] | 1388 | for (MachineFunction::iterator I = MF.begin(), E = MF.end(); I != E; ++I) { |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1389 | MachineBasicBlock *MBB = &*I; |
Andrew Trick | 9e76199 | 2012-02-08 21:22:43 +0000 | [diff] [blame] | 1390 | |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1391 | bool SeenMoveImm = false; |
Mehdi Amini | 22e5974 | 2015-01-13 07:07:13 +0000 | [diff] [blame] | 1392 | |
| 1393 | // During this forward scan, at some point it needs to answer the question |
| 1394 | // "given a pointer to an MI in the current BB, is it located before or |
| 1395 | // after the current instruction". |
| 1396 | // To perform this, the following set keeps track of the MIs already seen |
| 1397 | // during the scan, if a MI is not in the set, it is assumed to be located |
| 1398 | // after. Newly created MIs have to be inserted in the set as well. |
Hans Wennborg | 941a570 | 2014-08-11 02:50:43 +0000 | [diff] [blame] | 1399 | SmallPtrSet<MachineInstr*, 16> LocalMIs; |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1400 | SmallSet<unsigned, 4> ImmDefRegs; |
| 1401 | DenseMap<unsigned, MachineInstr*> ImmDefMIs; |
| 1402 | SmallSet<unsigned, 16> FoldAsLoadDefCandidates; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1403 | |
| 1404 | for (MachineBasicBlock::iterator |
Bill Wendling | aee679b | 2010-09-10 21:55:43 +0000 | [diff] [blame] | 1405 | MII = I->begin(), MIE = I->end(); MII != MIE; ) { |
Evan Cheng | 9bf3f8e | 2011-02-14 21:50:37 +0000 | [diff] [blame] | 1406 | MachineInstr *MI = &*MII; |
Jakob Stoklund Olesen | 714f595 | 2012-08-17 14:38:59 +0000 | [diff] [blame] | 1407 | // We may be erasing MI below, increment MII now. |
| 1408 | ++MII; |
Evan Cheng | 2ce016c | 2010-11-15 21:20:45 +0000 | [diff] [blame] | 1409 | LocalMIs.insert(MI); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1410 | |
Ekaterina Romanova | 8d62008 | 2014-03-13 18:47:12 +0000 | [diff] [blame] | 1411 | // Skip debug values. They should not affect this peephole optimization. |
| 1412 | if (MI->isDebugValue()) |
| 1413 | continue; |
| 1414 | |
Michael Kuperstein | bc7f99a | 2015-08-12 10:14:58 +0000 | [diff] [blame] | 1415 | // If we run into an instruction we can't fold across, discard |
| 1416 | // the load candidates. |
| 1417 | if (MI->isLoadFoldBarrier()) |
Michael Kuperstein | 82814f6 | 2015-08-11 08:19:43 +0000 | [diff] [blame] | 1418 | FoldAsLoadDefCandidates.clear(); |
| 1419 | |
Rafael Espindola | b1f25f1 | 2014-03-07 06:08:31 +0000 | [diff] [blame] | 1420 | if (MI->isPosition() || MI->isPHI() || MI->isImplicitDef() || |
Ekaterina Romanova | 8d62008 | 2014-03-13 18:47:12 +0000 | [diff] [blame] | 1421 | MI->isKill() || MI->isInlineAsm() || |
Michael Kuperstein | 82814f6 | 2015-08-11 08:19:43 +0000 | [diff] [blame] | 1422 | MI->hasUnmodeledSideEffects()) |
Evan Cheng | 2ce016c | 2010-11-15 21:20:45 +0000 | [diff] [blame] | 1423 | continue; |
| 1424 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1425 | if ((isUncoalescableCopy(*MI) && |
| 1426 | optimizeUncoalescableCopy(MI, LocalMIs)) || |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 1427 | (MI->isCompare() && optimizeCmpInstr(MI, MBB)) || |
Mehdi Amini | 22e5974 | 2015-01-13 07:07:13 +0000 | [diff] [blame] | 1428 | (MI->isSelect() && optimizeSelect(MI, LocalMIs))) { |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 1429 | // MI is deleted. |
| 1430 | LocalMIs.erase(MI); |
| 1431 | Changed = true; |
Jakob Stoklund Olesen | 2382d32 | 2012-08-16 23:11:47 +0000 | [diff] [blame] | 1432 | continue; |
Evan Cheng | 9bf3f8e | 2011-02-14 21:50:37 +0000 | [diff] [blame] | 1433 | } |
| 1434 | |
Gerolf Hoflehner | a4c96d0 | 2014-10-14 23:07:53 +0000 | [diff] [blame] | 1435 | if (MI->isConditionalBranch() && optimizeCondBranch(MI)) { |
| 1436 | Changed = true; |
| 1437 | continue; |
| 1438 | } |
| 1439 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1440 | if (isCoalescableCopy(*MI) && optimizeCoalescableCopy(MI)) { |
| 1441 | // MI is just rewritten. |
| 1442 | Changed = true; |
| 1443 | continue; |
| 1444 | } |
| 1445 | |
Evan Cheng | 9bf3f8e | 2011-02-14 21:50:37 +0000 | [diff] [blame] | 1446 | if (isMoveImmediate(MI, ImmDefRegs, ImmDefMIs)) { |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1447 | SeenMoveImm = true; |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1448 | } else { |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 1449 | Changed |= optimizeExtInstr(MI, MBB, LocalMIs); |
Rafael Espindola | 048405f | 2012-10-15 18:21:07 +0000 | [diff] [blame] | 1450 | // optimizeExtInstr might have created new instructions after MI |
| 1451 | // and before the already incremented MII. Adjust MII so that the |
| 1452 | // next iteration sees the new instructions. |
| 1453 | MII = MI; |
| 1454 | ++MII; |
Evan Cheng | 7f8ab6e | 2010-11-17 20:13:28 +0000 | [diff] [blame] | 1455 | if (SeenMoveImm) |
Jim Grosbach | edcb868 | 2012-05-01 23:21:41 +0000 | [diff] [blame] | 1456 | Changed |= foldImmediate(MI, MBB, ImmDefRegs, ImmDefMIs); |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1457 | } |
Evan Cheng | 98196b4 | 2011-02-15 05:00:24 +0000 | [diff] [blame] | 1458 | |
Manman Ren | 5759d01 | 2012-08-02 00:56:42 +0000 | [diff] [blame] | 1459 | // Check whether MI is a load candidate for folding into a later |
| 1460 | // instruction. If MI is not a candidate, check whether we can fold an |
| 1461 | // earlier load into MI. |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1462 | if (!isLoadFoldable(MI, FoldAsLoadDefCandidates) && |
| 1463 | !FoldAsLoadDefCandidates.empty()) { |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1464 | const MCInstrDesc &MIDesc = MI->getDesc(); |
| 1465 | for (unsigned i = MIDesc.getNumDefs(); i != MIDesc.getNumOperands(); |
| 1466 | ++i) { |
| 1467 | const MachineOperand &MOp = MI->getOperand(i); |
| 1468 | if (!MOp.isReg()) |
| 1469 | continue; |
Lang Hames | 3c0dc2a | 2014-04-03 05:03:20 +0000 | [diff] [blame] | 1470 | unsigned FoldAsLoadDefReg = MOp.getReg(); |
| 1471 | if (FoldAsLoadDefCandidates.count(FoldAsLoadDefReg)) { |
| 1472 | // We need to fold load after optimizeCmpInstr, since |
| 1473 | // optimizeCmpInstr can enable folding by converting SUB to CMP. |
| 1474 | // Save FoldAsLoadDefReg because optimizeLoadInstr() resets it and |
| 1475 | // we need it for markUsesInDebugValueAsUndef(). |
| 1476 | unsigned FoldedReg = FoldAsLoadDefReg; |
Craig Topper | c0196b1 | 2014-04-14 00:51:57 +0000 | [diff] [blame] | 1477 | MachineInstr *DefMI = nullptr; |
Lang Hames | 3c0dc2a | 2014-04-03 05:03:20 +0000 | [diff] [blame] | 1478 | MachineInstr *FoldMI = TII->optimizeLoadInstr(MI, MRI, |
| 1479 | FoldAsLoadDefReg, |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1480 | DefMI); |
| 1481 | if (FoldMI) { |
| 1482 | // Update LocalMIs since we replaced MI with FoldMI and deleted |
| 1483 | // DefMI. |
| 1484 | DEBUG(dbgs() << "Replacing: " << *MI); |
| 1485 | DEBUG(dbgs() << " With: " << *FoldMI); |
| 1486 | LocalMIs.erase(MI); |
| 1487 | LocalMIs.erase(DefMI); |
| 1488 | LocalMIs.insert(FoldMI); |
| 1489 | MI->eraseFromParent(); |
| 1490 | DefMI->eraseFromParent(); |
Lang Hames | 3c0dc2a | 2014-04-03 05:03:20 +0000 | [diff] [blame] | 1491 | MRI->markUsesInDebugValueAsUndef(FoldedReg); |
| 1492 | FoldAsLoadDefCandidates.erase(FoldedReg); |
Lang Hames | 5dc14bd | 2014-04-02 22:59:58 +0000 | [diff] [blame] | 1493 | ++NumLoadFold; |
| 1494 | // MI is replaced with FoldMI. |
| 1495 | Changed = true; |
| 1496 | break; |
| 1497 | } |
| 1498 | } |
Manman Ren | 5759d01 | 2012-08-02 00:56:42 +0000 | [diff] [blame] | 1499 | } |
| 1500 | } |
Bill Wendling | ca67835 | 2010-08-09 23:59:04 +0000 | [diff] [blame] | 1501 | } |
| 1502 | } |
| 1503 | |
| 1504 | return Changed; |
| 1505 | } |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1506 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1507 | ValueTrackerResult ValueTracker::getNextSourceFromCopy() { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1508 | assert(Def->isCopy() && "Invalid definition"); |
| 1509 | // Copy instruction are supposed to be: Def = Src. |
| 1510 | // If someone breaks this assumption, bad things will happen everywhere. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1511 | assert(Def->getNumOperands() == 2 && "Invalid number of operands"); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1512 | |
| 1513 | if (Def->getOperand(DefIdx).getSubReg() != DefSubReg) |
| 1514 | // If we look for a different subreg, it means we want a subreg of src. |
| 1515 | // Bails as we do not support composing subreg yet. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1516 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1517 | // Otherwise, we want the whole source. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1518 | const MachineOperand &Src = Def->getOperand(1); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1519 | return ValueTrackerResult(Src.getReg(), Src.getSubReg()); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1520 | } |
| 1521 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1522 | ValueTrackerResult ValueTracker::getNextSourceFromBitcast() { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1523 | assert(Def->isBitcast() && "Invalid definition"); |
| 1524 | |
| 1525 | // Bail if there are effects that a plain copy will not expose. |
| 1526 | if (Def->hasUnmodeledSideEffects()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1527 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1528 | |
| 1529 | // Bitcasts with more than one def are not supported. |
| 1530 | if (Def->getDesc().getNumDefs() != 1) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1531 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1532 | if (Def->getOperand(DefIdx).getSubReg() != DefSubReg) |
| 1533 | // If we look for a different subreg, it means we want a subreg of the src. |
| 1534 | // Bails as we do not support composing subreg yet. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1535 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1536 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1537 | unsigned SrcIdx = Def->getNumOperands(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1538 | for (unsigned OpIdx = DefIdx + 1, EndOpIdx = SrcIdx; OpIdx != EndOpIdx; |
| 1539 | ++OpIdx) { |
| 1540 | const MachineOperand &MO = Def->getOperand(OpIdx); |
| 1541 | if (!MO.isReg() || !MO.getReg()) |
| 1542 | continue; |
| 1543 | assert(!MO.isDef() && "We should have skipped all the definitions by now"); |
| 1544 | if (SrcIdx != EndOpIdx) |
| 1545 | // Multiple sources? |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1546 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1547 | SrcIdx = OpIdx; |
| 1548 | } |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1549 | const MachineOperand &Src = Def->getOperand(SrcIdx); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1550 | return ValueTrackerResult(Src.getReg(), Src.getSubReg()); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1551 | } |
| 1552 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1553 | ValueTrackerResult ValueTracker::getNextSourceFromRegSequence() { |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1554 | assert((Def->isRegSequence() || Def->isRegSequenceLike()) && |
| 1555 | "Invalid definition"); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1556 | |
| 1557 | if (Def->getOperand(DefIdx).getSubReg()) |
| 1558 | // If we are composing subreg, bails out. |
| 1559 | // The case we are checking is Def.<subreg> = REG_SEQUENCE. |
| 1560 | // This should almost never happen as the SSA property is tracked at |
| 1561 | // the register level (as opposed to the subreg level). |
| 1562 | // I.e., |
| 1563 | // Def.sub0 = |
| 1564 | // Def.sub1 = |
| 1565 | // is a valid SSA representation for Def.sub0 and Def.sub1, but not for |
| 1566 | // Def. Thus, it must not be generated. |
Quentin Colombet | 6d590d5 | 2014-07-01 16:23:44 +0000 | [diff] [blame] | 1567 | // However, some code could theoretically generates a single |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1568 | // Def.sub0 (i.e, not defining the other subregs) and we would |
| 1569 | // have this case. |
| 1570 | // If we can ascertain (or force) that this never happens, we could |
| 1571 | // turn that into an assertion. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1572 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1573 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1574 | if (!TII) |
| 1575 | // We could handle the REG_SEQUENCE here, but we do not want to |
| 1576 | // duplicate the code from the generic TII. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1577 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1578 | |
| 1579 | SmallVector<TargetInstrInfo::RegSubRegPairAndIdx, 8> RegSeqInputRegs; |
| 1580 | if (!TII->getRegSequenceInputs(*Def, DefIdx, RegSeqInputRegs)) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1581 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1582 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1583 | // We are looking at: |
| 1584 | // Def = REG_SEQUENCE v0, sub0, v1, sub1, ... |
| 1585 | // Check if one of the operand defines the subreg we are interested in. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1586 | for (auto &RegSeqInput : RegSeqInputRegs) { |
| 1587 | if (RegSeqInput.SubIdx == DefSubReg) { |
| 1588 | if (RegSeqInput.SubReg) |
| 1589 | // Bails if we have to compose sub registers. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1590 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1591 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1592 | return ValueTrackerResult(RegSeqInput.Reg, RegSeqInput.SubReg); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1593 | } |
| 1594 | } |
| 1595 | |
| 1596 | // If the subreg we are tracking is super-defined by another subreg, |
| 1597 | // we could follow this value. However, this would require to compose |
| 1598 | // the subreg and we do not do that for now. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1599 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1600 | } |
| 1601 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1602 | ValueTrackerResult ValueTracker::getNextSourceFromInsertSubreg() { |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 1603 | assert((Def->isInsertSubreg() || Def->isInsertSubregLike()) && |
| 1604 | "Invalid definition"); |
| 1605 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1606 | if (Def->getOperand(DefIdx).getSubReg()) |
| 1607 | // If we are composing subreg, bails out. |
| 1608 | // Same remark as getNextSourceFromRegSequence. |
| 1609 | // I.e., this may be turned into an assert. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1610 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1611 | |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 1612 | if (!TII) |
| 1613 | // We could handle the REG_SEQUENCE here, but we do not want to |
| 1614 | // duplicate the code from the generic TII. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1615 | return ValueTrackerResult(); |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 1616 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1617 | TargetInstrInfo::RegSubRegPair BaseReg; |
| 1618 | TargetInstrInfo::RegSubRegPairAndIdx InsertedReg; |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 1619 | if (!TII->getInsertSubregInputs(*Def, DefIdx, BaseReg, InsertedReg)) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1620 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1621 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1622 | // We are looking at: |
| 1623 | // Def = INSERT_SUBREG v0, v1, sub1 |
| 1624 | // There are two cases: |
| 1625 | // 1. DefSubReg == sub1, get v1. |
| 1626 | // 2. DefSubReg != sub1, the value may be available through v0. |
| 1627 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1628 | // #1 Check if the inserted register matches the required sub index. |
| 1629 | if (InsertedReg.SubIdx == DefSubReg) { |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1630 | return ValueTrackerResult(InsertedReg.Reg, InsertedReg.SubReg); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1631 | } |
| 1632 | // #2 Otherwise, if the sub register we are looking for is not partial |
| 1633 | // defined by the inserted element, we can look through the main |
| 1634 | // register (v0). |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1635 | const MachineOperand &MODef = Def->getOperand(DefIdx); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1636 | // If the result register (Def) and the base register (v0) do not |
| 1637 | // have the same register class or if we have to compose |
| 1638 | // subregisters, bails out. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1639 | if (MRI.getRegClass(MODef.getReg()) != MRI.getRegClass(BaseReg.Reg) || |
| 1640 | BaseReg.SubReg) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1641 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1642 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1643 | // Get the TRI and check if the inserted sub-register overlaps with the |
| 1644 | // sub-register we are tracking. |
| 1645 | const TargetRegisterInfo *TRI = MRI.getTargetRegisterInfo(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1646 | if (!TRI || |
| 1647 | (TRI->getSubRegIndexLaneMask(DefSubReg) & |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1648 | TRI->getSubRegIndexLaneMask(InsertedReg.SubIdx)) != 0) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1649 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1650 | // At this point, the value is available in v0 via the same subreg |
| 1651 | // we used for Def. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1652 | return ValueTrackerResult(BaseReg.Reg, DefSubReg); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1653 | } |
| 1654 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1655 | ValueTrackerResult ValueTracker::getNextSourceFromExtractSubreg() { |
Quentin Colombet | 67639df | 2014-08-20 23:13:02 +0000 | [diff] [blame] | 1656 | assert((Def->isExtractSubreg() || |
| 1657 | Def->isExtractSubregLike()) && "Invalid definition"); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1658 | // We are looking at: |
| 1659 | // Def = EXTRACT_SUBREG v0, sub0 |
| 1660 | |
| 1661 | // Bails if we have to compose sub registers. |
| 1662 | // Indeed, if DefSubReg != 0, we would have to compose it with sub0. |
| 1663 | if (DefSubReg) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1664 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1665 | |
Quentin Colombet | 67639df | 2014-08-20 23:13:02 +0000 | [diff] [blame] | 1666 | if (!TII) |
| 1667 | // We could handle the EXTRACT_SUBREG here, but we do not want to |
| 1668 | // duplicate the code from the generic TII. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1669 | return ValueTrackerResult(); |
Quentin Colombet | 67639df | 2014-08-20 23:13:02 +0000 | [diff] [blame] | 1670 | |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1671 | TargetInstrInfo::RegSubRegPairAndIdx ExtractSubregInputReg; |
Quentin Colombet | 67639df | 2014-08-20 23:13:02 +0000 | [diff] [blame] | 1672 | if (!TII->getExtractSubregInputs(*Def, DefIdx, ExtractSubregInputReg)) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1673 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1674 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1675 | // Bails if we have to compose sub registers. |
| 1676 | // Likewise, if v0.subreg != 0, we would have to compose v0.subreg with sub0. |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1677 | if (ExtractSubregInputReg.SubReg) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1678 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1679 | // Otherwise, the value is available in the v0.sub0. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1680 | return ValueTrackerResult(ExtractSubregInputReg.Reg, ExtractSubregInputReg.SubIdx); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1681 | } |
| 1682 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1683 | ValueTrackerResult ValueTracker::getNextSourceFromSubregToReg() { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1684 | assert(Def->isSubregToReg() && "Invalid definition"); |
| 1685 | // We are looking at: |
| 1686 | // Def = SUBREG_TO_REG Imm, v0, sub0 |
| 1687 | |
| 1688 | // Bails if we have to compose sub registers. |
| 1689 | // If DefSubReg != sub0, we would have to check that all the bits |
| 1690 | // we track are included in sub0 and if yes, we would have to |
| 1691 | // determine the right subreg in v0. |
| 1692 | if (DefSubReg != Def->getOperand(3).getImm()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1693 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1694 | // Bails if we have to compose sub registers. |
| 1695 | // Likewise, if v0.subreg != 0, we would have to compose it with sub0. |
| 1696 | if (Def->getOperand(2).getSubReg()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1697 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1698 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1699 | return ValueTrackerResult(Def->getOperand(2).getReg(), |
| 1700 | Def->getOperand(3).getImm()); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1701 | } |
| 1702 | |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1703 | /// \brief Explore each PHI incoming operand and return its sources |
| 1704 | ValueTrackerResult ValueTracker::getNextSourceFromPHI() { |
| 1705 | assert(Def->isPHI() && "Invalid definition"); |
| 1706 | ValueTrackerResult Res; |
| 1707 | |
| 1708 | // If we look for a different subreg, bails as we do not |
| 1709 | // support composing subreg yet. |
| 1710 | if (Def->getOperand(0).getSubReg() != DefSubReg) |
| 1711 | return ValueTrackerResult(); |
| 1712 | |
| 1713 | // Return all register sources for PHI instructions. |
| 1714 | for (unsigned i = 1, e = Def->getNumOperands(); i < e; i += 2) { |
| 1715 | auto &MO = Def->getOperand(i); |
| 1716 | assert(MO.isReg() && "Invalid PHI instruction"); |
| 1717 | Res.addSource(MO.getReg(), MO.getSubReg()); |
| 1718 | } |
| 1719 | |
| 1720 | return Res; |
| 1721 | } |
| 1722 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1723 | ValueTrackerResult ValueTracker::getNextSourceImpl() { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1724 | assert(Def && "This method needs a valid definition"); |
| 1725 | |
| 1726 | assert( |
| 1727 | (DefIdx < Def->getDesc().getNumDefs() || Def->getDesc().isVariadic()) && |
| 1728 | Def->getOperand(DefIdx).isDef() && "Invalid DefIdx"); |
| 1729 | if (Def->isCopy()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1730 | return getNextSourceFromCopy(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1731 | if (Def->isBitcast()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1732 | return getNextSourceFromBitcast(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1733 | // All the remaining cases involve "complex" instructions. |
| 1734 | // Bails if we did not ask for the advanced tracking. |
| 1735 | if (!UseAdvancedTracking) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1736 | return ValueTrackerResult(); |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1737 | if (Def->isRegSequence() || Def->isRegSequenceLike()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1738 | return getNextSourceFromRegSequence(); |
Quentin Colombet | 6896230 | 2014-08-21 00:19:16 +0000 | [diff] [blame] | 1739 | if (Def->isInsertSubreg() || Def->isInsertSubregLike()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1740 | return getNextSourceFromInsertSubreg(); |
Quentin Colombet | 67639df | 2014-08-20 23:13:02 +0000 | [diff] [blame] | 1741 | if (Def->isExtractSubreg() || Def->isExtractSubregLike()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1742 | return getNextSourceFromExtractSubreg(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1743 | if (Def->isSubregToReg()) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1744 | return getNextSourceFromSubregToReg(); |
Bruno Cardoso Lopes | 27fd069 | 2015-08-19 18:53:36 +0000 | [diff] [blame^] | 1745 | if (Def->isPHI()) |
| 1746 | return getNextSourceFromPHI(); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1747 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1748 | } |
| 1749 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1750 | ValueTrackerResult ValueTracker::getNextSource() { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1751 | // If we reach a point where we cannot move up in the use-def chain, |
| 1752 | // there is nothing we can get. |
| 1753 | if (!Def) |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1754 | return ValueTrackerResult(); |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1755 | |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1756 | ValueTrackerResult Res = getNextSourceImpl(); |
| 1757 | if (Res.isValid()) { |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1758 | // Update definition, definition index, and subregister for the |
| 1759 | // next call of getNextSource. |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1760 | // Update the current register. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1761 | bool OneRegSrc = Res.getNumSources() == 1; |
| 1762 | if (OneRegSrc) |
| 1763 | Reg = Res.getSrcReg(0); |
| 1764 | // Update the result before moving up in the use-def chain |
| 1765 | // with the instruction containing the last found sources. |
| 1766 | Res.setInst(Def); |
| 1767 | |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1768 | // If we can still move up in the use-def chain, move to the next |
Benjamin Kramer | df005cb | 2015-08-08 18:27:36 +0000 | [diff] [blame] | 1769 | // definition. |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1770 | if (!TargetRegisterInfo::isPhysicalRegister(Reg) && OneRegSrc) { |
Quentin Colombet | 03e43f8 | 2014-08-20 17:41:48 +0000 | [diff] [blame] | 1771 | Def = MRI.getVRegDef(Reg); |
| 1772 | DefIdx = MRI.def_begin(Reg).getOperandNo(); |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1773 | DefSubReg = Res.getSrcSubReg(0); |
| 1774 | return Res; |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1775 | } |
| 1776 | } |
| 1777 | // If we end up here, this means we will not be able to find another source |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1778 | // for the next iteration. Make sure any new call to getNextSource bails out |
| 1779 | // early by cutting the use-def chain. |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1780 | Def = nullptr; |
Bruno Cardoso Lopes | f16ec12 | 2015-07-22 21:30:16 +0000 | [diff] [blame] | 1781 | return Res; |
Quentin Colombet | 1111e6f | 2014-07-01 14:33:36 +0000 | [diff] [blame] | 1782 | } |