| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 1 | //===-- SIFixSGPRCopies.cpp - Remove potential VGPR => SGPR copies --------===// |
| 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: |
| 17 | /// %vreg0 <sgpr> = SCALAR_INST |
| 18 | /// %vreg1 <vsrc> = COPY %vreg0 <sgpr> |
| 19 | /// ... |
| 20 | /// BRANCH %cond BB1, BB2 |
| 21 | /// BB1: |
| 22 | /// %vreg2 <vgpr> = VECTOR_INST |
| 23 | /// %vreg3 <vsrc> = COPY %vreg2 <vgpr> |
| 24 | /// BB2: |
| 25 | /// %vreg4 <vsrc> = PHI %vreg1 <vsrc>, <BB#0>, %vreg3 <vrsc>, <BB#1> |
| NAKAMURA Takumi | 78e80cd | 2013-11-14 04:05:22 +0000 | [diff] [blame] | 26 | /// %vreg5 <vgpr> = VECTOR_INST %vreg4 <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: |
| 33 | /// %vreg0 <sgpr> = SCALAR_INST |
| 34 | /// ... |
| 35 | /// BRANCH %cond BB1, BB2 |
| 36 | /// BB1: |
| 37 | /// %vreg2 <vgpr> = VECTOR_INST |
| 38 | /// %vreg3 <vsrc> = COPY %vreg2 <vgpr> |
| 39 | /// BB2: |
| 40 | /// %vreg4 <sgpr> = PHI %vreg0 <sgpr>, <BB#0>, %vreg3 <vsrc>, <BB#1> |
| 41 | /// %vreg5 <vgpr> = VECTOR_INST %vreg4 <sgpr> |
| 42 | /// |
| 43 | /// Now that the result of the PHI instruction is an SGPR, the register |
| 44 | /// allocator is now forced to constrain the register class of %vreg3 to |
| 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: |
| 48 | /// %vreg0 <sgpr> = SCALAR_INST |
| 49 | /// ... |
| 50 | /// BRANCH %cond BB1, BB2 |
| 51 | /// BB1: |
| 52 | /// %vreg2 <vgpr> = VECTOR_INST |
| 53 | /// %vreg3 <sgpr> = COPY %vreg2 <vgpr> |
| 54 | /// BB2: |
| 55 | /// %vreg4 <sgpr> = PHI %vreg0 <sgpr>, <BB#0>, %vreg3 <sgpr>, <BB#1> |
| 56 | /// %vreg5 <vgpr> = VECTOR_INST %vreg4 <sgpr> |
| 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" |
| 69 | #include "SIInstrInfo.h" |
| 70 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 71 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 72 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 73 | #include "llvm/Support/Debug.h" |
| Hans Wennborg | a74fd70 | 2013-11-14 23:24:09 +0000 | [diff] [blame] | 74 | #include "llvm/Support/raw_ostream.h" |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 75 | #include "llvm/Target/TargetMachine.h" |
| 76 | |
| 77 | using namespace llvm; |
| 78 | |
| Chandler Carruth | 84e68b2 | 2014-04-22 02:41:26 +0000 | [diff] [blame^] | 79 | #define DEBUG_TYPE "sgpr-copies" |
| 80 | |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 81 | namespace { |
| 82 | |
| 83 | class SIFixSGPRCopies : public MachineFunctionPass { |
| 84 | |
| 85 | private: |
| 86 | static char ID; |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 87 | const TargetRegisterClass *inferRegClassFromUses(const SIRegisterInfo *TRI, |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 88 | const MachineRegisterInfo &MRI, |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 89 | unsigned Reg, |
| 90 | unsigned SubReg) const; |
| 91 | const TargetRegisterClass *inferRegClassFromDef(const SIRegisterInfo *TRI, |
| 92 | const MachineRegisterInfo &MRI, |
| 93 | unsigned Reg, |
| 94 | unsigned SubReg) const; |
| 95 | bool isVGPRToSGPRCopy(const MachineInstr &Copy, const SIRegisterInfo *TRI, |
| 96 | const MachineRegisterInfo &MRI) const; |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 97 | |
| 98 | public: |
| 99 | SIFixSGPRCopies(TargetMachine &tm) : MachineFunctionPass(ID) { } |
| 100 | |
| 101 | virtual bool runOnMachineFunction(MachineFunction &MF); |
| 102 | |
| 103 | const char *getPassName() const { |
| 104 | return "SI Fix SGPR copies"; |
| 105 | } |
| 106 | |
| 107 | }; |
| 108 | |
| 109 | } // End anonymous namespace |
| 110 | |
| 111 | char SIFixSGPRCopies::ID = 0; |
| 112 | |
| 113 | FunctionPass *llvm::createSIFixSGPRCopiesPass(TargetMachine &tm) { |
| 114 | return new SIFixSGPRCopies(tm); |
| 115 | } |
| 116 | |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 117 | static bool hasVGPROperands(const MachineInstr &MI, const SIRegisterInfo *TRI) { |
| 118 | const MachineRegisterInfo &MRI = MI.getParent()->getParent()->getRegInfo(); |
| 119 | for (unsigned i = 0, e = MI.getNumOperands(); i != e; ++i) { |
| 120 | if (!MI.getOperand(i).isReg() || |
| 121 | !TargetRegisterInfo::isVirtualRegister(MI.getOperand(i).getReg())) |
| 122 | continue; |
| 123 | |
| 124 | if (TRI->hasVGPRs(MRI.getRegClass(MI.getOperand(i).getReg()))) |
| 125 | return true; |
| 126 | } |
| 127 | return false; |
| 128 | } |
| 129 | |
| 130 | /// This functions walks the use list of Reg until it finds an Instruction |
| 131 | /// that isn't a COPY returns the register class of that instruction. |
| NAKAMURA Takumi | b88288f | 2013-11-14 04:05:28 +0000 | [diff] [blame] | 132 | /// \return The register defined by the first non-COPY instruction. |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 133 | const TargetRegisterClass *SIFixSGPRCopies::inferRegClassFromUses( |
| 134 | const SIRegisterInfo *TRI, |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 135 | const MachineRegisterInfo &MRI, |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 136 | unsigned Reg, |
| 137 | unsigned SubReg) const { |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 138 | // The Reg parameter to the function must always be defined by either a PHI |
| 139 | // or a COPY, therefore it cannot be a physical register. |
| 140 | assert(TargetRegisterInfo::isVirtualRegister(Reg) && |
| 141 | "Reg cannot be a physical register"); |
| 142 | |
| 143 | const TargetRegisterClass *RC = MRI.getRegClass(Reg); |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 144 | RC = TRI->getSubRegClass(RC, SubReg); |
| Owen Anderson | 16c6bf4 | 2014-03-13 23:12:04 +0000 | [diff] [blame] | 145 | for (MachineRegisterInfo::use_instr_iterator |
| 146 | I = MRI.use_instr_begin(Reg), E = MRI.use_instr_end(); I != E; ++I) { |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 147 | switch (I->getOpcode()) { |
| 148 | case AMDGPU::COPY: |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 149 | RC = TRI->getCommonSubClass(RC, inferRegClassFromUses(TRI, MRI, |
| 150 | I->getOperand(0).getReg(), |
| 151 | I->getOperand(0).getSubReg())); |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 152 | break; |
| 153 | } |
| 154 | } |
| 155 | |
| 156 | return RC; |
| 157 | } |
| 158 | |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 159 | const TargetRegisterClass *SIFixSGPRCopies::inferRegClassFromDef( |
| 160 | const SIRegisterInfo *TRI, |
| 161 | const MachineRegisterInfo &MRI, |
| 162 | unsigned Reg, |
| 163 | unsigned SubReg) const { |
| 164 | if (!TargetRegisterInfo::isVirtualRegister(Reg)) { |
| 165 | const TargetRegisterClass *RC = TRI->getPhysRegClass(Reg); |
| 166 | return TRI->getSubRegClass(RC, SubReg); |
| 167 | } |
| 168 | MachineInstr *Def = MRI.getVRegDef(Reg); |
| 169 | if (Def->getOpcode() != AMDGPU::COPY) { |
| 170 | return TRI->getSubRegClass(MRI.getRegClass(Reg), SubReg); |
| 171 | } |
| 172 | |
| 173 | return inferRegClassFromDef(TRI, MRI, Def->getOperand(1).getReg(), |
| 174 | Def->getOperand(1).getSubReg()); |
| 175 | } |
| 176 | |
| 177 | bool SIFixSGPRCopies::isVGPRToSGPRCopy(const MachineInstr &Copy, |
| 178 | const SIRegisterInfo *TRI, |
| 179 | const MachineRegisterInfo &MRI) const { |
| 180 | |
| 181 | unsigned DstReg = Copy.getOperand(0).getReg(); |
| 182 | unsigned SrcReg = Copy.getOperand(1).getReg(); |
| 183 | unsigned SrcSubReg = Copy.getOperand(1).getSubReg(); |
| 184 | const TargetRegisterClass *DstRC = MRI.getRegClass(DstReg); |
| Tom Stellard | 13de545 | 2013-11-18 18:50:15 +0000 | [diff] [blame] | 185 | const TargetRegisterClass *SrcRC; |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 186 | |
| 187 | if (!TargetRegisterInfo::isVirtualRegister(SrcReg) || |
| 188 | DstRC == &AMDGPU::M0RegRegClass) |
| 189 | return false; |
| 190 | |
| Tom Stellard | b8725d8 | 2014-02-04 17:18:42 +0000 | [diff] [blame] | 191 | SrcRC = TRI->getSubRegClass(MRI.getRegClass(SrcReg), SrcSubReg); |
| Tom Stellard | f340787 | 2013-11-18 18:50:20 +0000 | [diff] [blame] | 192 | return TRI->isSGPRClass(DstRC) && TRI->hasVGPRs(SrcRC); |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 193 | } |
| 194 | |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 195 | bool SIFixSGPRCopies::runOnMachineFunction(MachineFunction &MF) { |
| 196 | MachineRegisterInfo &MRI = MF.getRegInfo(); |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 197 | const SIRegisterInfo *TRI = static_cast<const SIRegisterInfo *>( |
| 198 | MF.getTarget().getRegisterInfo()); |
| 199 | const SIInstrInfo *TII = static_cast<const SIInstrInfo *>( |
| 200 | MF.getTarget().getInstrInfo()); |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 201 | for (MachineFunction::iterator BI = MF.begin(), BE = MF.end(); |
| 202 | BI != BE; ++BI) { |
| 203 | |
| 204 | MachineBasicBlock &MBB = *BI; |
| 205 | for (MachineBasicBlock::iterator I = MBB.begin(), E = MBB.end(); |
| 206 | I != E; ++I) { |
| 207 | MachineInstr &MI = *I; |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 208 | if (MI.getOpcode() == AMDGPU::COPY && isVGPRToSGPRCopy(MI, TRI, MRI)) { |
| 209 | DEBUG(dbgs() << "Fixing VGPR -> SGPR copy:\n"); |
| 210 | DEBUG(MI.print(dbgs())); |
| 211 | TII->moveToVALU(MI); |
| 212 | |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 213 | } |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 214 | |
| 215 | switch (MI.getOpcode()) { |
| 216 | default: continue; |
| 217 | case AMDGPU::PHI: { |
| 218 | DEBUG(dbgs() << " Fixing PHI:\n"); |
| 219 | DEBUG(MI.print(dbgs())); |
| 220 | |
| 221 | for (unsigned i = 1; i < MI.getNumOperands(); i+=2) { |
| 222 | unsigned Reg = MI.getOperand(i).getReg(); |
| 223 | const TargetRegisterClass *RC = inferRegClassFromDef(TRI, MRI, Reg, |
| 224 | MI.getOperand(0).getSubReg()); |
| 225 | MRI.constrainRegClass(Reg, RC); |
| 226 | } |
| 227 | unsigned Reg = MI.getOperand(0).getReg(); |
| 228 | const TargetRegisterClass *RC = inferRegClassFromUses(TRI, MRI, Reg, |
| 229 | MI.getOperand(0).getSubReg()); |
| 230 | if (TRI->getCommonSubClass(RC, &AMDGPU::VReg_32RegClass)) { |
| 231 | MRI.constrainRegClass(Reg, &AMDGPU::VReg_32RegClass); |
| 232 | } |
| 233 | |
| 234 | if (!TRI->isSGPRClass(MRI.getRegClass(Reg))) |
| 235 | break; |
| 236 | |
| 237 | // If a PHI node defines an SGPR and any of its operands are VGPRs, |
| 238 | // then we need to move it to the VALU. |
| 239 | for (unsigned i = 1; i < MI.getNumOperands(); i+=2) { |
| 240 | unsigned Reg = MI.getOperand(i).getReg(); |
| 241 | if (TRI->hasVGPRs(MRI.getRegClass(Reg))) { |
| 242 | TII->moveToVALU(MI); |
| 243 | break; |
| 244 | } |
| 245 | } |
| 246 | |
| 247 | break; |
| 248 | } |
| 249 | case AMDGPU::REG_SEQUENCE: { |
| 250 | if (TRI->hasVGPRs(TII->getOpRegClass(MI, 0)) || |
| 251 | !hasVGPROperands(MI, TRI)) |
| 252 | continue; |
| 253 | |
| Matt Arsenault | 269092d | 2013-11-14 08:06:35 +0000 | [diff] [blame] | 254 | DEBUG(dbgs() << "Fixing REG_SEQUENCE:\n"); |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 255 | DEBUG(MI.print(dbgs())); |
| 256 | |
| 257 | TII->moveToVALU(MI); |
| Tom Stellard | 8216602 | 2013-11-13 23:36:37 +0000 | [diff] [blame] | 258 | break; |
| 259 | } |
| Tom Stellard | 204e61b | 2014-04-07 19:45:45 +0000 | [diff] [blame] | 260 | case AMDGPU::INSERT_SUBREG: { |
| 261 | const TargetRegisterClass *DstRC, *SrcRC; |
| 262 | DstRC = MRI.getRegClass(MI.getOperand(0).getReg()); |
| 263 | SrcRC = MRI.getRegClass(MI.getOperand(1).getReg()); |
| 264 | if (!TRI->isSGPRClass(DstRC) || !TRI->hasVGPRs(SrcRC)) |
| 265 | break; |
| 266 | DEBUG(dbgs() << " Fixing INSERT_SUBREG:\n"); |
| 267 | DEBUG(MI.print(dbgs())); |
| 268 | TII->moveToVALU(MI); |
| 269 | } |
| Tom Stellard | 2f7cdda | 2013-08-06 23:08:28 +0000 | [diff] [blame] | 270 | } |
| 271 | } |
| 272 | } |
| 273 | return false; |
| 274 | } |