| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 1 | //===-- X86FixupLEAs.cpp - use or replace LEA instructions -----------===// |
| 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 |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | // |
| Sanjay Patel | 6360441 | 2014-07-16 20:18:49 +0000 | [diff] [blame] | 9 | // This file defines the pass that finds instructions that can be |
| 10 | // re-written as LEA instructions in order to reduce pipeline delays. |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 11 | // When optimizing for size it replaces suitable LEAs with INC or DEC. |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 12 | // |
| 13 | //===----------------------------------------------------------------------===// |
| 14 | |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 15 | #include "X86.h" |
| 16 | #include "X86InstrInfo.h" |
| 17 | #include "X86Subtarget.h" |
| 18 | #include "llvm/ADT/Statistic.h" |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 19 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 20 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 21 | #include "llvm/CodeGen/Passes.h" |
| Simon Pilgrim | a3a9d812 | 2018-04-13 15:09:39 +0000 | [diff] [blame] | 22 | #include "llvm/CodeGen/TargetSchedule.h" |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 23 | #include "llvm/Support/Debug.h" |
| 24 | #include "llvm/Support/raw_ostream.h" |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 25 | using namespace llvm; |
| 26 | |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 27 | #define FIXUPLEA_DESC "X86 LEA Fixup" |
| 28 | #define FIXUPLEA_NAME "x86-fixup-LEAs" |
| 29 | |
| 30 | #define DEBUG_TYPE FIXUPLEA_NAME |
| Chandler Carruth | 84e68b2 | 2014-04-22 02:41:26 +0000 | [diff] [blame] | 31 | |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 32 | STATISTIC(NumLEAs, "Number of LEA instructions created"); |
| 33 | |
| 34 | namespace { |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 35 | class FixupLEAPass : public MachineFunctionPass { |
| 36 | enum RegUsageState { RU_NotUsed, RU_Write, RU_Read }; |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 37 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 38 | /// Given a machine register, look for the instruction |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 39 | /// which writes it in the current basic block. If found, |
| 40 | /// try to replace it with an equivalent LEA instruction. |
| Eric Christopher | 572e03a | 2015-06-19 01:53:21 +0000 | [diff] [blame] | 41 | /// If replacement succeeds, then also process the newly created |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 42 | /// instruction. |
| 43 | void seekLEAFixup(MachineOperand &p, MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 44 | MachineBasicBlock &MBB); |
| Preston Gurd | 128920d | 2013-04-25 21:31:33 +0000 | [diff] [blame] | 45 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 46 | /// Given a memory access or LEA instruction |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 47 | /// whose address mode uses a base and/or index register, look for |
| 48 | /// an opportunity to replace the instruction which sets the base or index |
| 49 | /// register with an equivalent LEA instruction. |
| 50 | void processInstruction(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 51 | MachineBasicBlock &MBB); |
| Preston Gurd | 128920d | 2013-04-25 21:31:33 +0000 | [diff] [blame] | 52 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 53 | /// Given a LEA instruction which is unprofitable |
| Simon Pilgrim | d5d7224 | 2018-11-01 14:57:07 +0000 | [diff] [blame] | 54 | /// on SlowLEA targets try to replace it with an equivalent ADD instruction. |
| 55 | void processInstructionForSlowLEA(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 56 | MachineBasicBlock &MBB); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 57 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 58 | /// Given a LEA instruction which is unprofitable |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 59 | /// on SNB+ try to replace it with other instructions. |
| 60 | /// According to Intel's Optimization Reference Manual: |
| 61 | /// " For LEA instructions with three source operands and some specific |
| 62 | /// situations, instruction latency has increased to 3 cycles, and must |
| 63 | /// dispatch via port 1: |
| 64 | /// - LEA that has all three source operands: base, index, and offset |
| 65 | /// - LEA that uses base and index registers where the base is EBP, RBP, |
| 66 | /// or R13 |
| 67 | /// - LEA that uses RIP relative addressing mode |
| 68 | /// - LEA that uses 16-bit addressing mode " |
| 69 | /// This function currently handles the first 2 cases only. |
| 70 | MachineInstr *processInstrForSlow3OpLEA(MachineInstr &MI, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 71 | MachineBasicBlock &MBB); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 72 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 73 | /// Look for LEAs that add 1 to reg or subtract 1 from reg |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 74 | /// and convert them to INC or DEC respectively. |
| 75 | bool fixupIncDec(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 76 | MachineBasicBlock &MBB) const; |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 77 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 78 | /// Determine if an instruction references a machine register |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 79 | /// and, if so, whether it reads or writes the register. |
| 80 | RegUsageState usesRegister(MachineOperand &p, MachineBasicBlock::iterator I); |
| Preston Gurd | 128920d | 2013-04-25 21:31:33 +0000 | [diff] [blame] | 81 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 82 | /// Step backwards through a basic block, looking |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 83 | /// for an instruction which writes a register within |
| 84 | /// a maximum of INSTR_DISTANCE_THRESHOLD instruction latency cycles. |
| 85 | MachineBasicBlock::iterator searchBackwards(MachineOperand &p, |
| 86 | MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 87 | MachineBasicBlock &MBB); |
| Preston Gurd | 128920d | 2013-04-25 21:31:33 +0000 | [diff] [blame] | 88 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 89 | /// if an instruction can be converted to an |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 90 | /// equivalent LEA, insert the new instruction into the basic block |
| 91 | /// and return a pointer to it. Otherwise, return zero. |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 92 | MachineInstr *postRAConvertToLEA(MachineBasicBlock &MBB, |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 93 | MachineBasicBlock::iterator &MBBI) const; |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 94 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 95 | public: |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 96 | static char ID; |
| 97 | |
| 98 | StringRef getPassName() const override { return FIXUPLEA_DESC; } |
| 99 | |
| 100 | FixupLEAPass() : MachineFunctionPass(ID) { |
| 101 | initializeFixupLEAPassPass(*PassRegistry::getPassRegistry()); |
| 102 | } |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 103 | |
| Adrian Prantl | 5f8f34e4 | 2018-05-01 15:54:18 +0000 | [diff] [blame] | 104 | /// Loop over all of the basic blocks, |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 105 | /// replacing instructions by equivalent LEA instructions |
| 106 | /// if needed and when possible. |
| 107 | bool runOnMachineFunction(MachineFunction &MF) override; |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 108 | |
| Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 109 | // This pass runs after regalloc and doesn't support VReg operands. |
| 110 | MachineFunctionProperties getRequiredProperties() const override { |
| 111 | return MachineFunctionProperties().set( |
| Matthias Braun | 1eb4736 | 2016-08-25 01:27:13 +0000 | [diff] [blame] | 112 | MachineFunctionProperties::Property::NoVRegs); |
| Derek Schuff | 1dbf7a5 | 2016-04-04 17:09:25 +0000 | [diff] [blame] | 113 | } |
| 114 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 115 | private: |
| Simon Pilgrim | a3a9d812 | 2018-04-13 15:09:39 +0000 | [diff] [blame] | 116 | TargetSchedModel TSM; |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 117 | const X86InstrInfo *TII; // Machine instruction info. |
| 118 | }; |
| Reid Kleckner | 0ad69fc | 2017-05-16 19:55:03 +0000 | [diff] [blame] | 119 | } |
| Lama Saba | 52e8925 | 2017-05-16 16:01:36 +0000 | [diff] [blame] | 120 | |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 121 | char FixupLEAPass::ID = 0; |
| 122 | |
| 123 | INITIALIZE_PASS(FixupLEAPass, FIXUPLEA_NAME, FIXUPLEA_DESC, false, false) |
| 124 | |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 125 | MachineInstr * |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 126 | FixupLEAPass::postRAConvertToLEA(MachineBasicBlock &MBB, |
| Preston Gurd | 128920d | 2013-04-25 21:31:33 +0000 | [diff] [blame] | 127 | MachineBasicBlock::iterator &MBBI) const { |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 128 | MachineInstr &MI = *MBBI; |
| 129 | switch (MI.getOpcode()) { |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 130 | case X86::MOV32rr: |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 131 | case X86::MOV64rr: { |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 132 | const MachineOperand &Src = MI.getOperand(1); |
| 133 | const MachineOperand &Dest = MI.getOperand(0); |
| 134 | MachineInstr *NewMI = |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 135 | BuildMI(MBB, MBBI, MI.getDebugLoc(), |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 136 | TII->get(MI.getOpcode() == X86::MOV32rr ? X86::LEA32r |
| 137 | : X86::LEA64r)) |
| Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 138 | .add(Dest) |
| 139 | .add(Src) |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 140 | .addImm(1) |
| 141 | .addReg(0) |
| 142 | .addImm(0) |
| 143 | .addReg(0); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 144 | return NewMI; |
| 145 | } |
| Craig Topper | ffd8662 | 2019-04-02 20:52:10 +0000 | [diff] [blame] | 146 | } |
| 147 | |
| 148 | if (!MI.isConvertibleTo3Addr()) |
| 149 | return nullptr; |
| 150 | |
| 151 | switch (MI.getOpcode()) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 152 | case X86::ADD64ri32: |
| 153 | case X86::ADD64ri8: |
| 154 | case X86::ADD64ri32_DB: |
| 155 | case X86::ADD64ri8_DB: |
| 156 | case X86::ADD32ri: |
| 157 | case X86::ADD32ri8: |
| 158 | case X86::ADD32ri_DB: |
| 159 | case X86::ADD32ri8_DB: |
| 160 | case X86::ADD16ri: |
| 161 | case X86::ADD16ri8: |
| 162 | case X86::ADD16ri_DB: |
| 163 | case X86::ADD16ri8_DB: |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 164 | if (!MI.getOperand(2).isImm()) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 165 | // convertToThreeAddress will call getImm() |
| 166 | // which requires isImm() to be true |
| Craig Topper | 062a2ba | 2014-04-25 05:30:21 +0000 | [diff] [blame] | 167 | return nullptr; |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 168 | } |
| Preston Gurd | f03a6e7 | 2013-09-30 23:51:22 +0000 | [diff] [blame] | 169 | break; |
| Preston Gurd | f0b6288 | 2013-09-30 23:18:42 +0000 | [diff] [blame] | 170 | case X86::ADD16rr: |
| 171 | case X86::ADD16rr_DB: |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 172 | if (MI.getOperand(1).getReg() != MI.getOperand(2).getReg()) { |
| Preston Gurd | f0b6288 | 2013-09-30 23:18:42 +0000 | [diff] [blame] | 173 | // if src1 != src2, then convertToThreeAddress will |
| 174 | // need to create a Virtual register, which we cannot do |
| 175 | // after register allocation. |
| Craig Topper | 062a2ba | 2014-04-25 05:30:21 +0000 | [diff] [blame] | 176 | return nullptr; |
| Preston Gurd | f0b6288 | 2013-09-30 23:18:42 +0000 | [diff] [blame] | 177 | } |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 178 | } |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 179 | MachineFunction::iterator MFI = MBB.getIterator(); |
| Duncan P. N. Exon Smith | 9cfc75c | 2016-06-30 00:01:54 +0000 | [diff] [blame] | 180 | return TII->convertToThreeAddress(MFI, MI, nullptr); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 181 | } |
| 182 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 183 | FunctionPass *llvm::createX86FixupLEAs() { return new FixupLEAPass(); } |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 184 | |
| Craig Topper | dd66ace | 2019-05-01 06:53:03 +0000 | [diff] [blame] | 185 | static bool isLEA(unsigned Opcode) { |
| Craig Topper | bf29238 | 2019-05-02 22:46:23 +0000 | [diff] [blame] | 186 | return Opcode == X86::LEA32r || Opcode == X86::LEA64r || |
| 187 | Opcode == X86::LEA64_32r; |
| Craig Topper | dd66ace | 2019-05-01 06:53:03 +0000 | [diff] [blame] | 188 | } |
| 189 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 190 | bool FixupLEAPass::runOnMachineFunction(MachineFunction &MF) { |
| 191 | if (skipFunction(MF.getFunction())) |
| Andrew Kaylor | 2bee5ef | 2016-04-26 21:44:24 +0000 | [diff] [blame] | 192 | return false; |
| 193 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 194 | const X86Subtarget &ST = MF.getSubtarget<X86Subtarget>(); |
| Andrea Di Biagio | 4ae974e | 2018-11-07 12:26:00 +0000 | [diff] [blame] | 195 | bool IsSlowLEA = ST.slowLEA(); |
| 196 | bool IsSlow3OpsLEA = ST.slow3OpsLEA(); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 197 | bool LEAUsesAG = ST.LEAusesAG(); |
| Andrea Di Biagio | 4ae974e | 2018-11-07 12:26:00 +0000 | [diff] [blame] | 198 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 199 | bool OptIncDec = !ST.slowIncDec() || MF.getFunction().hasOptSize(); |
| 200 | bool OptLEA = LEAUsesAG || IsSlowLEA || IsSlow3OpsLEA; |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 201 | |
| 202 | if (!OptLEA && !OptIncDec) |
| Eric Christopher | 0d5c99e | 2014-05-22 01:46:02 +0000 | [diff] [blame] | 203 | return false; |
| 204 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 205 | TSM.init(&ST); |
| Eric Christopher | d361ff8 | 2015-02-05 19:27:01 +0000 | [diff] [blame] | 206 | TII = ST.getInstrInfo(); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 207 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 208 | LLVM_DEBUG(dbgs() << "Start X86FixupLEAs\n";); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 209 | for (MachineBasicBlock &MBB : MF) { |
| 210 | // First pass. Try to remove or optimize existing LEAs. |
| 211 | for (MachineBasicBlock::iterator I = MBB.begin(); I != MBB.end(); ++I) { |
| Craig Topper | dd66ace | 2019-05-01 06:53:03 +0000 | [diff] [blame] | 212 | if (!isLEA(I->getOpcode())) |
| 213 | continue; |
| 214 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 215 | if (OptIncDec && fixupIncDec(I, MBB)) |
| 216 | continue; |
| 217 | |
| 218 | if (IsSlowLEA) { |
| 219 | processInstructionForSlowLEA(I, MBB); |
| 220 | } else if (IsSlow3OpsLEA) { |
| 221 | if (auto *NewMI = processInstrForSlow3OpLEA(*I, MBB)) { |
| 222 | MBB.erase(I); |
| 223 | I = NewMI; |
| 224 | } |
| 225 | } |
| 226 | } |
| 227 | |
| 228 | // Second pass for creating LEAs. This may reverse some of the |
| 229 | // transformations above. |
| 230 | if (LEAUsesAG) { |
| 231 | for (MachineBasicBlock::iterator I = MBB.begin(); I != MBB.end(); ++I) |
| 232 | processInstruction(I, MBB); |
| 233 | } |
| 234 | } |
| 235 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 236 | LLVM_DEBUG(dbgs() << "End X86FixupLEAs\n";); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 237 | |
| 238 | return true; |
| 239 | } |
| 240 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 241 | FixupLEAPass::RegUsageState |
| 242 | FixupLEAPass::usesRegister(MachineOperand &p, MachineBasicBlock::iterator I) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 243 | RegUsageState RegUsage = RU_NotUsed; |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 244 | MachineInstr &MI = *I; |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 245 | |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 246 | for (unsigned i = 0; i < MI.getNumOperands(); ++i) { |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 247 | MachineOperand &opnd = MI.getOperand(i); |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 248 | if (opnd.isReg() && opnd.getReg() == p.getReg()) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 249 | if (opnd.isDef()) |
| 250 | return RU_Write; |
| 251 | RegUsage = RU_Read; |
| 252 | } |
| 253 | } |
| 254 | return RegUsage; |
| 255 | } |
| 256 | |
| 257 | /// getPreviousInstr - Given a reference to an instruction in a basic |
| 258 | /// block, return a reference to the previous instruction in the block, |
| 259 | /// wrapping around to the last instruction of the block if the block |
| 260 | /// branches to itself. |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 261 | static inline bool getPreviousInstr(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 262 | MachineBasicBlock &MBB) { |
| 263 | if (I == MBB.begin()) { |
| 264 | if (MBB.isPredecessor(&MBB)) { |
| 265 | I = --MBB.end(); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 266 | return true; |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 267 | } else |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 268 | return false; |
| 269 | } |
| 270 | --I; |
| 271 | return true; |
| 272 | } |
| 273 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 274 | MachineBasicBlock::iterator |
| 275 | FixupLEAPass::searchBackwards(MachineOperand &p, MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 276 | MachineBasicBlock &MBB) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 277 | int InstrDistance = 1; |
| 278 | MachineBasicBlock::iterator CurInst; |
| 279 | static const int INSTR_DISTANCE_THRESHOLD = 5; |
| 280 | |
| 281 | CurInst = I; |
| 282 | bool Found; |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 283 | Found = getPreviousInstr(CurInst, MBB); |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 284 | while (Found && I != CurInst) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 285 | if (CurInst->isCall() || CurInst->isInlineAsm()) |
| 286 | break; |
| 287 | if (InstrDistance > INSTR_DISTANCE_THRESHOLD) |
| 288 | break; // too far back to make a difference |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 289 | if (usesRegister(p, CurInst) == RU_Write) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 290 | return CurInst; |
| 291 | } |
| Simon Pilgrim | a3a9d812 | 2018-04-13 15:09:39 +0000 | [diff] [blame] | 292 | InstrDistance += TSM.computeInstrLatency(&*CurInst); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 293 | Found = getPreviousInstr(CurInst, MBB); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 294 | } |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 295 | return MachineBasicBlock::iterator(); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 296 | } |
| 297 | |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 298 | static inline bool isInefficientLEAReg(unsigned Reg) { |
| Craig Topper | b2cc9a1 | 2018-08-03 03:45:19 +0000 | [diff] [blame] | 299 | return Reg == X86::EBP || Reg == X86::RBP || |
| 300 | Reg == X86::R13D || Reg == X86::R13; |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 301 | } |
| 302 | |
| 303 | static inline bool isRegOperand(const MachineOperand &Op) { |
| 304 | return Op.isReg() && Op.getReg() != X86::NoRegister; |
| 305 | } |
| Andrea Di Biagio | 4ae974e | 2018-11-07 12:26:00 +0000 | [diff] [blame] | 306 | |
| 307 | /// Returns true if this LEA uses base an index registers, and the base register |
| 308 | /// is known to be inefficient for the subtarget. |
| Andrea Di Biagio | b6022aa | 2018-07-19 16:42:15 +0000 | [diff] [blame] | 309 | // TODO: use a variant scheduling class to model the latency profile |
| 310 | // of LEA instructions, and implement this logic as a scheduling predicate. |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 311 | static inline bool hasInefficientLEABaseReg(const MachineOperand &Base, |
| 312 | const MachineOperand &Index) { |
| 313 | return Base.isReg() && isInefficientLEAReg(Base.getReg()) && |
| 314 | isRegOperand(Index); |
| 315 | } |
| 316 | |
| 317 | static inline bool hasLEAOffset(const MachineOperand &Offset) { |
| 318 | return (Offset.isImm() && Offset.getImm() != 0) || Offset.isGlobal(); |
| 319 | } |
| 320 | |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 321 | static inline unsigned getADDrrFromLEA(unsigned LEAOpcode) { |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 322 | switch (LEAOpcode) { |
| 323 | default: |
| 324 | llvm_unreachable("Unexpected LEA instruction"); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 325 | case X86::LEA32r: |
| 326 | return X86::ADD32rr; |
| 327 | case X86::LEA64_32r: |
| 328 | case X86::LEA64r: |
| 329 | return X86::ADD64rr; |
| 330 | } |
| 331 | } |
| 332 | |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 333 | static inline unsigned getADDriFromLEA(unsigned LEAOpcode, |
| 334 | const MachineOperand &Offset) { |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 335 | bool IsInt8 = Offset.isImm() && isInt<8>(Offset.getImm()); |
| 336 | switch (LEAOpcode) { |
| 337 | default: |
| 338 | llvm_unreachable("Unexpected LEA instruction"); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 339 | case X86::LEA32r: |
| 340 | case X86::LEA64_32r: |
| 341 | return IsInt8 ? X86::ADD32ri8 : X86::ADD32ri; |
| 342 | case X86::LEA64r: |
| 343 | return IsInt8 ? X86::ADD64ri8 : X86::ADD64ri32; |
| 344 | } |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 345 | } |
| 346 | |
| 347 | /// isLEASimpleIncOrDec - Does this LEA have one these forms: |
| 348 | /// lea %reg, 1(%reg) |
| 349 | /// lea %reg, -1(%reg) |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 350 | static inline bool isLEASimpleIncOrDec(MachineInstr &LEA) { |
| 351 | unsigned SrcReg = LEA.getOperand(1 + X86::AddrBaseReg).getReg(); |
| 352 | unsigned DstReg = LEA.getOperand(0).getReg(); |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 353 | const MachineOperand &AddrDisp = LEA.getOperand(1 + X86::AddrDisp); |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 354 | return SrcReg == DstReg && |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 355 | LEA.getOperand(1 + X86::AddrIndexReg).getReg() == 0 && |
| 356 | LEA.getOperand(1 + X86::AddrSegmentReg).getReg() == 0 && |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 357 | AddrDisp.isImm() && |
| 358 | (AddrDisp.getImm() == 1 || AddrDisp.getImm() == -1); |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 359 | } |
| 360 | |
| 361 | bool FixupLEAPass::fixupIncDec(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 362 | MachineBasicBlock &MBB) const { |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 363 | MachineInstr &MI = *I; |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 364 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 365 | if (isLEASimpleIncOrDec(MI) && TII->isSafeToClobberEFLAGS(MBB, I)) { |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 366 | unsigned NewOpcode; |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 367 | bool isINC = MI.getOperand(1 + X86::AddrDisp).getImm() == 1; |
| Craig Topper | dd66ace | 2019-05-01 06:53:03 +0000 | [diff] [blame] | 368 | switch (MI.getOpcode()) { |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 369 | case X86::LEA32r: |
| 370 | case X86::LEA64_32r: |
| 371 | NewOpcode = isINC ? X86::INC32r : X86::DEC32r; |
| 372 | break; |
| 373 | case X86::LEA64r: |
| 374 | NewOpcode = isINC ? X86::INC64r : X86::DEC64r; |
| 375 | break; |
| 376 | } |
| 377 | |
| 378 | MachineInstr *NewMI = |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 379 | BuildMI(MBB, I, MI.getDebugLoc(), TII->get(NewOpcode)) |
| Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 380 | .add(MI.getOperand(0)) |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 381 | .add(MI.getOperand(1 + X86::AddrBaseReg)); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 382 | MBB.erase(I); |
| Michael Kuperstein | 12982a8 | 2015-11-11 11:44:31 +0000 | [diff] [blame] | 383 | I = static_cast<MachineBasicBlock::iterator>(NewMI); |
| 384 | return true; |
| 385 | } |
| 386 | return false; |
| 387 | } |
| 388 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 389 | void FixupLEAPass::processInstruction(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 390 | MachineBasicBlock &MBB) { |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 391 | // Process a load, store, or LEA instruction. |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 392 | MachineInstr &MI = *I; |
| 393 | const MCInstrDesc &Desc = MI.getDesc(); |
| Craig Topper | 477649a | 2016-04-28 05:58:46 +0000 | [diff] [blame] | 394 | int AddrOffset = X86II::getMemoryOperandNo(Desc.TSFlags); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 395 | if (AddrOffset >= 0) { |
| 396 | AddrOffset += X86II::getOperandBias(Desc); |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 397 | MachineOperand &p = MI.getOperand(AddrOffset + X86::AddrBaseReg); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 398 | if (p.isReg() && p.getReg() != X86::ESP) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 399 | seekLEAFixup(p, I, MBB); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 400 | } |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 401 | MachineOperand &q = MI.getOperand(AddrOffset + X86::AddrIndexReg); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 402 | if (q.isReg() && q.getReg() != X86::ESP) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 403 | seekLEAFixup(q, I, MBB); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 404 | } |
| 405 | } |
| 406 | } |
| 407 | |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 408 | void FixupLEAPass::seekLEAFixup(MachineOperand &p, |
| 409 | MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 410 | MachineBasicBlock &MBB) { |
| 411 | MachineBasicBlock::iterator MBI = searchBackwards(p, I, MBB); |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 412 | if (MBI != MachineBasicBlock::iterator()) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 413 | MachineInstr *NewMI = postRAConvertToLEA(MBB, MBI); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 414 | if (NewMI) { |
| 415 | ++NumLEAs; |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 416 | LLVM_DEBUG(dbgs() << "FixLEA: Candidate to replace:"; MBI->dump();); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 417 | // now to replace with an equivalent LEA... |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 418 | LLVM_DEBUG(dbgs() << "FixLEA: Replaced by: "; NewMI->dump();); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 419 | MBB.erase(MBI); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 420 | MachineBasicBlock::iterator J = |
| Eric Christopher | 31b81ce | 2014-06-03 21:01:35 +0000 | [diff] [blame] | 421 | static_cast<MachineBasicBlock::iterator>(NewMI); |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 422 | processInstruction(J, MBB); |
| Preston Gurd | 8b7ab4b | 2013-04-25 20:29:37 +0000 | [diff] [blame] | 423 | } |
| 424 | } |
| 425 | } |
| 426 | |
| Simon Pilgrim | d5d7224 | 2018-11-01 14:57:07 +0000 | [diff] [blame] | 427 | void FixupLEAPass::processInstructionForSlowLEA(MachineBasicBlock::iterator &I, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 428 | MachineBasicBlock &MBB) { |
| Duncan P. N. Exon Smith | 7b4c18e | 2016-07-12 03:18:50 +0000 | [diff] [blame] | 429 | MachineInstr &MI = *I; |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 430 | const unsigned Opcode = MI.getOpcode(); |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 431 | |
| 432 | const MachineOperand &Dst = MI.getOperand(0); |
| 433 | const MachineOperand &Base = MI.getOperand(1 + X86::AddrBaseReg); |
| 434 | const MachineOperand &Scale = MI.getOperand(1 + X86::AddrScaleAmt); |
| 435 | const MachineOperand &Index = MI.getOperand(1 + X86::AddrIndexReg); |
| 436 | const MachineOperand &Offset = MI.getOperand(1 + X86::AddrDisp); |
| 437 | const MachineOperand &Segment = MI.getOperand(1 + X86::AddrSegmentReg); |
| 438 | |
| 439 | if (Segment.getReg() != 0 || !Offset.isImm() || |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 440 | !TII->isSafeToClobberEFLAGS(MBB, I)) |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 441 | return; |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 442 | const unsigned DstR = Dst.getReg(); |
| 443 | const unsigned SrcR1 = Base.getReg(); |
| 444 | const unsigned SrcR2 = Index.getReg(); |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 445 | if ((SrcR1 == 0 || SrcR1 != DstR) && (SrcR2 == 0 || SrcR2 != DstR)) |
| 446 | return; |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 447 | if (Scale.getImm() > 1) |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 448 | return; |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 449 | LLVM_DEBUG(dbgs() << "FixLEA: Candidate to replace:"; I->dump();); |
| 450 | LLVM_DEBUG(dbgs() << "FixLEA: Replaced by: ";); |
| Craig Topper | 66f09ad | 2014-06-08 22:29:17 +0000 | [diff] [blame] | 451 | MachineInstr *NewMI = nullptr; |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 452 | // Make ADD instruction for two registers writing to LEA's destination |
| 453 | if (SrcR1 != 0 && SrcR2 != 0) { |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 454 | const MCInstrDesc &ADDrr = TII->get(getADDrrFromLEA(Opcode)); |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 455 | const MachineOperand &Src = SrcR1 == DstR ? Index : Base; |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 456 | NewMI = |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 457 | BuildMI(MBB, I, MI.getDebugLoc(), ADDrr, DstR).addReg(DstR).add(Src); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 458 | LLVM_DEBUG(NewMI->dump();); |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 459 | } |
| 460 | // Make ADD instruction for immediate |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 461 | if (Offset.getImm() != 0) { |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 462 | const MCInstrDesc &ADDri = |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 463 | TII->get(getADDriFromLEA(Opcode, Offset)); |
| 464 | const MachineOperand &SrcR = SrcR1 == DstR ? Base : Index; |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 465 | NewMI = BuildMI(MBB, I, MI.getDebugLoc(), ADDri, DstR) |
| Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 466 | .add(SrcR) |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 467 | .addImm(Offset.getImm()); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 468 | LLVM_DEBUG(NewMI->dump();); |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 469 | } |
| 470 | if (NewMI) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 471 | MBB.erase(I); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 472 | I = NewMI; |
| Alexey Volkov | 6226de6 | 2014-05-20 08:55:50 +0000 | [diff] [blame] | 473 | } |
| 474 | } |
| 475 | |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 476 | MachineInstr * |
| 477 | FixupLEAPass::processInstrForSlow3OpLEA(MachineInstr &MI, |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 478 | MachineBasicBlock &MBB) { |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 479 | const unsigned LEAOpcode = MI.getOpcode(); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 480 | |
| Craig Topper | 1f02ac3 | 2018-12-22 01:34:47 +0000 | [diff] [blame] | 481 | const MachineOperand &Dst = MI.getOperand(0); |
| 482 | const MachineOperand &Base = MI.getOperand(1 + X86::AddrBaseReg); |
| 483 | const MachineOperand &Scale = MI.getOperand(1 + X86::AddrScaleAmt); |
| 484 | const MachineOperand &Index = MI.getOperand(1 + X86::AddrIndexReg); |
| 485 | const MachineOperand &Offset = MI.getOperand(1 + X86::AddrDisp); |
| 486 | const MachineOperand &Segment = MI.getOperand(1 + X86::AddrSegmentReg); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 487 | |
| Andrea Di Biagio | b6022aa | 2018-07-19 16:42:15 +0000 | [diff] [blame] | 488 | if (!(TII->isThreeOperandsLEA(MI) || |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 489 | hasInefficientLEABaseReg(Base, Index)) || |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 490 | !TII->isSafeToClobberEFLAGS(MBB, MI) || |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 491 | Segment.getReg() != X86::NoRegister) |
| 492 | return nullptr; |
| 493 | |
| Craig Topper | c5903c9 | 2019-04-02 00:50:58 +0000 | [diff] [blame] | 494 | unsigned DstR = Dst.getReg(); |
| 495 | unsigned BaseR = Base.getReg(); |
| 496 | unsigned IndexR = Index.getReg(); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 497 | unsigned SSDstR = |
| 498 | (LEAOpcode == X86::LEA64_32r) ? getX86SubSuperRegister(DstR, 64) : DstR; |
| 499 | bool IsScale1 = Scale.getImm() == 1; |
| 500 | bool IsInefficientBase = isInefficientLEAReg(BaseR); |
| 501 | bool IsInefficientIndex = isInefficientLEAReg(IndexR); |
| 502 | |
| 503 | // Skip these cases since it takes more than 2 instructions |
| 504 | // to replace the LEA instruction. |
| 505 | if (IsInefficientBase && SSDstR == BaseR && !IsScale1) |
| 506 | return nullptr; |
| 507 | if (LEAOpcode == X86::LEA64_32r && IsInefficientBase && |
| 508 | (IsInefficientIndex || !IsScale1)) |
| 509 | return nullptr; |
| 510 | |
| 511 | const DebugLoc DL = MI.getDebugLoc(); |
| 512 | const MCInstrDesc &ADDrr = TII->get(getADDrrFromLEA(LEAOpcode)); |
| 513 | const MCInstrDesc &ADDri = TII->get(getADDriFromLEA(LEAOpcode, Offset)); |
| 514 | |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 515 | LLVM_DEBUG(dbgs() << "FixLEA: Candidate to replace:"; MI.dump();); |
| 516 | LLVM_DEBUG(dbgs() << "FixLEA: Replaced by: ";); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 517 | |
| 518 | // First try to replace LEA with one or two (for the 3-op LEA case) |
| 519 | // add instructions: |
| 520 | // 1.lea (%base,%index,1), %base => add %index,%base |
| 521 | // 2.lea (%base,%index,1), %index => add %base,%index |
| 522 | if (IsScale1 && (DstR == BaseR || DstR == IndexR)) { |
| 523 | const MachineOperand &Src = DstR == BaseR ? Index : Base; |
| 524 | MachineInstr *NewMI = |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 525 | BuildMI(MBB, MI, DL, ADDrr, DstR).addReg(DstR).add(Src); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 526 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 527 | // Create ADD instruction for the Offset in case of 3-Ops LEA. |
| 528 | if (hasLEAOffset(Offset)) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 529 | NewMI = BuildMI(MBB, MI, DL, ADDri, DstR).addReg(DstR).add(Offset); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 530 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 531 | } |
| 532 | return NewMI; |
| 533 | } |
| 534 | // If the base is inefficient try switching the index and base operands, |
| 535 | // otherwise just break the 3-Ops LEA inst into 2-Ops LEA + ADD instruction: |
| 536 | // lea offset(%base,%index,scale),%dst => |
| 537 | // lea (%base,%index,scale); add offset,%dst |
| 538 | if (!IsInefficientBase || (!IsInefficientIndex && IsScale1)) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 539 | MachineInstr *NewMI = BuildMI(MBB, MI, DL, TII->get(LEAOpcode)) |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 540 | .add(Dst) |
| 541 | .add(IsInefficientBase ? Index : Base) |
| 542 | .add(Scale) |
| 543 | .add(IsInefficientBase ? Base : Index) |
| 544 | .addImm(0) |
| 545 | .add(Segment); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 546 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 547 | // Create ADD instruction for the Offset in case of 3-Ops LEA. |
| 548 | if (hasLEAOffset(Offset)) { |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 549 | NewMI = BuildMI(MBB, MI, DL, ADDri, DstR).addReg(DstR).add(Offset); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 550 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 551 | } |
| 552 | return NewMI; |
| 553 | } |
| 554 | // Handle the rest of the cases with inefficient base register: |
| 555 | assert(SSDstR != BaseR && "SSDstR == BaseR should be handled already!"); |
| 556 | assert(IsInefficientBase && "efficient base should be handled already!"); |
| 557 | |
| 558 | // lea (%base,%index,1), %dst => mov %base,%dst; add %index,%dst |
| 559 | if (IsScale1 && !hasLEAOffset(Offset)) { |
| Serguei Katkov | 1ce7137 | 2018-01-26 04:49:26 +0000 | [diff] [blame] | 560 | bool BIK = Base.isKill() && BaseR != IndexR; |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 561 | TII->copyPhysReg(MBB, MI, DL, DstR, BaseR, BIK); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 562 | LLVM_DEBUG(MI.getPrevNode()->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 563 | |
| 564 | MachineInstr *NewMI = |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 565 | BuildMI(MBB, MI, DL, ADDrr, DstR).addReg(DstR).add(Index); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 566 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 567 | return NewMI; |
| 568 | } |
| 569 | // lea offset(%base,%index,scale), %dst => |
| 570 | // lea offset( ,%index,scale), %dst; add %base,%dst |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 571 | MachineInstr *NewMI = BuildMI(MBB, MI, DL, TII->get(LEAOpcode)) |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 572 | .add(Dst) |
| 573 | .addReg(0) |
| 574 | .add(Scale) |
| 575 | .add(Index) |
| 576 | .add(Offset) |
| 577 | .add(Segment); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 578 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 579 | |
| Craig Topper | 965d130 | 2019-04-30 17:56:28 +0000 | [diff] [blame] | 580 | NewMI = BuildMI(MBB, MI, DL, ADDrr, DstR).addReg(DstR).add(Base); |
| Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 581 | LLVM_DEBUG(NewMI->dump();); |
| Lama Saba | 2ea271b | 2017-05-18 08:11:50 +0000 | [diff] [blame] | 582 | return NewMI; |
| 583 | } |