blob: 30fc27b0150ec4d1be914362103ccd8e93f2faf0 [file] [log] [blame]
Evan Cheng197d19d2007-03-28 08:30:04 +00001//===---------------------------------------------------------------------===//
2
Evan Chengc3c70882007-03-20 22:22:38 +00003Common register allocation / spilling problem:
4
Anton Korobeynikovbed29462007-04-16 18:10:23 +00005 mul lr, r4, lr
6 str lr, [sp, #+52]
7 ldr lr, [r1, #+32]
8 sxth r3, r3
9 ldr r4, [sp, #+52]
10 mla r4, r3, lr, r4
Evan Chengc3c70882007-03-20 22:22:38 +000011
12can be:
13
Anton Korobeynikovbed29462007-04-16 18:10:23 +000014 mul lr, r4, lr
Evan Chengc3c70882007-03-20 22:22:38 +000015 mov r4, lr
Anton Korobeynikovbed29462007-04-16 18:10:23 +000016 str lr, [sp, #+52]
17 ldr lr, [r1, #+32]
18 sxth r3, r3
19 mla r4, r3, lr, r4
Evan Chengc3c70882007-03-20 22:22:38 +000020
21and then "merge" mul and mov:
22
Anton Korobeynikovbed29462007-04-16 18:10:23 +000023 mul r4, r4, lr
24 str lr, [sp, #+52]
25 ldr lr, [r1, #+32]
26 sxth r3, r3
27 mla r4, r3, lr, r4
Evan Chengc3c70882007-03-20 22:22:38 +000028
29It also increase the likelyhood the store may become dead.
Evan Cheng197d19d2007-03-28 08:30:04 +000030
31//===---------------------------------------------------------------------===//
32
33I think we should have a "hasSideEffects" flag (which is automatically set for
34stuff that "isLoad" "isCall" etc), and the remat pass should eventually be able
35to remat any instruction that has no side effects, if it can handle it and if
36profitable.
37
38For now, I'd suggest having the remat stuff work like this:
39
401. I need to spill/reload this thing.
412. Check to see if it has side effects.
423. Check to see if it is simple enough: e.g. it only has one register
43destination and no register input.
444. If so, clone the instruction, do the xform, etc.
45
46Advantages of this are:
47
481. the .td file describes the behavior of the instructions, not the way the
49 algorithm should work.
502. as remat gets smarter in the future, we shouldn't have to be changing the .td
51 files.
523. it is easier to explain what the flag means in the .td file, because you
53 don't have to pull in the explanation of how the current remat algo works.
54
55Some potential added complexities:
56
571. Some instructions have to be glued to it's predecessor or successor. All of
58 the PC relative instructions and condition code setting instruction. We could
59 mark them as hasSideEffects, but that's not quite right. PC relative loads
60 from constantpools can be remat'ed, for example. But it requires more than
61 just cloning the instruction. Some instructions can be remat'ed but it
62 expands to more than one instruction. But allocator will have to make a
63 decision.
64
654. As stated in 3, not as simple as cloning in some cases. The target will have
66 to decide how to remat it. For example, an ARM 2-piece constant generation
67 instruction is remat'ed as a load from constantpool.
Evan Cheng97477782007-03-29 02:48:56 +000068
69//===---------------------------------------------------------------------===//
70
71bb27 ...
72 ...
Anton Korobeynikovbed29462007-04-16 18:10:23 +000073 %reg1037 = ADDri %reg1039, 1
74 %reg1038 = ADDrs %reg1032, %reg1039, %NOREG, 10
Evan Cheng97477782007-03-29 02:48:56 +000075 Successors according to CFG: 0x8b03bf0 (#5)
76
77bb76 (0x8b03bf0, LLVM BB @0x8b032d0, ID#5):
78 Predecessors according to CFG: 0x8b0c5f0 (#3) 0x8b0a7c0 (#4)
Anton Korobeynikovbed29462007-04-16 18:10:23 +000079 %reg1039 = PHI %reg1070, mbb<bb76.outer,0x8b0c5f0>, %reg1037, mbb<bb27,0x8b0a7c0>
Evan Cheng97477782007-03-29 02:48:56 +000080
81Note ADDri is not a two-address instruction. However, its result %reg1037 is an
82operand of the PHI node in bb76 and its operand %reg1039 is the result of the
83PHI node. We should treat it as a two-address code and make sure the ADDri is
84scheduled after any node that reads %reg1039.
85
86//===---------------------------------------------------------------------===//
87
Evan Chenge47e75b2007-04-30 18:42:09 +000088Use local info (i.e. register scavenger) to assign it a free register to allow
89reuse:
90 ldr r3, [sp, #+4]
91 add r3, r3, #3
92 ldr r2, [sp, #+8]
93 add r2, r2, #2
94 ldr r1, [sp, #+4] <==
95 add r1, r1, #1
96 ldr r0, [sp, #+4]
97 add r0, r0, #2
98
99//===---------------------------------------------------------------------===//
100
101LLVM aggressively lift CSE out of loop. Sometimes this can be negative side-
102effects:
103
104R1 = X + 4
105R2 = X + 7
106R3 = X + 15
107
108loop:
109load [i + R1]
110...
111load [i + R2]
112...
113load [i + R3]
114
115Suppose there is high register pressure, R1, R2, R3, can be spilled. We need
116to implement proper re-materialization to handle this:
117
118R1 = X + 4
119R2 = X + 7
120R3 = X + 15
121
122loop:
123R1 = X + 4 @ re-materialized
124load [i + R1]
125...
126R2 = X + 7 @ re-materialized
127load [i + R2]
128...
129R3 = X + 15 @ re-materialized
130load [i + R3]
131
132Furthermore, with re-association, we can enable sharing:
133
134R1 = X + 4
135R2 = X + 7
136R3 = X + 15
137
138loop:
139T = i + X
140load [T + 4]
141...
142load [T + 7]
143...
144load [T + 15]