Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 1 | //===- HexagonExpandCondsets.cpp ------------------------------------------===// |
Krzysztof Parzyszek | 8b26fbf | 2015-07-09 15:40:25 +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 |
Krzysztof Parzyszek | 8b26fbf | 2015-07-09 15:40:25 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 9 | // Replace mux instructions with the corresponding legal instructions. |
| 10 | // It is meant to work post-SSA, but still on virtual registers. It was |
| 11 | // originally placed between register coalescing and machine instruction |
| 12 | // scheduler. |
| 13 | // In this place in the optimization sequence, live interval analysis had |
| 14 | // been performed, and the live intervals should be preserved. A large part |
| 15 | // of the code deals with preserving the liveness information. |
| 16 | // |
| 17 | // Liveness tracking aside, the main functionality of this pass is divided |
| 18 | // into two steps. The first step is to replace an instruction |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 19 | // %0 = C2_mux %1, %2, %3 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 20 | // with a pair of conditional transfers |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 21 | // %0 = A2_tfrt %1, %2 |
| 22 | // %0 = A2_tfrf %1, %3 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 23 | // It is the intention that the execution of this pass could be terminated |
| 24 | // after this step, and the code generated would be functionally correct. |
| 25 | // |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 26 | // If the uses of the source values %1 and %2 are kills, and their |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 27 | // definitions are predicable, then in the second step, the conditional |
| 28 | // transfers will then be rewritten as predicated instructions. E.g. |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 29 | // %0 = A2_or %1, %2 |
Francis Visoiu Mistrih | a8a83d1 | 2017-12-07 10:40:31 +0000 | [diff] [blame] | 30 | // %3 = A2_tfrt %99, killed %0 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 31 | // will be rewritten as |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 32 | // %3 = A2_port %99, %1, %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 33 | // |
| 34 | // This replacement has two variants: "up" and "down". Consider this case: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 35 | // %0 = A2_or %1, %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 36 | // ... [intervening instructions] ... |
Francis Visoiu Mistrih | a8a83d1 | 2017-12-07 10:40:31 +0000 | [diff] [blame] | 37 | // %3 = A2_tfrt %99, killed %0 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 38 | // variant "up": |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 39 | // %3 = A2_port %99, %1, %2 |
| 40 | // ... [intervening instructions, %0->vreg3] ... |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 41 | // [deleted] |
| 42 | // variant "down": |
| 43 | // [deleted] |
| 44 | // ... [intervening instructions] ... |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 45 | // %3 = A2_port %99, %1, %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 46 | // |
| 47 | // Both, one or none of these variants may be valid, and checks are made |
| 48 | // to rule out inapplicable variants. |
| 49 | // |
| 50 | // As an additional optimization, before either of the two steps above is |
| 51 | // executed, the pass attempts to coalesce the target register with one of |
| 52 | // the source registers, e.g. given an instruction |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 53 | // %3 = C2_mux %0, %1, %2 |
| 54 | // %3 will be coalesced with either %1 or %2. If this succeeds, |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 55 | // the instruction would then be (for example) |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 56 | // %3 = C2_mux %0, %3, %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 57 | // and, under certain circumstances, this could result in only one predicated |
| 58 | // instruction: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 59 | // %3 = A2_tfrf %0, %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 60 | // |
| 61 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 62 | // Splitting a definition of a register into two predicated transfers |
| 63 | // creates a complication in liveness tracking. Live interval computation |
| 64 | // will see both instructions as actual definitions, and will mark the |
| 65 | // first one as dead. The definition is not actually dead, and this |
| 66 | // situation will need to be fixed. For example: |
Francis Visoiu Mistrih | a8a83d1 | 2017-12-07 10:40:31 +0000 | [diff] [blame] | 67 | // dead %1 = A2_tfrt ... ; marked as dead |
| 68 | // %1 = A2_tfrf ... |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 69 | // |
| 70 | // Since any of the individual predicated transfers may end up getting |
| 71 | // removed (in case it is an identity copy), some pre-existing def may |
| 72 | // be marked as dead after live interval recomputation: |
Francis Visoiu Mistrih | a8a83d1 | 2017-12-07 10:40:31 +0000 | [diff] [blame] | 73 | // dead %1 = ... ; marked as dead |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 74 | // ... |
Francis Visoiu Mistrih | a8a83d1 | 2017-12-07 10:40:31 +0000 | [diff] [blame] | 75 | // %1 = A2_tfrf ... ; if A2_tfrt is removed |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 76 | // This case happens if %1 was used as a source in A2_tfrt, which means |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 77 | // that is it actually live at the A2_tfrf, and so the now dead definition |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 78 | // of %1 will need to be updated to non-dead at some point. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 79 | // |
| 80 | // This issue could be remedied by adding implicit uses to the predicated |
| 81 | // transfers, but this will create a problem with subsequent predication, |
| 82 | // since the transfers will no longer be possible to reorder. To avoid |
| 83 | // that, the initial splitting will not add any implicit uses. These |
| 84 | // implicit uses will be added later, after predication. The extra price, |
| 85 | // however, is that finding the locations where the implicit uses need |
| 86 | // to be added, and updating the live ranges will be more involved. |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 87 | |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 88 | #include "HexagonInstrInfo.h" |
Krzysztof Parzyszek | 5577297 | 2017-09-15 15:46:05 +0000 | [diff] [blame] | 89 | #include "HexagonRegisterInfo.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 90 | #include "llvm/ADT/DenseMap.h" |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 91 | #include "llvm/ADT/SetVector.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 92 | #include "llvm/ADT/SmallVector.h" |
| 93 | #include "llvm/ADT/StringRef.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 94 | #include "llvm/CodeGen/LiveInterval.h" |
Matthias Braun | f842297 | 2017-12-13 02:51:04 +0000 | [diff] [blame] | 95 | #include "llvm/CodeGen/LiveIntervals.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 96 | #include "llvm/CodeGen/MachineBasicBlock.h" |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 97 | #include "llvm/CodeGen/MachineDominators.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 98 | #include "llvm/CodeGen/MachineFunction.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 99 | #include "llvm/CodeGen/MachineFunctionPass.h" |
| 100 | #include "llvm/CodeGen/MachineInstr.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 101 | #include "llvm/CodeGen/MachineInstrBuilder.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 102 | #include "llvm/CodeGen/MachineOperand.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 103 | #include "llvm/CodeGen/MachineRegisterInfo.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 104 | #include "llvm/CodeGen/SlotIndexes.h" |
David Blaikie | b3bde2e | 2017-11-17 01:07:10 +0000 | [diff] [blame] | 105 | #include "llvm/CodeGen/TargetRegisterInfo.h" |
| 106 | #include "llvm/CodeGen/TargetSubtargetInfo.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 107 | #include "llvm/IR/DebugLoc.h" |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 108 | #include "llvm/IR/Function.h" |
Reid Kleckner | 05da2fe | 2019-11-13 13:15:01 -0800 | [diff] [blame^] | 109 | #include "llvm/InitializePasses.h" |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 110 | #include "llvm/MC/LaneBitmask.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 111 | #include "llvm/Pass.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 112 | #include "llvm/Support/CommandLine.h" |
| 113 | #include "llvm/Support/Debug.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 114 | #include "llvm/Support/ErrorHandling.h" |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 115 | #include "llvm/Support/raw_ostream.h" |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 116 | #include <cassert> |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 117 | #include <iterator> |
| 118 | #include <set> |
| 119 | #include <utility> |
| 120 | |
Jakub Kuderski | 34327d2 | 2017-07-13 20:26:45 +0000 | [diff] [blame] | 121 | #define DEBUG_TYPE "expand-condsets" |
| 122 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 123 | using namespace llvm; |
| 124 | |
| 125 | static cl::opt<unsigned> OptTfrLimit("expand-condsets-tfr-limit", |
| 126 | cl::init(~0U), cl::Hidden, cl::desc("Max number of mux expansions")); |
| 127 | static cl::opt<unsigned> OptCoaLimit("expand-condsets-coa-limit", |
| 128 | cl::init(~0U), cl::Hidden, cl::desc("Max number of segment coalescings")); |
| 129 | |
| 130 | namespace llvm { |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 131 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 132 | void initializeHexagonExpandCondsetsPass(PassRegistry&); |
| 133 | FunctionPass *createHexagonExpandCondsets(); |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 134 | |
| 135 | } // end namespace llvm |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 136 | |
| 137 | namespace { |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 138 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 139 | class HexagonExpandCondsets : public MachineFunctionPass { |
| 140 | public: |
| 141 | static char ID; |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 142 | |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 143 | HexagonExpandCondsets() : MachineFunctionPass(ID) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 144 | if (OptCoaLimit.getPosition()) |
| 145 | CoaLimitActive = true, CoaLimit = OptCoaLimit; |
| 146 | if (OptTfrLimit.getPosition()) |
| 147 | TfrLimitActive = true, TfrLimit = OptTfrLimit; |
| 148 | initializeHexagonExpandCondsetsPass(*PassRegistry::getPassRegistry()); |
| 149 | } |
| 150 | |
Mehdi Amini | 117296c | 2016-10-01 02:56:57 +0000 | [diff] [blame] | 151 | StringRef getPassName() const override { return "Hexagon Expand Condsets"; } |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 152 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 153 | void getAnalysisUsage(AnalysisUsage &AU) const override { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 154 | AU.addRequired<LiveIntervals>(); |
| 155 | AU.addPreserved<LiveIntervals>(); |
| 156 | AU.addPreserved<SlotIndexes>(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 157 | AU.addRequired<MachineDominatorTree>(); |
| 158 | AU.addPreserved<MachineDominatorTree>(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 159 | MachineFunctionPass::getAnalysisUsage(AU); |
| 160 | } |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 161 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 162 | bool runOnMachineFunction(MachineFunction &MF) override; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 163 | |
| 164 | private: |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 165 | const HexagonInstrInfo *HII = nullptr; |
| 166 | const TargetRegisterInfo *TRI = nullptr; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 167 | MachineDominatorTree *MDT; |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 168 | MachineRegisterInfo *MRI = nullptr; |
| 169 | LiveIntervals *LIS = nullptr; |
| 170 | bool CoaLimitActive = false; |
| 171 | bool TfrLimitActive = false; |
| 172 | unsigned CoaLimit; |
| 173 | unsigned TfrLimit; |
| 174 | unsigned CoaCounter = 0; |
| 175 | unsigned TfrCounter = 0; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 176 | |
| 177 | struct RegisterRef { |
| 178 | RegisterRef(const MachineOperand &Op) : Reg(Op.getReg()), |
| 179 | Sub(Op.getSubReg()) {} |
| 180 | RegisterRef(unsigned R = 0, unsigned S = 0) : Reg(R), Sub(S) {} |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 181 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 182 | bool operator== (RegisterRef RR) const { |
| 183 | return Reg == RR.Reg && Sub == RR.Sub; |
| 184 | } |
| 185 | bool operator!= (RegisterRef RR) const { return !operator==(RR); } |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 186 | bool operator< (RegisterRef RR) const { |
| 187 | return Reg < RR.Reg || (Reg == RR.Reg && Sub < RR.Sub); |
| 188 | } |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 189 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 190 | unsigned Reg, Sub; |
| 191 | }; |
| 192 | |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 193 | using ReferenceMap = DenseMap<unsigned, unsigned>; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 194 | enum { Sub_Low = 0x1, Sub_High = 0x2, Sub_None = (Sub_Low | Sub_High) }; |
| 195 | enum { Exec_Then = 0x10, Exec_Else = 0x20 }; |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 196 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 197 | unsigned getMaskForSub(unsigned Sub); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 198 | bool isCondset(const MachineInstr &MI); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 199 | LaneBitmask getLaneMask(unsigned Reg, unsigned Sub); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 200 | |
| 201 | void addRefToMap(RegisterRef RR, ReferenceMap &Map, unsigned Exec); |
| 202 | bool isRefInMap(RegisterRef, ReferenceMap &Map, unsigned Exec); |
| 203 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 204 | void updateDeadsInRange(unsigned Reg, LaneBitmask LM, LiveRange &Range); |
| 205 | void updateKillFlags(unsigned Reg); |
| 206 | void updateDeadFlags(unsigned Reg); |
| 207 | void recalculateLiveInterval(unsigned Reg); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 208 | void removeInstr(MachineInstr &MI); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 209 | void updateLiveness(std::set<unsigned> &RegSet, bool Recalc, |
| 210 | bool UpdateKills, bool UpdateDeads); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 211 | |
| 212 | unsigned getCondTfrOpcode(const MachineOperand &SO, bool Cond); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 213 | MachineInstr *genCondTfrFor(MachineOperand &SrcOp, |
| 214 | MachineBasicBlock::iterator At, unsigned DstR, |
| 215 | unsigned DstSR, const MachineOperand &PredOp, bool PredSense, |
| 216 | bool ReadUndef, bool ImpUse); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 217 | bool split(MachineInstr &MI, std::set<unsigned> &UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 218 | |
| 219 | bool isPredicable(MachineInstr *MI); |
| 220 | MachineInstr *getReachingDefForPred(RegisterRef RD, |
| 221 | MachineBasicBlock::iterator UseIt, unsigned PredR, bool Cond); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 222 | bool canMoveOver(MachineInstr &MI, ReferenceMap &Defs, ReferenceMap &Uses); |
| 223 | bool canMoveMemTo(MachineInstr &MI, MachineInstr &ToI, bool IsDown); |
| 224 | void predicateAt(const MachineOperand &DefOp, MachineInstr &MI, |
| 225 | MachineBasicBlock::iterator Where, |
| 226 | const MachineOperand &PredOp, bool Cond, |
| 227 | std::set<unsigned> &UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 228 | void renameInRange(RegisterRef RO, RegisterRef RN, unsigned PredR, |
| 229 | bool Cond, MachineBasicBlock::iterator First, |
| 230 | MachineBasicBlock::iterator Last); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 231 | bool predicate(MachineInstr &TfrI, bool Cond, std::set<unsigned> &UpdRegs); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 232 | bool predicateInBlock(MachineBasicBlock &B, |
| 233 | std::set<unsigned> &UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 234 | |
| 235 | bool isIntReg(RegisterRef RR, unsigned &BW); |
| 236 | bool isIntraBlocks(LiveInterval &LI); |
| 237 | bool coalesceRegisters(RegisterRef R1, RegisterRef R2); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 238 | bool coalesceSegments(const SmallVectorImpl<MachineInstr*> &Condsets, |
| 239 | std::set<unsigned> &UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 240 | }; |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 241 | |
| 242 | } // end anonymous namespace |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 243 | |
| 244 | char HexagonExpandCondsets::ID = 0; |
| 245 | |
Krzysztof Parzyszek | 951fb36 | 2016-08-24 22:27:36 +0000 | [diff] [blame] | 246 | namespace llvm { |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 247 | |
Krzysztof Parzyszek | 951fb36 | 2016-08-24 22:27:36 +0000 | [diff] [blame] | 248 | char &HexagonExpandCondsetsID = HexagonExpandCondsets::ID; |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 249 | |
| 250 | } // end namespace llvm |
Krzysztof Parzyszek | 951fb36 | 2016-08-24 22:27:36 +0000 | [diff] [blame] | 251 | |
Krzysztof Parzyszek | 764fed9 | 2016-05-27 21:15:34 +0000 | [diff] [blame] | 252 | INITIALIZE_PASS_BEGIN(HexagonExpandCondsets, "expand-condsets", |
| 253 | "Hexagon Expand Condsets", false, false) |
| 254 | INITIALIZE_PASS_DEPENDENCY(MachineDominatorTree) |
| 255 | INITIALIZE_PASS_DEPENDENCY(SlotIndexes) |
| 256 | INITIALIZE_PASS_DEPENDENCY(LiveIntervals) |
| 257 | INITIALIZE_PASS_END(HexagonExpandCondsets, "expand-condsets", |
| 258 | "Hexagon Expand Condsets", false, false) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 259 | |
| 260 | unsigned HexagonExpandCondsets::getMaskForSub(unsigned Sub) { |
| 261 | switch (Sub) { |
Krzysztof Parzyszek | a540997 | 2016-11-09 16:19:08 +0000 | [diff] [blame] | 262 | case Hexagon::isub_lo: |
| 263 | case Hexagon::vsub_lo: |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 264 | return Sub_Low; |
Krzysztof Parzyszek | a540997 | 2016-11-09 16:19:08 +0000 | [diff] [blame] | 265 | case Hexagon::isub_hi: |
| 266 | case Hexagon::vsub_hi: |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 267 | return Sub_High; |
| 268 | case Hexagon::NoSubRegister: |
| 269 | return Sub_None; |
| 270 | } |
| 271 | llvm_unreachable("Invalid subregister"); |
| 272 | } |
| 273 | |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 274 | bool HexagonExpandCondsets::isCondset(const MachineInstr &MI) { |
| 275 | unsigned Opc = MI.getOpcode(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 276 | switch (Opc) { |
| 277 | case Hexagon::C2_mux: |
| 278 | case Hexagon::C2_muxii: |
| 279 | case Hexagon::C2_muxir: |
| 280 | case Hexagon::C2_muxri: |
Krzysztof Parzyszek | 258af19 | 2016-08-11 19:12:18 +0000 | [diff] [blame] | 281 | case Hexagon::PS_pselect: |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 282 | return true; |
| 283 | break; |
| 284 | } |
| 285 | return false; |
| 286 | } |
| 287 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 288 | LaneBitmask HexagonExpandCondsets::getLaneMask(unsigned Reg, unsigned Sub) { |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 289 | assert(Register::isVirtualRegister(Reg)); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 290 | return Sub != 0 ? TRI->getSubRegIndexLaneMask(Sub) |
| 291 | : MRI->getMaxLaneMaskForVReg(Reg); |
| 292 | } |
| 293 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 294 | void HexagonExpandCondsets::addRefToMap(RegisterRef RR, ReferenceMap &Map, |
| 295 | unsigned Exec) { |
| 296 | unsigned Mask = getMaskForSub(RR.Sub) | Exec; |
| 297 | ReferenceMap::iterator F = Map.find(RR.Reg); |
| 298 | if (F == Map.end()) |
| 299 | Map.insert(std::make_pair(RR.Reg, Mask)); |
| 300 | else |
| 301 | F->second |= Mask; |
| 302 | } |
| 303 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 304 | bool HexagonExpandCondsets::isRefInMap(RegisterRef RR, ReferenceMap &Map, |
| 305 | unsigned Exec) { |
| 306 | ReferenceMap::iterator F = Map.find(RR.Reg); |
| 307 | if (F == Map.end()) |
| 308 | return false; |
| 309 | unsigned Mask = getMaskForSub(RR.Sub) | Exec; |
| 310 | if (Mask & F->second) |
| 311 | return true; |
| 312 | return false; |
| 313 | } |
| 314 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 315 | void HexagonExpandCondsets::updateKillFlags(unsigned Reg) { |
| 316 | auto KillAt = [this,Reg] (SlotIndex K, LaneBitmask LM) -> void { |
| 317 | // Set the <kill> flag on a use of Reg whose lane mask is contained in LM. |
| 318 | MachineInstr *MI = LIS->getInstructionFromIndex(K); |
Krzysztof Parzyszek | 6f503b9 | 2018-03-23 19:39:37 +0000 | [diff] [blame] | 319 | for (unsigned i = 0, e = MI->getNumOperands(); i != e; ++i) { |
| 320 | MachineOperand &Op = MI->getOperand(i); |
| 321 | if (!Op.isReg() || !Op.isUse() || Op.getReg() != Reg || |
| 322 | MI->isRegTiedToDefOperand(i)) |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 323 | continue; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 324 | LaneBitmask SLM = getLaneMask(Reg, Op.getSubReg()); |
| 325 | if ((SLM & LM) == SLM) { |
| 326 | // Only set the kill flag on the first encountered use of Reg in this |
| 327 | // instruction. |
| 328 | Op.setIsKill(true); |
| 329 | break; |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 330 | } |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 331 | } |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 332 | }; |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 333 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 334 | LiveInterval &LI = LIS->getInterval(Reg); |
| 335 | for (auto I = LI.begin(), E = LI.end(); I != E; ++I) { |
| 336 | if (!I->end.isRegister()) |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 337 | continue; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 338 | // Do not mark the end of the segment as <kill>, if the next segment |
| 339 | // starts with a predicated instruction. |
| 340 | auto NextI = std::next(I); |
| 341 | if (NextI != E && NextI->start.isRegister()) { |
| 342 | MachineInstr *DefI = LIS->getInstructionFromIndex(NextI->start); |
| 343 | if (HII->isPredicated(*DefI)) |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 344 | continue; |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 345 | } |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 346 | bool WholeReg = true; |
| 347 | if (LI.hasSubRanges()) { |
| 348 | auto EndsAtI = [I] (LiveInterval::SubRange &S) -> bool { |
| 349 | LiveRange::iterator F = S.find(I->end); |
| 350 | return F != S.end() && I->end == F->end; |
| 351 | }; |
| 352 | // Check if all subranges end at I->end. If so, make sure to kill |
| 353 | // the whole register. |
| 354 | for (LiveInterval::SubRange &S : LI.subranges()) { |
| 355 | if (EndsAtI(S)) |
| 356 | KillAt(I->end, S.LaneMask); |
| 357 | else |
| 358 | WholeReg = false; |
| 359 | } |
| 360 | } |
| 361 | if (WholeReg) |
| 362 | KillAt(I->end, MRI->getMaxLaneMaskForVReg(Reg)); |
| 363 | } |
| 364 | } |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 365 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 366 | void HexagonExpandCondsets::updateDeadsInRange(unsigned Reg, LaneBitmask LM, |
| 367 | LiveRange &Range) { |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 368 | assert(Register::isVirtualRegister(Reg)); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 369 | if (Range.empty()) |
| 370 | return; |
| 371 | |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 372 | // Return two booleans: { def-modifes-reg, def-covers-reg }. |
| 373 | auto IsRegDef = [this,Reg,LM] (MachineOperand &Op) -> std::pair<bool,bool> { |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 374 | if (!Op.isReg() || !Op.isDef()) |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 375 | return { false, false }; |
Daniel Sanders | 0c47611 | 2019-08-15 19:22:08 +0000 | [diff] [blame] | 376 | Register DR = Op.getReg(), DSR = Op.getSubReg(); |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 377 | if (!Register::isVirtualRegister(DR) || DR != Reg) |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 378 | return { false, false }; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 379 | LaneBitmask SLM = getLaneMask(DR, DSR); |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 380 | LaneBitmask A = SLM & LM; |
| 381 | return { A.any(), A == SLM }; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 382 | }; |
| 383 | |
| 384 | // The splitting step will create pairs of predicated definitions without |
| 385 | // any implicit uses (since implicit uses would interfere with predication). |
| 386 | // This can cause the reaching defs to become dead after live range |
| 387 | // recomputation, even though they are not really dead. |
| 388 | // We need to identify predicated defs that need implicit uses, and |
| 389 | // dead defs that are not really dead, and correct both problems. |
| 390 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 391 | auto Dominate = [this] (SetVector<MachineBasicBlock*> &Defs, |
| 392 | MachineBasicBlock *Dest) -> bool { |
| 393 | for (MachineBasicBlock *D : Defs) |
| 394 | if (D != Dest && MDT->dominates(D, Dest)) |
| 395 | return true; |
| 396 | |
| 397 | MachineBasicBlock *Entry = &Dest->getParent()->front(); |
| 398 | SetVector<MachineBasicBlock*> Work(Dest->pred_begin(), Dest->pred_end()); |
| 399 | for (unsigned i = 0; i < Work.size(); ++i) { |
| 400 | MachineBasicBlock *B = Work[i]; |
| 401 | if (Defs.count(B)) |
| 402 | continue; |
| 403 | if (B == Entry) |
| 404 | return false; |
| 405 | for (auto *P : B->predecessors()) |
| 406 | Work.insert(P); |
| 407 | } |
| 408 | return true; |
| 409 | }; |
| 410 | |
| 411 | // First, try to extend live range within individual basic blocks. This |
| 412 | // will leave us only with dead defs that do not reach any predicated |
| 413 | // defs in the same block. |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 414 | SetVector<MachineBasicBlock*> Defs; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 415 | SmallVector<SlotIndex,4> PredDefs; |
| 416 | for (auto &Seg : Range) { |
| 417 | if (!Seg.start.isRegister()) |
| 418 | continue; |
| 419 | MachineInstr *DefI = LIS->getInstructionFromIndex(Seg.start); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 420 | Defs.insert(DefI->getParent()); |
| 421 | if (HII->isPredicated(*DefI)) |
| 422 | PredDefs.push_back(Seg.start); |
| 423 | } |
Krzysztof Parzyszek | a7ed090 | 2016-08-24 13:37:55 +0000 | [diff] [blame] | 424 | |
| 425 | SmallVector<SlotIndex,8> Undefs; |
| 426 | LiveInterval &LI = LIS->getInterval(Reg); |
| 427 | LI.computeSubRangeUndefs(Undefs, LM, *MRI, *LIS->getSlotIndexes()); |
| 428 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 429 | for (auto &SI : PredDefs) { |
| 430 | MachineBasicBlock *BB = LIS->getMBBFromIndex(SI); |
Krzysztof Parzyszek | a7ed090 | 2016-08-24 13:37:55 +0000 | [diff] [blame] | 431 | auto P = Range.extendInBlock(Undefs, LIS->getMBBStartIdx(BB), SI); |
| 432 | if (P.first != nullptr || P.second) |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 433 | SI = SlotIndex(); |
| 434 | } |
| 435 | |
| 436 | // Calculate reachability for those predicated defs that were not handled |
| 437 | // by the in-block extension. |
| 438 | SmallVector<SlotIndex,4> ExtTo; |
| 439 | for (auto &SI : PredDefs) { |
| 440 | if (!SI.isValid()) |
| 441 | continue; |
| 442 | MachineBasicBlock *BB = LIS->getMBBFromIndex(SI); |
| 443 | if (BB->pred_empty()) |
| 444 | continue; |
| 445 | // If the defs from this range reach SI via all predecessors, it is live. |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 446 | // It can happen that SI is reached by the defs through some paths, but |
| 447 | // not all. In the IR coming into this optimization, SI would not be |
| 448 | // considered live, since the defs would then not jointly dominate SI. |
| 449 | // That means that SI is an overwriting def, and no implicit use is |
| 450 | // needed at this point. Do not add SI to the extension points, since |
| 451 | // extendToIndices will abort if there is no joint dominance. |
| 452 | // If the abort was avoided by adding extra undefs added to Undefs, |
| 453 | // extendToIndices could actually indicate that SI is live, contrary |
| 454 | // to the original IR. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 455 | if (Dominate(Defs, BB)) |
| 456 | ExtTo.push_back(SI); |
| 457 | } |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 458 | |
| 459 | if (!ExtTo.empty()) |
| 460 | LIS->extendToIndices(Range, ExtTo, Undefs); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 461 | |
| 462 | // Remove <dead> flags from all defs that are not dead after live range |
| 463 | // extension, and collect all def operands. They will be used to generate |
| 464 | // the necessary implicit uses. |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 465 | // At the same time, add <dead> flag to all defs that are actually dead. |
| 466 | // This can happen, for example, when a mux with identical inputs is |
| 467 | // replaced with a COPY: the use of the predicate register disappears and |
| 468 | // the dead can become dead. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 469 | std::set<RegisterRef> DefRegs; |
| 470 | for (auto &Seg : Range) { |
| 471 | if (!Seg.start.isRegister()) |
| 472 | continue; |
| 473 | MachineInstr *DefI = LIS->getInstructionFromIndex(Seg.start); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 474 | for (auto &Op : DefI->operands()) { |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 475 | auto P = IsRegDef(Op); |
| 476 | if (P.second && Seg.end.isDead()) { |
| 477 | Op.setIsDead(true); |
| 478 | } else if (P.first) { |
| 479 | DefRegs.insert(Op); |
| 480 | Op.setIsDead(false); |
| 481 | } |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 482 | } |
| 483 | } |
| 484 | |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 485 | // Now, add implicit uses to each predicated def that is reached |
Krzysztof Parzyszek | cbd559f | 2016-08-24 16:36:37 +0000 | [diff] [blame] | 486 | // by other defs. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 487 | for (auto &Seg : Range) { |
| 488 | if (!Seg.start.isRegister() || !Range.liveAt(Seg.start.getPrevSlot())) |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 489 | continue; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 490 | MachineInstr *DefI = LIS->getInstructionFromIndex(Seg.start); |
| 491 | if (!HII->isPredicated(*DefI)) |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 492 | continue; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 493 | // Construct the set of all necessary implicit uses, based on the def |
Krzysztof Parzyszek | 688843d | 2017-08-09 19:58:00 +0000 | [diff] [blame] | 494 | // operands in the instruction. We need to tie the implicit uses to |
| 495 | // the corresponding defs. |
| 496 | std::map<RegisterRef,unsigned> ImpUses; |
| 497 | for (unsigned i = 0, e = DefI->getNumOperands(); i != e; ++i) { |
| 498 | MachineOperand &Op = DefI->getOperand(i); |
| 499 | if (!Op.isReg() || !DefRegs.count(Op)) |
| 500 | continue; |
| 501 | if (Op.isDef()) { |
Krzysztof Parzyszek | c052451 | 2018-07-10 12:57:49 +0000 | [diff] [blame] | 502 | // Tied defs will always have corresponding uses, so no extra |
| 503 | // implicit uses are needed. |
| 504 | if (!Op.isTied()) |
| 505 | ImpUses.insert({Op, i}); |
Krzysztof Parzyszek | 688843d | 2017-08-09 19:58:00 +0000 | [diff] [blame] | 506 | } else { |
| 507 | // This function can be called for the same register with different |
| 508 | // lane masks. If the def in this instruction was for the whole |
| 509 | // register, we can get here more than once. Avoid adding multiple |
| 510 | // implicit uses (or adding an implicit use when an explicit one is |
| 511 | // present). |
Krzysztof Parzyszek | c052451 | 2018-07-10 12:57:49 +0000 | [diff] [blame] | 512 | if (Op.isTied()) |
| 513 | ImpUses.erase(Op); |
Krzysztof Parzyszek | 688843d | 2017-08-09 19:58:00 +0000 | [diff] [blame] | 514 | } |
| 515 | } |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 516 | if (ImpUses.empty()) |
| 517 | continue; |
| 518 | MachineFunction &MF = *DefI->getParent()->getParent(); |
Krzysztof Parzyszek | 688843d | 2017-08-09 19:58:00 +0000 | [diff] [blame] | 519 | for (std::pair<RegisterRef, unsigned> P : ImpUses) { |
| 520 | RegisterRef R = P.first; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 521 | MachineInstrBuilder(MF, DefI).addReg(R.Reg, RegState::Implicit, R.Sub); |
Krzysztof Parzyszek | 688843d | 2017-08-09 19:58:00 +0000 | [diff] [blame] | 522 | DefI->tieOperands(P.second, DefI->getNumOperands()-1); |
| 523 | } |
Krzysztof Parzyszek | 8b61759 | 2016-06-07 19:25:28 +0000 | [diff] [blame] | 524 | } |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 525 | } |
| 526 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 527 | void HexagonExpandCondsets::updateDeadFlags(unsigned Reg) { |
| 528 | LiveInterval &LI = LIS->getInterval(Reg); |
| 529 | if (LI.hasSubRanges()) { |
| 530 | for (LiveInterval::SubRange &S : LI.subranges()) { |
| 531 | updateDeadsInRange(Reg, S.LaneMask, S); |
| 532 | LIS->shrinkToUses(S, Reg); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 533 | } |
| 534 | LI.clear(); |
| 535 | LIS->constructMainRangeFromSubranges(LI); |
| 536 | } else { |
| 537 | updateDeadsInRange(Reg, MRI->getMaxLaneMaskForVReg(Reg), LI); |
| 538 | } |
| 539 | } |
| 540 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 541 | void HexagonExpandCondsets::recalculateLiveInterval(unsigned Reg) { |
| 542 | LIS->removeInterval(Reg); |
| 543 | LIS->createAndComputeVirtRegInterval(Reg); |
| 544 | } |
| 545 | |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 546 | void HexagonExpandCondsets::removeInstr(MachineInstr &MI) { |
| 547 | LIS->RemoveMachineInstrFromMaps(MI); |
| 548 | MI.eraseFromParent(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 549 | } |
| 550 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 551 | void HexagonExpandCondsets::updateLiveness(std::set<unsigned> &RegSet, |
| 552 | bool Recalc, bool UpdateKills, bool UpdateDeads) { |
| 553 | UpdateKills |= UpdateDeads; |
Krzysztof Parzyszek | af73d2b | 2018-05-04 13:59:05 +0000 | [diff] [blame] | 554 | for (unsigned R : RegSet) { |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 555 | if (!Register::isVirtualRegister(R)) { |
| 556 | assert(Register::isPhysicalRegister(R)); |
Krzysztof Parzyszek | af73d2b | 2018-05-04 13:59:05 +0000 | [diff] [blame] | 557 | // There shouldn't be any physical registers as operands, except |
| 558 | // possibly reserved registers. |
| 559 | assert(MRI->isReserved(R)); |
| 560 | continue; |
| 561 | } |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 562 | if (Recalc) |
| 563 | recalculateLiveInterval(R); |
| 564 | if (UpdateKills) |
| 565 | MRI->clearKillFlags(R); |
| 566 | if (UpdateDeads) |
| 567 | updateDeadFlags(R); |
| 568 | // Fixing <dead> flags may extend live ranges, so reset <kill> flags |
| 569 | // after that. |
| 570 | if (UpdateKills) |
| 571 | updateKillFlags(R); |
| 572 | LIS->getInterval(R).verify(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 573 | } |
| 574 | } |
| 575 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 576 | /// Get the opcode for a conditional transfer of the value in SO (source |
| 577 | /// operand). The condition (true/false) is given in Cond. |
| 578 | unsigned HexagonExpandCondsets::getCondTfrOpcode(const MachineOperand &SO, |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 579 | bool IfTrue) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 580 | using namespace Hexagon; |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 581 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 582 | if (SO.isReg()) { |
Daniel Sanders | e7694f3 | 2019-08-02 20:23:00 +0000 | [diff] [blame] | 583 | Register PhysR; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 584 | RegisterRef RS = SO; |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 585 | if (Register::isVirtualRegister(RS.Reg)) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 586 | const TargetRegisterClass *VC = MRI->getRegClass(RS.Reg); |
| 587 | assert(VC->begin() != VC->end() && "Empty register class"); |
| 588 | PhysR = *VC->begin(); |
| 589 | } else { |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 590 | assert(Register::isPhysicalRegister(RS.Reg)); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 591 | PhysR = RS.Reg; |
| 592 | } |
Daniel Sanders | 0c47611 | 2019-08-15 19:22:08 +0000 | [diff] [blame] | 593 | Register PhysS = (RS.Sub == 0) ? PhysR : TRI->getSubReg(PhysR, RS.Sub); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 594 | const TargetRegisterClass *RC = TRI->getMinimalPhysRegClass(PhysS); |
Krzysztof Parzyszek | 44e25f3 | 2017-04-24 18:55:33 +0000 | [diff] [blame] | 595 | switch (TRI->getRegSizeInBits(*RC)) { |
| 596 | case 32: |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 597 | return IfTrue ? A2_tfrt : A2_tfrf; |
Krzysztof Parzyszek | 44e25f3 | 2017-04-24 18:55:33 +0000 | [diff] [blame] | 598 | case 64: |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 599 | return IfTrue ? A2_tfrpt : A2_tfrpf; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 600 | } |
| 601 | llvm_unreachable("Invalid register operand"); |
| 602 | } |
Krzysztof Parzyszek | fd048cc | 2017-06-21 19:21:30 +0000 | [diff] [blame] | 603 | switch (SO.getType()) { |
| 604 | case MachineOperand::MO_Immediate: |
| 605 | case MachineOperand::MO_FPImmediate: |
| 606 | case MachineOperand::MO_ConstantPoolIndex: |
| 607 | case MachineOperand::MO_TargetIndex: |
| 608 | case MachineOperand::MO_JumpTableIndex: |
| 609 | case MachineOperand::MO_ExternalSymbol: |
| 610 | case MachineOperand::MO_GlobalAddress: |
| 611 | case MachineOperand::MO_BlockAddress: |
| 612 | return IfTrue ? C2_cmoveit : C2_cmoveif; |
| 613 | default: |
| 614 | break; |
| 615 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 616 | llvm_unreachable("Unexpected source operand"); |
| 617 | } |
| 618 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 619 | /// Generate a conditional transfer, copying the value SrcOp to the |
| 620 | /// destination register DstR:DstSR, and using the predicate register from |
| 621 | /// PredOp. The Cond argument specifies whether the predicate is to be |
| 622 | /// if(PredOp), or if(!PredOp). |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 623 | MachineInstr *HexagonExpandCondsets::genCondTfrFor(MachineOperand &SrcOp, |
| 624 | MachineBasicBlock::iterator At, |
| 625 | unsigned DstR, unsigned DstSR, const MachineOperand &PredOp, |
| 626 | bool PredSense, bool ReadUndef, bool ImpUse) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 627 | MachineInstr *MI = SrcOp.getParent(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 628 | MachineBasicBlock &B = *At->getParent(); |
Benjamin Kramer | 4ca41fd | 2016-06-12 17:30:47 +0000 | [diff] [blame] | 629 | const DebugLoc &DL = MI->getDebugLoc(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 630 | |
| 631 | // Don't avoid identity copies here (i.e. if the source and the destination |
| 632 | // are the same registers). It is actually better to generate them here, |
| 633 | // since this would cause the copy to potentially be predicated in the next |
| 634 | // step. The predication will remove such a copy if it is unable to |
| 635 | /// predicate. |
| 636 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 637 | unsigned Opc = getCondTfrOpcode(SrcOp, PredSense); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 638 | unsigned DstState = RegState::Define | (ReadUndef ? RegState::Undef : 0); |
| 639 | unsigned PredState = getRegState(PredOp) & ~RegState::Kill; |
| 640 | MachineInstrBuilder MIB; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 641 | |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 642 | if (SrcOp.isReg()) { |
| 643 | unsigned SrcState = getRegState(SrcOp); |
| 644 | if (RegisterRef(SrcOp) == RegisterRef(DstR, DstSR)) |
| 645 | SrcState &= ~RegState::Kill; |
| 646 | MIB = BuildMI(B, At, DL, HII->get(Opc)) |
| 647 | .addReg(DstR, DstState, DstSR) |
| 648 | .addReg(PredOp.getReg(), PredState, PredOp.getSubReg()) |
| 649 | .addReg(SrcOp.getReg(), SrcState, SrcOp.getSubReg()); |
| 650 | } else { |
| 651 | MIB = BuildMI(B, At, DL, HII->get(Opc)) |
Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 652 | .addReg(DstR, DstState, DstSR) |
| 653 | .addReg(PredOp.getReg(), PredState, PredOp.getSubReg()) |
| 654 | .add(SrcOp); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 655 | } |
| 656 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 657 | LLVM_DEBUG(dbgs() << "created an initial copy: " << *MIB); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 658 | return &*MIB; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 659 | } |
| 660 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 661 | /// Replace a MUX instruction MI with a pair A2_tfrt/A2_tfrf. This function |
| 662 | /// performs all necessary changes to complete the replacement. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 663 | bool HexagonExpandCondsets::split(MachineInstr &MI, |
| 664 | std::set<unsigned> &UpdRegs) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 665 | if (TfrLimitActive) { |
| 666 | if (TfrCounter >= TfrLimit) |
| 667 | return false; |
| 668 | TfrCounter++; |
| 669 | } |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 670 | LLVM_DEBUG(dbgs() << "\nsplitting " << printMBBReference(*MI.getParent()) |
| 671 | << ": " << MI); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 672 | MachineOperand &MD = MI.getOperand(0); // Definition |
| 673 | MachineOperand &MP = MI.getOperand(1); // Predicate register |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 674 | assert(MD.isDef()); |
Daniel Sanders | 0c47611 | 2019-08-15 19:22:08 +0000 | [diff] [blame] | 675 | Register DR = MD.getReg(), DSR = MD.getSubReg(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 676 | bool ReadUndef = MD.isUndef(); |
| 677 | MachineBasicBlock::iterator At = MI; |
| 678 | |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 679 | auto updateRegs = [&UpdRegs] (const MachineInstr &MI) -> void { |
| 680 | for (auto &Op : MI.operands()) |
| 681 | if (Op.isReg()) |
| 682 | UpdRegs.insert(Op.getReg()); |
| 683 | }; |
| 684 | |
Krzysztof Parzyszek | 22586dc | 2016-10-31 15:45:09 +0000 | [diff] [blame] | 685 | // If this is a mux of the same register, just replace it with COPY. |
| 686 | // Ideally, this would happen earlier, so that register coalescing would |
| 687 | // see it. |
| 688 | MachineOperand &ST = MI.getOperand(2); |
| 689 | MachineOperand &SF = MI.getOperand(3); |
| 690 | if (ST.isReg() && SF.isReg()) { |
| 691 | RegisterRef RT(ST); |
| 692 | if (RT == RegisterRef(SF)) { |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 693 | // Copy regs to update first. |
| 694 | updateRegs(MI); |
Krzysztof Parzyszek | 22586dc | 2016-10-31 15:45:09 +0000 | [diff] [blame] | 695 | MI.setDesc(HII->get(TargetOpcode::COPY)); |
| 696 | unsigned S = getRegState(ST); |
| 697 | while (MI.getNumOperands() > 1) |
| 698 | MI.RemoveOperand(MI.getNumOperands()-1); |
| 699 | MachineFunction &MF = *MI.getParent()->getParent(); |
| 700 | MachineInstrBuilder(MF, MI).addReg(RT.Reg, S, RT.Sub); |
| 701 | return true; |
| 702 | } |
| 703 | } |
| 704 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 705 | // First, create the two invididual conditional transfers, and add each |
| 706 | // of them to the live intervals information. Do that first and then remove |
| 707 | // the old instruction from live intervals. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 708 | MachineInstr *TfrT = |
Krzysztof Parzyszek | 22586dc | 2016-10-31 15:45:09 +0000 | [diff] [blame] | 709 | genCondTfrFor(ST, At, DR, DSR, MP, true, ReadUndef, false); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 710 | MachineInstr *TfrF = |
Krzysztof Parzyszek | 22586dc | 2016-10-31 15:45:09 +0000 | [diff] [blame] | 711 | genCondTfrFor(SF, At, DR, DSR, MP, false, ReadUndef, true); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 712 | LIS->InsertMachineInstrInMaps(*TfrT); |
| 713 | LIS->InsertMachineInstrInMaps(*TfrF); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 714 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 715 | // Will need to recalculate live intervals for all registers in MI. |
Krzysztof Parzyszek | e16ce15 | 2017-03-06 17:09:06 +0000 | [diff] [blame] | 716 | updateRegs(MI); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 717 | |
| 718 | removeInstr(MI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 719 | return true; |
| 720 | } |
| 721 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 722 | bool HexagonExpandCondsets::isPredicable(MachineInstr *MI) { |
Duncan P. N. Exon Smith | 6307eb5 | 2016-02-23 02:46:52 +0000 | [diff] [blame] | 723 | if (HII->isPredicated(*MI) || !HII->isPredicable(*MI)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 724 | return false; |
| 725 | if (MI->hasUnmodeledSideEffects() || MI->mayStore()) |
| 726 | return false; |
| 727 | // Reject instructions with multiple defs (e.g. post-increment loads). |
| 728 | bool HasDef = false; |
| 729 | for (auto &Op : MI->operands()) { |
| 730 | if (!Op.isReg() || !Op.isDef()) |
| 731 | continue; |
| 732 | if (HasDef) |
| 733 | return false; |
| 734 | HasDef = true; |
| 735 | } |
| 736 | for (auto &Mo : MI->memoperands()) |
Philip Reames | 33d7e49 | 2019-02-24 00:45:09 +0000 | [diff] [blame] | 737 | if (Mo->isVolatile() || Mo->isAtomic()) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 738 | return false; |
| 739 | return true; |
| 740 | } |
| 741 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 742 | /// Find the reaching definition for a predicated use of RD. The RD is used |
| 743 | /// under the conditions given by PredR and Cond, and this function will ignore |
| 744 | /// definitions that set RD under the opposite conditions. |
| 745 | MachineInstr *HexagonExpandCondsets::getReachingDefForPred(RegisterRef RD, |
| 746 | MachineBasicBlock::iterator UseIt, unsigned PredR, bool Cond) { |
| 747 | MachineBasicBlock &B = *UseIt->getParent(); |
| 748 | MachineBasicBlock::iterator I = UseIt, S = B.begin(); |
| 749 | if (I == S) |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 750 | return nullptr; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 751 | |
| 752 | bool PredValid = true; |
| 753 | do { |
| 754 | --I; |
| 755 | MachineInstr *MI = &*I; |
| 756 | // Check if this instruction can be ignored, i.e. if it is predicated |
| 757 | // on the complementary condition. |
Duncan P. N. Exon Smith | 6307eb5 | 2016-02-23 02:46:52 +0000 | [diff] [blame] | 758 | if (PredValid && HII->isPredicated(*MI)) { |
| 759 | if (MI->readsRegister(PredR) && (Cond != HII->isPredicatedTrue(*MI))) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 760 | continue; |
| 761 | } |
| 762 | |
| 763 | // Check the defs. If the PredR is defined, invalidate it. If RD is |
| 764 | // defined, return the instruction or 0, depending on the circumstances. |
| 765 | for (auto &Op : MI->operands()) { |
| 766 | if (!Op.isReg() || !Op.isDef()) |
| 767 | continue; |
| 768 | RegisterRef RR = Op; |
| 769 | if (RR.Reg == PredR) { |
| 770 | PredValid = false; |
| 771 | continue; |
| 772 | } |
| 773 | if (RR.Reg != RD.Reg) |
| 774 | continue; |
| 775 | // If the "Reg" part agrees, there is still the subregister to check. |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 776 | // If we are looking for %1:loreg, we can skip %1:hireg, but |
| 777 | // not %1 (w/o subregisters). |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 778 | if (RR.Sub == RD.Sub) |
| 779 | return MI; |
| 780 | if (RR.Sub == 0 || RD.Sub == 0) |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 781 | return nullptr; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 782 | // We have different subregisters, so we can continue looking. |
| 783 | } |
| 784 | } while (I != S); |
| 785 | |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 786 | return nullptr; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 787 | } |
| 788 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 789 | /// Check if the instruction MI can be safely moved over a set of instructions |
| 790 | /// whose side-effects (in terms of register defs and uses) are expressed in |
| 791 | /// the maps Defs and Uses. These maps reflect the conditional defs and uses |
| 792 | /// that depend on the same predicate register to allow moving instructions |
| 793 | /// over instructions predicated on the opposite condition. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 794 | bool HexagonExpandCondsets::canMoveOver(MachineInstr &MI, ReferenceMap &Defs, |
| 795 | ReferenceMap &Uses) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 796 | // In order to be able to safely move MI over instructions that define |
| 797 | // "Defs" and use "Uses", no def operand from MI can be defined or used |
| 798 | // and no use operand can be defined. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 799 | for (auto &Op : MI.operands()) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 800 | if (!Op.isReg()) |
| 801 | continue; |
| 802 | RegisterRef RR = Op; |
| 803 | // For physical register we would need to check register aliases, etc. |
| 804 | // and we don't want to bother with that. It would be of little value |
| 805 | // before the actual register rewriting (from virtual to physical). |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 806 | if (!Register::isVirtualRegister(RR.Reg)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 807 | return false; |
| 808 | // No redefs for any operand. |
| 809 | if (isRefInMap(RR, Defs, Exec_Then)) |
| 810 | return false; |
| 811 | // For defs, there cannot be uses. |
| 812 | if (Op.isDef() && isRefInMap(RR, Uses, Exec_Then)) |
| 813 | return false; |
| 814 | } |
| 815 | return true; |
| 816 | } |
| 817 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 818 | /// Check if the instruction accessing memory (TheI) can be moved to the |
| 819 | /// location ToI. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 820 | bool HexagonExpandCondsets::canMoveMemTo(MachineInstr &TheI, MachineInstr &ToI, |
| 821 | bool IsDown) { |
| 822 | bool IsLoad = TheI.mayLoad(), IsStore = TheI.mayStore(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 823 | if (!IsLoad && !IsStore) |
| 824 | return true; |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 825 | if (HII->areMemAccessesTriviallyDisjoint(TheI, ToI)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 826 | return true; |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 827 | if (TheI.hasUnmodeledSideEffects()) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 828 | return false; |
| 829 | |
| 830 | MachineBasicBlock::iterator StartI = IsDown ? TheI : ToI; |
| 831 | MachineBasicBlock::iterator EndI = IsDown ? ToI : TheI; |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 832 | bool Ordered = TheI.hasOrderedMemoryRef(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 833 | |
| 834 | // Search for aliased memory reference in (StartI, EndI). |
| 835 | for (MachineBasicBlock::iterator I = std::next(StartI); I != EndI; ++I) { |
| 836 | MachineInstr *MI = &*I; |
| 837 | if (MI->hasUnmodeledSideEffects()) |
| 838 | return false; |
| 839 | bool L = MI->mayLoad(), S = MI->mayStore(); |
| 840 | if (!L && !S) |
| 841 | continue; |
| 842 | if (Ordered && MI->hasOrderedMemoryRef()) |
| 843 | return false; |
| 844 | |
| 845 | bool Conflict = (L && IsStore) || S; |
| 846 | if (Conflict) |
| 847 | return false; |
| 848 | } |
| 849 | return true; |
| 850 | } |
| 851 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 852 | /// Generate a predicated version of MI (where the condition is given via |
| 853 | /// PredR and Cond) at the point indicated by Where. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 854 | void HexagonExpandCondsets::predicateAt(const MachineOperand &DefOp, |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 855 | MachineInstr &MI, |
| 856 | MachineBasicBlock::iterator Where, |
| 857 | const MachineOperand &PredOp, bool Cond, |
| 858 | std::set<unsigned> &UpdRegs) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 859 | // The problem with updating live intervals is that we can move one def |
| 860 | // past another def. In particular, this can happen when moving an A2_tfrt |
| 861 | // over an A2_tfrf defining the same register. From the point of view of |
| 862 | // live intervals, these two instructions are two separate definitions, |
| 863 | // and each one starts another live segment. LiveIntervals's "handleMove" |
| 864 | // does not allow such moves, so we need to handle it ourselves. To avoid |
| 865 | // invalidating liveness data while we are using it, the move will be |
| 866 | // implemented in 4 steps: (1) add a clone of the instruction MI at the |
| 867 | // target location, (2) update liveness, (3) delete the old instruction, |
| 868 | // and (4) update liveness again. |
| 869 | |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 870 | MachineBasicBlock &B = *MI.getParent(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 871 | DebugLoc DL = Where->getDebugLoc(); // "Where" points to an instruction. |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 872 | unsigned Opc = MI.getOpcode(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 873 | unsigned PredOpc = HII->getCondOpcode(Opc, !Cond); |
| 874 | MachineInstrBuilder MB = BuildMI(B, Where, DL, HII->get(PredOpc)); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 875 | unsigned Ox = 0, NP = MI.getNumOperands(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 876 | // Skip all defs from MI first. |
| 877 | while (Ox < NP) { |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 878 | MachineOperand &MO = MI.getOperand(Ox); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 879 | if (!MO.isReg() || !MO.isDef()) |
| 880 | break; |
| 881 | Ox++; |
| 882 | } |
| 883 | // Add the new def, then the predicate register, then the rest of the |
| 884 | // operands. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 885 | MB.addReg(DefOp.getReg(), getRegState(DefOp), DefOp.getSubReg()); |
| 886 | MB.addReg(PredOp.getReg(), PredOp.isUndef() ? RegState::Undef : 0, |
| 887 | PredOp.getSubReg()); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 888 | while (Ox < NP) { |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 889 | MachineOperand &MO = MI.getOperand(Ox); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 890 | if (!MO.isReg() || !MO.isImplicit()) |
Diana Picus | 116bbab | 2017-01-13 09:58:52 +0000 | [diff] [blame] | 891 | MB.add(MO); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 892 | Ox++; |
| 893 | } |
Chandler Carruth | c73c030 | 2018-08-16 21:30:05 +0000 | [diff] [blame] | 894 | MB.cloneMemRefs(MI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 895 | |
| 896 | MachineInstr *NewI = MB; |
| 897 | NewI->clearKillInfo(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 898 | LIS->InsertMachineInstrInMaps(*NewI); |
| 899 | |
| 900 | for (auto &Op : NewI->operands()) |
| 901 | if (Op.isReg()) |
| 902 | UpdRegs.insert(Op.getReg()); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 903 | } |
| 904 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 905 | /// In the range [First, Last], rename all references to the "old" register RO |
| 906 | /// to the "new" register RN, but only in instructions predicated on the given |
| 907 | /// condition. |
| 908 | void HexagonExpandCondsets::renameInRange(RegisterRef RO, RegisterRef RN, |
| 909 | unsigned PredR, bool Cond, MachineBasicBlock::iterator First, |
| 910 | MachineBasicBlock::iterator Last) { |
| 911 | MachineBasicBlock::iterator End = std::next(Last); |
| 912 | for (MachineBasicBlock::iterator I = First; I != End; ++I) { |
| 913 | MachineInstr *MI = &*I; |
| 914 | // Do not touch instructions that are not predicated, or are predicated |
| 915 | // on the opposite condition. |
Duncan P. N. Exon Smith | 6307eb5 | 2016-02-23 02:46:52 +0000 | [diff] [blame] | 916 | if (!HII->isPredicated(*MI)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 917 | continue; |
Duncan P. N. Exon Smith | 6307eb5 | 2016-02-23 02:46:52 +0000 | [diff] [blame] | 918 | if (!MI->readsRegister(PredR) || (Cond != HII->isPredicatedTrue(*MI))) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 919 | continue; |
| 920 | |
| 921 | for (auto &Op : MI->operands()) { |
| 922 | if (!Op.isReg() || RO != RegisterRef(Op)) |
| 923 | continue; |
| 924 | Op.setReg(RN.Reg); |
| 925 | Op.setSubReg(RN.Sub); |
| 926 | // In practice, this isn't supposed to see any defs. |
| 927 | assert(!Op.isDef() && "Not expecting a def"); |
| 928 | } |
| 929 | } |
| 930 | } |
| 931 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 932 | /// For a given conditional copy, predicate the definition of the source of |
| 933 | /// the copy under the given condition (using the same predicate register as |
| 934 | /// the copy). |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 935 | bool HexagonExpandCondsets::predicate(MachineInstr &TfrI, bool Cond, |
| 936 | std::set<unsigned> &UpdRegs) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 937 | // TfrI - A2_tfr[tf] Instruction (not A2_tfrsi). |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 938 | unsigned Opc = TfrI.getOpcode(); |
Simon Atanasyan | 772944a | 2015-03-31 19:43:47 +0000 | [diff] [blame] | 939 | (void)Opc; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 940 | assert(Opc == Hexagon::A2_tfrt || Opc == Hexagon::A2_tfrf); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 941 | LLVM_DEBUG(dbgs() << "\nattempt to predicate if-" << (Cond ? "true" : "false") |
| 942 | << ": " << TfrI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 943 | |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 944 | MachineOperand &MD = TfrI.getOperand(0); |
| 945 | MachineOperand &MP = TfrI.getOperand(1); |
| 946 | MachineOperand &MS = TfrI.getOperand(2); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 947 | // The source operand should be a <kill>. This is not strictly necessary, |
| 948 | // but it makes things a lot simpler. Otherwise, we would need to rename |
| 949 | // some registers, which would complicate the transformation considerably. |
| 950 | if (!MS.isKill()) |
| 951 | return false; |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 952 | // Avoid predicating instructions that define a subregister if subregister |
| 953 | // liveness tracking is not enabled. |
| 954 | if (MD.getSubReg() && !MRI->shouldTrackSubRegLiveness(MD.getReg())) |
Krzysztof Parzyszek | a580273 | 2016-05-31 14:27:10 +0000 | [diff] [blame] | 955 | return false; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 956 | |
| 957 | RegisterRef RT(MS); |
Daniel Sanders | 0c47611 | 2019-08-15 19:22:08 +0000 | [diff] [blame] | 958 | Register PredR = MP.getReg(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 959 | MachineInstr *DefI = getReachingDefForPred(RT, TfrI, PredR, Cond); |
| 960 | if (!DefI || !isPredicable(DefI)) |
| 961 | return false; |
| 962 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 963 | LLVM_DEBUG(dbgs() << "Source def: " << *DefI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 964 | |
| 965 | // Collect the information about registers defined and used between the |
| 966 | // DefI and the TfrI. |
| 967 | // Map: reg -> bitmask of subregs |
| 968 | ReferenceMap Uses, Defs; |
| 969 | MachineBasicBlock::iterator DefIt = DefI, TfrIt = TfrI; |
| 970 | |
| 971 | // Check if the predicate register is valid between DefI and TfrI. |
| 972 | // If it is, we can then ignore instructions predicated on the negated |
| 973 | // conditions when collecting def and use information. |
| 974 | bool PredValid = true; |
| 975 | for (MachineBasicBlock::iterator I = std::next(DefIt); I != TfrIt; ++I) { |
Eugene Zelenko | f9f8c68 | 2016-12-14 22:50:46 +0000 | [diff] [blame] | 976 | if (!I->modifiesRegister(PredR, nullptr)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 977 | continue; |
| 978 | PredValid = false; |
| 979 | break; |
| 980 | } |
| 981 | |
| 982 | for (MachineBasicBlock::iterator I = std::next(DefIt); I != TfrIt; ++I) { |
| 983 | MachineInstr *MI = &*I; |
| 984 | // If this instruction is predicated on the same register, it could |
| 985 | // potentially be ignored. |
| 986 | // By default assume that the instruction executes on the same condition |
| 987 | // as TfrI (Exec_Then), and also on the opposite one (Exec_Else). |
| 988 | unsigned Exec = Exec_Then | Exec_Else; |
Duncan P. N. Exon Smith | 6307eb5 | 2016-02-23 02:46:52 +0000 | [diff] [blame] | 989 | if (PredValid && HII->isPredicated(*MI) && MI->readsRegister(PredR)) |
| 990 | Exec = (Cond == HII->isPredicatedTrue(*MI)) ? Exec_Then : Exec_Else; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 991 | |
| 992 | for (auto &Op : MI->operands()) { |
| 993 | if (!Op.isReg()) |
| 994 | continue; |
| 995 | // We don't want to deal with physical registers. The reason is that |
| 996 | // they can be aliased with other physical registers. Aliased virtual |
| 997 | // registers must share the same register number, and can only differ |
| 998 | // in the subregisters, which we are keeping track of. Physical |
| 999 | // registers ters no longer have subregisters---their super- and |
| 1000 | // subregisters are other physical registers, and we are not checking |
| 1001 | // that. |
| 1002 | RegisterRef RR = Op; |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 1003 | if (!Register::isVirtualRegister(RR.Reg)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1004 | return false; |
| 1005 | |
| 1006 | ReferenceMap &Map = Op.isDef() ? Defs : Uses; |
Krzysztof Parzyszek | b7eb7fc | 2016-11-04 20:41:03 +0000 | [diff] [blame] | 1007 | if (Op.isDef() && Op.isUndef()) { |
| 1008 | assert(RR.Sub && "Expecting a subregister on <def,read-undef>"); |
| 1009 | // If this is a <def,read-undef>, then it invalidates the non-written |
| 1010 | // part of the register. For the purpose of checking the validity of |
| 1011 | // the move, assume that it modifies the whole register. |
| 1012 | RR.Sub = 0; |
| 1013 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1014 | addRefToMap(RR, Map, Exec); |
| 1015 | } |
| 1016 | } |
| 1017 | |
| 1018 | // The situation: |
| 1019 | // RT = DefI |
| 1020 | // ... |
| 1021 | // RD = TfrI ..., RT |
| 1022 | |
| 1023 | // If the register-in-the-middle (RT) is used or redefined between |
| 1024 | // DefI and TfrI, we may not be able proceed with this transformation. |
| 1025 | // We can ignore a def that will not execute together with TfrI, and a |
| 1026 | // use that will. If there is such a use (that does execute together with |
| 1027 | // TfrI), we will not be able to move DefI down. If there is a use that |
| 1028 | // executed if TfrI's condition is false, then RT must be available |
| 1029 | // unconditionally (cannot be predicated). |
| 1030 | // Essentially, we need to be able to rename RT to RD in this segment. |
| 1031 | if (isRefInMap(RT, Defs, Exec_Then) || isRefInMap(RT, Uses, Exec_Else)) |
| 1032 | return false; |
| 1033 | RegisterRef RD = MD; |
| 1034 | // If the predicate register is defined between DefI and TfrI, the only |
| 1035 | // potential thing to do would be to move the DefI down to TfrI, and then |
| 1036 | // predicate. The reaching def (DefI) must be movable down to the location |
| 1037 | // of the TfrI. |
| 1038 | // If the target register of the TfrI (RD) is not used or defined between |
| 1039 | // DefI and TfrI, consider moving TfrI up to DefI. |
| 1040 | bool CanUp = canMoveOver(TfrI, Defs, Uses); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1041 | bool CanDown = canMoveOver(*DefI, Defs, Uses); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1042 | // The TfrI does not access memory, but DefI could. Check if it's safe |
| 1043 | // to move DefI down to TfrI. |
| 1044 | if (DefI->mayLoad() || DefI->mayStore()) |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1045 | if (!canMoveMemTo(*DefI, TfrI, true)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1046 | CanDown = false; |
| 1047 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1048 | LLVM_DEBUG(dbgs() << "Can move up: " << (CanUp ? "yes" : "no") |
| 1049 | << ", can move down: " << (CanDown ? "yes\n" : "no\n")); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1050 | MachineBasicBlock::iterator PastDefIt = std::next(DefIt); |
| 1051 | if (CanUp) |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1052 | predicateAt(MD, *DefI, PastDefIt, MP, Cond, UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1053 | else if (CanDown) |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1054 | predicateAt(MD, *DefI, TfrIt, MP, Cond, UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1055 | else |
| 1056 | return false; |
| 1057 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1058 | if (RT != RD) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1059 | renameInRange(RT, RD, PredR, Cond, PastDefIt, TfrIt); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1060 | UpdRegs.insert(RT.Reg); |
| 1061 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1062 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1063 | removeInstr(TfrI); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1064 | removeInstr(*DefI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1065 | return true; |
| 1066 | } |
| 1067 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1068 | /// Predicate all cases of conditional copies in the specified block. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1069 | bool HexagonExpandCondsets::predicateInBlock(MachineBasicBlock &B, |
| 1070 | std::set<unsigned> &UpdRegs) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1071 | bool Changed = false; |
| 1072 | MachineBasicBlock::iterator I, E, NextI; |
| 1073 | for (I = B.begin(), E = B.end(); I != E; I = NextI) { |
| 1074 | NextI = std::next(I); |
| 1075 | unsigned Opc = I->getOpcode(); |
| 1076 | if (Opc == Hexagon::A2_tfrt || Opc == Hexagon::A2_tfrf) { |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1077 | bool Done = predicate(*I, (Opc == Hexagon::A2_tfrt), UpdRegs); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1078 | if (!Done) { |
| 1079 | // If we didn't predicate I, we may need to remove it in case it is |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 1080 | // an "identity" copy, e.g. %1 = A2_tfrt %2, %1. |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1081 | if (RegisterRef(I->getOperand(0)) == RegisterRef(I->getOperand(2))) { |
| 1082 | for (auto &Op : I->operands()) |
| 1083 | if (Op.isReg()) |
| 1084 | UpdRegs.insert(Op.getReg()); |
Duncan P. N. Exon Smith | 98226e3 | 2016-07-12 01:55:32 +0000 | [diff] [blame] | 1085 | removeInstr(*I); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1086 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1087 | } |
| 1088 | Changed |= Done; |
| 1089 | } |
| 1090 | } |
| 1091 | return Changed; |
| 1092 | } |
| 1093 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1094 | bool HexagonExpandCondsets::isIntReg(RegisterRef RR, unsigned &BW) { |
Daniel Sanders | 2bea69b | 2019-08-01 23:27:28 +0000 | [diff] [blame] | 1095 | if (!Register::isVirtualRegister(RR.Reg)) |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1096 | return false; |
| 1097 | const TargetRegisterClass *RC = MRI->getRegClass(RR.Reg); |
| 1098 | if (RC == &Hexagon::IntRegsRegClass) { |
| 1099 | BW = 32; |
| 1100 | return true; |
| 1101 | } |
| 1102 | if (RC == &Hexagon::DoubleRegsRegClass) { |
| 1103 | BW = (RR.Sub != 0) ? 32 : 64; |
| 1104 | return true; |
| 1105 | } |
| 1106 | return false; |
| 1107 | } |
| 1108 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1109 | bool HexagonExpandCondsets::isIntraBlocks(LiveInterval &LI) { |
| 1110 | for (LiveInterval::iterator I = LI.begin(), E = LI.end(); I != E; ++I) { |
| 1111 | LiveRange::Segment &LR = *I; |
| 1112 | // Range must start at a register... |
| 1113 | if (!LR.start.isRegister()) |
| 1114 | return false; |
| 1115 | // ...and end in a register or in a dead slot. |
| 1116 | if (!LR.end.isRegister() && !LR.end.isDead()) |
| 1117 | return false; |
| 1118 | } |
| 1119 | return true; |
| 1120 | } |
| 1121 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1122 | bool HexagonExpandCondsets::coalesceRegisters(RegisterRef R1, RegisterRef R2) { |
| 1123 | if (CoaLimitActive) { |
| 1124 | if (CoaCounter >= CoaLimit) |
| 1125 | return false; |
| 1126 | CoaCounter++; |
| 1127 | } |
| 1128 | unsigned BW1, BW2; |
| 1129 | if (!isIntReg(R1, BW1) || !isIntReg(R2, BW2) || BW1 != BW2) |
| 1130 | return false; |
| 1131 | if (MRI->isLiveIn(R1.Reg)) |
| 1132 | return false; |
| 1133 | if (MRI->isLiveIn(R2.Reg)) |
| 1134 | return false; |
| 1135 | |
| 1136 | LiveInterval &L1 = LIS->getInterval(R1.Reg); |
| 1137 | LiveInterval &L2 = LIS->getInterval(R2.Reg); |
Krzysztof Parzyszek | 66dd679 | 2016-08-19 14:29:43 +0000 | [diff] [blame] | 1138 | if (L2.empty()) |
| 1139 | return false; |
Krzysztof Parzyszek | ead7701 | 2016-11-02 17:59:54 +0000 | [diff] [blame] | 1140 | if (L1.hasSubRanges() || L2.hasSubRanges()) |
| 1141 | return false; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1142 | bool Overlap = L1.overlaps(L2); |
| 1143 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1144 | LLVM_DEBUG(dbgs() << "compatible registers: (" |
| 1145 | << (Overlap ? "overlap" : "disjoint") << ")\n " |
| 1146 | << printReg(R1.Reg, TRI, R1.Sub) << " " << L1 << "\n " |
| 1147 | << printReg(R2.Reg, TRI, R2.Sub) << " " << L2 << "\n"); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1148 | if (R1.Sub || R2.Sub) |
| 1149 | return false; |
| 1150 | if (Overlap) |
| 1151 | return false; |
| 1152 | |
| 1153 | // Coalescing could have a negative impact on scheduling, so try to limit |
| 1154 | // to some reasonable extent. Only consider coalescing segments, when one |
| 1155 | // of them does not cross basic block boundaries. |
| 1156 | if (!isIntraBlocks(L1) && !isIntraBlocks(L2)) |
| 1157 | return false; |
| 1158 | |
| 1159 | MRI->replaceRegWith(R2.Reg, R1.Reg); |
| 1160 | |
| 1161 | // Move all live segments from L2 to L1. |
Eugene Zelenko | e4fc6ee | 2017-07-26 23:20:35 +0000 | [diff] [blame] | 1162 | using ValueInfoMap = DenseMap<VNInfo *, VNInfo *>; |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1163 | ValueInfoMap VM; |
| 1164 | for (LiveInterval::iterator I = L2.begin(), E = L2.end(); I != E; ++I) { |
| 1165 | VNInfo *NewVN, *OldVN = I->valno; |
| 1166 | ValueInfoMap::iterator F = VM.find(OldVN); |
| 1167 | if (F == VM.end()) { |
| 1168 | NewVN = L1.getNextValue(I->valno->def, LIS->getVNInfoAllocator()); |
| 1169 | VM.insert(std::make_pair(OldVN, NewVN)); |
| 1170 | } else { |
| 1171 | NewVN = F->second; |
| 1172 | } |
| 1173 | L1.addSegment(LiveRange::Segment(I->start, I->end, NewVN)); |
| 1174 | } |
| 1175 | while (L2.begin() != L2.end()) |
| 1176 | L2.removeSegment(*L2.begin()); |
Krzysztof Parzyszek | ead7701 | 2016-11-02 17:59:54 +0000 | [diff] [blame] | 1177 | LIS->removeInterval(R2.Reg); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1178 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1179 | updateKillFlags(R1.Reg); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1180 | LLVM_DEBUG(dbgs() << "coalesced: " << L1 << "\n"); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1181 | L1.verify(); |
| 1182 | |
| 1183 | return true; |
| 1184 | } |
| 1185 | |
Simon Pilgrim | 6ba672e | 2016-11-17 19:21:20 +0000 | [diff] [blame] | 1186 | /// Attempt to coalesce one of the source registers to a MUX instruction with |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1187 | /// the destination register. This could lead to having only one predicated |
| 1188 | /// instruction in the end instead of two. |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1189 | bool HexagonExpandCondsets::coalesceSegments( |
| 1190 | const SmallVectorImpl<MachineInstr*> &Condsets, |
| 1191 | std::set<unsigned> &UpdRegs) { |
| 1192 | SmallVector<MachineInstr*,16> TwoRegs; |
| 1193 | for (MachineInstr *MI : Condsets) { |
| 1194 | MachineOperand &S1 = MI->getOperand(2), &S2 = MI->getOperand(3); |
| 1195 | if (!S1.isReg() && !S2.isReg()) |
| 1196 | continue; |
| 1197 | TwoRegs.push_back(MI); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1198 | } |
| 1199 | |
| 1200 | bool Changed = false; |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1201 | for (MachineInstr *CI : TwoRegs) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1202 | RegisterRef RD = CI->getOperand(0); |
| 1203 | RegisterRef RP = CI->getOperand(1); |
| 1204 | MachineOperand &S1 = CI->getOperand(2), &S2 = CI->getOperand(3); |
| 1205 | bool Done = false; |
| 1206 | // Consider this case: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 1207 | // %1 = instr1 ... |
| 1208 | // %2 = instr2 ... |
| 1209 | // %0 = C2_mux ..., %1, %2 |
| 1210 | // If %0 was coalesced with %1, we could end up with the following |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1211 | // code: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 1212 | // %0 = instr1 ... |
| 1213 | // %2 = instr2 ... |
| 1214 | // %0 = A2_tfrf ..., %2 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1215 | // which will later become: |
Francis Visoiu Mistrih | 93ef145 | 2017-11-30 12:12:19 +0000 | [diff] [blame] | 1216 | // %0 = instr1 ... |
| 1217 | // %0 = instr2_cNotPt ... |
| 1218 | // i.e. there will be an unconditional definition (instr1) of %0 |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1219 | // followed by a conditional one. The output dependency was there before |
| 1220 | // and it unavoidable, but if instr1 is predicable, we will no longer be |
| 1221 | // able to predicate it here. |
| 1222 | // To avoid this scenario, don't coalesce the destination register with |
| 1223 | // a source register that is defined by a predicable instruction. |
| 1224 | if (S1.isReg()) { |
| 1225 | RegisterRef RS = S1; |
| 1226 | MachineInstr *RDef = getReachingDefForPred(RS, CI, RP.Reg, true); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1227 | if (!RDef || !HII->isPredicable(*RDef)) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1228 | Done = coalesceRegisters(RD, RegisterRef(S1)); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1229 | if (Done) { |
| 1230 | UpdRegs.insert(RD.Reg); |
| 1231 | UpdRegs.insert(S1.getReg()); |
| 1232 | } |
| 1233 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1234 | } |
| 1235 | if (!Done && S2.isReg()) { |
| 1236 | RegisterRef RS = S2; |
| 1237 | MachineInstr *RDef = getReachingDefForPred(RS, CI, RP.Reg, false); |
Kirill Bobyrev | 1f17511 | 2016-11-02 10:00:40 +0000 | [diff] [blame] | 1238 | if (!RDef || !HII->isPredicable(*RDef)) { |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1239 | Done = coalesceRegisters(RD, RegisterRef(S2)); |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1240 | if (Done) { |
| 1241 | UpdRegs.insert(RD.Reg); |
| 1242 | UpdRegs.insert(S2.getReg()); |
| 1243 | } |
Kirill Bobyrev | 1f17511 | 2016-11-02 10:00:40 +0000 | [diff] [blame] | 1244 | } |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1245 | } |
| 1246 | Changed |= Done; |
| 1247 | } |
| 1248 | return Changed; |
| 1249 | } |
| 1250 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1251 | bool HexagonExpandCondsets::runOnMachineFunction(MachineFunction &MF) { |
Matthias Braun | f1caa28 | 2017-12-15 22:22:58 +0000 | [diff] [blame] | 1252 | if (skipFunction(MF.getFunction())) |
Andrew Kaylor | 5b444a2 | 2016-04-26 19:46:28 +0000 | [diff] [blame] | 1253 | return false; |
| 1254 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1255 | HII = static_cast<const HexagonInstrInfo*>(MF.getSubtarget().getInstrInfo()); |
| 1256 | TRI = MF.getSubtarget().getRegisterInfo(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1257 | MDT = &getAnalysis<MachineDominatorTree>(); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1258 | LIS = &getAnalysis<LiveIntervals>(); |
| 1259 | MRI = &MF.getRegInfo(); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1260 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1261 | LLVM_DEBUG(LIS->print(dbgs() << "Before expand-condsets\n", |
| 1262 | MF.getFunction().getParent())); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1263 | |
| 1264 | bool Changed = false; |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1265 | std::set<unsigned> CoalUpd, PredUpd; |
| 1266 | |
| 1267 | SmallVector<MachineInstr*,16> Condsets; |
| 1268 | for (auto &B : MF) |
| 1269 | for (auto &I : B) |
| 1270 | if (isCondset(I)) |
| 1271 | Condsets.push_back(&I); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1272 | |
| 1273 | // Try to coalesce the target of a mux with one of its sources. |
| 1274 | // This could eliminate a register copy in some circumstances. |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1275 | Changed |= coalesceSegments(Condsets, CoalUpd); |
| 1276 | |
| 1277 | // Update kill flags on all source operands. This is done here because |
| 1278 | // at this moment (when expand-condsets runs), there are no kill flags |
| 1279 | // in the IR (they have been removed by live range analysis). |
| 1280 | // Updating them right before we split is the easiest, because splitting |
| 1281 | // adds definitions which would interfere with updating kills afterwards. |
| 1282 | std::set<unsigned> KillUpd; |
| 1283 | for (MachineInstr *MI : Condsets) |
| 1284 | for (MachineOperand &Op : MI->operands()) |
| 1285 | if (Op.isReg() && Op.isUse()) |
| 1286 | if (!CoalUpd.count(Op.getReg())) |
| 1287 | KillUpd.insert(Op.getReg()); |
| 1288 | updateLiveness(KillUpd, false, true, false); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1289 | LLVM_DEBUG( |
| 1290 | LIS->print(dbgs() << "After coalescing\n", MF.getFunction().getParent())); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1291 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1292 | // First, simply split all muxes into a pair of conditional transfers |
| 1293 | // and update the live intervals to reflect the new arrangement. The |
| 1294 | // goal is to update the kill flags, since predication will rely on |
| 1295 | // them. |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1296 | for (MachineInstr *MI : Condsets) |
| 1297 | Changed |= split(*MI, PredUpd); |
| 1298 | Condsets.clear(); // The contents of Condsets are invalid here anyway. |
| 1299 | |
| 1300 | // Do not update live ranges after splitting. Recalculation of live |
| 1301 | // intervals removes kill flags, which were preserved by splitting on |
| 1302 | // the source operands of condsets. These kill flags are needed by |
| 1303 | // predication, and after splitting they are difficult to recalculate |
| 1304 | // (because of predicated defs), so make sure they are left untouched. |
| 1305 | // Predication does not use live intervals. |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1306 | LLVM_DEBUG( |
| 1307 | LIS->print(dbgs() << "After splitting\n", MF.getFunction().getParent())); |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1308 | |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1309 | // Traverse all blocks and collapse predicable instructions feeding |
| 1310 | // conditional transfers into predicated instructions. |
| 1311 | // Walk over all the instructions again, so we may catch pre-existing |
| 1312 | // cases that were not created in the previous step. |
| 1313 | for (auto &B : MF) |
| 1314 | Changed |= predicateInBlock(B, PredUpd); |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1315 | LLVM_DEBUG(LIS->print(dbgs() << "After predicating\n", |
| 1316 | MF.getFunction().getParent())); |
Krzysztof Parzyszek | 9062b75 | 2016-04-22 16:47:01 +0000 | [diff] [blame] | 1317 | |
Krzysztof Parzyszek | 87a47be | 2016-10-28 15:50:22 +0000 | [diff] [blame] | 1318 | PredUpd.insert(CoalUpd.begin(), CoalUpd.end()); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1319 | updateLiveness(PredUpd, true, true, true); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1320 | |
Nicola Zaghen | d34e60c | 2018-05-14 12:53:11 +0000 | [diff] [blame] | 1321 | LLVM_DEBUG({ |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1322 | if (Changed) |
| 1323 | LIS->print(dbgs() << "After expand-condsets\n", |
Matthias Braun | f1caa28 | 2017-12-15 22:22:58 +0000 | [diff] [blame] | 1324 | MF.getFunction().getParent()); |
Krzysztof Parzyszek | b16882d | 2016-06-08 12:31:16 +0000 | [diff] [blame] | 1325 | }); |
Krzysztof Parzyszek | 9062b75 | 2016-04-22 16:47:01 +0000 | [diff] [blame] | 1326 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1327 | return Changed; |
| 1328 | } |
| 1329 | |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1330 | //===----------------------------------------------------------------------===// |
| 1331 | // Public Constructor Functions |
| 1332 | //===----------------------------------------------------------------------===// |
Krzysztof Parzyszek | c05dff1 | 2015-03-31 13:35:12 +0000 | [diff] [blame] | 1333 | FunctionPass *llvm::createHexagonExpandCondsets() { |
| 1334 | return new HexagonExpandCondsets(); |
| 1335 | } |