Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 1 | //===-- SIShrinkInstructions.cpp - Shrink Instructions --------------------===// |
| 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 | /// The pass tries to use the 32-bit encoding for instructions when possible. |
| 9 | //===----------------------------------------------------------------------===// |
| 10 | // |
| 11 | |
| 12 | #include "AMDGPU.h" |
Marek Olsak | a93603d | 2015-01-15 18:42:51 +0000 | [diff] [blame] | 13 | #include "AMDGPUMCInstLower.h" |
Eric Christopher | d913448 | 2014-08-04 21:25:23 +0000 | [diff] [blame] | 14 | #include "AMDGPUSubtarget.h" |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 15 | #include "SIInstrInfo.h" |
| 16 | #include "llvm/ADT/Statistic.h" |
| 17 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 18 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| 19 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 20 | #include "llvm/IR/Constants.h" |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 21 | #include "llvm/IR/Function.h" |
Benjamin Kramer | 16132e6 | 2015-03-23 18:07:13 +0000 | [diff] [blame] | 22 | #include "llvm/IR/LLVMContext.h" |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 23 | #include "llvm/Support/Debug.h" |
Benjamin Kramer | 16132e6 | 2015-03-23 18:07:13 +0000 | [diff] [blame] | 24 | #include "llvm/Support/raw_ostream.h" |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 25 | #include "llvm/Target/TargetMachine.h" |
| 26 | |
| 27 | #define DEBUG_TYPE "si-shrink-instructions" |
| 28 | |
| 29 | STATISTIC(NumInstructionsShrunk, |
| 30 | "Number of 64-bit instruction reduced to 32-bit."); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 31 | STATISTIC(NumLiteralConstantsFolded, |
| 32 | "Number of literal constants folded into 32-bit instructions."); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 33 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 34 | using namespace llvm; |
| 35 | |
| 36 | namespace { |
| 37 | |
| 38 | class SIShrinkInstructions : public MachineFunctionPass { |
| 39 | public: |
| 40 | static char ID; |
| 41 | |
| 42 | public: |
| 43 | SIShrinkInstructions() : MachineFunctionPass(ID) { |
| 44 | } |
| 45 | |
Craig Topper | fd38cbe | 2014-08-30 16:48:34 +0000 | [diff] [blame] | 46 | bool runOnMachineFunction(MachineFunction &MF) override; |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 47 | |
Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 48 | StringRef getPassName() const override { return "SI Shrink Instructions"; } |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 49 | |
Craig Topper | fd38cbe | 2014-08-30 16:48:34 +0000 | [diff] [blame] | 50 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 51 | AU.setPreservesCFG(); |
| 52 | MachineFunctionPass::getAnalysisUsage(AU); |
| 53 | } |
| 54 | }; |
| 55 | |
| 56 | } // End anonymous namespace. |
| 57 | |
Matt Arsenault | c3a01ec | 2016-06-09 23:18:47 +0000 | [diff] [blame] | 58 | INITIALIZE_PASS(SIShrinkInstructions, DEBUG_TYPE, |
| 59 | "SI Shrink Instructions", false, false) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 60 | |
| 61 | char SIShrinkInstructions::ID = 0; |
| 62 | |
| 63 | FunctionPass *llvm::createSIShrinkInstructionsPass() { |
| 64 | return new SIShrinkInstructions(); |
| 65 | } |
| 66 | |
| 67 | static bool isVGPR(const MachineOperand *MO, const SIRegisterInfo &TRI, |
| 68 | const MachineRegisterInfo &MRI) { |
| 69 | if (!MO->isReg()) |
| 70 | return false; |
| 71 | |
| 72 | if (TargetRegisterInfo::isVirtualRegister(MO->getReg())) |
| 73 | return TRI.hasVGPRs(MRI.getRegClass(MO->getReg())); |
| 74 | |
| 75 | return TRI.hasVGPRs(TRI.getPhysRegClass(MO->getReg())); |
| 76 | } |
| 77 | |
| 78 | static bool canShrink(MachineInstr &MI, const SIInstrInfo *TII, |
| 79 | const SIRegisterInfo &TRI, |
| 80 | const MachineRegisterInfo &MRI) { |
| 81 | |
| 82 | const MachineOperand *Src2 = TII->getNamedOperand(MI, AMDGPU::OpName::src2); |
| 83 | // Can't shrink instruction with three operands. |
Tom Stellard | 5224df3 | 2015-03-10 16:16:44 +0000 | [diff] [blame] | 84 | // FIXME: v_cndmask_b32 has 3 operands and is shrinkable, but we need to add |
| 85 | // a special case for it. It can only be shrunk if the third operand |
| 86 | // is vcc. We should handle this the same way we handle vopc, by addding |
| 87 | // a register allocation hint pre-regalloc and then do the shrining |
| 88 | // post-regalloc. |
Tom Stellard | db5a11f | 2015-07-13 15:47:57 +0000 | [diff] [blame] | 89 | if (Src2) { |
Tom Stellard | e48fe2a | 2015-07-14 14:15:03 +0000 | [diff] [blame] | 90 | switch (MI.getOpcode()) { |
| 91 | default: return false; |
Tom Stellard | db5a11f | 2015-07-13 15:47:57 +0000 | [diff] [blame] | 92 | |
Tom Stellard | e48fe2a | 2015-07-14 14:15:03 +0000 | [diff] [blame] | 93 | case AMDGPU::V_MAC_F32_e64: |
| 94 | if (!isVGPR(Src2, TRI, MRI) || |
| 95 | TII->hasModifiersSet(MI, AMDGPU::OpName::src2_modifiers)) |
| 96 | return false; |
| 97 | break; |
| 98 | |
| 99 | case AMDGPU::V_CNDMASK_B32_e64: |
| 100 | break; |
| 101 | } |
Tom Stellard | db5a11f | 2015-07-13 15:47:57 +0000 | [diff] [blame] | 102 | } |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 103 | |
| 104 | const MachineOperand *Src1 = TII->getNamedOperand(MI, AMDGPU::OpName::src1); |
| 105 | const MachineOperand *Src1Mod = |
| 106 | TII->getNamedOperand(MI, AMDGPU::OpName::src1_modifiers); |
| 107 | |
Tom Stellard | b4a313a | 2014-08-01 00:32:39 +0000 | [diff] [blame] | 108 | if (Src1 && (!isVGPR(Src1, TRI, MRI) || (Src1Mod && Src1Mod->getImm() != 0))) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 109 | return false; |
| 110 | |
Matt Arsenault | 8943d24 | 2014-10-17 18:00:45 +0000 | [diff] [blame] | 111 | // We don't need to check src0, all input types are legal, so just make sure |
| 112 | // src0 isn't using any modifiers. |
| 113 | if (TII->hasModifiersSet(MI, AMDGPU::OpName::src0_modifiers)) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 114 | return false; |
| 115 | |
| 116 | // Check output modifiers |
Matt Arsenault | 8943d24 | 2014-10-17 18:00:45 +0000 | [diff] [blame] | 117 | if (TII->hasModifiersSet(MI, AMDGPU::OpName::omod)) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 118 | return false; |
| 119 | |
Matt Arsenault | 8226fc4 | 2016-03-02 23:00:21 +0000 | [diff] [blame] | 120 | return !TII->hasModifiersSet(MI, AMDGPU::OpName::clamp); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 121 | } |
| 122 | |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 123 | /// \brief This function checks \p MI for operands defined by a move immediate |
| 124 | /// instruction and then folds the literal constant into the instruction if it |
| 125 | /// can. This function assumes that \p MI is a VOP1, VOP2, or VOPC instruction |
| 126 | /// and will only fold literal constants if we are still in SSA. |
| 127 | static void foldImmediates(MachineInstr &MI, const SIInstrInfo *TII, |
| 128 | MachineRegisterInfo &MRI, bool TryToCommute = true) { |
| 129 | |
| 130 | if (!MRI.isSSA()) |
| 131 | return; |
| 132 | |
Matt Arsenault | 3add643 | 2015-10-20 04:35:43 +0000 | [diff] [blame] | 133 | assert(TII->isVOP1(MI) || TII->isVOP2(MI) || TII->isVOPC(MI)); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 134 | |
Matt Arsenault | 11a4d67 | 2015-02-13 19:05:03 +0000 | [diff] [blame] | 135 | int Src0Idx = AMDGPU::getNamedOperandIdx(MI.getOpcode(), AMDGPU::OpName::src0); |
| 136 | MachineOperand &Src0 = MI.getOperand(Src0Idx); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 137 | |
| 138 | // Only one literal constant is allowed per instruction, so if src0 is a |
| 139 | // literal constant then we can't do any folding. |
Matt Arsenault | 11a4d67 | 2015-02-13 19:05:03 +0000 | [diff] [blame] | 140 | if (Src0.isImm() && |
| 141 | TII->isLiteralConstant(Src0, TII->getOpSize(MI, Src0Idx))) |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 142 | return; |
| 143 | |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 144 | // Try to fold Src0 |
Tom Stellard | ab6e9c0 | 2015-07-09 16:30:36 +0000 | [diff] [blame] | 145 | if (Src0.isReg() && MRI.hasOneUse(Src0.getReg())) { |
Matt Arsenault | 11a4d67 | 2015-02-13 19:05:03 +0000 | [diff] [blame] | 146 | unsigned Reg = Src0.getReg(); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 147 | MachineInstr *Def = MRI.getUniqueVRegDef(Reg); |
| 148 | if (Def && Def->isMoveImmediate()) { |
| 149 | MachineOperand &MovSrc = Def->getOperand(1); |
| 150 | bool ConstantFolded = false; |
| 151 | |
Matt Arsenault | 124384f | 2016-09-09 23:32:53 +0000 | [diff] [blame] | 152 | if (MovSrc.isImm() && (isInt<32>(MovSrc.getImm()) || |
| 153 | isUInt<32>(MovSrc.getImm()))) { |
Matt Arsenault | 11a4d67 | 2015-02-13 19:05:03 +0000 | [diff] [blame] | 154 | Src0.ChangeToImmediate(MovSrc.getImm()); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 155 | ConstantFolded = true; |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 156 | } |
| 157 | if (ConstantFolded) { |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 158 | if (MRI.use_empty(Reg)) |
| 159 | Def->eraseFromParent(); |
| 160 | ++NumLiteralConstantsFolded; |
| 161 | return; |
| 162 | } |
| 163 | } |
| 164 | } |
| 165 | |
| 166 | // We have failed to fold src0, so commute the instruction and try again. |
Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 167 | if (TryToCommute && MI.isCommutable() && TII->commuteInstruction(MI)) |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 168 | foldImmediates(MI, TII, MRI, false); |
| 169 | |
| 170 | } |
| 171 | |
Matt Arsenault | 6942d1a | 2015-08-08 00:41:45 +0000 | [diff] [blame] | 172 | // Copy MachineOperand with all flags except setting it as implicit. |
Matt Arsenault | 2209625 | 2016-06-20 18:34:00 +0000 | [diff] [blame] | 173 | static void copyFlagsToImplicitVCC(MachineInstr &MI, |
| 174 | const MachineOperand &Orig) { |
| 175 | |
| 176 | for (MachineOperand &Use : MI.implicit_operands()) { |
| 177 | if (Use.getReg() == AMDGPU::VCC) { |
| 178 | Use.setIsUndef(Orig.isUndef()); |
| 179 | Use.setIsKill(Orig.isKill()); |
| 180 | return; |
| 181 | } |
| 182 | } |
Matt Arsenault | 6942d1a | 2015-08-08 00:41:45 +0000 | [diff] [blame] | 183 | } |
| 184 | |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 185 | static bool isKImmOperand(const SIInstrInfo *TII, const MachineOperand &Src) { |
| 186 | return isInt<16>(Src.getImm()) && !TII->isInlineConstant(Src, 4); |
| 187 | } |
| 188 | |
Matt Arsenault | 7ccf6cd | 2016-09-16 21:41:16 +0000 | [diff] [blame] | 189 | static bool isKUImmOperand(const SIInstrInfo *TII, const MachineOperand &Src) { |
| 190 | return isUInt<16>(Src.getImm()) && !TII->isInlineConstant(Src, 4); |
| 191 | } |
| 192 | |
| 193 | static bool isKImmOrKUImmOperand(const SIInstrInfo *TII, |
| 194 | const MachineOperand &Src, |
| 195 | bool &IsUnsigned) { |
| 196 | if (isInt<16>(Src.getImm())) { |
| 197 | IsUnsigned = false; |
| 198 | return !TII->isInlineConstant(Src, 4); |
| 199 | } |
| 200 | |
| 201 | if (isUInt<16>(Src.getImm())) { |
| 202 | IsUnsigned = true; |
| 203 | return !TII->isInlineConstant(Src, 4); |
| 204 | } |
| 205 | |
| 206 | return false; |
| 207 | } |
| 208 | |
Matt Arsenault | 663ab8c | 2016-11-01 23:14:20 +0000 | [diff] [blame] | 209 | /// \returns true if the constant in \p Src should be replaced with a bitreverse |
| 210 | /// of an inline immediate. |
| 211 | static bool isReverseInlineImm(const SIInstrInfo *TII, |
| 212 | const MachineOperand &Src, |
| 213 | int32_t &ReverseImm) { |
| 214 | if (!isInt<32>(Src.getImm()) || TII->isInlineConstant(Src, 4)) |
| 215 | return false; |
| 216 | |
| 217 | ReverseImm = reverseBits<int32_t>(static_cast<int32_t>(Src.getImm())); |
| 218 | return ReverseImm >= -16 && ReverseImm <= 64; |
| 219 | } |
| 220 | |
Matt Arsenault | 5ffe3e1 | 2016-09-03 17:25:39 +0000 | [diff] [blame] | 221 | /// Copy implicit register operands from specified instruction to this |
| 222 | /// instruction that are not part of the instruction definition. |
| 223 | static void copyExtraImplicitOps(MachineInstr &NewMI, MachineFunction &MF, |
| 224 | const MachineInstr &MI) { |
| 225 | for (unsigned i = MI.getDesc().getNumOperands() + |
| 226 | MI.getDesc().getNumImplicitUses() + |
| 227 | MI.getDesc().getNumImplicitDefs(), e = MI.getNumOperands(); |
| 228 | i != e; ++i) { |
| 229 | const MachineOperand &MO = MI.getOperand(i); |
| 230 | if ((MO.isReg() && MO.isImplicit()) || MO.isRegMask()) |
| 231 | NewMI.addOperand(MF, MO); |
| 232 | } |
| 233 | } |
| 234 | |
Matt Arsenault | 7ccf6cd | 2016-09-16 21:41:16 +0000 | [diff] [blame] | 235 | static void shrinkScalarCompare(const SIInstrInfo *TII, MachineInstr &MI) { |
| 236 | // cmpk instructions do scc = dst <cc op> imm16, so commute the instruction to |
| 237 | // get constants on the RHS. |
| 238 | if (!MI.getOperand(0).isReg()) |
| 239 | TII->commuteInstruction(MI, false, 0, 1); |
| 240 | |
| 241 | const MachineOperand &Src1 = MI.getOperand(1); |
| 242 | if (!Src1.isImm()) |
| 243 | return; |
| 244 | |
| 245 | int SOPKOpc = AMDGPU::getSOPKOp(MI.getOpcode()); |
| 246 | if (SOPKOpc == -1) |
| 247 | return; |
| 248 | |
| 249 | // eq/ne is special because the imm16 can be treated as signed or unsigned, |
Matt Arsenault | 5d8eb25 | 2016-09-30 01:50:20 +0000 | [diff] [blame] | 250 | // and initially selectd to the unsigned versions. |
| 251 | if (SOPKOpc == AMDGPU::S_CMPK_EQ_U32 || SOPKOpc == AMDGPU::S_CMPK_LG_U32) { |
Matt Arsenault | 7ccf6cd | 2016-09-16 21:41:16 +0000 | [diff] [blame] | 252 | bool HasUImm; |
| 253 | if (isKImmOrKUImmOperand(TII, Src1, HasUImm)) { |
Matt Arsenault | 5d8eb25 | 2016-09-30 01:50:20 +0000 | [diff] [blame] | 254 | if (!HasUImm) { |
| 255 | SOPKOpc = (SOPKOpc == AMDGPU::S_CMPK_EQ_U32) ? |
| 256 | AMDGPU::S_CMPK_EQ_I32 : AMDGPU::S_CMPK_LG_I32; |
Matt Arsenault | 7ccf6cd | 2016-09-16 21:41:16 +0000 | [diff] [blame] | 257 | } |
| 258 | |
| 259 | MI.setDesc(TII->get(SOPKOpc)); |
| 260 | } |
| 261 | |
| 262 | return; |
| 263 | } |
| 264 | |
| 265 | const MCInstrDesc &NewDesc = TII->get(SOPKOpc); |
| 266 | |
| 267 | if ((TII->sopkIsZext(SOPKOpc) && isKUImmOperand(TII, Src1)) || |
| 268 | (!TII->sopkIsZext(SOPKOpc) && isKImmOperand(TII, Src1))) { |
| 269 | MI.setDesc(NewDesc); |
| 270 | } |
| 271 | } |
| 272 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 273 | bool SIShrinkInstructions::runOnMachineFunction(MachineFunction &MF) { |
Andrew Kaylor | 7de74af | 2016-04-25 22:23:44 +0000 | [diff] [blame] | 274 | if (skipFunction(*MF.getFunction())) |
| 275 | return false; |
| 276 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 277 | MachineRegisterInfo &MRI = MF.getRegInfo(); |
Matt Arsenault | 43e92fe | 2016-06-24 06:30:11 +0000 | [diff] [blame] | 278 | const SISubtarget &ST = MF.getSubtarget<SISubtarget>(); |
| 279 | const SIInstrInfo *TII = ST.getInstrInfo(); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 280 | const SIRegisterInfo &TRI = TII->getRegisterInfo(); |
Matt Arsenault | 43e92fe | 2016-06-24 06:30:11 +0000 | [diff] [blame] | 281 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 282 | std::vector<unsigned> I1Defs; |
| 283 | |
| 284 | for (MachineFunction::iterator BI = MF.begin(), BE = MF.end(); |
| 285 | BI != BE; ++BI) { |
| 286 | |
| 287 | MachineBasicBlock &MBB = *BI; |
| 288 | MachineBasicBlock::iterator I, Next; |
| 289 | for (I = MBB.begin(); I != MBB.end(); I = Next) { |
| 290 | Next = std::next(I); |
| 291 | MachineInstr &MI = *I; |
| 292 | |
Matt Arsenault | 9a19c24 | 2016-03-11 07:42:49 +0000 | [diff] [blame] | 293 | if (MI.getOpcode() == AMDGPU::V_MOV_B32_e32) { |
| 294 | // If this has a literal constant source that is the same as the |
| 295 | // reversed bits of an inline immediate, replace with a bitreverse of |
| 296 | // that constant. This saves 4 bytes in the common case of materializing |
| 297 | // sign bits. |
| 298 | |
| 299 | // Test if we are after regalloc. We only want to do this after any |
| 300 | // optimizations happen because this will confuse them. |
| 301 | // XXX - not exactly a check for post-regalloc run. |
| 302 | MachineOperand &Src = MI.getOperand(1); |
| 303 | if (Src.isImm() && |
| 304 | TargetRegisterInfo::isPhysicalRegister(MI.getOperand(0).getReg())) { |
Matt Arsenault | 663ab8c | 2016-11-01 23:14:20 +0000 | [diff] [blame] | 305 | int32_t ReverseImm; |
| 306 | if (isReverseInlineImm(TII, Src, ReverseImm)) { |
| 307 | MI.setDesc(TII->get(AMDGPU::V_BFREV_B32_e32)); |
| 308 | Src.setImm(ReverseImm); |
| 309 | continue; |
Matt Arsenault | 9a19c24 | 2016-03-11 07:42:49 +0000 | [diff] [blame] | 310 | } |
| 311 | } |
| 312 | } |
| 313 | |
Matt Arsenault | 074ea28 | 2016-04-25 19:53:22 +0000 | [diff] [blame] | 314 | // Combine adjacent s_nops to use the immediate operand encoding how long |
| 315 | // to wait. |
| 316 | // |
| 317 | // s_nop N |
| 318 | // s_nop M |
| 319 | // => |
| 320 | // s_nop (N + M) |
| 321 | if (MI.getOpcode() == AMDGPU::S_NOP && |
| 322 | Next != MBB.end() && |
| 323 | (*Next).getOpcode() == AMDGPU::S_NOP) { |
| 324 | |
| 325 | MachineInstr &NextMI = *Next; |
| 326 | // The instruction encodes the amount to wait with an offset of 1, |
| 327 | // i.e. 0 is wait 1 cycle. Convert both to cycles and then convert back |
| 328 | // after adding. |
| 329 | uint8_t Nop0 = MI.getOperand(0).getImm() + 1; |
| 330 | uint8_t Nop1 = NextMI.getOperand(0).getImm() + 1; |
| 331 | |
| 332 | // Make sure we don't overflow the bounds. |
| 333 | if (Nop0 + Nop1 <= 8) { |
| 334 | NextMI.getOperand(0).setImm(Nop0 + Nop1 - 1); |
| 335 | MI.eraseFromParent(); |
| 336 | } |
| 337 | |
| 338 | continue; |
| 339 | } |
| 340 | |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 341 | // FIXME: We also need to consider movs of constant operands since |
| 342 | // immediate operands are not folded if they have more than one use, and |
| 343 | // the operand folding pass is unaware if the immediate will be free since |
| 344 | // it won't know if the src == dest constraint will end up being |
| 345 | // satisfied. |
| 346 | if (MI.getOpcode() == AMDGPU::S_ADD_I32 || |
| 347 | MI.getOpcode() == AMDGPU::S_MUL_I32) { |
Matt Arsenault | be90f70 | 2016-09-08 17:35:41 +0000 | [diff] [blame] | 348 | const MachineOperand *Dest = &MI.getOperand(0); |
| 349 | MachineOperand *Src0 = &MI.getOperand(1); |
| 350 | MachineOperand *Src1 = &MI.getOperand(2); |
| 351 | |
| 352 | if (!Src0->isReg() && Src1->isReg()) { |
| 353 | if (TII->commuteInstruction(MI, false, 1, 2)) |
| 354 | std::swap(Src0, Src1); |
| 355 | } |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 356 | |
| 357 | // FIXME: This could work better if hints worked with subregisters. If |
| 358 | // we have a vector add of a constant, we usually don't get the correct |
| 359 | // allocation due to the subregister usage. |
Matt Arsenault | be90f70 | 2016-09-08 17:35:41 +0000 | [diff] [blame] | 360 | if (TargetRegisterInfo::isVirtualRegister(Dest->getReg()) && |
| 361 | Src0->isReg()) { |
| 362 | MRI.setRegAllocationHint(Dest->getReg(), 0, Src0->getReg()); |
| 363 | MRI.setRegAllocationHint(Src0->getReg(), 0, Dest->getReg()); |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 364 | continue; |
| 365 | } |
| 366 | |
Matt Arsenault | be90f70 | 2016-09-08 17:35:41 +0000 | [diff] [blame] | 367 | if (Src0->isReg() && Src0->getReg() == Dest->getReg()) { |
| 368 | if (Src1->isImm() && isKImmOperand(TII, *Src1)) { |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 369 | unsigned Opc = (MI.getOpcode() == AMDGPU::S_ADD_I32) ? |
| 370 | AMDGPU::S_ADDK_I32 : AMDGPU::S_MULK_I32; |
| 371 | |
| 372 | MI.setDesc(TII->get(Opc)); |
| 373 | MI.tieOperands(0, 1); |
| 374 | } |
| 375 | } |
| 376 | } |
| 377 | |
Matt Arsenault | 7ccf6cd | 2016-09-16 21:41:16 +0000 | [diff] [blame] | 378 | // Try to use s_cmpk_* |
| 379 | if (MI.isCompare() && TII->isSOPC(MI)) { |
| 380 | shrinkScalarCompare(TII, MI); |
| 381 | continue; |
| 382 | } |
| 383 | |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 384 | // Try to use S_MOVK_I32, which will save 4 bytes for small immediates. |
| 385 | if (MI.getOpcode() == AMDGPU::S_MOV_B32) { |
Matt Arsenault | 663ab8c | 2016-11-01 23:14:20 +0000 | [diff] [blame] | 386 | const MachineOperand &Dst = MI.getOperand(0); |
| 387 | MachineOperand &Src = MI.getOperand(1); |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 388 | |
Matt Arsenault | 663ab8c | 2016-11-01 23:14:20 +0000 | [diff] [blame] | 389 | if (Src.isImm() && |
| 390 | TargetRegisterInfo::isPhysicalRegister(Dst.getReg())) { |
| 391 | int32_t ReverseImm; |
| 392 | if (isKImmOperand(TII, Src)) |
| 393 | MI.setDesc(TII->get(AMDGPU::S_MOVK_I32)); |
| 394 | else if (isReverseInlineImm(TII, Src, ReverseImm)) { |
| 395 | MI.setDesc(TII->get(AMDGPU::S_BREV_B32)); |
| 396 | Src.setImm(ReverseImm); |
| 397 | } |
| 398 | } |
Matt Arsenault | b6be202 | 2016-04-16 01:46:49 +0000 | [diff] [blame] | 399 | |
| 400 | continue; |
| 401 | } |
| 402 | |
Tom Stellard | 86d12eb | 2014-08-01 00:32:28 +0000 | [diff] [blame] | 403 | if (!TII->hasVALU32BitEncoding(MI.getOpcode())) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 404 | continue; |
| 405 | |
| 406 | if (!canShrink(MI, TII, TRI, MRI)) { |
Matt Arsenault | 6652403 | 2014-09-16 18:00:23 +0000 | [diff] [blame] | 407 | // Try commuting the instruction and see if that enables us to shrink |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 408 | // it. |
Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 409 | if (!MI.isCommutable() || !TII->commuteInstruction(MI) || |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 410 | !canShrink(MI, TII, TRI, MRI)) |
| 411 | continue; |
| 412 | } |
| 413 | |
Marek Olsak | a93603d | 2015-01-15 18:42:51 +0000 | [diff] [blame] | 414 | // getVOPe32 could be -1 here if we started with an instruction that had |
Tom Stellard | 86d12eb | 2014-08-01 00:32:28 +0000 | [diff] [blame] | 415 | // a 32-bit encoding and then commuted it to an instruction that did not. |
Marek Olsak | a93603d | 2015-01-15 18:42:51 +0000 | [diff] [blame] | 416 | if (!TII->hasVALU32BitEncoding(MI.getOpcode())) |
Tom Stellard | 86d12eb | 2014-08-01 00:32:28 +0000 | [diff] [blame] | 417 | continue; |
| 418 | |
Marek Olsak | a93603d | 2015-01-15 18:42:51 +0000 | [diff] [blame] | 419 | int Op32 = AMDGPU::getVOPe32(MI.getOpcode()); |
| 420 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 421 | if (TII->isVOPC(Op32)) { |
| 422 | unsigned DstReg = MI.getOperand(0).getReg(); |
| 423 | if (TargetRegisterInfo::isVirtualRegister(DstReg)) { |
Matt Arsenault | 6942d1a | 2015-08-08 00:41:45 +0000 | [diff] [blame] | 424 | // VOPC instructions can only write to the VCC register. We can't |
| 425 | // force them to use VCC here, because this is only one register and |
| 426 | // cannot deal with sequences which would require multiple copies of |
| 427 | // VCC, e.g. S_AND_B64 (vcc = V_CMP_...), (vcc = V_CMP_...) |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 428 | // |
Matt Arsenault | a9627ae | 2014-09-21 17:27:32 +0000 | [diff] [blame] | 429 | // So, instead of forcing the instruction to write to VCC, we provide |
| 430 | // a hint to the register allocator to use VCC and then we we will run |
| 431 | // this pass again after RA and shrink it if it outputs to VCC. |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 432 | MRI.setRegAllocationHint(MI.getOperand(0).getReg(), 0, AMDGPU::VCC); |
| 433 | continue; |
| 434 | } |
| 435 | if (DstReg != AMDGPU::VCC) |
| 436 | continue; |
| 437 | } |
| 438 | |
Tom Stellard | e48fe2a | 2015-07-14 14:15:03 +0000 | [diff] [blame] | 439 | if (Op32 == AMDGPU::V_CNDMASK_B32_e32) { |
| 440 | // We shrink V_CNDMASK_B32_e64 using regalloc hints like we do for VOPC |
| 441 | // instructions. |
| 442 | const MachineOperand *Src2 = |
| 443 | TII->getNamedOperand(MI, AMDGPU::OpName::src2); |
| 444 | if (!Src2->isReg()) |
| 445 | continue; |
| 446 | unsigned SReg = Src2->getReg(); |
| 447 | if (TargetRegisterInfo::isVirtualRegister(SReg)) { |
| 448 | MRI.setRegAllocationHint(SReg, 0, AMDGPU::VCC); |
| 449 | continue; |
| 450 | } |
| 451 | if (SReg != AMDGPU::VCC) |
| 452 | continue; |
| 453 | } |
| 454 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 455 | // We can shrink this instruction |
Matt Arsenault | e0b4404 | 2015-09-10 21:51:19 +0000 | [diff] [blame] | 456 | DEBUG(dbgs() << "Shrinking " << MI); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 457 | |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 458 | MachineInstrBuilder Inst32 = |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 459 | BuildMI(MBB, I, MI.getDebugLoc(), TII->get(Op32)); |
| 460 | |
Tom Stellard | cc4c871 | 2016-02-16 18:14:56 +0000 | [diff] [blame] | 461 | // Add the dst operand if the 32-bit encoding also has an explicit $vdst. |
Matt Arsenault | 4635915 | 2015-08-08 00:41:48 +0000 | [diff] [blame] | 462 | // For VOPC instructions, this is replaced by an implicit def of vcc. |
Tom Stellard | cc4c871 | 2016-02-16 18:14:56 +0000 | [diff] [blame] | 463 | int Op32DstIdx = AMDGPU::getNamedOperandIdx(Op32, AMDGPU::OpName::vdst); |
Matt Arsenault | 4635915 | 2015-08-08 00:41:48 +0000 | [diff] [blame] | 464 | if (Op32DstIdx != -1) { |
| 465 | // dst |
| 466 | Inst32.addOperand(MI.getOperand(0)); |
| 467 | } else { |
| 468 | assert(MI.getOperand(0).getReg() == AMDGPU::VCC && |
| 469 | "Unexpected case"); |
| 470 | } |
| 471 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 472 | |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 473 | Inst32.addOperand(*TII->getNamedOperand(MI, AMDGPU::OpName::src0)); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 474 | |
| 475 | const MachineOperand *Src1 = |
| 476 | TII->getNamedOperand(MI, AMDGPU::OpName::src1); |
| 477 | if (Src1) |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 478 | Inst32.addOperand(*Src1); |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 479 | |
Tom Stellard | db5a11f | 2015-07-13 15:47:57 +0000 | [diff] [blame] | 480 | const MachineOperand *Src2 = |
Matt Arsenault | 6942d1a | 2015-08-08 00:41:45 +0000 | [diff] [blame] | 481 | TII->getNamedOperand(MI, AMDGPU::OpName::src2); |
| 482 | if (Src2) { |
| 483 | int Op32Src2Idx = AMDGPU::getNamedOperandIdx(Op32, AMDGPU::OpName::src2); |
| 484 | if (Op32Src2Idx != -1) { |
| 485 | Inst32.addOperand(*Src2); |
| 486 | } else { |
| 487 | // In the case of V_CNDMASK_B32_e32, the explicit operand src2 is |
Matt Arsenault | 2209625 | 2016-06-20 18:34:00 +0000 | [diff] [blame] | 488 | // replaced with an implicit read of vcc. This was already added |
| 489 | // during the initial BuildMI, so find it to preserve the flags. |
| 490 | copyFlagsToImplicitVCC(*Inst32, *Src2); |
Matt Arsenault | 6942d1a | 2015-08-08 00:41:45 +0000 | [diff] [blame] | 491 | } |
| 492 | } |
Tom Stellard | db5a11f | 2015-07-13 15:47:57 +0000 | [diff] [blame] | 493 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 494 | ++NumInstructionsShrunk; |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 495 | |
Matt Arsenault | cb540bc | 2016-07-19 00:35:03 +0000 | [diff] [blame] | 496 | // Copy extra operands not present in the instruction definition. |
Matt Arsenault | 5ffe3e1 | 2016-09-03 17:25:39 +0000 | [diff] [blame] | 497 | copyExtraImplicitOps(*Inst32, MF, MI); |
Matt Arsenault | cb540bc | 2016-07-19 00:35:03 +0000 | [diff] [blame] | 498 | |
| 499 | MI.eraseFromParent(); |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 500 | foldImmediates(*Inst32, TII, MRI); |
Matt Arsenault | cb540bc | 2016-07-19 00:35:03 +0000 | [diff] [blame] | 501 | |
Tom Stellard | 6407e1e | 2014-08-01 00:32:33 +0000 | [diff] [blame] | 502 | DEBUG(dbgs() << "e32 MI = " << *Inst32 << '\n'); |
| 503 | |
| 504 | |
Tom Stellard | 1aaad69 | 2014-07-21 16:55:33 +0000 | [diff] [blame] | 505 | } |
| 506 | } |
| 507 | return false; |
| 508 | } |