Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 1 | //=- AArch64RedundantCopyElimination.cpp - Remove useless copy for AArch64 -=// |
| 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 | // |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 8 | // This pass removes unnecessary copies/moves in BBs based on a dominating |
| 9 | // condition. |
| 10 | // |
| 11 | // We handle three cases: |
| 12 | // 1. For BBs that are targets of CBZ/CBNZ instructions, we know the value of |
| 13 | // the CBZ/CBNZ source register is zero on the taken/not-taken path. For |
| 14 | // instance, the copy instruction in the code below can be removed because |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 15 | // the CBZW jumps to %bb.2 when w0 is zero. |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 16 | // |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 17 | // %bb.1: |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 18 | // cbz w0, .LBB0_2 |
| 19 | // .LBB0_2: |
| 20 | // mov w0, wzr ; <-- redundant |
| 21 | // |
| 22 | // 2. If the flag setting instruction defines a register other than WZR/XZR, we |
| 23 | // can remove a zero copy in some cases. |
| 24 | // |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 25 | // %bb.0: |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 26 | // subs w0, w1, w2 |
| 27 | // str w0, [x1] |
| 28 | // b.ne .LBB0_2 |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 29 | // %bb.1: |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 30 | // mov w0, wzr ; <-- redundant |
| 31 | // str w0, [x2] |
| 32 | // .LBB0_2 |
| 33 | // |
| 34 | // 3. Finally, if the flag setting instruction is a comparison against a |
| 35 | // constant (i.e., ADDS[W|X]ri, SUBS[W|X]ri), we can remove a mov immediate |
| 36 | // in some cases. |
| 37 | // |
Francis Visoiu Mistrih | 25528d6 | 2017-12-04 17:18:51 +0000 | [diff] [blame] | 38 | // %bb.0: |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 39 | // subs xzr, x0, #1 |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 40 | // b.eq .LBB0_1 |
| 41 | // .LBB0_1: |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 42 | // orr x0, xzr, #0x1 ; <-- redundant |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 43 | // |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 44 | // This pass should be run after register allocation. |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 45 | // |
| 46 | // FIXME: This could also be extended to check the whole dominance subtree below |
| 47 | // the comparison if the compile time regression is acceptable. |
| 48 | // |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 49 | // FIXME: Add support for handling CCMP instructions. |
| 50 | // FIXME: If the known register value is zero, we should be able to rewrite uses |
| 51 | // to use WZR/XZR directly in some cases. |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 52 | //===----------------------------------------------------------------------===// |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 53 | #include "AArch64.h" |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 54 | #include "llvm/ADT/Optional.h" |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 55 | #include "llvm/ADT/SetVector.h" |
| 56 | #include "llvm/ADT/Statistic.h" |
| 57 | #include "llvm/ADT/iterator_range.h" |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 58 | #include "llvm/CodeGen/LiveRegUnits.h" |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 59 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 60 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
| 61 | #include "llvm/Support/Debug.h" |
| 62 | |
| 63 | using namespace llvm; |
| 64 | |
| 65 | #define DEBUG_TYPE "aarch64-copyelim" |
| 66 | |
| 67 | STATISTIC(NumCopiesRemoved, "Number of copies removed."); |
| 68 | |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 69 | namespace { |
| 70 | class AArch64RedundantCopyElimination : public MachineFunctionPass { |
| 71 | const MachineRegisterInfo *MRI; |
| 72 | const TargetRegisterInfo *TRI; |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 73 | |
| 74 | // DomBBClobberedRegs is used when computing known values in the dominating |
| 75 | // BB. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 76 | LiveRegUnits DomBBClobberedRegs, DomBBUsedRegs; |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 77 | |
| 78 | // OptBBClobberedRegs is used when optimizing away redundant copies/moves. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 79 | LiveRegUnits OptBBClobberedRegs, OptBBUsedRegs; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 80 | |
| 81 | public: |
| 82 | static char ID; |
Diana Picus | 850043b | 2016-08-01 05:56:57 +0000 | [diff] [blame] | 83 | AArch64RedundantCopyElimination() : MachineFunctionPass(ID) { |
| 84 | initializeAArch64RedundantCopyEliminationPass( |
| 85 | *PassRegistry::getPassRegistry()); |
| 86 | } |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 87 | |
| 88 | struct RegImm { |
| 89 | MCPhysReg Reg; |
| 90 | int32_t Imm; |
| 91 | RegImm(MCPhysReg Reg, int32_t Imm) : Reg(Reg), Imm(Imm) {} |
| 92 | }; |
| 93 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 94 | bool knownRegValInBlock(MachineInstr &CondBr, MachineBasicBlock *MBB, |
| 95 | SmallVectorImpl<RegImm> &KnownRegs, |
| 96 | MachineBasicBlock::iterator &FirstUse); |
| 97 | bool optimizeBlock(MachineBasicBlock *MBB); |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 98 | bool runOnMachineFunction(MachineFunction &MF) override; |
Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 99 | MachineFunctionProperties getRequiredProperties() const override { |
| 100 | return MachineFunctionProperties().set( |
Matthias Braun | 1eb4736 | 2016-08-25 01:27:13 +0000 | [diff] [blame] | 101 | MachineFunctionProperties::Property::NoVRegs); |
Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 102 | } |
Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 103 | StringRef getPassName() const override { |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 104 | return "AArch64 Redundant Copy Elimination"; |
| 105 | } |
| 106 | }; |
| 107 | char AArch64RedundantCopyElimination::ID = 0; |
| 108 | } |
| 109 | |
| 110 | INITIALIZE_PASS(AArch64RedundantCopyElimination, "aarch64-copyelim", |
| 111 | "AArch64 redundant copy elimination pass", false, false) |
| 112 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 113 | /// It's possible to determine the value of a register based on a dominating |
| 114 | /// condition. To do so, this function checks to see if the basic block \p MBB |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 115 | /// is the target of a conditional branch \p CondBr with an equality comparison. |
| 116 | /// If the branch is a CBZ/CBNZ, we know the value of its source operand is zero |
| 117 | /// in \p MBB for some cases. Otherwise, we find and inspect the NZCV setting |
| 118 | /// instruction (e.g., SUBS, ADDS). If this instruction defines a register |
| 119 | /// other than WZR/XZR, we know the value of the destination register is zero in |
| 120 | /// \p MMB for some cases. In addition, if the NZCV setting instruction is |
| 121 | /// comparing against a constant we know the other source register is equal to |
| 122 | /// the constant in \p MBB for some cases. If we find any constant values, push |
| 123 | /// a physical register and constant value pair onto the KnownRegs vector and |
| 124 | /// return true. Otherwise, return false if no known values were found. |
| 125 | bool AArch64RedundantCopyElimination::knownRegValInBlock( |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 126 | MachineInstr &CondBr, MachineBasicBlock *MBB, |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 127 | SmallVectorImpl<RegImm> &KnownRegs, MachineBasicBlock::iterator &FirstUse) { |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 128 | unsigned Opc = CondBr.getOpcode(); |
| 129 | |
| 130 | // Check if the current basic block is the target block to which the |
| 131 | // CBZ/CBNZ instruction jumps when its Wt/Xt is zero. |
| 132 | if (((Opc == AArch64::CBZW || Opc == AArch64::CBZX) && |
| 133 | MBB == CondBr.getOperand(1).getMBB()) || |
| 134 | ((Opc == AArch64::CBNZW || Opc == AArch64::CBNZX) && |
| 135 | MBB != CondBr.getOperand(1).getMBB())) { |
| 136 | FirstUse = CondBr; |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 137 | KnownRegs.push_back(RegImm(CondBr.getOperand(0).getReg(), 0)); |
| 138 | return true; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 139 | } |
| 140 | |
| 141 | // Otherwise, must be a conditional branch. |
| 142 | if (Opc != AArch64::Bcc) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 143 | return false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 144 | |
| 145 | // Must be an equality check (i.e., == or !=). |
| 146 | AArch64CC::CondCode CC = (AArch64CC::CondCode)CondBr.getOperand(0).getImm(); |
| 147 | if (CC != AArch64CC::EQ && CC != AArch64CC::NE) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 148 | return false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 149 | |
| 150 | MachineBasicBlock *BrTarget = CondBr.getOperand(1).getMBB(); |
| 151 | if ((CC == AArch64CC::EQ && BrTarget != MBB) || |
| 152 | (CC == AArch64CC::NE && BrTarget == MBB)) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 153 | return false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 154 | |
| 155 | // Stop if we get to the beginning of PredMBB. |
| 156 | MachineBasicBlock *PredMBB = *MBB->pred_begin(); |
| 157 | assert(PredMBB == CondBr.getParent() && |
| 158 | "Conditional branch not in predecessor block!"); |
| 159 | if (CondBr == PredMBB->begin()) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 160 | return false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 161 | |
| 162 | // Registers clobbered in PredMBB between CondBr instruction and current |
| 163 | // instruction being checked in loop. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 164 | DomBBClobberedRegs.clear(); |
| 165 | DomBBUsedRegs.clear(); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 166 | |
| 167 | // Find compare instruction that sets NZCV used by CondBr. |
| 168 | MachineBasicBlock::reverse_iterator RIt = CondBr.getReverseIterator(); |
| 169 | for (MachineInstr &PredI : make_range(std::next(RIt), PredMBB->rend())) { |
| 170 | |
Chad Rosier | 9a70c7c | 2017-03-06 21:20:00 +0000 | [diff] [blame] | 171 | bool IsCMN = false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 172 | switch (PredI.getOpcode()) { |
| 173 | default: |
| 174 | break; |
| 175 | |
Chad Rosier | 9a70c7c | 2017-03-06 21:20:00 +0000 | [diff] [blame] | 176 | // CMN is an alias for ADDS with a dead destination register. |
| 177 | case AArch64::ADDSWri: |
| 178 | case AArch64::ADDSXri: |
| 179 | IsCMN = true; |
Simon Pilgrim | 8b4dc53 | 2017-07-07 13:03:28 +0000 | [diff] [blame] | 180 | LLVM_FALLTHROUGH; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 181 | // CMP is an alias for SUBS with a dead destination register. |
| 182 | case AArch64::SUBSWri: |
| 183 | case AArch64::SUBSXri: { |
Tim Northover | 350a87e | 2017-10-17 21:43:52 +0000 | [diff] [blame] | 184 | // Sometimes the first operand is a FrameIndex. Bail if tht happens. |
| 185 | if (!PredI.getOperand(1).isReg()) |
| 186 | return false; |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 187 | MCPhysReg DstReg = PredI.getOperand(0).getReg(); |
Chad Rosier | 9a70c7c | 2017-03-06 21:20:00 +0000 | [diff] [blame] | 188 | MCPhysReg SrcReg = PredI.getOperand(1).getReg(); |
| 189 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 190 | bool Res = false; |
| 191 | // If we're comparing against a non-symbolic immediate and the source |
| 192 | // register of the compare is not modified (including a self-clobbering |
| 193 | // compare) between the compare and conditional branch we known the value |
| 194 | // of the 1st source operand. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 195 | if (PredI.getOperand(2).isImm() && DomBBClobberedRegs.available(SrcReg) && |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 196 | SrcReg != DstReg) { |
| 197 | // We've found the instruction that sets NZCV. |
| 198 | int32_t KnownImm = PredI.getOperand(2).getImm(); |
| 199 | int32_t Shift = PredI.getOperand(3).getImm(); |
| 200 | KnownImm <<= Shift; |
| 201 | if (IsCMN) |
| 202 | KnownImm = -KnownImm; |
| 203 | FirstUse = PredI; |
| 204 | KnownRegs.push_back(RegImm(SrcReg, KnownImm)); |
| 205 | Res = true; |
| 206 | } |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 207 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 208 | // If this instructions defines something other than WZR/XZR, we know it's |
| 209 | // result is zero in some cases. |
| 210 | if (DstReg == AArch64::WZR || DstReg == AArch64::XZR) |
| 211 | return Res; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 212 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 213 | // The destination register must not be modified between the NZCV setting |
| 214 | // instruction and the conditional branch. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 215 | if (!DomBBClobberedRegs.available(DstReg)) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 216 | return Res; |
| 217 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 218 | FirstUse = PredI; |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 219 | KnownRegs.push_back(RegImm(DstReg, 0)); |
| 220 | return true; |
| 221 | } |
| 222 | |
| 223 | // Look for NZCV setting instructions that define something other than |
| 224 | // WZR/XZR. |
| 225 | case AArch64::ADCSWr: |
| 226 | case AArch64::ADCSXr: |
| 227 | case AArch64::ADDSWrr: |
| 228 | case AArch64::ADDSWrs: |
| 229 | case AArch64::ADDSWrx: |
| 230 | case AArch64::ADDSXrr: |
| 231 | case AArch64::ADDSXrs: |
| 232 | case AArch64::ADDSXrx: |
| 233 | case AArch64::ADDSXrx64: |
| 234 | case AArch64::ANDSWri: |
| 235 | case AArch64::ANDSWrr: |
| 236 | case AArch64::ANDSWrs: |
| 237 | case AArch64::ANDSXri: |
| 238 | case AArch64::ANDSXrr: |
| 239 | case AArch64::ANDSXrs: |
| 240 | case AArch64::BICSWrr: |
| 241 | case AArch64::BICSWrs: |
| 242 | case AArch64::BICSXrs: |
| 243 | case AArch64::BICSXrr: |
| 244 | case AArch64::SBCSWr: |
| 245 | case AArch64::SBCSXr: |
| 246 | case AArch64::SUBSWrr: |
| 247 | case AArch64::SUBSWrs: |
| 248 | case AArch64::SUBSWrx: |
| 249 | case AArch64::SUBSXrr: |
| 250 | case AArch64::SUBSXrs: |
| 251 | case AArch64::SUBSXrx: |
| 252 | case AArch64::SUBSXrx64: { |
| 253 | MCPhysReg DstReg = PredI.getOperand(0).getReg(); |
| 254 | if (DstReg == AArch64::WZR || DstReg == AArch64::XZR) |
| 255 | return false; |
| 256 | |
| 257 | // The destination register of the NZCV setting instruction must not be |
| 258 | // modified before the conditional branch. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 259 | if (!DomBBClobberedRegs.available(DstReg)) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 260 | return false; |
| 261 | |
| 262 | // We've found the instruction that sets NZCV whose DstReg == 0. |
| 263 | FirstUse = PredI; |
| 264 | KnownRegs.push_back(RegImm(DstReg, 0)); |
| 265 | return true; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 266 | } |
| 267 | } |
| 268 | |
| 269 | // Bail if we see an instruction that defines NZCV that we don't handle. |
| 270 | if (PredI.definesRegister(AArch64::NZCV)) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 271 | return false; |
| 272 | |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 273 | // Track clobbered and used registers. |
| 274 | LiveRegUnits::accumulateUsedDefed(PredI, DomBBClobberedRegs, DomBBUsedRegs, |
| 275 | TRI); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 276 | } |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 277 | return false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 278 | } |
| 279 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 280 | bool AArch64RedundantCopyElimination::optimizeBlock(MachineBasicBlock *MBB) { |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 281 | // Check if the current basic block has a single predecessor. |
| 282 | if (MBB->pred_size() != 1) |
| 283 | return false; |
| 284 | |
Chad Rosier | 072e70b | 2017-01-25 15:56:59 +0000 | [diff] [blame] | 285 | // Check if the predecessor has two successors, implying the block ends in a |
| 286 | // conditional branch. |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 287 | MachineBasicBlock *PredMBB = *MBB->pred_begin(); |
Chad Rosier | 072e70b | 2017-01-25 15:56:59 +0000 | [diff] [blame] | 288 | if (PredMBB->succ_size() != 2) |
| 289 | return false; |
| 290 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 291 | MachineBasicBlock::iterator CondBr = PredMBB->getLastNonDebugInstr(); |
| 292 | if (CondBr == PredMBB->end()) |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 293 | return false; |
| 294 | |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 295 | // Keep track of the earliest point in the PredMBB block where kill markers |
| 296 | // need to be removed if a COPY is removed. |
| 297 | MachineBasicBlock::iterator FirstUse; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 298 | // After calling knownRegValInBlock, FirstUse will either point to a CBZ/CBNZ |
| 299 | // or a compare (i.e., SUBS). In the latter case, we must take care when |
| 300 | // updating FirstUse when scanning for COPY instructions. In particular, if |
| 301 | // there's a COPY in between the compare and branch the COPY should not |
| 302 | // update FirstUse. |
| 303 | bool SeenFirstUse = false; |
| 304 | // Registers that contain a known value at the start of MBB. |
| 305 | SmallVector<RegImm, 4> KnownRegs; |
| 306 | |
| 307 | MachineBasicBlock::iterator Itr = std::next(CondBr); |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 308 | do { |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 309 | --Itr; |
| 310 | |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 311 | if (!knownRegValInBlock(*Itr, MBB, KnownRegs, FirstUse)) |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 312 | continue; |
| 313 | |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 314 | // Reset the clobbered and used register units. |
| 315 | OptBBClobberedRegs.clear(); |
| 316 | OptBBUsedRegs.clear(); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 317 | |
| 318 | // Look backward in PredMBB for COPYs from the known reg to find other |
| 319 | // registers that are known to be a constant value. |
| 320 | for (auto PredI = Itr;; --PredI) { |
| 321 | if (FirstUse == PredI) |
| 322 | SeenFirstUse = true; |
| 323 | |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 324 | if (PredI->isCopy()) { |
| 325 | MCPhysReg CopyDstReg = PredI->getOperand(0).getReg(); |
| 326 | MCPhysReg CopySrcReg = PredI->getOperand(1).getReg(); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 327 | for (auto &KnownReg : KnownRegs) { |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 328 | if (!OptBBClobberedRegs.available(KnownReg.Reg)) |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 329 | continue; |
| 330 | // If we have X = COPY Y, and Y is known to be zero, then now X is |
| 331 | // known to be zero. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 332 | if (CopySrcReg == KnownReg.Reg && |
| 333 | OptBBClobberedRegs.available(CopyDstReg)) { |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 334 | KnownRegs.push_back(RegImm(CopyDstReg, KnownReg.Imm)); |
| 335 | if (SeenFirstUse) |
| 336 | FirstUse = PredI; |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 337 | break; |
| 338 | } |
| 339 | // If we have X = COPY Y, and X is known to be zero, then now Y is |
| 340 | // known to be zero. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 341 | if (CopyDstReg == KnownReg.Reg && |
| 342 | OptBBClobberedRegs.available(CopySrcReg)) { |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 343 | KnownRegs.push_back(RegImm(CopySrcReg, KnownReg.Imm)); |
| 344 | if (SeenFirstUse) |
| 345 | FirstUse = PredI; |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 346 | break; |
| 347 | } |
| 348 | } |
| 349 | } |
| 350 | |
| 351 | // Stop if we get to the beginning of PredMBB. |
| 352 | if (PredI == PredMBB->begin()) |
| 353 | break; |
| 354 | |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 355 | LiveRegUnits::accumulateUsedDefed(*PredI, OptBBClobberedRegs, |
| 356 | OptBBUsedRegs, TRI); |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 357 | // Stop if all of the known-zero regs have been clobbered. |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 358 | if (all_of(KnownRegs, [&](RegImm KnownReg) { |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 359 | return !OptBBClobberedRegs.available(KnownReg.Reg); |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 360 | })) |
| 361 | break; |
| 362 | } |
| 363 | break; |
| 364 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 365 | } while (Itr != PredMBB->begin() && Itr->isTerminator()); |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 366 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 367 | // We've not found a registers with a known value, time to bail out. |
| 368 | if (KnownRegs.empty()) |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 369 | return false; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 370 | |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 371 | bool Changed = false; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 372 | // UsedKnownRegs is the set of KnownRegs that have had uses added to MBB. |
| 373 | SmallSetVector<unsigned, 4> UsedKnownRegs; |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 374 | MachineBasicBlock::iterator LastChange = MBB->begin(); |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 375 | // Remove redundant copy/move instructions unless KnownReg is modified. |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 376 | for (MachineBasicBlock::iterator I = MBB->begin(), E = MBB->end(); I != E;) { |
| 377 | MachineInstr *MI = &*I; |
| 378 | ++I; |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 379 | bool RemovedMI = false; |
| 380 | bool IsCopy = MI->isCopy(); |
| 381 | bool IsMoveImm = MI->isMoveImmediate(); |
| 382 | if (IsCopy || IsMoveImm) { |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 383 | MCPhysReg DefReg = MI->getOperand(0).getReg(); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 384 | MCPhysReg SrcReg = IsCopy ? MI->getOperand(1).getReg() : 0; |
| 385 | int64_t SrcImm = IsMoveImm ? MI->getOperand(1).getImm() : 0; |
| 386 | if (!MRI->isReserved(DefReg) && |
| 387 | ((IsCopy && (SrcReg == AArch64::XZR || SrcReg == AArch64::WZR)) || |
| 388 | IsMoveImm)) { |
| 389 | for (RegImm &KnownReg : KnownRegs) { |
| 390 | if (KnownReg.Reg != DefReg && |
| 391 | !TRI->isSuperRegister(DefReg, KnownReg.Reg)) |
| 392 | continue; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 393 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 394 | // For a copy, the known value must be a zero. |
| 395 | if (IsCopy && KnownReg.Imm != 0) |
| 396 | continue; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 397 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 398 | if (IsMoveImm) { |
| 399 | // For a move immediate, the known immediate must match the source |
| 400 | // immediate. |
| 401 | if (KnownReg.Imm != SrcImm) |
| 402 | continue; |
| 403 | |
| 404 | // Don't remove a move immediate that implicitly defines the upper |
| 405 | // bits when only the lower 32 bits are known. |
| 406 | MCPhysReg CmpReg = KnownReg.Reg; |
| 407 | if (any_of(MI->implicit_operands(), [CmpReg](MachineOperand &O) { |
| 408 | return !O.isDead() && O.isReg() && O.isDef() && |
| 409 | O.getReg() != CmpReg; |
| 410 | })) |
| 411 | continue; |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 412 | } |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 413 | |
| 414 | if (IsCopy) |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 415 | LLVM_DEBUG(dbgs() << "Remove redundant Copy : " << *MI); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 416 | else |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 417 | LLVM_DEBUG(dbgs() << "Remove redundant Move : " << *MI); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 418 | |
| 419 | MI->eraseFromParent(); |
| 420 | Changed = true; |
| 421 | LastChange = I; |
| 422 | NumCopiesRemoved++; |
| 423 | UsedKnownRegs.insert(KnownReg.Reg); |
| 424 | RemovedMI = true; |
| 425 | break; |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 426 | } |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 427 | } |
| 428 | } |
| 429 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 430 | // Skip to the next instruction if we removed the COPY/MovImm. |
| 431 | if (RemovedMI) |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 432 | continue; |
| 433 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 434 | // Remove any regs the MI clobbers from the KnownConstRegs set. |
| 435 | for (unsigned RI = 0; RI < KnownRegs.size();) |
| 436 | if (MI->modifiesRegister(KnownRegs[RI].Reg, TRI)) { |
| 437 | std::swap(KnownRegs[RI], KnownRegs[KnownRegs.size() - 1]); |
| 438 | KnownRegs.pop_back(); |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 439 | // Don't increment RI since we need to now check the swapped-in |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 440 | // KnownRegs[RI]. |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 441 | } else { |
| 442 | ++RI; |
| 443 | } |
| 444 | |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 445 | // Continue until the KnownRegs set is empty. |
| 446 | if (KnownRegs.empty()) |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 447 | break; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 448 | } |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 449 | |
| 450 | if (!Changed) |
| 451 | return false; |
| 452 | |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 453 | // Add newly used regs to the block's live-in list if they aren't there |
| 454 | // already. |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 455 | for (MCPhysReg KnownReg : UsedKnownRegs) |
| 456 | if (!MBB->isLiveIn(KnownReg)) |
| 457 | MBB->addLiveIn(KnownReg); |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 458 | |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 459 | // Clear kills in the range where changes were made. This is conservative, |
| 460 | // but should be okay since kill markers are being phased out. |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 461 | LLVM_DEBUG(dbgs() << "Clearing kill flags.\n\tFirstUse: " << *FirstUse |
| 462 | << "\tLastChange: " << *LastChange); |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 463 | for (MachineInstr &MMI : make_range(FirstUse, PredMBB->end())) |
| 464 | MMI.clearKillInfo(); |
Duncan P. N. Exon Smith | 84c2da4 | 2016-08-18 17:58:09 +0000 | [diff] [blame] | 465 | for (MachineInstr &MMI : make_range(MBB->begin(), LastChange)) |
Geoff Berry | 6bb7915 | 2017-02-22 19:10:45 +0000 | [diff] [blame] | 466 | MMI.clearKillInfo(); |
Tim Northover | 7687bce | 2016-02-17 23:07:04 +0000 | [diff] [blame] | 467 | |
Tim Northover | 3f22856 | 2016-02-17 21:16:53 +0000 | [diff] [blame] | 468 | return true; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 469 | } |
| 470 | |
| 471 | bool AArch64RedundantCopyElimination::runOnMachineFunction( |
| 472 | MachineFunction &MF) { |
Matthias Braun | f1caa28 | 2017-12-15 22:22:58 +0000 | [diff] [blame] | 473 | if (skipFunction(MF.getFunction())) |
Andrew Kaylor | 1ac98bb | 2016-04-25 21:58:52 +0000 | [diff] [blame] | 474 | return false; |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 475 | TRI = MF.getSubtarget().getRegisterInfo(); |
| 476 | MRI = &MF.getRegInfo(); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 477 | |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 478 | // Resize the clobbered and used register unit trackers. We do this once per |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 479 | // function. |
Chad Rosier | 3f66363 | 2018-05-23 17:49:38 +0000 | [diff] [blame] | 480 | DomBBClobberedRegs.init(*TRI); |
| 481 | DomBBUsedRegs.init(*TRI); |
| 482 | OptBBClobberedRegs.init(*TRI); |
| 483 | OptBBUsedRegs.init(*TRI); |
Chad Rosier | ea25eca | 2017-03-02 20:48:11 +0000 | [diff] [blame] | 484 | |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 485 | bool Changed = false; |
| 486 | for (MachineBasicBlock &MBB : MF) |
Chad Rosier | 9b2b4c9 | 2017-07-23 16:38:08 +0000 | [diff] [blame] | 487 | Changed |= optimizeBlock(&MBB); |
Jun Bum Lim | b389d9b | 2016-02-16 20:02:39 +0000 | [diff] [blame] | 488 | return Changed; |
| 489 | } |
| 490 | |
| 491 | FunctionPass *llvm::createAArch64RedundantCopyEliminationPass() { |
| 492 | return new AArch64RedundantCopyElimination(); |
| 493 | } |