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