| 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 | } |