Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 1 | //===- SIFixSGPRCopies.cpp - Remove potential VGPR => SGPR copies ---------===// |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 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 | /// \file |
| 11 | /// Copies from VGPR to SGPR registers are illegal and the register coalescer |
| 12 | /// will sometimes generate these illegal copies in situations like this: |
| 13 | /// |
| 14 | /// Register Class <vsrc> is the union of <vgpr> and <sgpr> |
| 15 | /// |
| 16 | /// BB0: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 17 | /// %0 <sgpr> = SCALAR_INST |
| 18 | /// %1 <vsrc> = COPY %0 <sgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 19 | /// ... |
| 20 | /// BRANCH %cond BB1, BB2 |
| 21 | /// BB1: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 22 | /// %2 <vgpr> = VECTOR_INST |
| 23 | /// %3 <vsrc> = COPY %2 <vgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 24 | /// BB2: |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 25 | /// %4 <vsrc> = PHI %1 <vsrc>, <%bb.0>, %3 <vrsc>, <%bb.1> |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 26 | /// %5 <vgpr> = VECTOR_INST %4 <vsrc> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 27 | /// |
NAKAMURA Takumi | 78e80cd | 2013-11-14 04:05:22 +0000 | [diff] [blame] | 28 | /// |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 29 | /// The coalescer will begin at BB0 and eliminate its copy, then the resulting |
| 30 | /// code will look like this: |
| 31 | /// |
| 32 | /// BB0: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 33 | /// %0 <sgpr> = SCALAR_INST |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 34 | /// ... |
| 35 | /// BRANCH %cond BB1, BB2 |
| 36 | /// BB1: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 37 | /// %2 <vgpr> = VECTOR_INST |
| 38 | /// %3 <vsrc> = COPY %2 <vgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 39 | /// BB2: |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 40 | /// %4 <sgpr> = PHI %0 <sgpr>, <%bb.0>, %3 <vsrc>, <%bb.1> |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 41 | /// %5 <vgpr> = VECTOR_INST %4 <sgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 42 | /// |
| 43 | /// Now that the result of the PHI instruction is an SGPR, the register |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 44 | /// allocator is now forced to constrain the register class of %3 to |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 45 | /// <sgpr> so we end up with final code like this: |
NAKAMURA Takumi | 78e80cd | 2013-11-14 04:05:22 +0000 | [diff] [blame] | 46 | /// |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 47 | /// BB0: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 48 | /// %0 <sgpr> = SCALAR_INST |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 49 | /// ... |
| 50 | /// BRANCH %cond BB1, BB2 |
| 51 | /// BB1: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 52 | /// %2 <vgpr> = VECTOR_INST |
| 53 | /// %3 <sgpr> = COPY %2 <vgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 54 | /// BB2: |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 55 | /// %4 <sgpr> = PHI %0 <sgpr>, <%bb.0>, %3 <sgpr>, <%bb.1> |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 56 | /// %5 <vgpr> = VECTOR_INST %4 <sgpr> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 57 | /// |
NAKAMURA Takumi | 78e80cd | 2013-11-14 04:05:22 +0000 | [diff] [blame] | 58 | /// Now this code contains an illegal copy from a VGPR to an SGPR. |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 59 | /// |
| 60 | /// In order to avoid this problem, this pass searches for PHI instructions |
| 61 | /// which define a <vsrc> register and constrains its definition class to |
| 62 | /// <vgpr> if the user of the PHI's definition register is a vector instruction. |
| 63 | /// If the PHI's definition class is constrained to <vgpr> then the coalescer |
| 64 | /// will be unable to perform the COPY removal from the above example which |
| 65 | /// ultimately led to the creation of an illegal COPY. |
| 66 | //===----------------------------------------------------------------------===// |
| 67 | |
| 68 | #include "AMDGPU.h" |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 69 | #include "AMDGPUSubtarget.h" |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 70 | #include "SIInstrInfo.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 71 | #include "SIRegisterInfo.h" |
Chandler Carruth | 6bda14b | 2017-06-06 11:49:48 +0000 | [diff] [blame] | 72 | #include "llvm/ADT/DenseSet.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 73 | #include "llvm/ADT/STLExtras.h" |
| 74 | #include "llvm/ADT/SmallSet.h" |
| 75 | #include "llvm/ADT/SmallVector.h" |
| 76 | #include "llvm/CodeGen/MachineBasicBlock.h" |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 77 | #include "llvm/CodeGen/MachineDominators.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 78 | #include "llvm/CodeGen/MachineFunction.h" |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 79 | #include "llvm/CodeGen/MachineFunctionPass.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 80 | #include "llvm/CodeGen/MachineInstr.h" |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 81 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 82 | #include "llvm/CodeGen/MachineOperand.h" |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 83 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 84 | #include "llvm/CodeGen/MachinePostDominators.h" |
David Blaikie | b3bde2e | 2017-11-17 01:07:10 +0000 | [diff] [blame] | 85 | #include "llvm/CodeGen/TargetRegisterInfo.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 86 | #include "llvm/Pass.h" |
| 87 | #include "llvm/Support/CodeGen.h" |
| 88 | #include "llvm/Support/CommandLine.h" |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 89 | #include "llvm/Support/Debug.h" |
Hans Wennborg | a74fd70 | 2013-11-14 23:24:09 +0000 | [diff] [blame] | 90 | #include "llvm/Support/raw_ostream.h" |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 91 | #include "llvm/Target/TargetMachine.h" |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 92 | #include <cassert> |
| 93 | #include <cstdint> |
| 94 | #include <iterator> |
| 95 | #include <list> |
| 96 | #include <map> |
| 97 | #include <tuple> |
| 98 | #include <utility> |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 99 | |
| 100 | using namespace llvm; |
| 101 | |
Matt Arsenault | 98f8394 | 2016-04-21 18:21:54 +0000 | [diff] [blame] | 102 | #define DEBUG_TYPE "si-fix-sgpr-copies" |
Chandler Carruth | 84e68b2 | 2014-04-22 02:41:26 +0000 | [diff] [blame] | 103 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 104 | static cl::opt<bool> EnableM0Merge( |
| 105 | "amdgpu-enable-merge-m0", |
| 106 | cl::desc("Merge and hoist M0 initializations"), |
| 107 | cl::init(false)); |
| 108 | |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 109 | namespace { |
| 110 | |
| 111 | class SIFixSGPRCopies : public MachineFunctionPass { |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 112 | MachineDominatorTree *MDT; |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 113 | MachinePostDominatorTree *MPDT; |
| 114 | DenseMap<MachineBasicBlock *, SetVector<MachineBasicBlock*>> PDF; |
| 115 | void computePDF(MachineFunction * MF); |
| 116 | #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) |
| 117 | void printPDF(); |
| 118 | #endif |
Matt Arsenault | 782c03b | 2015-11-03 22:30:13 +0000 | [diff] [blame] | 119 | public: |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 120 | static char ID; |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 121 | |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 122 | SIFixSGPRCopies() : MachineFunctionPass(ID) {} |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 123 | |
Craig Topper | 5656db4 | 2014-04-29 07:57:24 +0000 | [diff] [blame] | 124 | bool runOnMachineFunction(MachineFunction &MF) override; |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 125 | |
Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 126 | StringRef getPassName() const override { return "SI Fix SGPR copies"; } |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 127 | |
Matt Arsenault | 0cb8517 | 2015-09-25 17:21:28 +0000 | [diff] [blame] | 128 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 129 | AU.addRequired<MachineDominatorTree>(); |
| 130 | AU.addPreserved<MachineDominatorTree>(); |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 131 | AU.addRequired<MachinePostDominatorTree>(); |
| 132 | AU.addPreserved<MachinePostDominatorTree>(); |
Matt Arsenault | 0cb8517 | 2015-09-25 17:21:28 +0000 | [diff] [blame] | 133 | AU.setPreservesCFG(); |
| 134 | MachineFunctionPass::getAnalysisUsage(AU); |
| 135 | } |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 136 | }; |
| 137 | |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 138 | } // end anonymous namespace |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 139 | |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 140 | INITIALIZE_PASS_BEGIN(SIFixSGPRCopies, DEBUG_TYPE, |
| 141 | "SI Fix SGPR copies", false, false) |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 142 | INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree) |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 143 | INITIALIZE_PASS_END(SIFixSGPRCopies, DEBUG_TYPE, |
| 144 | "SI Fix SGPR copies", false, false) |
| 145 | |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 146 | char SIFixSGPRCopies::ID = 0; |
| 147 | |
Matt Arsenault | 782c03b | 2015-11-03 22:30:13 +0000 | [diff] [blame] | 148 | char &llvm::SIFixSGPRCopiesID = SIFixSGPRCopies::ID; |
| 149 | |
| 150 | FunctionPass *llvm::createSIFixSGPRCopiesPass() { |
| 151 | return new SIFixSGPRCopies(); |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 152 | } |
| 153 | |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 154 | static bool hasVGPROperands(const MachineInstr &MI, const SIRegisterInfo *TRI) { |
| 155 | const MachineRegisterInfo &MRI = MI.getParent()->getParent()->getRegInfo(); |
| 156 | for (unsigned i = 0, e = MI.getNumOperands(); i != e; ++i) { |
| 157 | if (!MI.getOperand(i).isReg() || |
| 158 | !TargetRegisterInfo::isVirtualRegister(MI.getOperand(i).getReg())) |
| 159 | continue; |
| 160 | |
| 161 | if (TRI->hasVGPRs(MRI.getRegClass(MI.getOperand(i).getReg()))) |
| 162 | return true; |
| 163 | } |
| 164 | return false; |
| 165 | } |
| 166 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 167 | static std::pair<const TargetRegisterClass *, const TargetRegisterClass *> |
| 168 | getCopyRegClasses(const MachineInstr &Copy, |
| 169 | const SIRegisterInfo &TRI, |
| 170 | const MachineRegisterInfo &MRI) { |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 171 | unsigned DstReg = Copy.getOperand(0).getReg(); |
| 172 | unsigned SrcReg = Copy.getOperand(1).getReg(); |
Matt Arsenault | 120a0c9 | 2014-12-03 05:22:39 +0000 | [diff] [blame] | 173 | |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 174 | const TargetRegisterClass *SrcRC = |
| 175 | TargetRegisterInfo::isVirtualRegister(SrcReg) ? |
| 176 | MRI.getRegClass(SrcReg) : |
| 177 | TRI.getPhysRegClass(SrcReg); |
Tom Stellard | d33d7f1 | 2015-05-12 14:18:11 +0000 | [diff] [blame] | 178 | |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 179 | // We don't really care about the subregister here. |
| 180 | // SrcRC = TRI.getSubRegClass(SrcRC, Copy.getOperand(1).getSubReg()); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 181 | |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 182 | const TargetRegisterClass *DstRC = |
| 183 | TargetRegisterInfo::isVirtualRegister(DstReg) ? |
| 184 | MRI.getRegClass(DstReg) : |
| 185 | TRI.getPhysRegClass(DstReg); |
| 186 | |
| 187 | return std::make_pair(SrcRC, DstRC); |
| 188 | } |
| 189 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 190 | static bool isVGPRToSGPRCopy(const TargetRegisterClass *SrcRC, |
| 191 | const TargetRegisterClass *DstRC, |
| 192 | const SIRegisterInfo &TRI) { |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 193 | return TRI.isSGPRClass(DstRC) && TRI.hasVGPRs(SrcRC); |
| 194 | } |
| 195 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 196 | static bool isSGPRToVGPRCopy(const TargetRegisterClass *SrcRC, |
| 197 | const TargetRegisterClass *DstRC, |
| 198 | const SIRegisterInfo &TRI) { |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 199 | return TRI.isSGPRClass(SrcRC) && TRI.hasVGPRs(DstRC); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 200 | } |
| 201 | |
Stanislav Mekhanoshin | 465a1ff | 2017-06-20 18:32:42 +0000 | [diff] [blame] | 202 | static bool tryChangeVGPRtoSGPRinCopy(MachineInstr &MI, |
| 203 | const SIRegisterInfo *TRI, |
| 204 | const SIInstrInfo *TII) { |
| 205 | MachineRegisterInfo &MRI = MI.getParent()->getParent()->getRegInfo(); |
| 206 | auto &Src = MI.getOperand(1); |
| 207 | unsigned DstReg = MI.getOperand(0).getReg(); |
| 208 | unsigned SrcReg = Src.getReg(); |
| 209 | if (!TargetRegisterInfo::isVirtualRegister(SrcReg) || |
| 210 | !TargetRegisterInfo::isVirtualRegister(DstReg)) |
| 211 | return false; |
| 212 | |
| 213 | for (const auto &MO : MRI.reg_nodbg_operands(DstReg)) { |
| 214 | const auto *UseMI = MO.getParent(); |
| 215 | if (UseMI == &MI) |
| 216 | continue; |
| 217 | if (MO.isDef() || UseMI->getParent() != MI.getParent() || |
| 218 | UseMI->getOpcode() <= TargetOpcode::GENERIC_OP_END || |
| 219 | !TII->isOperandLegal(*UseMI, UseMI->getOperandNo(&MO), &Src)) |
| 220 | return false; |
| 221 | } |
| 222 | // Change VGPR to SGPR destination. |
| 223 | MRI.setRegClass(DstReg, TRI->getEquivalentSGPRClass(MRI.getRegClass(DstReg))); |
| 224 | return true; |
| 225 | } |
| 226 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 227 | // Distribute an SGPR->VGPR copy of a REG_SEQUENCE into a VGPR REG_SEQUENCE. |
| 228 | // |
| 229 | // SGPRx = ... |
| 230 | // SGPRy = REG_SEQUENCE SGPRx, sub0 ... |
| 231 | // VGPRz = COPY SGPRy |
| 232 | // |
| 233 | // ==> |
| 234 | // |
| 235 | // VGPRx = COPY SGPRx |
| 236 | // VGPRz = REG_SEQUENCE VGPRx, sub0 |
| 237 | // |
| 238 | // This exposes immediate folding opportunities when materializing 64-bit |
| 239 | // immediates. |
| 240 | static bool foldVGPRCopyIntoRegSequence(MachineInstr &MI, |
| 241 | const SIRegisterInfo *TRI, |
| 242 | const SIInstrInfo *TII, |
| 243 | MachineRegisterInfo &MRI) { |
| 244 | assert(MI.isRegSequence()); |
| 245 | |
| 246 | unsigned DstReg = MI.getOperand(0).getReg(); |
| 247 | if (!TRI->isSGPRClass(MRI.getRegClass(DstReg))) |
| 248 | return false; |
| 249 | |
| 250 | if (!MRI.hasOneUse(DstReg)) |
| 251 | return false; |
| 252 | |
| 253 | MachineInstr &CopyUse = *MRI.use_instr_begin(DstReg); |
| 254 | if (!CopyUse.isCopy()) |
| 255 | return false; |
| 256 | |
Matt Arsenault | fe78ffb | 2017-04-11 22:29:19 +0000 | [diff] [blame] | 257 | // It is illegal to have vreg inputs to a physreg defining reg_sequence. |
| 258 | if (TargetRegisterInfo::isPhysicalRegister(CopyUse.getOperand(0).getReg())) |
| 259 | return false; |
| 260 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 261 | const TargetRegisterClass *SrcRC, *DstRC; |
| 262 | std::tie(SrcRC, DstRC) = getCopyRegClasses(CopyUse, *TRI, MRI); |
| 263 | |
| 264 | if (!isSGPRToVGPRCopy(SrcRC, DstRC, *TRI)) |
| 265 | return false; |
| 266 | |
Stanislav Mekhanoshin | 465a1ff | 2017-06-20 18:32:42 +0000 | [diff] [blame] | 267 | if (tryChangeVGPRtoSGPRinCopy(CopyUse, TRI, TII)) |
| 268 | return true; |
| 269 | |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 270 | // TODO: Could have multiple extracts? |
| 271 | unsigned SubReg = CopyUse.getOperand(1).getSubReg(); |
| 272 | if (SubReg != AMDGPU::NoSubRegister) |
| 273 | return false; |
| 274 | |
| 275 | MRI.setRegClass(DstReg, DstRC); |
| 276 | |
| 277 | // SGPRx = ... |
| 278 | // SGPRy = REG_SEQUENCE SGPRx, sub0 ... |
| 279 | // VGPRz = COPY SGPRy |
| 280 | |
| 281 | // => |
| 282 | // VGPRx = COPY SGPRx |
| 283 | // VGPRz = REG_SEQUENCE VGPRx, sub0 |
| 284 | |
| 285 | MI.getOperand(0).setReg(CopyUse.getOperand(0).getReg()); |
| 286 | |
| 287 | for (unsigned I = 1, N = MI.getNumOperands(); I != N; I += 2) { |
| 288 | unsigned SrcReg = MI.getOperand(I).getReg(); |
Nicolai Haehnle | 82fc962 | 2016-01-07 17:10:29 +0000 | [diff] [blame] | 289 | unsigned SrcSubReg = MI.getOperand(I).getSubReg(); |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 290 | |
| 291 | const TargetRegisterClass *SrcRC = MRI.getRegClass(SrcReg); |
| 292 | assert(TRI->isSGPRClass(SrcRC) && |
| 293 | "Expected SGPR REG_SEQUENCE to only have SGPR inputs"); |
| 294 | |
| 295 | SrcRC = TRI->getSubRegClass(SrcRC, SrcSubReg); |
| 296 | const TargetRegisterClass *NewSrcRC = TRI->getEquivalentVGPRClass(SrcRC); |
| 297 | |
| 298 | unsigned TmpReg = MRI.createVirtualRegister(NewSrcRC); |
| 299 | |
Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 300 | BuildMI(*MI.getParent(), &MI, MI.getDebugLoc(), TII->get(AMDGPU::COPY), |
| 301 | TmpReg) |
| 302 | .add(MI.getOperand(I)); |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 303 | |
| 304 | MI.getOperand(I).setReg(TmpReg); |
| 305 | } |
| 306 | |
| 307 | CopyUse.eraseFromParent(); |
| 308 | return true; |
| 309 | } |
| 310 | |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 311 | static bool phiHasVGPROperands(const MachineInstr &PHI, |
| 312 | const MachineRegisterInfo &MRI, |
| 313 | const SIRegisterInfo *TRI, |
| 314 | const SIInstrInfo *TII) { |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 315 | for (unsigned i = 1; i < PHI.getNumOperands(); i += 2) { |
| 316 | unsigned Reg = PHI.getOperand(i).getReg(); |
| 317 | if (TRI->hasVGPRs(MRI.getRegClass(Reg))) |
| 318 | return true; |
| 319 | } |
| 320 | return false; |
| 321 | } |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 322 | |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 323 | static bool phiHasBreakDef(const MachineInstr &PHI, |
| 324 | const MachineRegisterInfo &MRI, |
| 325 | SmallSet<unsigned, 8> &Visited) { |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 326 | for (unsigned i = 1; i < PHI.getNumOperands(); i += 2) { |
| 327 | unsigned Reg = PHI.getOperand(i).getReg(); |
| 328 | if (Visited.count(Reg)) |
| 329 | continue; |
| 330 | |
| 331 | Visited.insert(Reg); |
| 332 | |
Matt Arsenault | 2a80369 | 2017-04-29 01:26:34 +0000 | [diff] [blame] | 333 | MachineInstr *DefInstr = MRI.getVRegDef(Reg); |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 334 | switch (DefInstr->getOpcode()) { |
| 335 | default: |
| 336 | break; |
| 337 | case AMDGPU::SI_BREAK: |
| 338 | case AMDGPU::SI_IF_BREAK: |
| 339 | case AMDGPU::SI_ELSE_BREAK: |
| 340 | return true; |
| 341 | case AMDGPU::PHI: |
| 342 | if (phiHasBreakDef(*DefInstr, MRI, Visited)) |
| 343 | return true; |
| 344 | } |
| 345 | } |
| 346 | return false; |
| 347 | } |
| 348 | |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 349 | static bool hasTerminatorThatModifiesExec(const MachineBasicBlock &MBB, |
| 350 | const TargetRegisterInfo &TRI) { |
| 351 | for (MachineBasicBlock::const_iterator I = MBB.getFirstTerminator(), |
| 352 | E = MBB.end(); I != E; ++I) { |
| 353 | if (I->modifiesRegister(AMDGPU::EXEC, &TRI)) |
| 354 | return true; |
| 355 | } |
| 356 | return false; |
| 357 | } |
| 358 | |
Tom Stellard | 00cfa74 | 2016-12-06 21:13:30 +0000 | [diff] [blame] | 359 | static bool isSafeToFoldImmIntoCopy(const MachineInstr *Copy, |
| 360 | const MachineInstr *MoveImm, |
| 361 | const SIInstrInfo *TII, |
| 362 | unsigned &SMovOp, |
| 363 | int64_t &Imm) { |
Connor Abbott | 8c217d0 | 2017-08-04 18:36:49 +0000 | [diff] [blame] | 364 | if (Copy->getOpcode() != AMDGPU::COPY) |
| 365 | return false; |
| 366 | |
Tom Stellard | 00cfa74 | 2016-12-06 21:13:30 +0000 | [diff] [blame] | 367 | if (!MoveImm->isMoveImmediate()) |
| 368 | return false; |
| 369 | |
| 370 | const MachineOperand *ImmOp = |
| 371 | TII->getNamedOperand(*MoveImm, AMDGPU::OpName::src0); |
| 372 | if (!ImmOp->isImm()) |
| 373 | return false; |
| 374 | |
| 375 | // FIXME: Handle copies with sub-regs. |
| 376 | if (Copy->getOperand(0).getSubReg()) |
| 377 | return false; |
| 378 | |
| 379 | switch (MoveImm->getOpcode()) { |
| 380 | default: |
| 381 | return false; |
| 382 | case AMDGPU::V_MOV_B32_e32: |
| 383 | SMovOp = AMDGPU::S_MOV_B32; |
| 384 | break; |
| 385 | case AMDGPU::V_MOV_B64_PSEUDO: |
| 386 | SMovOp = AMDGPU::S_MOV_B64; |
| 387 | break; |
| 388 | } |
| 389 | Imm = ImmOp->getImm(); |
| 390 | return true; |
| 391 | } |
| 392 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 393 | template <class UnaryPredicate> |
| 394 | bool searchPredecessors(const MachineBasicBlock *MBB, |
| 395 | const MachineBasicBlock *CutOff, |
| 396 | UnaryPredicate Predicate) { |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 397 | if (MBB == CutOff) |
| 398 | return false; |
| 399 | |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 400 | DenseSet<const MachineBasicBlock *> Visited; |
| 401 | SmallVector<MachineBasicBlock *, 4> Worklist(MBB->pred_begin(), |
| 402 | MBB->pred_end()); |
Wei Ding | 74da350 | 2017-04-12 23:51:47 +0000 | [diff] [blame] | 403 | |
| 404 | while (!Worklist.empty()) { |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 405 | MachineBasicBlock *MBB = Worklist.pop_back_val(); |
Wei Ding | 74da350 | 2017-04-12 23:51:47 +0000 | [diff] [blame] | 406 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 407 | if (!Visited.insert(MBB).second) |
Wei Ding | 74da350 | 2017-04-12 23:51:47 +0000 | [diff] [blame] | 408 | continue; |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 409 | if (MBB == CutOff) |
| 410 | continue; |
| 411 | if (Predicate(MBB)) |
Wei Ding | 74da350 | 2017-04-12 23:51:47 +0000 | [diff] [blame] | 412 | return true; |
| 413 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 414 | Worklist.append(MBB->pred_begin(), MBB->pred_end()); |
Wei Ding | 74da350 | 2017-04-12 23:51:47 +0000 | [diff] [blame] | 415 | } |
| 416 | |
| 417 | return false; |
| 418 | } |
| 419 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 420 | // Checks if there is potential path From instruction To instruction. |
| 421 | // If CutOff is specified and it sits in between of that path we ignore |
| 422 | // a higher portion of the path and report it is not reachable. |
| 423 | static bool isReachable(const MachineInstr *From, |
| 424 | const MachineInstr *To, |
| 425 | const MachineBasicBlock *CutOff, |
| 426 | MachineDominatorTree &MDT) { |
| 427 | // If either From block dominates To block or instructions are in the same |
| 428 | // block and From is higher. |
| 429 | if (MDT.dominates(From, To)) |
| 430 | return true; |
| 431 | |
| 432 | const MachineBasicBlock *MBBFrom = From->getParent(); |
| 433 | const MachineBasicBlock *MBBTo = To->getParent(); |
| 434 | if (MBBFrom == MBBTo) |
| 435 | return false; |
| 436 | |
| 437 | // Instructions are in different blocks, do predecessor search. |
| 438 | // We should almost never get here since we do not usually produce M0 stores |
| 439 | // other than -1. |
| 440 | return searchPredecessors(MBBTo, CutOff, [MBBFrom] |
| 441 | (const MachineBasicBlock *MBB) { return MBB == MBBFrom; }); |
| 442 | } |
| 443 | |
| 444 | // Hoist and merge identical SGPR initializations into a common predecessor. |
| 445 | // This is intended to combine M0 initializations, but can work with any |
| 446 | // SGPR. A VGPR cannot be processed since we cannot guarantee vector |
| 447 | // executioon. |
| 448 | static bool hoistAndMergeSGPRInits(unsigned Reg, |
| 449 | const MachineRegisterInfo &MRI, |
| 450 | MachineDominatorTree &MDT) { |
| 451 | // List of inits by immediate value. |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 452 | using InitListMap = std::map<unsigned, std::list<MachineInstr *>>; |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 453 | InitListMap Inits; |
| 454 | // List of clobbering instructions. |
| 455 | SmallVector<MachineInstr*, 8> Clobbers; |
| 456 | bool Changed = false; |
| 457 | |
| 458 | for (auto &MI : MRI.def_instructions(Reg)) { |
| 459 | MachineOperand *Imm = nullptr; |
| 460 | for (auto &MO: MI.operands()) { |
| 461 | if ((MO.isReg() && ((MO.isDef() && MO.getReg() != Reg) || !MO.isDef())) || |
| 462 | (!MO.isImm() && !MO.isReg()) || (MO.isImm() && Imm)) { |
| 463 | Imm = nullptr; |
| 464 | break; |
| 465 | } else if (MO.isImm()) |
| 466 | Imm = &MO; |
| 467 | } |
| 468 | if (Imm) |
| 469 | Inits[Imm->getImm()].push_front(&MI); |
| 470 | else |
| 471 | Clobbers.push_back(&MI); |
| 472 | } |
| 473 | |
| 474 | for (auto &Init : Inits) { |
| 475 | auto &Defs = Init.second; |
| 476 | |
| 477 | for (auto I1 = Defs.begin(), E = Defs.end(); I1 != E; ) { |
| 478 | MachineInstr *MI1 = *I1; |
| 479 | |
| 480 | for (auto I2 = std::next(I1); I2 != E; ) { |
| 481 | MachineInstr *MI2 = *I2; |
| 482 | |
| 483 | // Check any possible interference |
| 484 | auto intereferes = [&](MachineBasicBlock::iterator From, |
| 485 | MachineBasicBlock::iterator To) -> bool { |
| 486 | |
| 487 | assert(MDT.dominates(&*To, &*From)); |
| 488 | |
| 489 | auto interferes = [&MDT, From, To](MachineInstr* &Clobber) -> bool { |
| 490 | const MachineBasicBlock *MBBFrom = From->getParent(); |
| 491 | const MachineBasicBlock *MBBTo = To->getParent(); |
| 492 | bool MayClobberFrom = isReachable(Clobber, &*From, MBBTo, MDT); |
| 493 | bool MayClobberTo = isReachable(Clobber, &*To, MBBTo, MDT); |
| 494 | if (!MayClobberFrom && !MayClobberTo) |
| 495 | return false; |
| 496 | if ((MayClobberFrom && !MayClobberTo) || |
| 497 | (!MayClobberFrom && MayClobberTo)) |
| 498 | return true; |
| 499 | // Both can clobber, this is not an interference only if both are |
| 500 | // dominated by Clobber and belong to the same block or if Clobber |
| 501 | // properly dominates To, given that To >> From, so it dominates |
| 502 | // both and located in a common dominator. |
| 503 | return !((MBBFrom == MBBTo && |
| 504 | MDT.dominates(Clobber, &*From) && |
| 505 | MDT.dominates(Clobber, &*To)) || |
| 506 | MDT.properlyDominates(Clobber->getParent(), MBBTo)); |
| 507 | }; |
| 508 | |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 509 | return (llvm::any_of(Clobbers, interferes)) || |
| 510 | (llvm::any_of(Inits, [&](InitListMap::value_type &C) { |
| 511 | return C.first != Init.first && |
| 512 | llvm::any_of(C.second, interferes); |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 513 | })); |
| 514 | }; |
| 515 | |
| 516 | if (MDT.dominates(MI1, MI2)) { |
| 517 | if (!intereferes(MI2, MI1)) { |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 518 | DEBUG(dbgs() << "Erasing from " |
| 519 | << printMBBReference(*MI2->getParent()) << " " |
| 520 | << *MI2); |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 521 | MI2->eraseFromParent(); |
| 522 | Defs.erase(I2++); |
| 523 | Changed = true; |
| 524 | continue; |
| 525 | } |
| 526 | } else if (MDT.dominates(MI2, MI1)) { |
| 527 | if (!intereferes(MI1, MI2)) { |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 528 | DEBUG(dbgs() << "Erasing from " |
| 529 | << printMBBReference(*MI1->getParent()) << " " |
| 530 | << *MI1); |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 531 | MI1->eraseFromParent(); |
| 532 | Defs.erase(I1++); |
| 533 | Changed = true; |
| 534 | break; |
| 535 | } |
| 536 | } else { |
| 537 | auto *MBB = MDT.findNearestCommonDominator(MI1->getParent(), |
| 538 | MI2->getParent()); |
| 539 | if (!MBB) { |
| 540 | ++I2; |
| 541 | continue; |
| 542 | } |
| 543 | |
| 544 | MachineBasicBlock::iterator I = MBB->getFirstNonPHI(); |
| 545 | if (!intereferes(MI1, I) && !intereferes(MI2, I)) { |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 546 | DEBUG(dbgs() << "Erasing from " |
| 547 | << printMBBReference(*MI1->getParent()) << " " << *MI1 |
| 548 | << "and moving from " |
| 549 | << printMBBReference(*MI2->getParent()) << " to " |
| 550 | << printMBBReference(*I->getParent()) << " " << *MI2); |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 551 | I->getParent()->splice(I, MI2->getParent(), MI2); |
| 552 | MI1->eraseFromParent(); |
| 553 | Defs.erase(I1++); |
| 554 | Changed = true; |
| 555 | break; |
| 556 | } |
| 557 | } |
| 558 | ++I2; |
| 559 | } |
| 560 | ++I1; |
| 561 | } |
| 562 | } |
| 563 | |
| 564 | if (Changed) |
| 565 | MRI.clearKillFlags(Reg); |
| 566 | |
| 567 | return Changed; |
| 568 | } |
| 569 | |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 570 | void SIFixSGPRCopies::computePDF(MachineFunction *MF) { |
| 571 | MachineFunction::iterator B = MF->begin(); |
| 572 | MachineFunction::iterator E = MF->end(); |
| 573 | for (; B != E; ++B) { |
| 574 | if (B->succ_size() > 1) { |
| 575 | for (auto S : B->successors()) { |
| 576 | MachineDomTreeNode *runner = MPDT->getNode(&*S); |
| 577 | MachineDomTreeNode *sentinel = MPDT->getNode(&*B)->getIDom(); |
| 578 | while (runner && runner != sentinel) { |
| 579 | PDF[runner->getBlock()].insert(&*B); |
| 580 | runner = runner->getIDom(); |
| 581 | } |
| 582 | } |
| 583 | } |
| 584 | } |
| 585 | } |
| 586 | |
| 587 | #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) |
| 588 | void SIFixSGPRCopies::printPDF() { |
| 589 | dbgs() << "\n######## PostDominanceFrontiers set #########\n"; |
| 590 | for (auto &I : PDF) { |
| 591 | dbgs() << "PDF[ " << I.first->getNumber() << "] : "; |
| 592 | for (auto &J : I.second) { |
| 593 | dbgs() << J->getNumber() << ' '; |
| 594 | } |
| 595 | dbgs() << '\n'; |
| 596 | } |
| 597 | dbgs() << "\n##############################################\n"; |
| 598 | } |
| 599 | #endif |
| 600 | |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 601 | bool SIFixSGPRCopies::runOnMachineFunction(MachineFunction &MF) { |
Matt Arsenault | 43e92fe | 2016-06-24 06:30:11 +0000 | [diff] [blame] | 602 | const SISubtarget &ST = MF.getSubtarget<SISubtarget>(); |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 603 | MachineRegisterInfo &MRI = MF.getRegInfo(); |
Matt Arsenault | 43e92fe | 2016-06-24 06:30:11 +0000 | [diff] [blame] | 604 | const SIRegisterInfo *TRI = ST.getRegisterInfo(); |
| 605 | const SIInstrInfo *TII = ST.getInstrInfo(); |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 606 | MDT = &getAnalysis<MachineDominatorTree>(); |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 607 | MPDT = &getAnalysis<MachinePostDominatorTree>(); |
| 608 | PDF.clear(); |
| 609 | computePDF(&MF); |
| 610 | DEBUG(printPDF()); |
Matt Arsenault | f1aebbf | 2015-11-02 23:30:48 +0000 | [diff] [blame] | 611 | |
| 612 | SmallVector<MachineInstr *, 16> Worklist; |
| 613 | |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 614 | for (MachineFunction::iterator BI = MF.begin(), BE = MF.end(); |
| 615 | BI != BE; ++BI) { |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 616 | MachineBasicBlock &MBB = *BI; |
| 617 | for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end(); |
Matt Arsenault | f1aebbf | 2015-11-02 23:30:48 +0000 | [diff] [blame] | 618 | I != E; ++I) { |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 619 | MachineInstr &MI = *I; |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 620 | |
| 621 | switch (MI.getOpcode()) { |
Matt Arsenault | 85441dd | 2015-09-21 16:27:22 +0000 | [diff] [blame] | 622 | default: |
| 623 | continue; |
Connor Abbott | 8c217d0 | 2017-08-04 18:36:49 +0000 | [diff] [blame] | 624 | case AMDGPU::COPY: |
Connor Abbott | 92638ab | 2017-08-04 18:36:52 +0000 | [diff] [blame] | 625 | case AMDGPU::WQM: |
| 626 | case AMDGPU::WWM: { |
Matt Arsenault | f0d9e47 | 2015-10-13 00:07:54 +0000 | [diff] [blame] | 627 | // If the destination register is a physical register there isn't really |
| 628 | // much we can do to fix this. |
| 629 | if (!TargetRegisterInfo::isVirtualRegister(MI.getOperand(0).getReg())) |
| 630 | continue; |
| 631 | |
| 632 | const TargetRegisterClass *SrcRC, *DstRC; |
| 633 | std::tie(SrcRC, DstRC) = getCopyRegClasses(MI, *TRI, MRI); |
| 634 | if (isVGPRToSGPRCopy(SrcRC, DstRC, *TRI)) { |
Matt Arsenault | 2a80369 | 2017-04-29 01:26:34 +0000 | [diff] [blame] | 635 | unsigned SrcReg = MI.getOperand(1).getReg(); |
| 636 | if (!TargetRegisterInfo::isVirtualRegister(SrcReg)) { |
| 637 | TII->moveToVALU(MI); |
| 638 | break; |
| 639 | } |
| 640 | |
| 641 | MachineInstr *DefMI = MRI.getVRegDef(SrcReg); |
Tom Stellard | 00cfa74 | 2016-12-06 21:13:30 +0000 | [diff] [blame] | 642 | unsigned SMovOp; |
| 643 | int64_t Imm; |
| 644 | // If we are just copying an immediate, we can replace the copy with |
| 645 | // s_mov_b32. |
| 646 | if (isSafeToFoldImmIntoCopy(&MI, DefMI, TII, SMovOp, Imm)) { |
| 647 | MI.getOperand(1).ChangeToImmediate(Imm); |
| 648 | MI.addImplicitDefUseOperands(MF); |
| 649 | MI.setDesc(TII->get(SMovOp)); |
| 650 | break; |
| 651 | } |
Matt Arsenault | 85441dd | 2015-09-21 16:27:22 +0000 | [diff] [blame] | 652 | TII->moveToVALU(MI); |
Stanislav Mekhanoshin | 465a1ff | 2017-06-20 18:32:42 +0000 | [diff] [blame] | 653 | } else if (isSGPRToVGPRCopy(SrcRC, DstRC, *TRI)) { |
| 654 | tryChangeVGPRtoSGPRinCopy(MI, TRI, TII); |
Matt Arsenault | 85441dd | 2015-09-21 16:27:22 +0000 | [diff] [blame] | 655 | } |
| 656 | |
| 657 | break; |
| 658 | } |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 659 | case AMDGPU::PHI: { |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 660 | unsigned Reg = MI.getOperand(0).getReg(); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 661 | if (!TRI->isSGPRClass(MRI.getRegClass(Reg))) |
| 662 | break; |
| 663 | |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 664 | // We don't need to fix the PHI if all the source blocks |
| 665 | // have no divergent control dependecies |
Changpeng Fang | ef4dbb4 | 2017-08-03 16:37:02 +0000 | [diff] [blame] | 666 | bool HasVGPROperand = phiHasVGPROperands(MI, MRI, TRI, TII); |
Alexander Timofeev | c1425c9d | 2017-12-01 11:56:34 +0000 | [diff] [blame] | 667 | if (!HasVGPROperand) { |
| 668 | bool Uniform = true; |
| 669 | MachineBasicBlock * Join = MI.getParent(); |
| 670 | for (auto &O : MI.explicit_operands()) { |
| 671 | if (O.isMBB()) { |
| 672 | MachineBasicBlock * Source = O.getMBB(); |
| 673 | SetVector<MachineBasicBlock*> &SourcePDF = PDF[Source]; |
| 674 | SetVector<MachineBasicBlock*> &JoinPDF = PDF[Join]; |
| 675 | SetVector<MachineBasicBlock*> CDList; |
| 676 | for (auto &I : SourcePDF) { |
| 677 | if (!JoinPDF.count(I) || /* back edge */MDT->dominates(Join, I)) { |
| 678 | if (hasTerminatorThatModifiesExec(*I, *TRI)) |
| 679 | Uniform = false; |
| 680 | } |
| 681 | } |
| 682 | } |
| 683 | } |
| 684 | if (Uniform) { |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 685 | DEBUG(dbgs() << "Not fixing PHI for uniform branch: " << MI << '\n'); |
| 686 | break; |
| 687 | } |
| 688 | } |
| 689 | |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 690 | // If a PHI node defines an SGPR and any of its operands are VGPRs, |
| 691 | // then we need to move it to the VALU. |
Tom Stellard | deb3f9e | 2014-09-24 01:33:26 +0000 | [diff] [blame] | 692 | // |
| 693 | // Also, if a PHI node defines an SGPR and has all SGPR operands |
| 694 | // we must move it to the VALU, because the SGPR operands will |
| 695 | // all end up being assigned the same register, which means |
| 696 | // there is a potential for a conflict if different threads take |
Matt Arsenault | bfaab76 | 2014-10-17 00:36:20 +0000 | [diff] [blame] | 697 | // different control flow paths. |
Tom Stellard | deb3f9e | 2014-09-24 01:33:26 +0000 | [diff] [blame] | 698 | // |
| 699 | // For Example: |
| 700 | // |
| 701 | // sgpr0 = def; |
| 702 | // ... |
| 703 | // sgpr1 = def; |
| 704 | // ... |
| 705 | // sgpr2 = PHI sgpr0, sgpr1 |
| 706 | // use sgpr2; |
| 707 | // |
| 708 | // Will Become: |
| 709 | // |
| 710 | // sgpr2 = def; |
| 711 | // ... |
| 712 | // sgpr2 = def; |
| 713 | // ... |
| 714 | // use sgpr2 |
| 715 | // |
Tom Stellard | deb3f9e | 2014-09-24 01:33:26 +0000 | [diff] [blame] | 716 | // The one exception to this rule is when one of the operands |
| 717 | // is defined by a SI_BREAK, SI_IF_BREAK, or SI_ELSE_BREAK |
| 718 | // instruction. In this case, there we know the program will |
| 719 | // never enter the second block (the loop) without entering |
| 720 | // the first block (where the condition is computed), so there |
| 721 | // is no chance for values to be over-written. |
| 722 | |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 723 | SmallSet<unsigned, 8> Visited; |
Changpeng Fang | ef4dbb4 | 2017-08-03 16:37:02 +0000 | [diff] [blame] | 724 | if (HasVGPROperand || !phiHasBreakDef(MI, MRI, Visited)) { |
Tom Stellard | 0bc6881 | 2016-11-29 00:46:46 +0000 | [diff] [blame] | 725 | DEBUG(dbgs() << "Fixing PHI: " << MI); |
Tom Stellard | deb3f9e | 2014-09-24 01:33:26 +0000 | [diff] [blame] | 726 | TII->moveToVALU(MI); |
Tom Stellard | 9fdbec8 | 2016-11-11 23:35:42 +0000 | [diff] [blame] | 727 | } |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 728 | break; |
| 729 | } |
Eugene Zelenko | 59e1282 | 2017-08-08 00:47:13 +0000 | [diff] [blame] | 730 | case AMDGPU::REG_SEQUENCE: |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 731 | if (TRI->hasVGPRs(TII->getOpRegClass(MI, 0)) || |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 732 | !hasVGPROperands(MI, TRI)) { |
| 733 | foldVGPRCopyIntoRegSequence(MI, TRI, TII, MRI); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 734 | continue; |
Matt Arsenault | 0de924b | 2015-11-02 23:15:42 +0000 | [diff] [blame] | 735 | } |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 736 | |
Matt Arsenault | bfaab76 | 2014-10-17 00:36:20 +0000 | [diff] [blame] | 737 | DEBUG(dbgs() << "Fixing REG_SEQUENCE: " << MI); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 738 | |
| 739 | TII->moveToVALU(MI); |
Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 740 | break; |
Tom Stellard | 204e61b | 2014-04-07 19:45:45 +0000 | [diff] [blame] | 741 | case AMDGPU::INSERT_SUBREG: { |
Tom Stellard | a568738 | 2014-05-15 14:41:55 +0000 | [diff] [blame] | 742 | const TargetRegisterClass *DstRC, *Src0RC, *Src1RC; |
Tom Stellard | 204e61b | 2014-04-07 19:45:45 +0000 | [diff] [blame] | 743 | DstRC = MRI.getRegClass(MI.getOperand(0).getReg()); |
Tom Stellard | a568738 | 2014-05-15 14:41:55 +0000 | [diff] [blame] | 744 | Src0RC = MRI.getRegClass(MI.getOperand(1).getReg()); |
| 745 | Src1RC = MRI.getRegClass(MI.getOperand(2).getReg()); |
| 746 | if (TRI->isSGPRClass(DstRC) && |
| 747 | (TRI->hasVGPRs(Src0RC) || TRI->hasVGPRs(Src1RC))) { |
Matt Arsenault | bfaab76 | 2014-10-17 00:36:20 +0000 | [diff] [blame] | 748 | DEBUG(dbgs() << " Fixing INSERT_SUBREG: " << MI); |
Tom Stellard | a568738 | 2014-05-15 14:41:55 +0000 | [diff] [blame] | 749 | TII->moveToVALU(MI); |
| 750 | } |
| 751 | break; |
Tom Stellard | 204e61b | 2014-04-07 19:45:45 +0000 | [diff] [blame] | 752 | } |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 753 | } |
| 754 | } |
| 755 | } |
Matt Arsenault | 6f67978 | 2014-11-17 21:11:34 +0000 | [diff] [blame] | 756 | |
Stanislav Mekhanoshin | bd5394b | 2017-04-24 19:37:54 +0000 | [diff] [blame] | 757 | if (MF.getTarget().getOptLevel() > CodeGenOpt::None && EnableM0Merge) |
| 758 | hoistAndMergeSGPRInits(AMDGPU::M0, MRI, *MDT); |
| 759 | |
Matt Arsenault | 6f67978 | 2014-11-17 21:11:34 +0000 | [diff] [blame] | 760 | return true; |
Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 761 | } |