121
|
1 //===- CalcSpillWeights.cpp -----------------------------------------------===//
|
0
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 #include "llvm/CodeGen/CalcSpillWeights.h"
|
121
|
11 #include "llvm/ADT/SmallPtrSet.h"
|
|
12 #include "llvm/CodeGen/LiveInterval.h"
|
134
|
13 #include "llvm/CodeGen/LiveIntervals.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
14 #include "llvm/CodeGen/MachineFunction.h"
|
121
|
15 #include "llvm/CodeGen/MachineInstr.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
16 #include "llvm/CodeGen/MachineLoopInfo.h"
|
121
|
17 #include "llvm/CodeGen/MachineOperand.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
18 #include "llvm/CodeGen/MachineRegisterInfo.h"
|
134
|
19 #include "llvm/CodeGen/TargetInstrInfo.h"
|
|
20 #include "llvm/CodeGen/TargetRegisterInfo.h"
|
|
21 #include "llvm/CodeGen/TargetSubtargetInfo.h"
|
121
|
22 #include "llvm/CodeGen/VirtRegMap.h"
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
23 #include "llvm/Support/Debug.h"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
24 #include "llvm/Support/raw_ostream.h"
|
121
|
25 #include <cassert>
|
|
26 #include <tuple>
|
|
27
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
28 using namespace llvm;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
29
|
77
|
30 #define DEBUG_TYPE "calcspillweights"
|
|
31
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
32 void llvm::calculateSpillWeightsAndHints(LiveIntervals &LIS,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
33 MachineFunction &MF,
|
95
|
34 VirtRegMap *VRM,
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
35 const MachineLoopInfo &MLI,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
36 const MachineBlockFrequencyInfo &MBFI,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
37 VirtRegAuxInfo::NormalizingFn norm) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
38 DEBUG(dbgs() << "********** Compute Spill Weights **********\n"
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
39 << "********** Function: " << MF.getName() << '\n');
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
40
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
41 MachineRegisterInfo &MRI = MF.getRegInfo();
|
95
|
42 VirtRegAuxInfo VRAI(MF, LIS, VRM, MLI, MBFI, norm);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
43 for (unsigned i = 0, e = MRI.getNumVirtRegs(); i != e; ++i) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
44 unsigned Reg = TargetRegisterInfo::index2VirtReg(i);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
45 if (MRI.reg_nodbg_empty(Reg))
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
46 continue;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
47 VRAI.calculateSpillWeightAndHint(LIS.getInterval(Reg));
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
48 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
49 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
50
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
51 // Return the preferred allocation register for reg, given a COPY instruction.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
52 static unsigned copyHint(const MachineInstr *mi, unsigned reg,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
53 const TargetRegisterInfo &tri,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
54 const MachineRegisterInfo &mri) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
55 unsigned sub, hreg, hsub;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
56 if (mi->getOperand(0).getReg() == reg) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
57 sub = mi->getOperand(0).getSubReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
58 hreg = mi->getOperand(1).getReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
59 hsub = mi->getOperand(1).getSubReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
60 } else {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
61 sub = mi->getOperand(1).getSubReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
62 hreg = mi->getOperand(0).getReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
63 hsub = mi->getOperand(0).getSubReg();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
64 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
65
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
66 if (!hreg)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
67 return 0;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
68
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
69 if (TargetRegisterInfo::isVirtualRegister(hreg))
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
70 return sub == hsub ? hreg : 0;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
71
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
72 const TargetRegisterClass *rc = mri.getRegClass(reg);
|
134
|
73 if (!tri.enableMultipleCopyHints()) {
|
|
74 // Only allow physreg hints in rc.
|
|
75 if (sub == 0)
|
|
76 return rc->contains(hreg) ? hreg : 0;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
77
|
134
|
78 // reg:sub should match the physreg hreg.
|
|
79 return tri.getMatchingSuperReg(hreg, sub, rc);
|
|
80 }
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
81
|
134
|
82 unsigned CopiedPReg = (hsub ? tri.getSubReg(hreg, hsub) : hreg);
|
|
83 if (rc->contains(CopiedPReg))
|
|
84 return CopiedPReg;
|
|
85
|
|
86 // Check if reg:sub matches so that a super register could be hinted.
|
|
87 if (sub)
|
|
88 return tri.getMatchingSuperReg(CopiedPReg, sub, rc);
|
|
89
|
|
90 return 0;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
91 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
92
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
93 // Check if all values in LI are rematerializable
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
94 static bool isRematerializable(const LiveInterval &LI,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
95 const LiveIntervals &LIS,
|
95
|
96 VirtRegMap *VRM,
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
97 const TargetInstrInfo &TII) {
|
95
|
98 unsigned Reg = LI.reg;
|
|
99 unsigned Original = VRM ? VRM->getOriginal(Reg) : 0;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
100 for (LiveInterval::const_vni_iterator I = LI.vni_begin(), E = LI.vni_end();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
101 I != E; ++I) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
102 const VNInfo *VNI = *I;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
103 if (VNI->isUnused())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
104 continue;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
105 if (VNI->isPHIDef())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
106 return false;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
107
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
108 MachineInstr *MI = LIS.getInstructionFromIndex(VNI->def);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
109 assert(MI && "Dead valno in interval");
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
110
|
95
|
111 // Trace copies introduced by live range splitting. The inline
|
|
112 // spiller can rematerialize through these copies, so the spill
|
|
113 // weight must reflect this.
|
|
114 if (VRM) {
|
|
115 while (MI->isFullCopy()) {
|
|
116 // The copy destination must match the interval register.
|
|
117 if (MI->getOperand(0).getReg() != Reg)
|
|
118 return false;
|
|
119
|
|
120 // Get the source register.
|
|
121 Reg = MI->getOperand(1).getReg();
|
|
122
|
|
123 // If the original (pre-splitting) registers match this
|
|
124 // copy came from a split.
|
|
125 if (!TargetRegisterInfo::isVirtualRegister(Reg) ||
|
|
126 VRM->getOriginal(Reg) != Original)
|
|
127 return false;
|
|
128
|
|
129 // Follow the copy live-in value.
|
|
130 const LiveInterval &SrcLI = LIS.getInterval(Reg);
|
|
131 LiveQueryResult SrcQ = SrcLI.Query(VNI->def);
|
|
132 VNI = SrcQ.valueIn();
|
|
133 assert(VNI && "Copy from non-existing value");
|
|
134 if (VNI->isPHIDef())
|
|
135 return false;
|
|
136 MI = LIS.getInstructionFromIndex(VNI->def);
|
|
137 assert(MI && "Dead valno in interval");
|
|
138 }
|
|
139 }
|
|
140
|
120
|
141 if (!TII.isTriviallyReMaterializable(*MI, LIS.getAliasAnalysis()))
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
142 return false;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
143 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
144 return true;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
145 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
146
|
121
|
147 void VirtRegAuxInfo::calculateSpillWeightAndHint(LiveInterval &li) {
|
|
148 float weight = weightCalcHelper(li);
|
|
149 // Check if unspillable.
|
|
150 if (weight < 0)
|
|
151 return;
|
|
152 li.weight = weight;
|
|
153 }
|
|
154
|
|
155 float VirtRegAuxInfo::futureWeight(LiveInterval &li, SlotIndex start,
|
|
156 SlotIndex end) {
|
|
157 return weightCalcHelper(li, &start, &end);
|
|
158 }
|
|
159
|
|
160 float VirtRegAuxInfo::weightCalcHelper(LiveInterval &li, SlotIndex *start,
|
|
161 SlotIndex *end) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
162 MachineRegisterInfo &mri = MF.getRegInfo();
|
77
|
163 const TargetRegisterInfo &tri = *MF.getSubtarget().getRegisterInfo();
|
|
164 MachineBasicBlock *mbb = nullptr;
|
|
165 MachineLoop *loop = nullptr;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
166 bool isExiting = false;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
167 float totalWeight = 0;
|
83
|
168 unsigned numInstr = 0; // Number of instructions using li
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
169 SmallPtrSet<MachineInstr*, 8> visited;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
170
|
134
|
171 std::pair<unsigned, unsigned> TargetHint = mri.getRegAllocationHint(li.reg);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
172
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
173 // Don't recompute spill weight for an unspillable register.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
174 bool Spillable = li.isSpillable();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
175
|
121
|
176 bool localSplitArtifact = start && end;
|
|
177
|
|
178 // Do not update future local split artifacts.
|
|
179 bool updateLI = !localSplitArtifact;
|
|
180
|
|
181 if (localSplitArtifact) {
|
|
182 MachineBasicBlock *localMBB = LIS.getMBBFromIndex(*end);
|
|
183 assert(localMBB == LIS.getMBBFromIndex(*start) &&
|
|
184 "start and end are expected to be in the same basic block");
|
|
185
|
|
186 // Local split artifact will have 2 additional copy instructions and they
|
|
187 // will be in the same BB.
|
|
188 // localLI = COPY other
|
|
189 // ...
|
|
190 // other = COPY localLI
|
|
191 totalWeight += LiveIntervals::getSpillWeight(true, false, &MBFI, localMBB);
|
|
192 totalWeight += LiveIntervals::getSpillWeight(false, true, &MBFI, localMBB);
|
|
193
|
|
194 numInstr += 2;
|
|
195 }
|
|
196
|
134
|
197 // CopyHint is a sortable hint derived from a COPY instruction.
|
|
198 struct CopyHint {
|
|
199 unsigned Reg;
|
|
200 float Weight;
|
|
201 bool IsPhys;
|
|
202 unsigned HintOrder;
|
|
203 CopyHint(unsigned R, float W, bool P, unsigned HR) :
|
|
204 Reg(R), Weight(W), IsPhys(P), HintOrder(HR) {}
|
|
205 bool operator<(const CopyHint &rhs) const {
|
|
206 // Always prefer any physreg hint.
|
|
207 if (IsPhys != rhs.IsPhys)
|
|
208 return (IsPhys && !rhs.IsPhys);
|
|
209 if (Weight != rhs.Weight)
|
|
210 return (Weight > rhs.Weight);
|
|
211
|
|
212 // This is just a temporary way to achive NFC for targets that don't
|
|
213 // enable multiple copy hints. HintOrder should be removed when all
|
|
214 // targets return true in enableMultipleCopyHints().
|
|
215 return (HintOrder < rhs.HintOrder);
|
|
216
|
|
217 #if 0 // Should replace the HintOrder check, see above.
|
|
218 // (just for the purpose of maintaining the set)
|
|
219 return Reg < rhs.Reg;
|
|
220 #endif
|
|
221 }
|
|
222 };
|
|
223 std::set<CopyHint> CopyHints;
|
|
224
|
|
225 // Temporary: see comment for HintOrder above.
|
|
226 unsigned CopyHintOrder = 0;
|
77
|
227 for (MachineRegisterInfo::reg_instr_iterator
|
|
228 I = mri.reg_instr_begin(li.reg), E = mri.reg_instr_end();
|
|
229 I != E; ) {
|
|
230 MachineInstr *mi = &*(I++);
|
121
|
231
|
|
232 // For local split artifacts, we are interested only in instructions between
|
|
233 // the expected start and end of the range.
|
|
234 SlotIndex si = LIS.getInstructionIndex(*mi);
|
|
235 if (localSplitArtifact && ((si < *start) || (si > *end)))
|
|
236 continue;
|
|
237
|
83
|
238 numInstr++;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
239 if (mi->isIdentityCopy() || mi->isImplicitDef() || mi->isDebugValue())
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
240 continue;
|
83
|
241 if (!visited.insert(mi).second)
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
242 continue;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
243
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
244 float weight = 1.0f;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
245 if (Spillable) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
246 // Get loop info for mi.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
247 if (mi->getParent() != mbb) {
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
248 mbb = mi->getParent();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
249 loop = Loops.getLoopFor(mbb);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
250 isExiting = loop ? loop->isLoopExiting(mbb) : false;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
251 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
252
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
253 // Calculate instr weight.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
254 bool reads, writes;
|
77
|
255 std::tie(reads, writes) = mi->readsWritesVirtualRegister(li.reg);
|
120
|
256 weight = LiveIntervals::getSpillWeight(writes, reads, &MBFI, *mi);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
257
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
258 // Give extra weight to what looks like a loop induction variable update.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
259 if (writes && isExiting && LIS.isLiveOutOfMBB(li, mbb))
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
260 weight *= 3;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
261
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
262 totalWeight += weight;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
263 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
264
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
265 // Get allocation hints from copies.
|
134
|
266 if (!mi->isCopy() ||
|
|
267 (TargetHint.first != 0 && !tri.enableMultipleCopyHints()))
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
268 continue;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
269 unsigned hint = copyHint(mi, li.reg, tri, mri);
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
270 if (!hint)
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
271 continue;
|
77
|
272 // Force hweight onto the stack so that x86 doesn't add hidden precision,
|
|
273 // making the comparison incorrectly pass (i.e., 1 > 1 == true??).
|
|
274 //
|
|
275 // FIXME: we probably shouldn't use floats at all.
|
|
276 volatile float hweight = Hint[hint] += weight;
|
134
|
277 if (TargetRegisterInfo::isVirtualRegister(hint) || mri.isAllocatable(hint))
|
|
278 CopyHints.insert(CopyHint(hint, hweight, tri.isPhysicalRegister(hint),
|
|
279 (tri.enableMultipleCopyHints() ? hint : CopyHintOrder++)));
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
280 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
281
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
282 Hint.clear();
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
283
|
134
|
284 // Pass all the sorted copy hints to mri.
|
|
285 if (updateLI && CopyHints.size()) {
|
|
286 // Remove a generic hint if previously added by target.
|
|
287 if (TargetHint.first == 0 && TargetHint.second)
|
|
288 mri.clearSimpleHint(li.reg);
|
|
289
|
|
290 for (auto &Hint : CopyHints) {
|
|
291 if (TargetHint.first != 0 && Hint.Reg == TargetHint.second)
|
|
292 // Don't add again the target-type hint.
|
|
293 continue;
|
|
294 mri.addRegAllocationHint(li.reg, Hint.Reg);
|
|
295 if (!tri.enableMultipleCopyHints())
|
|
296 break;
|
121
|
297 }
|
134
|
298
|
|
299 // Weakly boost the spill weight of hinted registers.
|
|
300 totalWeight *= 1.01F;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
301 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
302
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
303 // If the live interval was already unspillable, leave it that way.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
304 if (!Spillable)
|
121
|
305 return -1.0;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
306
|
120
|
307 // Mark li as unspillable if all live ranges are tiny and the interval
|
|
308 // is not live at any reg mask. If the interval is live at a reg mask
|
|
309 // spilling may be required.
|
121
|
310 if (updateLI && li.isZeroLength(LIS.getSlotIndexes()) &&
|
120
|
311 !li.isLiveAtIndexes(LIS.getRegMaskSlots())) {
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
312 li.markNotSpillable();
|
121
|
313 return -1.0;
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
314 }
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
315
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
316 // If all of the definitions of the interval are re-materializable,
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
317 // it is a preferred candidate for spilling.
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
318 // FIXME: this gets much more complicated once we support non-trivial
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
319 // re-materialization.
|
95
|
320 if (isRematerializable(li, LIS, VRM, *MF.getSubtarget().getInstrInfo()))
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
321 totalWeight *= 0.5F;
|
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
322
|
121
|
323 if (localSplitArtifact)
|
|
324 return normalize(totalWeight, start->distance(*end), numInstr);
|
|
325 return normalize(totalWeight, li.getSize(), numInstr);
|
0
Kaito Tokumori <e105711@ie.u-ryukyu.ac.jp>
parents:
diff
changeset
|
326 }
|