Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 1 | //===-------- InlineSpiller.cpp - Insert spills and restores inline -------===// |
| 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 | // The inline spiller modifies the machine function directly instead of |
| 11 | // inserting spills and restores in VirtRegMap. |
| 12 | // |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
Jakob Stoklund Olesen | 376dcbd | 2010-11-03 20:39:23 +0000 | [diff] [blame] | 15 | #define DEBUG_TYPE "regalloc" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 16 | #include "Spiller.h" |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 17 | #include "LiveRangeEdit.h" |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 18 | #include "SplitKit.h" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 19 | #include "VirtRegMap.h" |
Jakob Stoklund Olesen | e93198a | 2010-11-10 23:55:56 +0000 | [diff] [blame] | 20 | #include "llvm/Analysis/AliasAnalysis.h" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 21 | #include "llvm/CodeGen/LiveIntervalAnalysis.h" |
Jakob Stoklund Olesen | 0a12b80 | 2010-10-26 00:11:35 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/LiveStackAnalysis.h" |
Jakob Stoklund Olesen | d68f458 | 2010-10-28 20:34:50 +0000 | [diff] [blame] | 23 | #include "llvm/CodeGen/MachineDominators.h" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 24 | #include "llvm/CodeGen/MachineFrameInfo.h" |
| 25 | #include "llvm/CodeGen/MachineFunction.h" |
Jakob Stoklund Olesen | 9529a1c | 2010-07-19 18:41:20 +0000 | [diff] [blame] | 26 | #include "llvm/CodeGen/MachineLoopInfo.h" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 27 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
| 28 | #include "llvm/Target/TargetMachine.h" |
| 29 | #include "llvm/Target/TargetInstrInfo.h" |
Jakob Stoklund Olesen | 26b92be | 2010-10-28 20:34:47 +0000 | [diff] [blame] | 30 | #include "llvm/Support/CommandLine.h" |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 31 | #include "llvm/Support/Debug.h" |
| 32 | #include "llvm/Support/raw_ostream.h" |
| 33 | |
| 34 | using namespace llvm; |
| 35 | |
Jakob Stoklund Olesen | 26b92be | 2010-10-28 20:34:47 +0000 | [diff] [blame] | 36 | static cl::opt<bool> |
| 37 | VerifySpills("verify-spills", cl::desc("Verify after each spill/split")); |
| 38 | |
Jakob Stoklund Olesen | 8c42f48 | 2010-11-04 00:32:32 +0000 | [diff] [blame] | 39 | static cl::opt<bool> |
| 40 | ExtraSpillerSplits("extra-spiller-splits", |
| 41 | cl::desc("Enable additional splitting during splitting")); |
| 42 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 43 | namespace { |
| 44 | class InlineSpiller : public Spiller { |
Jakob Stoklund Olesen | 6d108e2 | 2010-08-06 18:47:06 +0000 | [diff] [blame] | 45 | MachineFunctionPass &pass_; |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 46 | MachineFunction &mf_; |
| 47 | LiveIntervals &lis_; |
Jakob Stoklund Olesen | 0a12b80 | 2010-10-26 00:11:35 +0000 | [diff] [blame] | 48 | LiveStacks &lss_; |
Jakob Stoklund Olesen | d68f458 | 2010-10-28 20:34:50 +0000 | [diff] [blame] | 49 | MachineDominatorTree &mdt_; |
Jakob Stoklund Olesen | 9529a1c | 2010-07-19 18:41:20 +0000 | [diff] [blame] | 50 | MachineLoopInfo &loops_; |
Jakob Stoklund Olesen | e93198a | 2010-11-10 23:55:56 +0000 | [diff] [blame] | 51 | AliasAnalysis *aa_; |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 52 | VirtRegMap &vrm_; |
| 53 | MachineFrameInfo &mfi_; |
| 54 | MachineRegisterInfo &mri_; |
| 55 | const TargetInstrInfo &tii_; |
| 56 | const TargetRegisterInfo &tri_; |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 57 | const BitVector reserved_; |
| 58 | |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 59 | SplitAnalysis splitAnalysis_; |
| 60 | |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 61 | // Variables that are valid during spill(), but used by multiple methods. |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 62 | LiveRangeEdit *edit_; |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 63 | const TargetRegisterClass *rc_; |
| 64 | int stackSlot_; |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 65 | |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 66 | // Values that failed to remat at some point. |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 67 | SmallPtrSet<VNInfo*, 8> usedValues_; |
| 68 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 69 | ~InlineSpiller() {} |
| 70 | |
| 71 | public: |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 72 | InlineSpiller(MachineFunctionPass &pass, |
| 73 | MachineFunction &mf, |
| 74 | VirtRegMap &vrm) |
Jakob Stoklund Olesen | 6d108e2 | 2010-08-06 18:47:06 +0000 | [diff] [blame] | 75 | : pass_(pass), |
| 76 | mf_(mf), |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 77 | lis_(pass.getAnalysis<LiveIntervals>()), |
Jakob Stoklund Olesen | 0a12b80 | 2010-10-26 00:11:35 +0000 | [diff] [blame] | 78 | lss_(pass.getAnalysis<LiveStacks>()), |
Jakob Stoklund Olesen | d68f458 | 2010-10-28 20:34:50 +0000 | [diff] [blame] | 79 | mdt_(pass.getAnalysis<MachineDominatorTree>()), |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 80 | loops_(pass.getAnalysis<MachineLoopInfo>()), |
Jakob Stoklund Olesen | e93198a | 2010-11-10 23:55:56 +0000 | [diff] [blame] | 81 | aa_(&pass.getAnalysis<AliasAnalysis>()), |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 82 | vrm_(vrm), |
| 83 | mfi_(*mf.getFrameInfo()), |
| 84 | mri_(mf.getRegInfo()), |
| 85 | tii_(*mf.getTarget().getInstrInfo()), |
| 86 | tri_(*mf.getTarget().getRegisterInfo()), |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 87 | reserved_(tri_.getReservedRegs(mf_)), |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 88 | splitAnalysis_(mf, lis_, loops_) {} |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 89 | |
| 90 | void spill(LiveInterval *li, |
Jakob Stoklund Olesen | 0a2b2a1 | 2010-08-13 22:56:53 +0000 | [diff] [blame] | 91 | SmallVectorImpl<LiveInterval*> &newIntervals, |
Andrew Trick | f4baeaf | 2010-11-10 19:18:47 +0000 | [diff] [blame] | 92 | const SmallVectorImpl<LiveInterval*> &spillIs); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 93 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 94 | void spill(LiveRangeEdit &); |
| 95 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 96 | private: |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 97 | bool split(); |
| 98 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 99 | bool reMaterializeFor(MachineBasicBlock::iterator MI); |
| 100 | void reMaterializeAll(); |
| 101 | |
Jakob Stoklund Olesen | 1a0f91b | 2010-08-04 22:35:11 +0000 | [diff] [blame] | 102 | bool coalesceStackAccess(MachineInstr *MI); |
Jakob Stoklund Olesen | e72a5c5 | 2010-07-01 00:13:04 +0000 | [diff] [blame] | 103 | bool foldMemoryOperand(MachineBasicBlock::iterator MI, |
| 104 | const SmallVectorImpl<unsigned> &Ops); |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 105 | void insertReload(LiveInterval &NewLI, MachineBasicBlock::iterator MI); |
| 106 | void insertSpill(LiveInterval &NewLI, MachineBasicBlock::iterator MI); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 107 | }; |
| 108 | } |
| 109 | |
| 110 | namespace llvm { |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 111 | Spiller *createInlineSpiller(MachineFunctionPass &pass, |
| 112 | MachineFunction &mf, |
| 113 | VirtRegMap &vrm) { |
Jakob Stoklund Olesen | 1f46a0a | 2010-10-29 00:40:55 +0000 | [diff] [blame] | 114 | if (VerifySpills) |
| 115 | mf.verify(&pass); |
Jakob Stoklund Olesen | f2c6e36 | 2010-07-20 23:50:15 +0000 | [diff] [blame] | 116 | return new InlineSpiller(pass, mf, vrm); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 117 | } |
| 118 | } |
| 119 | |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 120 | /// split - try splitting the current interval into pieces that may allocate |
| 121 | /// separately. Return true if successful. |
| 122 | bool InlineSpiller::split() { |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 123 | splitAnalysis_.analyze(&edit_->getParent()); |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 124 | |
Jakob Stoklund Olesen | 57d0f2d | 2010-10-05 22:19:33 +0000 | [diff] [blame] | 125 | // Try splitting around loops. |
Jakob Stoklund Olesen | 8c42f48 | 2010-11-04 00:32:32 +0000 | [diff] [blame] | 126 | if (ExtraSpillerSplits) { |
| 127 | const MachineLoop *loop = splitAnalysis_.getBestSplitLoop(); |
| 128 | if (loop) { |
| 129 | SplitEditor(splitAnalysis_, lis_, vrm_, mdt_, *edit_) |
| 130 | .splitAroundLoop(loop); |
| 131 | return true; |
| 132 | } |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 133 | } |
Jakob Stoklund Olesen | f1b05f2 | 2010-08-12 17:07:14 +0000 | [diff] [blame] | 134 | |
| 135 | // Try splitting into single block intervals. |
| 136 | SplitAnalysis::BlockPtrSet blocks; |
| 137 | if (splitAnalysis_.getMultiUseBlocks(blocks)) { |
Jakob Stoklund Olesen | d68f458 | 2010-10-28 20:34:50 +0000 | [diff] [blame] | 138 | SplitEditor(splitAnalysis_, lis_, vrm_, mdt_, *edit_) |
Jakob Stoklund Olesen | 57d0f2d | 2010-10-05 22:19:33 +0000 | [diff] [blame] | 139 | .splitSingleBlocks(blocks); |
| 140 | return true; |
Jakob Stoklund Olesen | f1b05f2 | 2010-08-12 17:07:14 +0000 | [diff] [blame] | 141 | } |
| 142 | |
Jakob Stoklund Olesen | fc412d8 | 2010-08-13 21:18:48 +0000 | [diff] [blame] | 143 | // Try splitting inside a basic block. |
Jakob Stoklund Olesen | 8c42f48 | 2010-11-04 00:32:32 +0000 | [diff] [blame] | 144 | if (ExtraSpillerSplits) { |
| 145 | const MachineBasicBlock *MBB = splitAnalysis_.getBlockForInsideSplit(); |
| 146 | if (MBB){ |
| 147 | SplitEditor(splitAnalysis_, lis_, vrm_, mdt_, *edit_) |
| 148 | .splitInsideBlock(MBB); |
| 149 | return true; |
| 150 | } |
Jakob Stoklund Olesen | fc412d8 | 2010-08-13 21:18:48 +0000 | [diff] [blame] | 151 | } |
| 152 | |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 153 | return false; |
| 154 | } |
| 155 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 156 | /// reMaterializeFor - Attempt to rematerialize edit_->getReg() before MI instead of |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 157 | /// reloading it. |
| 158 | bool InlineSpiller::reMaterializeFor(MachineBasicBlock::iterator MI) { |
| 159 | SlotIndex UseIdx = lis_.getInstructionIndex(MI).getUseIndex(); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 160 | VNInfo *OrigVNI = edit_->getParent().getVNInfoAt(UseIdx); |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 161 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 162 | if (!OrigVNI) { |
| 163 | DEBUG(dbgs() << "\tadding <undef> flags: "); |
| 164 | for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) { |
| 165 | MachineOperand &MO = MI->getOperand(i); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 166 | if (MO.isReg() && MO.isUse() && MO.getReg() == edit_->getReg()) |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 167 | MO.setIsUndef(); |
| 168 | } |
| 169 | DEBUG(dbgs() << UseIdx << '\t' << *MI); |
| 170 | return true; |
| 171 | } |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 172 | |
Jakob Stoklund Olesen | b80e973 | 2010-11-10 01:05:12 +0000 | [diff] [blame] | 173 | LiveRangeEdit::Remat RM(OrigVNI); |
| 174 | if (!edit_->canRematerializeAt(RM, UseIdx, false, lis_)) { |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 175 | usedValues_.insert(OrigVNI); |
| 176 | DEBUG(dbgs() << "\tcannot remat for " << UseIdx << '\t' << *MI); |
| 177 | return false; |
| 178 | } |
| 179 | |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 180 | // If the instruction also writes edit_->getReg(), it had better not require |
| 181 | // the same register for uses and defs. |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 182 | bool Reads, Writes; |
| 183 | SmallVector<unsigned, 8> Ops; |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 184 | tie(Reads, Writes) = MI->readsWritesVirtualRegister(edit_->getReg(), &Ops); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 185 | if (Writes) { |
| 186 | for (unsigned i = 0, e = Ops.size(); i != e; ++i) { |
| 187 | MachineOperand &MO = MI->getOperand(Ops[i]); |
| 188 | if (MO.isUse() ? MI->isRegTiedToDefOperand(Ops[i]) : MO.getSubReg()) { |
| 189 | usedValues_.insert(OrigVNI); |
| 190 | DEBUG(dbgs() << "\tcannot remat tied reg: " << UseIdx << '\t' << *MI); |
| 191 | return false; |
| 192 | } |
| 193 | } |
| 194 | } |
| 195 | |
| 196 | // Alocate a new register for the remat. |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 197 | LiveInterval &NewLI = edit_->create(mri_, lis_, vrm_); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 198 | NewLI.markNotSpillable(); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 199 | |
| 200 | // Finally we can rematerialize OrigMI before MI. |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 201 | SlotIndex DefIdx = edit_->rematerializeAt(*MI->getParent(), MI, NewLI.reg, RM, |
| 202 | lis_, tii_, tri_); |
| 203 | DEBUG(dbgs() << "\tremat: " << DefIdx << '\n'); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 204 | |
| 205 | // Replace operands |
| 206 | for (unsigned i = 0, e = Ops.size(); i != e; ++i) { |
| 207 | MachineOperand &MO = MI->getOperand(Ops[i]); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 208 | if (MO.isReg() && MO.isUse() && MO.getReg() == edit_->getReg()) { |
| 209 | MO.setReg(NewLI.reg); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 210 | MO.setIsKill(); |
| 211 | } |
| 212 | } |
| 213 | DEBUG(dbgs() << "\t " << UseIdx << '\t' << *MI); |
| 214 | |
Lang Hames | 6e2968c | 2010-09-25 12:04:16 +0000 | [diff] [blame] | 215 | VNInfo *DefVNI = NewLI.getNextValue(DefIdx, 0, lis_.getVNInfoAllocator()); |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 216 | NewLI.addRange(LiveRange(DefIdx, UseIdx.getDefIndex(), DefVNI)); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 217 | DEBUG(dbgs() << "\tinterval: " << NewLI << '\n'); |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 218 | return true; |
| 219 | } |
| 220 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 221 | /// reMaterializeAll - Try to rematerialize as many uses as possible, |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 222 | /// and trim the live ranges after. |
| 223 | void InlineSpiller::reMaterializeAll() { |
| 224 | // Do a quick scan of the interval values to find if any are remattable. |
Jakob Stoklund Olesen | e93198a | 2010-11-10 23:55:56 +0000 | [diff] [blame] | 225 | if (!edit_->anyRematerializable(lis_, tii_, aa_)) |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 226 | return; |
| 227 | |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 228 | usedValues_.clear(); |
| 229 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 230 | // Try to remat before all uses of edit_->getReg(). |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 231 | bool anyRemat = false; |
| 232 | for (MachineRegisterInfo::use_nodbg_iterator |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 233 | RI = mri_.use_nodbg_begin(edit_->getReg()); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 234 | MachineInstr *MI = RI.skipInstruction();) |
| 235 | anyRemat |= reMaterializeFor(MI); |
| 236 | |
| 237 | if (!anyRemat) |
| 238 | return; |
| 239 | |
| 240 | // Remove any values that were completely rematted. |
| 241 | bool anyRemoved = false; |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 242 | for (LiveInterval::vni_iterator I = edit_->getParent().vni_begin(), |
| 243 | E = edit_->getParent().vni_end(); I != E; ++I) { |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 244 | VNInfo *VNI = *I; |
Jakob Stoklund Olesen | 080c316 | 2010-10-20 22:00:51 +0000 | [diff] [blame] | 245 | if (VNI->hasPHIKill() || !edit_->didRematerialize(VNI) || |
| 246 | usedValues_.count(VNI)) |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 247 | continue; |
| 248 | MachineInstr *DefMI = lis_.getInstructionFromIndex(VNI->def); |
| 249 | DEBUG(dbgs() << "\tremoving dead def: " << VNI->def << '\t' << *DefMI); |
| 250 | lis_.RemoveMachineInstrFromMaps(DefMI); |
| 251 | vrm_.RemoveMachineInstrFromMaps(DefMI); |
| 252 | DefMI->eraseFromParent(); |
Lang Hames | cec2945 | 2010-09-26 03:37:09 +0000 | [diff] [blame] | 253 | VNI->def = SlotIndex(); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 254 | anyRemoved = true; |
| 255 | } |
| 256 | |
| 257 | if (!anyRemoved) |
| 258 | return; |
| 259 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 260 | // Removing values may cause debug uses where parent is not live. |
| 261 | for (MachineRegisterInfo::use_iterator RI = mri_.use_begin(edit_->getReg()); |
Jakob Stoklund Olesen | 3b9c7eb | 2010-07-02 19:54:40 +0000 | [diff] [blame] | 262 | MachineInstr *MI = RI.skipInstruction();) { |
| 263 | if (!MI->isDebugValue()) |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 264 | continue; |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 265 | // Try to preserve the debug value if parent is live immediately after it. |
Jakob Stoklund Olesen | 3b9c7eb | 2010-07-02 19:54:40 +0000 | [diff] [blame] | 266 | MachineBasicBlock::iterator NextMI = MI; |
| 267 | ++NextMI; |
| 268 | if (NextMI != MI->getParent()->end() && !lis_.isNotInMIMap(NextMI)) { |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 269 | SlotIndex Idx = lis_.getInstructionIndex(NextMI); |
| 270 | VNInfo *VNI = edit_->getParent().getVNInfoAt(Idx); |
Jakob Stoklund Olesen | b67b12e | 2010-08-10 20:45:07 +0000 | [diff] [blame] | 271 | if (VNI && (VNI->hasPHIKill() || usedValues_.count(VNI))) |
Jakob Stoklund Olesen | 3b9c7eb | 2010-07-02 19:54:40 +0000 | [diff] [blame] | 272 | continue; |
| 273 | } |
| 274 | DEBUG(dbgs() << "Removing debug info due to remat:" << "\t" << *MI); |
Jakob Stoklund Olesen | 3b9c7eb | 2010-07-02 19:54:40 +0000 | [diff] [blame] | 275 | MI->eraseFromParent(); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 276 | } |
| 277 | } |
| 278 | |
Jakob Stoklund Olesen | 1a0f91b | 2010-08-04 22:35:11 +0000 | [diff] [blame] | 279 | /// If MI is a load or store of stackSlot_, it can be removed. |
| 280 | bool InlineSpiller::coalesceStackAccess(MachineInstr *MI) { |
| 281 | int FI = 0; |
| 282 | unsigned reg; |
| 283 | if (!(reg = tii_.isLoadFromStackSlot(MI, FI)) && |
| 284 | !(reg = tii_.isStoreToStackSlot(MI, FI))) |
| 285 | return false; |
| 286 | |
| 287 | // We have a stack access. Is it the right register and slot? |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 288 | if (reg != edit_->getReg() || FI != stackSlot_) |
Jakob Stoklund Olesen | 1a0f91b | 2010-08-04 22:35:11 +0000 | [diff] [blame] | 289 | return false; |
| 290 | |
| 291 | DEBUG(dbgs() << "Coalescing stack access: " << *MI); |
| 292 | lis_.RemoveMachineInstrFromMaps(MI); |
| 293 | MI->eraseFromParent(); |
| 294 | return true; |
| 295 | } |
| 296 | |
Jakob Stoklund Olesen | e72a5c5 | 2010-07-01 00:13:04 +0000 | [diff] [blame] | 297 | /// foldMemoryOperand - Try folding stack slot references in Ops into MI. |
| 298 | /// Return true on success, and MI will be erased. |
| 299 | bool InlineSpiller::foldMemoryOperand(MachineBasicBlock::iterator MI, |
| 300 | const SmallVectorImpl<unsigned> &Ops) { |
| 301 | // TargetInstrInfo::foldMemoryOperand only expects explicit, non-tied |
| 302 | // operands. |
| 303 | SmallVector<unsigned, 8> FoldOps; |
| 304 | for (unsigned i = 0, e = Ops.size(); i != e; ++i) { |
| 305 | unsigned Idx = Ops[i]; |
| 306 | MachineOperand &MO = MI->getOperand(Idx); |
| 307 | if (MO.isImplicit()) |
| 308 | continue; |
| 309 | // FIXME: Teach targets to deal with subregs. |
| 310 | if (MO.getSubReg()) |
| 311 | return false; |
| 312 | // Tied use operands should not be passed to foldMemoryOperand. |
| 313 | if (!MI->isRegTiedToDefOperand(Idx)) |
| 314 | FoldOps.push_back(Idx); |
| 315 | } |
| 316 | |
Jakob Stoklund Olesen | e05442d | 2010-07-09 17:29:08 +0000 | [diff] [blame] | 317 | MachineInstr *FoldMI = tii_.foldMemoryOperand(MI, FoldOps, stackSlot_); |
Jakob Stoklund Olesen | e72a5c5 | 2010-07-01 00:13:04 +0000 | [diff] [blame] | 318 | if (!FoldMI) |
| 319 | return false; |
Jakob Stoklund Olesen | e72a5c5 | 2010-07-01 00:13:04 +0000 | [diff] [blame] | 320 | lis_.ReplaceMachineInstrInMaps(MI, FoldMI); |
| 321 | vrm_.addSpillSlotUse(stackSlot_, FoldMI); |
Jakob Stoklund Olesen | e05442d | 2010-07-09 17:29:08 +0000 | [diff] [blame] | 322 | MI->eraseFromParent(); |
Jakob Stoklund Olesen | e72a5c5 | 2010-07-01 00:13:04 +0000 | [diff] [blame] | 323 | DEBUG(dbgs() << "\tfolded: " << *FoldMI); |
| 324 | return true; |
| 325 | } |
| 326 | |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 327 | /// insertReload - Insert a reload of NewLI.reg before MI. |
| 328 | void InlineSpiller::insertReload(LiveInterval &NewLI, |
| 329 | MachineBasicBlock::iterator MI) { |
| 330 | MachineBasicBlock &MBB = *MI->getParent(); |
| 331 | SlotIndex Idx = lis_.getInstructionIndex(MI).getDefIndex(); |
| 332 | tii_.loadRegFromStackSlot(MBB, MI, NewLI.reg, stackSlot_, rc_, &tri_); |
| 333 | --MI; // Point to load instruction. |
| 334 | SlotIndex LoadIdx = lis_.InsertMachineInstrInMaps(MI).getDefIndex(); |
| 335 | vrm_.addSpillSlotUse(stackSlot_, MI); |
| 336 | DEBUG(dbgs() << "\treload: " << LoadIdx << '\t' << *MI); |
Lang Hames | 6e2968c | 2010-09-25 12:04:16 +0000 | [diff] [blame] | 337 | VNInfo *LoadVNI = NewLI.getNextValue(LoadIdx, 0, |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 338 | lis_.getVNInfoAllocator()); |
| 339 | NewLI.addRange(LiveRange(LoadIdx, Idx, LoadVNI)); |
| 340 | } |
| 341 | |
| 342 | /// insertSpill - Insert a spill of NewLI.reg after MI. |
| 343 | void InlineSpiller::insertSpill(LiveInterval &NewLI, |
| 344 | MachineBasicBlock::iterator MI) { |
| 345 | MachineBasicBlock &MBB = *MI->getParent(); |
Jakob Stoklund Olesen | 68257e6 | 2010-11-15 20:55:49 +0000 | [diff] [blame^] | 346 | |
| 347 | // Get the defined value. It could be an early clobber so keep the def index. |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 348 | SlotIndex Idx = lis_.getInstructionIndex(MI).getDefIndex(); |
Jakob Stoklund Olesen | 68257e6 | 2010-11-15 20:55:49 +0000 | [diff] [blame^] | 349 | VNInfo *VNI = edit_->getParent().getVNInfoAt(Idx); |
| 350 | assert(VNI && VNI->def.getDefIndex() == Idx && "Inconsistent VNInfo"); |
| 351 | Idx = VNI->def; |
| 352 | |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 353 | tii_.storeRegToStackSlot(MBB, ++MI, NewLI.reg, true, stackSlot_, rc_, &tri_); |
| 354 | --MI; // Point to store instruction. |
| 355 | SlotIndex StoreIdx = lis_.InsertMachineInstrInMaps(MI).getDefIndex(); |
| 356 | vrm_.addSpillSlotUse(stackSlot_, MI); |
| 357 | DEBUG(dbgs() << "\tspilled: " << StoreIdx << '\t' << *MI); |
Lang Hames | 6e2968c | 2010-09-25 12:04:16 +0000 | [diff] [blame] | 358 | VNInfo *StoreVNI = NewLI.getNextValue(Idx, 0, lis_.getVNInfoAllocator()); |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 359 | NewLI.addRange(LiveRange(Idx, StoreIdx, StoreVNI)); |
| 360 | } |
| 361 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 362 | void InlineSpiller::spill(LiveInterval *li, |
Jakob Stoklund Olesen | 0a2b2a1 | 2010-08-13 22:56:53 +0000 | [diff] [blame] | 363 | SmallVectorImpl<LiveInterval*> &newIntervals, |
Andrew Trick | f4baeaf | 2010-11-10 19:18:47 +0000 | [diff] [blame] | 364 | const SmallVectorImpl<LiveInterval*> &spillIs) { |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 365 | LiveRangeEdit edit(*li, newIntervals, spillIs); |
| 366 | spill(edit); |
Jakob Stoklund Olesen | 26b92be | 2010-10-28 20:34:47 +0000 | [diff] [blame] | 367 | if (VerifySpills) |
| 368 | mf_.verify(&pass_); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 369 | } |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 370 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 371 | void InlineSpiller::spill(LiveRangeEdit &edit) { |
| 372 | edit_ = &edit; |
Jakob Stoklund Olesen | 7d57753 | 2010-10-30 01:26:09 +0000 | [diff] [blame] | 373 | assert(!edit.getParent().isStackSlot() && "Trying to spill a stack slot."); |
| 374 | DEBUG(dbgs() << "Inline spilling " |
| 375 | << mri_.getRegClass(edit.getReg())->getName() |
| 376 | << ':' << edit.getParent() << "\n"); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 377 | assert(edit.getParent().isSpillable() && |
| 378 | "Attempting to spill already spilled value."); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 379 | |
Jakob Stoklund Olesen | 8ae0263 | 2010-07-20 15:41:07 +0000 | [diff] [blame] | 380 | if (split()) |
| 381 | return; |
| 382 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 383 | reMaterializeAll(); |
| 384 | |
| 385 | // Remat may handle everything. |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 386 | if (edit_->getParent().empty()) |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 387 | return; |
| 388 | |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 389 | rc_ = mri_.getRegClass(edit.getReg()); |
Jakob Stoklund Olesen | a37d5cf | 2010-11-01 19:49:57 +0000 | [diff] [blame] | 390 | stackSlot_ = vrm_.assignVirt2StackSlot(edit_->getReg()); |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 391 | |
Jakob Stoklund Olesen | 0a12b80 | 2010-10-26 00:11:35 +0000 | [diff] [blame] | 392 | // Update LiveStacks now that we are committed to spilling. |
| 393 | LiveInterval &stacklvr = lss_.getOrCreateInterval(stackSlot_, rc_); |
Jakob Stoklund Olesen | a37d5cf | 2010-11-01 19:49:57 +0000 | [diff] [blame] | 394 | assert(stacklvr.empty() && "Just created stack slot not empty"); |
| 395 | stacklvr.getNextValue(SlotIndex(), 0, lss_.getVNInfoAllocator()); |
Jakob Stoklund Olesen | 0a12b80 | 2010-10-26 00:11:35 +0000 | [diff] [blame] | 396 | stacklvr.MergeRangesInAsValue(edit_->getParent(), stacklvr.getValNumInfo(0)); |
| 397 | |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 398 | // Iterate over instructions using register. |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 399 | for (MachineRegisterInfo::reg_iterator RI = mri_.reg_begin(edit.getReg()); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 400 | MachineInstr *MI = RI.skipInstruction();) { |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 401 | |
Jakob Stoklund Olesen | 3b9c7eb | 2010-07-02 19:54:40 +0000 | [diff] [blame] | 402 | // Debug values are not allowed to affect codegen. |
| 403 | if (MI->isDebugValue()) { |
| 404 | // Modify DBG_VALUE now that the value is in a spill slot. |
| 405 | uint64_t Offset = MI->getOperand(1).getImm(); |
| 406 | const MDNode *MDPtr = MI->getOperand(2).getMetadata(); |
| 407 | DebugLoc DL = MI->getDebugLoc(); |
| 408 | if (MachineInstr *NewDV = tii_.emitFrameIndexDebugValue(mf_, stackSlot_, |
| 409 | Offset, MDPtr, DL)) { |
| 410 | DEBUG(dbgs() << "Modifying debug info due to spill:" << "\t" << *MI); |
| 411 | MachineBasicBlock *MBB = MI->getParent(); |
| 412 | MBB->insert(MBB->erase(MI), NewDV); |
| 413 | } else { |
| 414 | DEBUG(dbgs() << "Removing debug info due to spill:" << "\t" << *MI); |
| 415 | MI->eraseFromParent(); |
| 416 | } |
| 417 | continue; |
| 418 | } |
| 419 | |
Jakob Stoklund Olesen | 1a0f91b | 2010-08-04 22:35:11 +0000 | [diff] [blame] | 420 | // Stack slot accesses may coalesce away. |
| 421 | if (coalesceStackAccess(MI)) |
| 422 | continue; |
| 423 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 424 | // Analyze instruction. |
| 425 | bool Reads, Writes; |
| 426 | SmallVector<unsigned, 8> Ops; |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 427 | tie(Reads, Writes) = MI->readsWritesVirtualRegister(edit.getReg(), &Ops); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 428 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 429 | // Attempt to fold memory ops. |
| 430 | if (foldMemoryOperand(MI, Ops)) |
| 431 | continue; |
| 432 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 433 | // Allocate interval around instruction. |
| 434 | // FIXME: Infer regclass from instruction alone. |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 435 | LiveInterval &NewLI = edit.create(mri_, lis_, vrm_); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 436 | NewLI.markNotSpillable(); |
| 437 | |
Jakob Stoklund Olesen | 8de3b1e | 2010-07-02 17:44:57 +0000 | [diff] [blame] | 438 | if (Reads) |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 439 | insertReload(NewLI, MI); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 440 | |
| 441 | // Rewrite instruction operands. |
| 442 | bool hasLiveDef = false; |
| 443 | for (unsigned i = 0, e = Ops.size(); i != e; ++i) { |
| 444 | MachineOperand &MO = MI->getOperand(Ops[i]); |
Jakob Stoklund Olesen | a17768f | 2010-10-14 23:49:52 +0000 | [diff] [blame] | 445 | MO.setReg(NewLI.reg); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 446 | if (MO.isUse()) { |
| 447 | if (!MI->isRegTiedToDefOperand(Ops[i])) |
| 448 | MO.setIsKill(); |
| 449 | } else { |
| 450 | if (!MO.isDead()) |
| 451 | hasLiveDef = true; |
| 452 | } |
| 453 | } |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 454 | |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 455 | // FIXME: Use a second vreg if instruction has no tied ops. |
Jakob Stoklund Olesen | 9e55afb | 2010-06-30 23:03:52 +0000 | [diff] [blame] | 456 | if (Writes && hasLiveDef) |
| 457 | insertSpill(NewLI, MI); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 458 | |
| 459 | DEBUG(dbgs() << "\tinterval: " << NewLI << '\n'); |
Jakob Stoklund Olesen | 914f2ff | 2010-06-29 23:58:39 +0000 | [diff] [blame] | 460 | } |
| 461 | } |