| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 1 | //===- X86OptimizeLEAs.cpp - optimize usage of LEA instructions -----------===// |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 2 | // |
| Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | // |
| 9 | // This file defines the pass that performs some optimizations with LEA |
| Andrey Turetskiy | 45b22a4 | 2016-05-19 10:18:29 +0000 | [diff] [blame] | 10 | // instructions in order to improve performance and code size. |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 11 | // Currently, it does two things: |
| 12 | // 1) If there are two LEA instructions calculating addresses which only differ |
| 13 | // by displacement inside a basic block, one of them is removed. |
| 14 | // 2) Address calculations in load and store instructions are replaced by |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 15 | // existing LEA def registers where possible. |
| 16 | // |
| 17 | //===----------------------------------------------------------------------===// |
| 18 | |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 19 | #include "MCTargetDesc/X86BaseInfo.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 20 | #include "X86.h" |
| 21 | #include "X86InstrInfo.h" |
| 22 | #include "X86Subtarget.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 23 | #include "llvm/ADT/DenseMap.h" |
| 24 | #include "llvm/ADT/DenseMapInfo.h" |
| 25 | #include "llvm/ADT/Hashing.h" |
| 26 | #include "llvm/ADT/SmallVector.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 27 | #include "llvm/ADT/Statistic.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 28 | #include "llvm/CodeGen/MachineBasicBlock.h" |
| 29 | #include "llvm/CodeGen/MachineFunction.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 30 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 31 | #include "llvm/CodeGen/MachineInstr.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 32 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 33 | #include "llvm/CodeGen/MachineOperand.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 34 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
| David Blaikie | b3bde2e | 2017-11-17 01:07:10 +0000 | [diff] [blame] | 35 | #include "llvm/CodeGen/TargetOpcodes.h" |
| 36 | #include "llvm/CodeGen/TargetRegisterInfo.h" |
| Chandler Carruth | 6bda14b | 2017-06-06 11:49:48 +0000 | [diff] [blame] | 37 | #include "llvm/IR/DebugInfoMetadata.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 38 | #include "llvm/IR/DebugLoc.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 39 | #include "llvm/IR/Function.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 40 | #include "llvm/MC/MCInstrDesc.h" |
| 41 | #include "llvm/Support/CommandLine.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 42 | #include "llvm/Support/Debug.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 43 | #include "llvm/Support/ErrorHandling.h" |
| 44 | #include "llvm/Support/MathExtras.h" |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 45 | #include "llvm/Support/raw_ostream.h" |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 46 | #include <cassert> |
| 47 | #include <cstdint> |
| 48 | #include <iterator> |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 49 | |
| 50 | using namespace llvm; |
| 51 | |
| 52 | #define DEBUG_TYPE "x86-optimize-LEAs" |
| 53 | |
| Andrey Turetskiy | 9994b88 | 2016-02-20 11:11:55 +0000 | [diff] [blame] | 54 | static cl::opt<bool> |
| 55 | DisableX86LEAOpt("disable-x86-lea-opt", cl::Hidden, |
| 56 | cl::desc("X86: Disable LEA optimizations."), |
| 57 | cl::init(false)); |
| Alexey Bataev | 7b72b65 | 2015-12-17 07:34:39 +0000 | [diff] [blame] | 58 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 59 | STATISTIC(NumSubstLEAs, "Number of LEA instruction substitutions"); |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 60 | STATISTIC(NumRedundantLEAs, "Number of redundant LEA instructions removed"); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 61 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 62 | /// Returns true if two machine operands are identical and they are not |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 63 | /// physical registers. |
| 64 | static inline bool isIdenticalOp(const MachineOperand &MO1, |
| 65 | const MachineOperand &MO2); |
| 66 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 67 | /// Returns true if two address displacement operands are of the same |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 68 | /// type and use the same symbol/index/address regardless of the offset. |
| 69 | static bool isSimilarDispOp(const MachineOperand &MO1, |
| 70 | const MachineOperand &MO2); |
| 71 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 72 | /// Returns true if the instruction is LEA. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 73 | static inline bool isLEA(const MachineInstr &MI); |
| 74 | |
| Benjamin Kramer | b7d3311 | 2016-08-06 11:13:10 +0000 | [diff] [blame] | 75 | namespace { |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 76 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 77 | /// A key based on instruction's memory operands. |
| 78 | class MemOpKey { |
| 79 | public: |
| 80 | MemOpKey(const MachineOperand *Base, const MachineOperand *Scale, |
| 81 | const MachineOperand *Index, const MachineOperand *Segment, |
| Matt Morehouse | 9e658c9 | 2017-12-01 22:20:26 +0000 | [diff] [blame] | 82 | const MachineOperand *Disp) |
| 83 | : Disp(Disp) { |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 84 | Operands[0] = Base; |
| 85 | Operands[1] = Scale; |
| 86 | Operands[2] = Index; |
| 87 | Operands[3] = Segment; |
| 88 | } |
| 89 | |
| 90 | bool operator==(const MemOpKey &Other) const { |
| 91 | // Addresses' bases, scales, indices and segments must be identical. |
| 92 | for (int i = 0; i < 4; ++i) |
| 93 | if (!isIdenticalOp(*Operands[i], *Other.Operands[i])) |
| 94 | return false; |
| 95 | |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 96 | // Addresses' displacements don't have to be exactly the same. It only |
| 97 | // matters that they use the same symbol/index/address. Immediates' or |
| 98 | // offsets' differences will be taken care of during instruction |
| 99 | // substitution. |
| 100 | return isSimilarDispOp(*Disp, *Other.Disp); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 101 | } |
| 102 | |
| 103 | // Address' base, scale, index and segment operands. |
| 104 | const MachineOperand *Operands[4]; |
| 105 | |
| 106 | // Address' displacement operand. |
| 107 | const MachineOperand *Disp; |
| 108 | }; |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 109 | |
| Benjamin Kramer | b7d3311 | 2016-08-06 11:13:10 +0000 | [diff] [blame] | 110 | } // end anonymous namespace |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 111 | |
| 112 | /// Provide DenseMapInfo for MemOpKey. |
| 113 | namespace llvm { |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 114 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 115 | template <> struct DenseMapInfo<MemOpKey> { |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 116 | using PtrInfo = DenseMapInfo<const MachineOperand *>; |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 117 | |
| 118 | static inline MemOpKey getEmptyKey() { |
| 119 | return MemOpKey(PtrInfo::getEmptyKey(), PtrInfo::getEmptyKey(), |
| 120 | PtrInfo::getEmptyKey(), PtrInfo::getEmptyKey(), |
| 121 | PtrInfo::getEmptyKey()); |
| 122 | } |
| 123 | |
| 124 | static inline MemOpKey getTombstoneKey() { |
| 125 | return MemOpKey(PtrInfo::getTombstoneKey(), PtrInfo::getTombstoneKey(), |
| 126 | PtrInfo::getTombstoneKey(), PtrInfo::getTombstoneKey(), |
| 127 | PtrInfo::getTombstoneKey()); |
| 128 | } |
| 129 | |
| 130 | static unsigned getHashValue(const MemOpKey &Val) { |
| 131 | // Checking any field of MemOpKey is enough to determine if the key is |
| 132 | // empty or tombstone. |
| 133 | assert(Val.Disp != PtrInfo::getEmptyKey() && "Cannot hash the empty key"); |
| 134 | assert(Val.Disp != PtrInfo::getTombstoneKey() && |
| 135 | "Cannot hash the tombstone key"); |
| 136 | |
| Matt Morehouse | 9e658c9 | 2017-12-01 22:20:26 +0000 | [diff] [blame] | 137 | hash_code Hash = hash_combine(*Val.Operands[0], *Val.Operands[1], |
| 138 | *Val.Operands[2], *Val.Operands[3]); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 139 | |
| 140 | // If the address displacement is an immediate, it should not affect the |
| 141 | // hash so that memory operands which differ only be immediate displacement |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 142 | // would have the same hash. If the address displacement is something else, |
| 143 | // we should reflect symbol/index/address in the hash. |
| 144 | switch (Val.Disp->getType()) { |
| 145 | case MachineOperand::MO_Immediate: |
| 146 | break; |
| 147 | case MachineOperand::MO_ConstantPoolIndex: |
| 148 | case MachineOperand::MO_JumpTableIndex: |
| 149 | Hash = hash_combine(Hash, Val.Disp->getIndex()); |
| 150 | break; |
| 151 | case MachineOperand::MO_ExternalSymbol: |
| 152 | Hash = hash_combine(Hash, Val.Disp->getSymbolName()); |
| 153 | break; |
| 154 | case MachineOperand::MO_GlobalAddress: |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 155 | Hash = hash_combine(Hash, Val.Disp->getGlobal()); |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 156 | break; |
| 157 | case MachineOperand::MO_BlockAddress: |
| 158 | Hash = hash_combine(Hash, Val.Disp->getBlockAddress()); |
| 159 | break; |
| 160 | case MachineOperand::MO_MCSymbol: |
| 161 | Hash = hash_combine(Hash, Val.Disp->getMCSymbol()); |
| 162 | break; |
| Andrey Turetskiy | b405606 | 2016-04-26 12:18:12 +0000 | [diff] [blame] | 163 | case MachineOperand::MO_MachineBasicBlock: |
| 164 | Hash = hash_combine(Hash, Val.Disp->getMBB()); |
| 165 | break; |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 166 | default: |
| 167 | llvm_unreachable("Invalid address displacement operand"); |
| 168 | } |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 169 | |
| 170 | return (unsigned)Hash; |
| 171 | } |
| 172 | |
| 173 | static bool isEqual(const MemOpKey &LHS, const MemOpKey &RHS) { |
| 174 | // Checking any field of MemOpKey is enough to determine if the key is |
| 175 | // empty or tombstone. |
| 176 | if (RHS.Disp == PtrInfo::getEmptyKey()) |
| 177 | return LHS.Disp == PtrInfo::getEmptyKey(); |
| 178 | if (RHS.Disp == PtrInfo::getTombstoneKey()) |
| 179 | return LHS.Disp == PtrInfo::getTombstoneKey(); |
| 180 | return LHS == RHS; |
| 181 | } |
| 182 | }; |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 183 | |
| 184 | } // end namespace llvm |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 185 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 186 | /// Returns a hash table key based on memory operands of \p MI. The |
| Benjamin Kramer | b7d3311 | 2016-08-06 11:13:10 +0000 | [diff] [blame] | 187 | /// number of the first memory operand of \p MI is specified through \p N. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 188 | static inline MemOpKey getMemOpKey(const MachineInstr &MI, unsigned N) { |
| 189 | assert((isLEA(MI) || MI.mayLoadOrStore()) && |
| 190 | "The instruction must be a LEA, a load or a store"); |
| 191 | return MemOpKey(&MI.getOperand(N + X86::AddrBaseReg), |
| 192 | &MI.getOperand(N + X86::AddrScaleAmt), |
| 193 | &MI.getOperand(N + X86::AddrIndexReg), |
| 194 | &MI.getOperand(N + X86::AddrSegmentReg), |
| 195 | &MI.getOperand(N + X86::AddrDisp)); |
| 196 | } |
| 197 | |
| 198 | static inline bool isIdenticalOp(const MachineOperand &MO1, |
| 199 | const MachineOperand &MO2) { |
| 200 | return MO1.isIdenticalTo(MO2) && |
| Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 201 | (!MO1.isReg() || !Register::isPhysicalRegister(MO1.getReg())); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 202 | } |
| 203 | |
| Justin Bogner | 38e5217 | 2016-02-24 07:58:02 +0000 | [diff] [blame] | 204 | #ifndef NDEBUG |
| 205 | static bool isValidDispOp(const MachineOperand &MO) { |
| 206 | return MO.isImm() || MO.isCPI() || MO.isJTI() || MO.isSymbol() || |
| Andrey Turetskiy | b405606 | 2016-04-26 12:18:12 +0000 | [diff] [blame] | 207 | MO.isGlobal() || MO.isBlockAddress() || MO.isMCSymbol() || MO.isMBB(); |
| Justin Bogner | 38e5217 | 2016-02-24 07:58:02 +0000 | [diff] [blame] | 208 | } |
| 209 | #endif |
| 210 | |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 211 | static bool isSimilarDispOp(const MachineOperand &MO1, |
| 212 | const MachineOperand &MO2) { |
| 213 | assert(isValidDispOp(MO1) && isValidDispOp(MO2) && |
| 214 | "Address displacement operand is not valid"); |
| 215 | return (MO1.isImm() && MO2.isImm()) || |
| 216 | (MO1.isCPI() && MO2.isCPI() && MO1.getIndex() == MO2.getIndex()) || |
| 217 | (MO1.isJTI() && MO2.isJTI() && MO1.getIndex() == MO2.getIndex()) || |
| 218 | (MO1.isSymbol() && MO2.isSymbol() && |
| 219 | MO1.getSymbolName() == MO2.getSymbolName()) || |
| 220 | (MO1.isGlobal() && MO2.isGlobal() && |
| 221 | MO1.getGlobal() == MO2.getGlobal()) || |
| 222 | (MO1.isBlockAddress() && MO2.isBlockAddress() && |
| 223 | MO1.getBlockAddress() == MO2.getBlockAddress()) || |
| 224 | (MO1.isMCSymbol() && MO2.isMCSymbol() && |
| Andrey Turetskiy | b405606 | 2016-04-26 12:18:12 +0000 | [diff] [blame] | 225 | MO1.getMCSymbol() == MO2.getMCSymbol()) || |
| 226 | (MO1.isMBB() && MO2.isMBB() && MO1.getMBB() == MO2.getMBB()); |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 227 | } |
| 228 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 229 | static inline bool isLEA(const MachineInstr &MI) { |
| 230 | unsigned Opcode = MI.getOpcode(); |
| 231 | return Opcode == X86::LEA16r || Opcode == X86::LEA32r || |
| 232 | Opcode == X86::LEA64r || Opcode == X86::LEA64_32r; |
| 233 | } |
| 234 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 235 | namespace { |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 236 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 237 | class X86OptimizeLEAPass : public MachineFunctionPass { |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 238 | public: |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 239 | X86OptimizeLEAPass() : MachineFunctionPass(ID) {} |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 240 | |
| Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 241 | StringRef getPassName() const override { return "X86 LEA Optimize"; } |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 242 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 243 | /// Loop over all of the basic blocks, replacing address |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 244 | /// calculations in load and store instructions, if it's already |
| 245 | /// been calculated by LEA. Also, remove redundant LEAs. |
| 246 | bool runOnMachineFunction(MachineFunction &MF) override; |
| 247 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 248 | static char ID; |
| 249 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 250 | private: |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 251 | using MemOpMap = DenseMap<MemOpKey, SmallVector<MachineInstr *, 16>>; |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 252 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 253 | /// Returns a distance between two instructions inside one basic block. |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 254 | /// Negative result means, that instructions occur in reverse order. |
| 255 | int calcInstrDist(const MachineInstr &First, const MachineInstr &Last); |
| 256 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 257 | /// Choose the best \p LEA instruction from the \p List to replace |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 258 | /// address calculation in \p MI instruction. Return the address displacement |
| Simon Pilgrim | 9d15fb3 | 2016-11-17 19:03:05 +0000 | [diff] [blame] | 259 | /// and the distance between \p MI and the chosen \p BestLEA in |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 260 | /// \p AddrDispShift and \p Dist. |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 261 | bool chooseBestLEA(const SmallVectorImpl<MachineInstr *> &List, |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 262 | const MachineInstr &MI, MachineInstr *&BestLEA, |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 263 | int64_t &AddrDispShift, int &Dist); |
| 264 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 265 | /// Returns the difference between addresses' displacements of \p MI1 |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 266 | /// and \p MI2. The numbers of the first memory operands for the instructions |
| 267 | /// are specified through \p N1 and \p N2. |
| 268 | int64_t getAddrDispShift(const MachineInstr &MI1, unsigned N1, |
| 269 | const MachineInstr &MI2, unsigned N2) const; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 270 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 271 | /// Returns true if the \p Last LEA instruction can be replaced by the |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 272 | /// \p First. The difference between displacements of the addresses calculated |
| 273 | /// by these LEAs is returned in \p AddrDispShift. It'll be used for proper |
| 274 | /// replacement of the \p Last LEA's uses with the \p First's def register. |
| 275 | bool isReplaceable(const MachineInstr &First, const MachineInstr &Last, |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 276 | int64_t &AddrDispShift) const; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 277 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 278 | /// Find all LEA instructions in the basic block. Also, assign position |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 279 | /// numbers to all instructions in the basic block to speed up calculation of |
| 280 | /// distance between them. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 281 | void findLEAs(const MachineBasicBlock &MBB, MemOpMap &LEAs); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 282 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 283 | /// Removes redundant address calculations. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 284 | bool removeRedundantAddrCalc(MemOpMap &LEAs); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 285 | |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 286 | /// Replace debug value MI with a new debug value instruction using register |
| 287 | /// VReg with an appropriate offset and DIExpression to incorporate the |
| 288 | /// address displacement AddrDispShift. Return new debug value instruction. |
| 289 | MachineInstr *replaceDebugValue(MachineInstr &MI, unsigned VReg, |
| 290 | int64_t AddrDispShift); |
| 291 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 292 | /// Removes LEAs which calculate similar addresses. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 293 | bool removeRedundantLEAs(MemOpMap &LEAs); |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 294 | |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 295 | DenseMap<const MachineInstr *, unsigned> InstrPos; |
| 296 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 297 | MachineRegisterInfo *MRI; |
| 298 | const X86InstrInfo *TII; |
| 299 | const X86RegisterInfo *TRI; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 300 | }; |
| Eugene Zelenko | 60433b6 | 2017-10-05 00:33:50 +0000 | [diff] [blame] | 301 | |
| 302 | } // end anonymous namespace |
| 303 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 304 | char X86OptimizeLEAPass::ID = 0; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 305 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 306 | FunctionPass *llvm::createX86OptimizeLEAs() { return new X86OptimizeLEAPass(); } |
| 307 | INITIALIZE_PASS(X86OptimizeLEAPass, DEBUG_TYPE, "X86 optimize LEA pass", false, |
| 308 | false) |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 309 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 310 | int X86OptimizeLEAPass::calcInstrDist(const MachineInstr &First, |
| 311 | const MachineInstr &Last) { |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 312 | // Both instructions must be in the same basic block and they must be |
| 313 | // presented in InstrPos. |
| 314 | assert(Last.getParent() == First.getParent() && |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 315 | "Instructions are in different basic blocks"); |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 316 | assert(InstrPos.find(&First) != InstrPos.end() && |
| 317 | InstrPos.find(&Last) != InstrPos.end() && |
| 318 | "Instructions' positions are undefined"); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 319 | |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 320 | return InstrPos[&Last] - InstrPos[&First]; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 321 | } |
| 322 | |
| 323 | // Find the best LEA instruction in the List to replace address recalculation in |
| 324 | // MI. Such LEA must meet these requirements: |
| 325 | // 1) The address calculated by the LEA differs only by the displacement from |
| 326 | // the address used in MI. |
| 327 | // 2) The register class of the definition of the LEA is compatible with the |
| 328 | // register class of the address base register of MI. |
| 329 | // 3) Displacement of the new memory operand should fit in 1 byte if possible. |
| 330 | // 4) The LEA should be as close to MI as possible, and prior to it if |
| 331 | // possible. |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 332 | bool X86OptimizeLEAPass::chooseBestLEA( |
| 333 | const SmallVectorImpl<MachineInstr *> &List, const MachineInstr &MI, |
| 334 | MachineInstr *&BestLEA, int64_t &AddrDispShift, int &Dist) { |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 335 | const MachineFunction *MF = MI.getParent()->getParent(); |
| 336 | const MCInstrDesc &Desc = MI.getDesc(); |
| Craig Topper | 477649a | 2016-04-28 05:58:46 +0000 | [diff] [blame] | 337 | int MemOpNo = X86II::getMemoryOperandNo(Desc.TSFlags) + |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 338 | X86II::getOperandBias(Desc); |
| 339 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 340 | BestLEA = nullptr; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 341 | |
| 342 | // Loop over all LEA instructions. |
| 343 | for (auto DefMI : List) { |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 344 | // Get new address displacement. |
| 345 | int64_t AddrDispShiftTemp = getAddrDispShift(MI, MemOpNo, *DefMI, 1); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 346 | |
| 347 | // Make sure address displacement fits 4 bytes. |
| 348 | if (!isInt<32>(AddrDispShiftTemp)) |
| 349 | continue; |
| 350 | |
| 351 | // Check that LEA def register can be used as MI address base. Some |
| 352 | // instructions can use a limited set of registers as address base, for |
| 353 | // example MOV8mr_NOREX. We could constrain the register class of the LEA |
| 354 | // def to suit MI, however since this case is very rare and hard to |
| 355 | // reproduce in a test it's just more reliable to skip the LEA. |
| 356 | if (TII->getRegClass(Desc, MemOpNo + X86::AddrBaseReg, TRI, *MF) != |
| 357 | MRI->getRegClass(DefMI->getOperand(0).getReg())) |
| 358 | continue; |
| 359 | |
| 360 | // Choose the closest LEA instruction from the list, prior to MI if |
| 361 | // possible. Note that we took into account resulting address displacement |
| 362 | // as well. Also note that the list is sorted by the order in which the LEAs |
| 363 | // occur, so the break condition is pretty simple. |
| 364 | int DistTemp = calcInstrDist(*DefMI, MI); |
| 365 | assert(DistTemp != 0 && |
| 366 | "The distance between two different instructions cannot be zero"); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 367 | if (DistTemp > 0 || BestLEA == nullptr) { |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 368 | // Do not update return LEA, if the current one provides a displacement |
| 369 | // which fits in 1 byte, while the new candidate does not. |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 370 | if (BestLEA != nullptr && !isInt<8>(AddrDispShiftTemp) && |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 371 | isInt<8>(AddrDispShift)) |
| 372 | continue; |
| 373 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 374 | BestLEA = DefMI; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 375 | AddrDispShift = AddrDispShiftTemp; |
| 376 | Dist = DistTemp; |
| 377 | } |
| 378 | |
| 379 | // FIXME: Maybe we should not always stop at the first LEA after MI. |
| 380 | if (DistTemp < 0) |
| 381 | break; |
| 382 | } |
| 383 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 384 | return BestLEA != nullptr; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 385 | } |
| 386 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 387 | // Get the difference between the addresses' displacements of the two |
| 388 | // instructions \p MI1 and \p MI2. The numbers of the first memory operands are |
| 389 | // passed through \p N1 and \p N2. |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 390 | int64_t X86OptimizeLEAPass::getAddrDispShift(const MachineInstr &MI1, |
| 391 | unsigned N1, |
| 392 | const MachineInstr &MI2, |
| 393 | unsigned N2) const { |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 394 | const MachineOperand &Op1 = MI1.getOperand(N1 + X86::AddrDisp); |
| 395 | const MachineOperand &Op2 = MI2.getOperand(N2 + X86::AddrDisp); |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 396 | |
| 397 | assert(isSimilarDispOp(Op1, Op2) && |
| 398 | "Address displacement operands are not compatible"); |
| 399 | |
| 400 | // After the assert above we can be sure that both operands are of the same |
| 401 | // valid type and use the same symbol/index/address, thus displacement shift |
| 402 | // calculation is rather simple. |
| 403 | if (Op1.isJTI()) |
| 404 | return 0; |
| 405 | return Op1.isImm() ? Op1.getImm() - Op2.getImm() |
| 406 | : Op1.getOffset() - Op2.getOffset(); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 407 | } |
| 408 | |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 409 | // Check that the Last LEA can be replaced by the First LEA. To be so, |
| 410 | // these requirements must be met: |
| 411 | // 1) Addresses calculated by LEAs differ only by displacement. |
| 412 | // 2) Def registers of LEAs belong to the same class. |
| 413 | // 3) All uses of the Last LEA def register are replaceable, thus the |
| 414 | // register is used only as address base. |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 415 | bool X86OptimizeLEAPass::isReplaceable(const MachineInstr &First, |
| 416 | const MachineInstr &Last, |
| 417 | int64_t &AddrDispShift) const { |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 418 | assert(isLEA(First) && isLEA(Last) && |
| 419 | "The function works only with LEA instructions"); |
| 420 | |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 421 | // Make sure that LEA def registers belong to the same class. There may be |
| 422 | // instructions (like MOV8mr_NOREX) which allow a limited set of registers to |
| 423 | // be used as their operands, so we must be sure that replacing one LEA |
| 424 | // with another won't lead to putting a wrong register in the instruction. |
| 425 | if (MRI->getRegClass(First.getOperand(0).getReg()) != |
| 426 | MRI->getRegClass(Last.getOperand(0).getReg())) |
| 427 | return false; |
| 428 | |
| Andrea Di Biagio | 7937be7 | 2017-03-21 11:36:21 +0000 | [diff] [blame] | 429 | // Get new address displacement. |
| 430 | AddrDispShift = getAddrDispShift(Last, 1, First, 1); |
| 431 | |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 432 | // Loop over all uses of the Last LEA to check that its def register is |
| 433 | // used only as address base for memory accesses. If so, it can be |
| 434 | // replaced, otherwise - no. |
| Andrea Di Biagio | 7937be7 | 2017-03-21 11:36:21 +0000 | [diff] [blame] | 435 | for (auto &MO : MRI->use_nodbg_operands(Last.getOperand(0).getReg())) { |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 436 | MachineInstr &MI = *MO.getParent(); |
| 437 | |
| 438 | // Get the number of the first memory operand. |
| 439 | const MCInstrDesc &Desc = MI.getDesc(); |
| Craig Topper | 477649a | 2016-04-28 05:58:46 +0000 | [diff] [blame] | 440 | int MemOpNo = X86II::getMemoryOperandNo(Desc.TSFlags); |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 441 | |
| 442 | // If the use instruction has no memory operand - the LEA is not |
| 443 | // replaceable. |
| 444 | if (MemOpNo < 0) |
| 445 | return false; |
| 446 | |
| 447 | MemOpNo += X86II::getOperandBias(Desc); |
| 448 | |
| 449 | // If the address base of the use instruction is not the LEA def register - |
| 450 | // the LEA is not replaceable. |
| 451 | if (!isIdenticalOp(MI.getOperand(MemOpNo + X86::AddrBaseReg), MO)) |
| 452 | return false; |
| 453 | |
| 454 | // If the LEA def register is used as any other operand of the use |
| 455 | // instruction - the LEA is not replaceable. |
| 456 | for (unsigned i = 0; i < MI.getNumOperands(); i++) |
| 457 | if (i != (unsigned)(MemOpNo + X86::AddrBaseReg) && |
| 458 | isIdenticalOp(MI.getOperand(i), MO)) |
| 459 | return false; |
| 460 | |
| 461 | // Check that the new address displacement will fit 4 bytes. |
| 462 | if (MI.getOperand(MemOpNo + X86::AddrDisp).isImm() && |
| 463 | !isInt<32>(MI.getOperand(MemOpNo + X86::AddrDisp).getImm() + |
| 464 | AddrDispShift)) |
| 465 | return false; |
| 466 | } |
| 467 | |
| 468 | return true; |
| 469 | } |
| 470 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 471 | void X86OptimizeLEAPass::findLEAs(const MachineBasicBlock &MBB, |
| 472 | MemOpMap &LEAs) { |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 473 | unsigned Pos = 0; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 474 | for (auto &MI : MBB) { |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 475 | // Assign the position number to the instruction. Note that we are going to |
| 476 | // move some instructions during the optimization however there will never |
| 477 | // be a need to move two instructions before any selected instruction. So to |
| 478 | // avoid multiple positions' updates during moves we just increase position |
| 479 | // counter by two leaving a free space for instructions which will be moved. |
| 480 | InstrPos[&MI] = Pos += 2; |
| 481 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 482 | if (isLEA(MI)) |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 483 | LEAs[getMemOpKey(MI, 1)].push_back(const_cast<MachineInstr *>(&MI)); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 484 | } |
| 485 | } |
| 486 | |
| 487 | // Try to find load and store instructions which recalculate addresses already |
| 488 | // calculated by some LEA and replace their memory operands with its def |
| 489 | // register. |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 490 | bool X86OptimizeLEAPass::removeRedundantAddrCalc(MemOpMap &LEAs) { |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 491 | bool Changed = false; |
| 492 | |
| Matt Morehouse | 9e658c9 | 2017-12-01 22:20:26 +0000 | [diff] [blame] | 493 | assert(!LEAs.empty()); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 494 | MachineBasicBlock *MBB = (*LEAs.begin()->second.begin())->getParent(); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 495 | |
| 496 | // Process all instructions in basic block. |
| 497 | for (auto I = MBB->begin(), E = MBB->end(); I != E;) { |
| 498 | MachineInstr &MI = *I++; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 499 | |
| 500 | // Instruction must be load or store. |
| 501 | if (!MI.mayLoadOrStore()) |
| 502 | continue; |
| 503 | |
| 504 | // Get the number of the first memory operand. |
| 505 | const MCInstrDesc &Desc = MI.getDesc(); |
| Craig Topper | 477649a | 2016-04-28 05:58:46 +0000 | [diff] [blame] | 506 | int MemOpNo = X86II::getMemoryOperandNo(Desc.TSFlags); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 507 | |
| 508 | // If instruction has no memory operand - skip it. |
| 509 | if (MemOpNo < 0) |
| 510 | continue; |
| 511 | |
| 512 | MemOpNo += X86II::getOperandBias(Desc); |
| 513 | |
| Craig Topper | 87f78cf | 2018-08-22 18:24:13 +0000 | [diff] [blame] | 514 | // Do not call chooseBestLEA if there was no matching LEA |
| 515 | auto Insns = LEAs.find(getMemOpKey(MI, MemOpNo)); |
| 516 | if (Insns == LEAs.end()) |
| 517 | continue; |
| 518 | |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 519 | // Get the best LEA instruction to replace address calculation. |
| 520 | MachineInstr *DefMI; |
| 521 | int64_t AddrDispShift; |
| 522 | int Dist; |
| Craig Topper | 87f78cf | 2018-08-22 18:24:13 +0000 | [diff] [blame] | 523 | if (!chooseBestLEA(Insns->second, MI, DefMI, AddrDispShift, Dist)) |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 524 | continue; |
| 525 | |
| 526 | // If LEA occurs before current instruction, we can freely replace |
| 527 | // the instruction. If LEA occurs after, we can lift LEA above the |
| 528 | // instruction and this way to be able to replace it. Since LEA and the |
| 529 | // instruction have similar memory operands (thus, the same def |
| 530 | // instructions for these operands), we can always do that, without |
| 531 | // worries of using registers before their defs. |
| 532 | if (Dist < 0) { |
| 533 | DefMI->removeFromParent(); |
| 534 | MBB->insert(MachineBasicBlock::iterator(&MI), DefMI); |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 535 | InstrPos[DefMI] = InstrPos[&MI] - 1; |
| 536 | |
| 537 | // Make sure the instructions' position numbers are sane. |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 538 | assert(((InstrPos[DefMI] == 1 && |
| 539 | MachineBasicBlock::iterator(DefMI) == MBB->begin()) || |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 540 | InstrPos[DefMI] > |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 541 | InstrPos[&*std::prev(MachineBasicBlock::iterator(DefMI))]) && |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 542 | "Instruction positioning is broken"); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 543 | } |
| 544 | |
| 545 | // Since we can possibly extend register lifetime, clear kill flags. |
| 546 | MRI->clearKillFlags(DefMI->getOperand(0).getReg()); |
| 547 | |
| 548 | ++NumSubstLEAs; |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 549 | LLVM_DEBUG(dbgs() << "OptimizeLEAs: Candidate to replace: "; MI.dump();); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 550 | |
| 551 | // Change instruction operands. |
| 552 | MI.getOperand(MemOpNo + X86::AddrBaseReg) |
| 553 | .ChangeToRegister(DefMI->getOperand(0).getReg(), false); |
| 554 | MI.getOperand(MemOpNo + X86::AddrScaleAmt).ChangeToImmediate(1); |
| 555 | MI.getOperand(MemOpNo + X86::AddrIndexReg) |
| 556 | .ChangeToRegister(X86::NoRegister, false); |
| 557 | MI.getOperand(MemOpNo + X86::AddrDisp).ChangeToImmediate(AddrDispShift); |
| 558 | MI.getOperand(MemOpNo + X86::AddrSegmentReg) |
| 559 | .ChangeToRegister(X86::NoRegister, false); |
| 560 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 561 | LLVM_DEBUG(dbgs() << "OptimizeLEAs: Replaced by: "; MI.dump();); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 562 | |
| 563 | Changed = true; |
| 564 | } |
| 565 | |
| 566 | return Changed; |
| 567 | } |
| 568 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 569 | MachineInstr *X86OptimizeLEAPass::replaceDebugValue(MachineInstr &MI, |
| 570 | unsigned VReg, |
| 571 | int64_t AddrDispShift) { |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 572 | DIExpression *Expr = const_cast<DIExpression *>(MI.getDebugExpression()); |
| Adrian Prantl | 109b236 | 2017-04-28 17:51:05 +0000 | [diff] [blame] | 573 | if (AddrDispShift != 0) |
| Petar Jovanovic | e85bbf5 | 2019-05-20 10:35:57 +0000 | [diff] [blame] | 574 | Expr = DIExpression::prepend(Expr, DIExpression::StackValue, AddrDispShift); |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 575 | |
| 576 | // Replace DBG_VALUE instruction with modified version. |
| 577 | MachineBasicBlock *MBB = MI.getParent(); |
| 578 | DebugLoc DL = MI.getDebugLoc(); |
| 579 | bool IsIndirect = MI.isIndirectDebugValue(); |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 580 | const MDNode *Var = MI.getDebugVariable(); |
| Adrian Prantl | 8b9bb53 | 2017-07-28 23:00:45 +0000 | [diff] [blame] | 581 | if (IsIndirect) |
| 582 | assert(MI.getOperand(1).getImm() == 0 && "DBG_VALUE with nonzero offset"); |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 583 | return BuildMI(*MBB, MBB->erase(&MI), DL, TII->get(TargetOpcode::DBG_VALUE), |
| Adrian Prantl | 8b9bb53 | 2017-07-28 23:00:45 +0000 | [diff] [blame] | 584 | IsIndirect, VReg, Var, Expr); |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 585 | } |
| 586 | |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 587 | // Try to find similar LEAs in the list and replace one with another. |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 588 | bool X86OptimizeLEAPass::removeRedundantLEAs(MemOpMap &LEAs) { |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 589 | bool Changed = false; |
| 590 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 591 | // Loop over all entries in the table. |
| 592 | for (auto &E : LEAs) { |
| 593 | auto &List = E.second; |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 594 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 595 | // Loop over all LEA pairs. |
| 596 | auto I1 = List.begin(); |
| 597 | while (I1 != List.end()) { |
| 598 | MachineInstr &First = **I1; |
| 599 | auto I2 = std::next(I1); |
| 600 | while (I2 != List.end()) { |
| 601 | MachineInstr &Last = **I2; |
| 602 | int64_t AddrDispShift; |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 603 | |
| Simon Pilgrim | 9d15fb3 | 2016-11-17 19:03:05 +0000 | [diff] [blame] | 604 | // LEAs should be in occurrence order in the list, so we can freely |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 605 | // replace later LEAs with earlier ones. |
| 606 | assert(calcInstrDist(First, Last) > 0 && |
| Simon Pilgrim | 9d15fb3 | 2016-11-17 19:03:05 +0000 | [diff] [blame] | 607 | "LEAs must be in occurrence order in the list"); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 608 | |
| 609 | // Check that the Last LEA instruction can be replaced by the First. |
| 610 | if (!isReplaceable(First, Last, AddrDispShift)) { |
| 611 | ++I2; |
| 612 | continue; |
| 613 | } |
| 614 | |
| 615 | // Loop over all uses of the Last LEA and update their operands. Note |
| 616 | // that the correctness of this has already been checked in the |
| 617 | // isReplaceable function. |
| Daniel Sanders | 0c47611 | 2019-08-15 19:22:08 +0000 | [diff] [blame] | 618 | Register FirstVReg = First.getOperand(0).getReg(); |
| 619 | Register LastVReg = Last.getOperand(0).getReg(); |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 620 | for (auto UI = MRI->use_begin(LastVReg), UE = MRI->use_end(); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 621 | UI != UE;) { |
| 622 | MachineOperand &MO = *UI++; |
| 623 | MachineInstr &MI = *MO.getParent(); |
| 624 | |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 625 | if (MI.isDebugValue()) { |
| 626 | // Replace DBG_VALUE instruction with modified version using the |
| 627 | // register from the replacing LEA and the address displacement |
| 628 | // between the LEA instructions. |
| 629 | replaceDebugValue(MI, FirstVReg, AddrDispShift); |
| 630 | continue; |
| 631 | } |
| 632 | |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 633 | // Get the number of the first memory operand. |
| 634 | const MCInstrDesc &Desc = MI.getDesc(); |
| 635 | int MemOpNo = |
| Craig Topper | 477649a | 2016-04-28 05:58:46 +0000 | [diff] [blame] | 636 | X86II::getMemoryOperandNo(Desc.TSFlags) + |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 637 | X86II::getOperandBias(Desc); |
| 638 | |
| 639 | // Update address base. |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 640 | MO.setReg(FirstVReg); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 641 | |
| 642 | // Update address disp. |
| Andrey Turetskiy | 0babd26 | 2016-02-20 10:58:28 +0000 | [diff] [blame] | 643 | MachineOperand &Op = MI.getOperand(MemOpNo + X86::AddrDisp); |
| 644 | if (Op.isImm()) |
| 645 | Op.setImm(Op.getImm() + AddrDispShift); |
| 646 | else if (!Op.isJTI()) |
| 647 | Op.setOffset(Op.getOffset() + AddrDispShift); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 648 | } |
| 649 | |
| 650 | // Since we can possibly extend register lifetime, clear kill flags. |
| Andrew Ng | 03e35b6 | 2017-04-28 08:44:30 +0000 | [diff] [blame] | 651 | MRI->clearKillFlags(FirstVReg); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 652 | |
| 653 | ++NumRedundantLEAs; |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 654 | LLVM_DEBUG(dbgs() << "OptimizeLEAs: Remove redundant LEA: "; |
| 655 | Last.dump();); |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 656 | |
| 657 | // By this moment, all of the Last LEA's uses must be replaced. So we |
| 658 | // can freely remove it. |
| Andrea Di Biagio | 7937be7 | 2017-03-21 11:36:21 +0000 | [diff] [blame] | 659 | assert(MRI->use_empty(LastVReg) && |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 660 | "The LEA's def register must have no uses"); |
| 661 | Last.eraseFromParent(); |
| 662 | |
| 663 | // Erase removed LEA from the list. |
| 664 | I2 = List.erase(I2); |
| 665 | |
| 666 | Changed = true; |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 667 | } |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 668 | ++I1; |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 669 | } |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 670 | } |
| 671 | |
| 672 | return Changed; |
| 673 | } |
| 674 | |
| Pengfei Wang | 7630e24 | 2019-08-22 02:29:27 +0000 | [diff] [blame] | 675 | bool X86OptimizeLEAPass::runOnMachineFunction(MachineFunction &MF) { |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 676 | bool Changed = false; |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 677 | |
| Matthias Braun | f1caa28 | 2017-12-15 22:22:58 +0000 | [diff] [blame] | 678 | if (DisableX86LEAOpt || skipFunction(MF.getFunction())) |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 679 | return false; |
| 680 | |
| 681 | MRI = &MF.getRegInfo(); |
| 682 | TII = MF.getSubtarget<X86Subtarget>().getInstrInfo(); |
| 683 | TRI = MF.getSubtarget<X86Subtarget>().getRegisterInfo(); |
| 684 | |
| 685 | // Process all basic blocks. |
| 686 | for (auto &MBB : MF) { |
| Andrey Turetskiy | bca0f99 | 2016-02-04 08:57:03 +0000 | [diff] [blame] | 687 | MemOpMap LEAs; |
| Alexey Bataev | 28f0c5e | 2016-01-11 11:52:29 +0000 | [diff] [blame] | 688 | InstrPos.clear(); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 689 | |
| 690 | // Find all LEA instructions in basic block. |
| 691 | findLEAs(MBB, LEAs); |
| 692 | |
| 693 | // If current basic block has no LEAs, move on to the next one. |
| 694 | if (LEAs.empty()) |
| 695 | continue; |
| 696 | |
| Andrey Turetskiy | 45b22a4 | 2016-05-19 10:18:29 +0000 | [diff] [blame] | 697 | // Remove redundant LEA instructions. |
| 698 | Changed |= removeRedundantLEAs(LEAs); |
| Andrey Turetskiy | 1ce2c99 | 2016-01-13 11:30:44 +0000 | [diff] [blame] | 699 | |
| Andrey Turetskiy | 45b22a4 | 2016-05-19 10:18:29 +0000 | [diff] [blame] | 700 | // Remove redundant address calculations. Do it only for -Os/-Oz since only |
| 701 | // a code size gain is expected from this part of the pass. |
| Evandro Menezes | 85bd397 | 2019-04-04 22:40:06 +0000 | [diff] [blame] | 702 | if (MF.getFunction().hasOptSize()) |
| Andrey Turetskiy | 45b22a4 | 2016-05-19 10:18:29 +0000 | [diff] [blame] | 703 | Changed |= removeRedundantAddrCalc(LEAs); |
| Alexey Bataev | 7cf3247 | 2015-12-04 10:53:15 +0000 | [diff] [blame] | 704 | } |
| 705 | |
| 706 | return Changed; |
| 707 | } |