]>
Commit | Line | Data |
---|---|---|
26cd9add | 1 | /* Redundant Extension Elimination pass for the GNU compiler. |
5624e564 | 2 | Copyright (C) 2010-2015 Free Software Foundation, Inc. |
282bc7b4 | 3 | Contributed by Ilya Enkovich (ilya.enkovich@intel.com) |
26cd9add | 4 | |
282bc7b4 EB |
5 | Based on the Redundant Zero-extension elimination pass contributed by |
6 | Sriraman Tallam (tmsriram@google.com) and Silvius Rus (rus@google.com). | |
87a0ebfd ST |
7 | |
8 | This file is part of GCC. | |
9 | ||
10 | GCC is free software; you can redistribute it and/or modify it under | |
11 | the terms of the GNU General Public License as published by the Free | |
12 | Software Foundation; either version 3, or (at your option) any later | |
13 | version. | |
14 | ||
15 | GCC is distributed in the hope that it will be useful, but WITHOUT ANY | |
16 | WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
17 | FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
18 | for more details. | |
19 | ||
20 | You should have received a copy of the GNU General Public License | |
21 | along with GCC; see the file COPYING3. If not see | |
22 | <http://www.gnu.org/licenses/>. */ | |
23 | ||
24 | ||
25 | /* Problem Description : | |
26 | -------------------- | |
26cd9add EI |
27 | This pass is intended to remove redundant extension instructions. |
28 | Such instructions appear for different reasons. We expect some of | |
29 | them due to implicit zero-extension in 64-bit registers after writing | |
30 | to their lower 32-bit half (e.g. for the x86-64 architecture). | |
31 | Another possible reason is a type cast which follows a load (for | |
32 | instance a register restore) and which can be combined into a single | |
33 | instruction, and for which earlier local passes, e.g. the combiner, | |
34 | weren't able to optimize. | |
87a0ebfd ST |
35 | |
36 | How does this pass work ? | |
37 | -------------------------- | |
38 | ||
39 | This pass is run after register allocation. Hence, all registers that | |
26cd9add EI |
40 | this pass deals with are hard registers. This pass first looks for an |
41 | extension instruction that could possibly be redundant. Such extension | |
42 | instructions show up in RTL with the pattern : | |
43 | (set (reg:<SWI248> x) (any_extend:<SWI248> (reg:<SWI124> x))), | |
44 | where x can be any hard register. | |
87a0ebfd | 45 | Now, this pass tries to eliminate this instruction by merging the |
26cd9add | 46 | extension with the definitions of register x. For instance, if |
87a0ebfd ST |
47 | one of the definitions of register x was : |
48 | (set (reg:SI x) (plus:SI (reg:SI z1) (reg:SI z2))), | |
26cd9add EI |
49 | followed by extension : |
50 | (set (reg:DI x) (zero_extend:DI (reg:SI x))) | |
87a0ebfd ST |
51 | then the combination converts this into : |
52 | (set (reg:DI x) (zero_extend:DI (plus:SI (reg:SI z1) (reg:SI z2)))). | |
53 | If all the merged definitions are recognizable assembly instructions, | |
26cd9add EI |
54 | the extension is effectively eliminated. |
55 | ||
56 | For example, for the x86-64 architecture, implicit zero-extensions | |
57 | are captured with appropriate patterns in the i386.md file. Hence, | |
58 | these merged definition can be matched to a single assembly instruction. | |
59 | The original extension instruction is then deleted if all the | |
60 | definitions can be merged. | |
87a0ebfd ST |
61 | |
62 | However, there are cases where the definition instruction cannot be | |
26cd9add EI |
63 | merged with an extension. Examples are CALL instructions. In such |
64 | cases, the original extension is not redundant and this pass does | |
87a0ebfd ST |
65 | not delete it. |
66 | ||
67 | Handling conditional moves : | |
68 | ---------------------------- | |
69 | ||
26cd9add EI |
70 | Architectures like x86-64 support conditional moves whose semantics for |
71 | extension differ from the other instructions. For instance, the | |
87a0ebfd ST |
72 | instruction *cmov ebx, eax* |
73 | zero-extends eax onto rax only when the move from ebx to eax happens. | |
282bc7b4 | 74 | Otherwise, eax may not be zero-extended. Consider conditional moves as |
87a0ebfd ST |
75 | RTL instructions of the form |
76 | (set (reg:SI x) (if_then_else (cond) (reg:SI y) (reg:SI z))). | |
26cd9add | 77 | This pass tries to merge an extension with a conditional move by |
282bc7b4 | 78 | actually merging the definitions of y and z with an extension and then |
87a0ebfd ST |
79 | converting the conditional move into : |
80 | (set (reg:DI x) (if_then_else (cond) (reg:DI y) (reg:DI z))). | |
26cd9add EI |
81 | Since registers y and z are extended, register x will also be extended |
82 | after the conditional move. Note that this step has to be done | |
83 | transitively since the definition of a conditional copy can be | |
87a0ebfd ST |
84 | another conditional copy. |
85 | ||
86 | Motivating Example I : | |
87 | --------------------- | |
88 | For this program : | |
89 | ********************************************** | |
90 | bad_code.c | |
91 | ||
92 | int mask[1000]; | |
93 | ||
94 | int foo(unsigned x) | |
95 | { | |
96 | if (x < 10) | |
97 | x = x * 45; | |
98 | else | |
99 | x = x * 78; | |
100 | return mask[x]; | |
101 | } | |
102 | ********************************************** | |
103 | ||
26cd9add | 104 | $ gcc -O2 bad_code.c |
87a0ebfd ST |
105 | ........ |
106 | 400315: b8 4e 00 00 00 mov $0x4e,%eax | |
107 | 40031a: 0f af f8 imul %eax,%edi | |
282bc7b4 | 108 | 40031d: 89 ff mov %edi,%edi - useless extension |
87a0ebfd ST |
109 | 40031f: 8b 04 bd 60 19 40 00 mov 0x401960(,%rdi,4),%eax |
110 | 400326: c3 retq | |
111 | ...... | |
112 | 400330: ba 2d 00 00 00 mov $0x2d,%edx | |
113 | 400335: 0f af fa imul %edx,%edi | |
282bc7b4 | 114 | 400338: 89 ff mov %edi,%edi - useless extension |
87a0ebfd ST |
115 | 40033a: 8b 04 bd 60 19 40 00 mov 0x401960(,%rdi,4),%eax |
116 | 400341: c3 retq | |
117 | ||
26cd9add | 118 | $ gcc -O2 -free bad_code.c |
87a0ebfd ST |
119 | ...... |
120 | 400315: 6b ff 4e imul $0x4e,%edi,%edi | |
121 | 400318: 8b 04 bd 40 19 40 00 mov 0x401940(,%rdi,4),%eax | |
122 | 40031f: c3 retq | |
123 | 400320: 6b ff 2d imul $0x2d,%edi,%edi | |
124 | 400323: 8b 04 bd 40 19 40 00 mov 0x401940(,%rdi,4),%eax | |
125 | 40032a: c3 retq | |
126 | ||
127 | Motivating Example II : | |
128 | --------------------- | |
129 | ||
130 | Here is an example with a conditional move. | |
131 | ||
132 | For this program : | |
133 | ********************************************** | |
134 | ||
135 | unsigned long long foo(unsigned x , unsigned y) | |
136 | { | |
137 | unsigned z; | |
138 | if (x > 100) | |
139 | z = x + y; | |
140 | else | |
141 | z = x - y; | |
142 | return (unsigned long long)(z); | |
143 | } | |
144 | ||
26cd9add | 145 | $ gcc -O2 bad_code.c |
87a0ebfd ST |
146 | ............ |
147 | 400360: 8d 14 3e lea (%rsi,%rdi,1),%edx | |
148 | 400363: 89 f8 mov %edi,%eax | |
149 | 400365: 29 f0 sub %esi,%eax | |
150 | 400367: 83 ff 65 cmp $0x65,%edi | |
151 | 40036a: 0f 43 c2 cmovae %edx,%eax | |
282bc7b4 | 152 | 40036d: 89 c0 mov %eax,%eax - useless extension |
87a0ebfd ST |
153 | 40036f: c3 retq |
154 | ||
26cd9add | 155 | $ gcc -O2 -free bad_code.c |
87a0ebfd ST |
156 | ............. |
157 | 400360: 89 fa mov %edi,%edx | |
158 | 400362: 8d 04 3e lea (%rsi,%rdi,1),%eax | |
159 | 400365: 29 f2 sub %esi,%edx | |
160 | 400367: 83 ff 65 cmp $0x65,%edi | |
161 | 40036a: 89 d6 mov %edx,%esi | |
162 | 40036c: 48 0f 42 c6 cmovb %rsi,%rax | |
163 | 400370: c3 retq | |
164 | ||
26cd9add EI |
165 | Motivating Example III : |
166 | --------------------- | |
167 | ||
168 | Here is an example with a type cast. | |
169 | ||
170 | For this program : | |
171 | ********************************************** | |
172 | ||
173 | void test(int size, unsigned char *in, unsigned char *out) | |
174 | { | |
175 | int i; | |
176 | unsigned char xr, xg, xy=0; | |
177 | ||
178 | for (i = 0; i < size; i++) { | |
179 | xr = *in++; | |
180 | xg = *in++; | |
181 | xy = (unsigned char) ((19595*xr + 38470*xg) >> 16); | |
182 | *out++ = xy; | |
183 | } | |
184 | } | |
185 | ||
186 | $ gcc -O2 bad_code.c | |
187 | ............ | |
188 | 10: 0f b6 0e movzbl (%rsi),%ecx | |
189 | 13: 0f b6 46 01 movzbl 0x1(%rsi),%eax | |
190 | 17: 48 83 c6 02 add $0x2,%rsi | |
282bc7b4 EB |
191 | 1b: 0f b6 c9 movzbl %cl,%ecx - useless extension |
192 | 1e: 0f b6 c0 movzbl %al,%eax - useless extension | |
26cd9add EI |
193 | 21: 69 c9 8b 4c 00 00 imul $0x4c8b,%ecx,%ecx |
194 | 27: 69 c0 46 96 00 00 imul $0x9646,%eax,%eax | |
195 | ||
196 | $ gcc -O2 -free bad_code.c | |
197 | ............. | |
198 | 10: 0f b6 0e movzbl (%rsi),%ecx | |
199 | 13: 0f b6 46 01 movzbl 0x1(%rsi),%eax | |
200 | 17: 48 83 c6 02 add $0x2,%rsi | |
201 | 1b: 69 c9 8b 4c 00 00 imul $0x4c8b,%ecx,%ecx | |
202 | 21: 69 c0 46 96 00 00 imul $0x9646,%eax,%eax | |
87a0ebfd ST |
203 | |
204 | Usefulness : | |
205 | ---------- | |
206 | ||
26cd9add EI |
207 | The original redundant zero-extension elimination pass reported reduction |
208 | of the dynamic instruction count of a compression benchmark by 2.8% and | |
209 | improvement of its run time by about 1%. | |
87a0ebfd | 210 | |
26cd9add EI |
211 | The additional performance gain with the enhanced pass is mostly expected |
212 | on in-order architectures where redundancy cannot be compensated by out of | |
213 | order execution. Measurements showed up to 10% performance gain (reduced | |
214 | run time) on EEMBC 2.0 benchmarks on Atom processor with geomean performance | |
215 | gain 1%. */ | |
87a0ebfd ST |
216 | |
217 | ||
218 | #include "config.h" | |
219 | #include "system.h" | |
220 | #include "coretypes.h" | |
c7131fb2 AM |
221 | #include "backend.h" |
222 | #include "tree.h" | |
87a0ebfd | 223 | #include "rtl.h" |
c7131fb2 | 224 | #include "df.h" |
40e23961 | 225 | #include "alias.h" |
87a0ebfd ST |
226 | #include "tm_p.h" |
227 | #include "flags.h" | |
228 | #include "regs.h" | |
60393bbc | 229 | #include "cfgrtl.h" |
60393bbc | 230 | #include "insn-config.h" |
36566b39 PK |
231 | #include "expmed.h" |
232 | #include "dojump.h" | |
233 | #include "explow.h" | |
234 | #include "calls.h" | |
235 | #include "emit-rtl.h" | |
236 | #include "varasm.h" | |
237 | #include "stmt.h" | |
87a0ebfd ST |
238 | #include "expr.h" |
239 | #include "insn-attr.h" | |
240 | #include "recog.h" | |
718f9c0f | 241 | #include "diagnostic-core.h" |
87a0ebfd | 242 | #include "target.h" |
87a0ebfd | 243 | #include "insn-codes.h" |
b0710fe1 | 244 | #include "optabs.h" |
87a0ebfd | 245 | #include "rtlhooks-def.h" |
87a0ebfd | 246 | #include "params.h" |
87a0ebfd | 247 | #include "tree-pass.h" |
87a0ebfd ST |
248 | #include "cgraph.h" |
249 | ||
282bc7b4 | 250 | /* This structure represents a candidate for elimination. */ |
87a0ebfd | 251 | |
8a1239ac | 252 | typedef struct ext_cand |
87a0ebfd | 253 | { |
282bc7b4 EB |
254 | /* The expression. */ |
255 | const_rtx expr; | |
87a0ebfd | 256 | |
282bc7b4 EB |
257 | /* The kind of extension. */ |
258 | enum rtx_code code; | |
26cd9add | 259 | |
282bc7b4 | 260 | /* The destination mode. */ |
ef4bddc2 | 261 | machine_mode mode; |
282bc7b4 EB |
262 | |
263 | /* The instruction where it lives. */ | |
59a0c032 | 264 | rtx_insn *insn; |
282bc7b4 | 265 | } ext_cand; |
26cd9add | 266 | |
26cd9add | 267 | |
87a0ebfd ST |
268 | static int max_insn_uid; |
269 | ||
73c49bf5 JJ |
270 | /* Update or remove REG_EQUAL or REG_EQUIV notes for INSN. */ |
271 | ||
272 | static bool | |
273 | update_reg_equal_equiv_notes (rtx_insn *insn, machine_mode new_mode, | |
274 | machine_mode old_mode, enum rtx_code code) | |
275 | { | |
276 | rtx *loc = ®_NOTES (insn); | |
277 | while (*loc) | |
278 | { | |
279 | enum reg_note kind = REG_NOTE_KIND (*loc); | |
280 | if (kind == REG_EQUAL || kind == REG_EQUIV) | |
281 | { | |
282 | rtx orig_src = XEXP (*loc, 0); | |
283 | /* Update equivalency constants. Recall that RTL constants are | |
284 | sign-extended. */ | |
285 | if (GET_CODE (orig_src) == CONST_INT | |
286 | && HOST_BITS_PER_WIDE_INT >= GET_MODE_BITSIZE (new_mode)) | |
287 | { | |
288 | if (INTVAL (orig_src) >= 0 || code == SIGN_EXTEND) | |
289 | /* Nothing needed. */; | |
290 | else | |
291 | { | |
292 | /* Zero-extend the negative constant by masking out the | |
293 | bits outside the source mode. */ | |
294 | rtx new_const_int | |
295 | = gen_int_mode (INTVAL (orig_src) | |
296 | & GET_MODE_MASK (old_mode), | |
297 | new_mode); | |
298 | if (!validate_change (insn, &XEXP (*loc, 0), | |
299 | new_const_int, true)) | |
300 | return false; | |
301 | } | |
302 | loc = &XEXP (*loc, 1); | |
303 | } | |
304 | /* Drop all other notes, they assume a wrong mode. */ | |
305 | else if (!validate_change (insn, loc, XEXP (*loc, 1), true)) | |
306 | return false; | |
307 | } | |
308 | else | |
309 | loc = &XEXP (*loc, 1); | |
310 | } | |
311 | return true; | |
312 | } | |
313 | ||
26cd9add EI |
314 | /* Given a insn (CURR_INSN), an extension candidate for removal (CAND) |
315 | and a pointer to the SET rtx (ORIG_SET) that needs to be modified, | |
316 | this code modifies the SET rtx to a new SET rtx that extends the | |
317 | right hand expression into a register on the left hand side. Note | |
318 | that multiple assumptions are made about the nature of the set that | |
319 | needs to be true for this to work and is called from merge_def_and_ext. | |
87a0ebfd ST |
320 | |
321 | Original : | |
26cd9add | 322 | (set (reg a) (expression)) |
87a0ebfd ST |
323 | |
324 | Transform : | |
282bc7b4 | 325 | (set (reg a) (any_extend (expression))) |
87a0ebfd ST |
326 | |
327 | Special Cases : | |
282bc7b4 | 328 | If the expression is a constant or another extension, then directly |
26cd9add | 329 | assign it to the register. */ |
87a0ebfd ST |
330 | |
331 | static bool | |
59a0c032 | 332 | combine_set_extension (ext_cand *cand, rtx_insn *curr_insn, rtx *orig_set) |
87a0ebfd | 333 | { |
282bc7b4 | 334 | rtx orig_src = SET_SRC (*orig_set); |
73c49bf5 | 335 | machine_mode orig_mode = GET_MODE (SET_DEST (*orig_set)); |
282bc7b4 | 336 | rtx new_set; |
3c92da90 JL |
337 | rtx cand_pat = PATTERN (cand->insn); |
338 | ||
339 | /* If the extension's source/destination registers are not the same | |
340 | then we need to change the original load to reference the destination | |
341 | of the extension. Then we need to emit a copy from that destination | |
342 | to the original destination of the load. */ | |
343 | rtx new_reg; | |
344 | bool copy_needed | |
345 | = (REGNO (SET_DEST (cand_pat)) != REGNO (XEXP (SET_SRC (cand_pat), 0))); | |
346 | if (copy_needed) | |
347 | new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (cand_pat))); | |
348 | else | |
349 | new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (*orig_set))); | |
87a0ebfd | 350 | |
2043135a JL |
351 | #if 0 |
352 | /* Rethinking test. Temporarily disabled. */ | |
a6a2d67b JL |
353 | /* We're going to be widening the result of DEF_INSN, ensure that doing so |
354 | doesn't change the number of hard registers needed for the result. */ | |
355 | if (HARD_REGNO_NREGS (REGNO (new_reg), cand->mode) | |
2043135a JL |
356 | != HARD_REGNO_NREGS (REGNO (SET_DEST (*orig_set)), |
357 | GET_MODE (SET_DEST (*orig_set)))) | |
a6a2d67b | 358 | return false; |
2043135a | 359 | #endif |
a6a2d67b | 360 | |
282bc7b4 EB |
361 | /* Merge constants by directly moving the constant into the register under |
362 | some conditions. Recall that RTL constants are sign-extended. */ | |
26cd9add | 363 | if (GET_CODE (orig_src) == CONST_INT |
282bc7b4 | 364 | && HOST_BITS_PER_WIDE_INT >= GET_MODE_BITSIZE (cand->mode)) |
26cd9add | 365 | { |
282bc7b4 | 366 | if (INTVAL (orig_src) >= 0 || cand->code == SIGN_EXTEND) |
f7df4a84 | 367 | new_set = gen_rtx_SET (new_reg, orig_src); |
87a0ebfd | 368 | else |
26cd9add EI |
369 | { |
370 | /* Zero-extend the negative constant by masking out the bits outside | |
371 | the source mode. */ | |
282bc7b4 | 372 | rtx new_const_int |
73c49bf5 | 373 | = gen_int_mode (INTVAL (orig_src) & GET_MODE_MASK (orig_mode), |
69db2d57 | 374 | GET_MODE (new_reg)); |
f7df4a84 | 375 | new_set = gen_rtx_SET (new_reg, new_const_int); |
26cd9add EI |
376 | } |
377 | } | |
378 | else if (GET_MODE (orig_src) == VOIDmode) | |
379 | { | |
282bc7b4 | 380 | /* This is mostly due to a call insn that should not be optimized. */ |
26cd9add | 381 | return false; |
87a0ebfd | 382 | } |
282bc7b4 | 383 | else if (GET_CODE (orig_src) == cand->code) |
87a0ebfd | 384 | { |
282bc7b4 EB |
385 | /* Here is a sequence of two extensions. Try to merge them. */ |
386 | rtx temp_extension | |
387 | = gen_rtx_fmt_e (cand->code, cand->mode, XEXP (orig_src, 0)); | |
388 | rtx simplified_temp_extension = simplify_rtx (temp_extension); | |
87a0ebfd ST |
389 | if (simplified_temp_extension) |
390 | temp_extension = simplified_temp_extension; | |
f7df4a84 | 391 | new_set = gen_rtx_SET (new_reg, temp_extension); |
87a0ebfd ST |
392 | } |
393 | else if (GET_CODE (orig_src) == IF_THEN_ELSE) | |
394 | { | |
26cd9add | 395 | /* Only IF_THEN_ELSE of phi-type copies are combined. Otherwise, |
87a0ebfd | 396 | in general, IF_THEN_ELSE should not be combined. */ |
87a0ebfd ST |
397 | return false; |
398 | } | |
399 | else | |
400 | { | |
282bc7b4 EB |
401 | /* This is the normal case. */ |
402 | rtx temp_extension | |
403 | = gen_rtx_fmt_e (cand->code, cand->mode, orig_src); | |
404 | rtx simplified_temp_extension = simplify_rtx (temp_extension); | |
87a0ebfd ST |
405 | if (simplified_temp_extension) |
406 | temp_extension = simplified_temp_extension; | |
f7df4a84 | 407 | new_set = gen_rtx_SET (new_reg, temp_extension); |
87a0ebfd ST |
408 | } |
409 | ||
26cd9add | 410 | /* This change is a part of a group of changes. Hence, |
87a0ebfd | 411 | validate_change will not try to commit the change. */ |
73c49bf5 JJ |
412 | if (validate_change (curr_insn, orig_set, new_set, true) |
413 | && update_reg_equal_equiv_notes (curr_insn, cand->mode, orig_mode, | |
414 | cand->code)) | |
87a0ebfd ST |
415 | { |
416 | if (dump_file) | |
417 | { | |
ca10595c | 418 | fprintf (dump_file, |
3c92da90 JL |
419 | "Tentatively merged extension with definition %s:\n", |
420 | (copy_needed) ? "(copy needed)" : ""); | |
87a0ebfd ST |
421 | print_rtl_single (dump_file, curr_insn); |
422 | } | |
423 | return true; | |
424 | } | |
282bc7b4 | 425 | |
87a0ebfd ST |
426 | return false; |
427 | } | |
428 | ||
87a0ebfd | 429 | /* Treat if_then_else insns, where the operands of both branches |
26cd9add | 430 | are registers, as copies. For instance, |
87a0ebfd ST |
431 | Original : |
432 | (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c))) | |
433 | Transformed : | |
434 | (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c))) | |
435 | DEF_INSN is the if_then_else insn. */ | |
436 | ||
437 | static bool | |
59a0c032 | 438 | transform_ifelse (ext_cand *cand, rtx_insn *def_insn) |
87a0ebfd ST |
439 | { |
440 | rtx set_insn = PATTERN (def_insn); | |
441 | rtx srcreg, dstreg, srcreg2; | |
442 | rtx map_srcreg, map_dstreg, map_srcreg2; | |
443 | rtx ifexpr; | |
444 | rtx cond; | |
445 | rtx new_set; | |
446 | ||
447 | gcc_assert (GET_CODE (set_insn) == SET); | |
282bc7b4 | 448 | |
87a0ebfd ST |
449 | cond = XEXP (SET_SRC (set_insn), 0); |
450 | dstreg = SET_DEST (set_insn); | |
451 | srcreg = XEXP (SET_SRC (set_insn), 1); | |
452 | srcreg2 = XEXP (SET_SRC (set_insn), 2); | |
b57cca0b JJ |
453 | /* If the conditional move already has the right or wider mode, |
454 | there is nothing to do. */ | |
455 | if (GET_MODE_SIZE (GET_MODE (dstreg)) >= GET_MODE_SIZE (cand->mode)) | |
456 | return true; | |
457 | ||
282bc7b4 EB |
458 | map_srcreg = gen_rtx_REG (cand->mode, REGNO (srcreg)); |
459 | map_srcreg2 = gen_rtx_REG (cand->mode, REGNO (srcreg2)); | |
460 | map_dstreg = gen_rtx_REG (cand->mode, REGNO (dstreg)); | |
461 | ifexpr = gen_rtx_IF_THEN_ELSE (cand->mode, cond, map_srcreg, map_srcreg2); | |
f7df4a84 | 462 | new_set = gen_rtx_SET (map_dstreg, ifexpr); |
87a0ebfd | 463 | |
73c49bf5 JJ |
464 | if (validate_change (def_insn, &PATTERN (def_insn), new_set, true) |
465 | && update_reg_equal_equiv_notes (def_insn, cand->mode, GET_MODE (dstreg), | |
466 | cand->code)) | |
87a0ebfd ST |
467 | { |
468 | if (dump_file) | |
469 | { | |
282bc7b4 EB |
470 | fprintf (dump_file, |
471 | "Mode of conditional move instruction extended:\n"); | |
87a0ebfd ST |
472 | print_rtl_single (dump_file, def_insn); |
473 | } | |
474 | return true; | |
475 | } | |
282bc7b4 EB |
476 | |
477 | return false; | |
87a0ebfd ST |
478 | } |
479 | ||
282bc7b4 EB |
480 | /* Get all the reaching definitions of an instruction. The definitions are |
481 | desired for REG used in INSN. Return the definition list or NULL if a | |
482 | definition is missing. If DEST is non-NULL, additionally push the INSN | |
483 | of the definitions onto DEST. */ | |
87a0ebfd | 484 | |
282bc7b4 | 485 | static struct df_link * |
59a0c032 | 486 | get_defs (rtx_insn *insn, rtx reg, vec<rtx_insn *> *dest) |
87a0ebfd | 487 | { |
bfac633a | 488 | df_ref use; |
282bc7b4 | 489 | struct df_link *ref_chain, *ref_link; |
87a0ebfd | 490 | |
bfac633a | 491 | FOR_EACH_INSN_USE (use, insn) |
87a0ebfd | 492 | { |
bfac633a | 493 | if (GET_CODE (DF_REF_REG (use)) == SUBREG) |
282bc7b4 | 494 | return NULL; |
bfac633a RS |
495 | if (REGNO (DF_REF_REG (use)) == REGNO (reg)) |
496 | break; | |
87a0ebfd ST |
497 | } |
498 | ||
bfac633a | 499 | gcc_assert (use != NULL); |
87a0ebfd | 500 | |
bfac633a | 501 | ref_chain = DF_REF_CHAIN (use); |
282bc7b4 EB |
502 | |
503 | for (ref_link = ref_chain; ref_link; ref_link = ref_link->next) | |
87a0ebfd ST |
504 | { |
505 | /* Problem getting some definition for this instruction. */ | |
282bc7b4 EB |
506 | if (ref_link->ref == NULL) |
507 | return NULL; | |
508 | if (DF_REF_INSN_INFO (ref_link->ref) == NULL) | |
509 | return NULL; | |
87a0ebfd ST |
510 | } |
511 | ||
282bc7b4 EB |
512 | if (dest) |
513 | for (ref_link = ref_chain; ref_link; ref_link = ref_link->next) | |
9771b263 | 514 | dest->safe_push (DF_REF_INSN (ref_link->ref)); |
87a0ebfd | 515 | |
282bc7b4 | 516 | return ref_chain; |
87a0ebfd ST |
517 | } |
518 | ||
282bc7b4 EB |
519 | /* Return true if INSN is |
520 | (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2))) | |
521 | and store x1 and x2 in REG_1 and REG_2. */ | |
87a0ebfd | 522 | |
282bc7b4 | 523 | static bool |
59a0c032 | 524 | is_cond_copy_insn (rtx_insn *insn, rtx *reg1, rtx *reg2) |
87a0ebfd | 525 | { |
282bc7b4 | 526 | rtx expr = single_set (insn); |
87a0ebfd | 527 | |
282bc7b4 EB |
528 | if (expr != NULL_RTX |
529 | && GET_CODE (expr) == SET | |
87a0ebfd | 530 | && GET_CODE (SET_DEST (expr)) == REG |
87a0ebfd ST |
531 | && GET_CODE (SET_SRC (expr)) == IF_THEN_ELSE |
532 | && GET_CODE (XEXP (SET_SRC (expr), 1)) == REG | |
26cd9add | 533 | && GET_CODE (XEXP (SET_SRC (expr), 2)) == REG) |
87a0ebfd | 534 | { |
282bc7b4 EB |
535 | *reg1 = XEXP (SET_SRC (expr), 1); |
536 | *reg2 = XEXP (SET_SRC (expr), 2); | |
537 | return true; | |
87a0ebfd ST |
538 | } |
539 | ||
282bc7b4 | 540 | return false; |
87a0ebfd ST |
541 | } |
542 | ||
b57cca0b JJ |
543 | enum ext_modified_kind |
544 | { | |
545 | /* The insn hasn't been modified by ree pass yet. */ | |
546 | EXT_MODIFIED_NONE, | |
547 | /* Changed into zero extension. */ | |
548 | EXT_MODIFIED_ZEXT, | |
549 | /* Changed into sign extension. */ | |
550 | EXT_MODIFIED_SEXT | |
551 | }; | |
552 | ||
925e30ff | 553 | struct ATTRIBUTE_PACKED ext_modified |
b57cca0b JJ |
554 | { |
555 | /* Mode from which ree has zero or sign extended the destination. */ | |
556 | ENUM_BITFIELD(machine_mode) mode : 8; | |
557 | ||
558 | /* Kind of modification of the insn. */ | |
559 | ENUM_BITFIELD(ext_modified_kind) kind : 2; | |
560 | ||
0d732cca JL |
561 | unsigned int do_not_reextend : 1; |
562 | ||
b57cca0b JJ |
563 | /* True if the insn is scheduled to be deleted. */ |
564 | unsigned int deleted : 1; | |
565 | }; | |
566 | ||
567 | /* Vectors used by combine_reaching_defs and its helpers. */ | |
568 | typedef struct ext_state | |
569 | { | |
9771b263 | 570 | /* In order to avoid constant alloc/free, we keep these |
b57cca0b | 571 | 4 vectors live through the entire find_and_remove_re and just |
9771b263 | 572 | truncate them each time. */ |
59a0c032 DM |
573 | vec<rtx_insn *> defs_list; |
574 | vec<rtx_insn *> copies_list; | |
575 | vec<rtx_insn *> modified_list; | |
576 | vec<rtx_insn *> work_list; | |
b57cca0b JJ |
577 | |
578 | /* For instructions that have been successfully modified, this is | |
579 | the original mode from which the insn is extending and | |
580 | kind of extension. */ | |
581 | struct ext_modified *modified; | |
582 | } ext_state; | |
583 | ||
26cd9add EI |
584 | /* Reaching Definitions of the extended register could be conditional copies |
585 | or regular definitions. This function separates the two types into two | |
b57cca0b JJ |
586 | lists, STATE->DEFS_LIST and STATE->COPIES_LIST. This is necessary because, |
587 | if a reaching definition is a conditional copy, merging the extension with | |
588 | this definition is wrong. Conditional copies are merged by transitively | |
589 | merging their definitions. The defs_list is populated with all the reaching | |
590 | definitions of the extension instruction (EXTEND_INSN) which must be merged | |
591 | with an extension. The copies_list contains all the conditional moves that | |
592 | will later be extended into a wider mode conditional move if all the merges | |
593 | are successful. The function returns false upon failure, true upon | |
594 | success. */ | |
595 | ||
596 | static bool | |
59a0c032 | 597 | make_defs_and_copies_lists (rtx_insn *extend_insn, const_rtx set_pat, |
b57cca0b | 598 | ext_state *state) |
87a0ebfd | 599 | { |
282bc7b4 | 600 | rtx src_reg = XEXP (SET_SRC (set_pat), 0); |
87a0ebfd | 601 | bool *is_insn_visited; |
b57cca0b JJ |
602 | bool ret = true; |
603 | ||
9771b263 | 604 | state->work_list.truncate (0); |
87a0ebfd | 605 | |
282bc7b4 | 606 | /* Initialize the work list. */ |
b57cca0b JJ |
607 | if (!get_defs (extend_insn, src_reg, &state->work_list)) |
608 | gcc_unreachable (); | |
87a0ebfd | 609 | |
282bc7b4 | 610 | is_insn_visited = XCNEWVEC (bool, max_insn_uid); |
87a0ebfd ST |
611 | |
612 | /* Perform transitive closure for conditional copies. */ | |
9771b263 | 613 | while (!state->work_list.is_empty ()) |
87a0ebfd | 614 | { |
59a0c032 | 615 | rtx_insn *def_insn = state->work_list.pop (); |
282bc7b4 EB |
616 | rtx reg1, reg2; |
617 | ||
87a0ebfd ST |
618 | gcc_assert (INSN_UID (def_insn) < max_insn_uid); |
619 | ||
620 | if (is_insn_visited[INSN_UID (def_insn)]) | |
282bc7b4 | 621 | continue; |
87a0ebfd | 622 | is_insn_visited[INSN_UID (def_insn)] = true; |
87a0ebfd | 623 | |
282bc7b4 EB |
624 | if (is_cond_copy_insn (def_insn, ®1, ®2)) |
625 | { | |
626 | /* Push it onto the copy list first. */ | |
9771b263 | 627 | state->copies_list.safe_push (def_insn); |
282bc7b4 EB |
628 | |
629 | /* Now perform the transitive closure. */ | |
b57cca0b JJ |
630 | if (!get_defs (def_insn, reg1, &state->work_list) |
631 | || !get_defs (def_insn, reg2, &state->work_list)) | |
282bc7b4 | 632 | { |
b57cca0b | 633 | ret = false; |
282bc7b4 EB |
634 | break; |
635 | } | |
87a0ebfd ST |
636 | } |
637 | else | |
9771b263 | 638 | state->defs_list.safe_push (def_insn); |
87a0ebfd ST |
639 | } |
640 | ||
87a0ebfd | 641 | XDELETEVEC (is_insn_visited); |
282bc7b4 EB |
642 | |
643 | return ret; | |
87a0ebfd ST |
644 | } |
645 | ||
650c4c85 JL |
646 | /* If DEF_INSN has single SET expression, possibly buried inside |
647 | a PARALLEL, return the address of the SET expression, else | |
648 | return NULL. This is similar to single_set, except that | |
649 | single_set allows multiple SETs when all but one is dead. */ | |
650 | static rtx * | |
59a0c032 | 651 | get_sub_rtx (rtx_insn *def_insn) |
87a0ebfd | 652 | { |
650c4c85 JL |
653 | enum rtx_code code = GET_CODE (PATTERN (def_insn)); |
654 | rtx *sub_rtx = NULL; | |
87a0ebfd ST |
655 | |
656 | if (code == PARALLEL) | |
657 | { | |
650c4c85 | 658 | for (int i = 0; i < XVECLEN (PATTERN (def_insn), 0); i++) |
87a0ebfd | 659 | { |
650c4c85 | 660 | rtx s_expr = XVECEXP (PATTERN (def_insn), 0, i); |
87a0ebfd ST |
661 | if (GET_CODE (s_expr) != SET) |
662 | continue; | |
663 | ||
664 | if (sub_rtx == NULL) | |
665 | sub_rtx = &XVECEXP (PATTERN (def_insn), 0, i); | |
666 | else | |
667 | { | |
668 | /* PARALLEL with multiple SETs. */ | |
650c4c85 | 669 | return NULL; |
87a0ebfd ST |
670 | } |
671 | } | |
672 | } | |
673 | else if (code == SET) | |
674 | sub_rtx = &PATTERN (def_insn); | |
675 | else | |
676 | { | |
677 | /* It is not a PARALLEL or a SET, what could it be ? */ | |
650c4c85 | 678 | return NULL; |
87a0ebfd ST |
679 | } |
680 | ||
681 | gcc_assert (sub_rtx != NULL); | |
650c4c85 JL |
682 | return sub_rtx; |
683 | } | |
684 | ||
685 | /* Merge the DEF_INSN with an extension. Calls combine_set_extension | |
686 | on the SET pattern. */ | |
687 | ||
688 | static bool | |
59a0c032 | 689 | merge_def_and_ext (ext_cand *cand, rtx_insn *def_insn, ext_state *state) |
650c4c85 | 690 | { |
ef4bddc2 | 691 | machine_mode ext_src_mode; |
650c4c85 JL |
692 | rtx *sub_rtx; |
693 | ||
694 | ext_src_mode = GET_MODE (XEXP (SET_SRC (cand->expr), 0)); | |
695 | sub_rtx = get_sub_rtx (def_insn); | |
696 | ||
697 | if (sub_rtx == NULL) | |
698 | return false; | |
87a0ebfd | 699 | |
b57cca0b JJ |
700 | if (REG_P (SET_DEST (*sub_rtx)) |
701 | && (GET_MODE (SET_DEST (*sub_rtx)) == ext_src_mode | |
702 | || ((state->modified[INSN_UID (def_insn)].kind | |
703 | == (cand->code == ZERO_EXTEND | |
704 | ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT)) | |
705 | && state->modified[INSN_UID (def_insn)].mode | |
706 | == ext_src_mode))) | |
87a0ebfd | 707 | { |
b57cca0b JJ |
708 | if (GET_MODE_SIZE (GET_MODE (SET_DEST (*sub_rtx))) |
709 | >= GET_MODE_SIZE (cand->mode)) | |
710 | return true; | |
711 | /* If def_insn is already scheduled to be deleted, don't attempt | |
712 | to modify it. */ | |
713 | if (state->modified[INSN_UID (def_insn)].deleted) | |
714 | return false; | |
715 | if (combine_set_extension (cand, def_insn, sub_rtx)) | |
716 | { | |
717 | if (state->modified[INSN_UID (def_insn)].kind == EXT_MODIFIED_NONE) | |
718 | state->modified[INSN_UID (def_insn)].mode = ext_src_mode; | |
719 | return true; | |
720 | } | |
87a0ebfd | 721 | } |
26cd9add EI |
722 | |
723 | return false; | |
87a0ebfd ST |
724 | } |
725 | ||
059742a4 JL |
726 | /* Given SRC, which should be one or more extensions of a REG, strip |
727 | away the extensions and return the REG. */ | |
728 | ||
729 | static inline rtx | |
730 | get_extended_src_reg (rtx src) | |
731 | { | |
732 | while (GET_CODE (src) == SIGN_EXTEND || GET_CODE (src) == ZERO_EXTEND) | |
733 | src = XEXP (src, 0); | |
734 | gcc_assert (REG_P (src)); | |
735 | return src; | |
736 | } | |
737 | ||
87a0ebfd | 738 | /* This function goes through all reaching defs of the source |
26cd9add EI |
739 | of the candidate for elimination (CAND) and tries to combine |
740 | the extension with the definition instruction. The changes | |
741 | are made as a group so that even if one definition cannot be | |
742 | merged, all reaching definitions end up not being merged. | |
743 | When a conditional copy is encountered, merging is attempted | |
744 | transitively on its definitions. It returns true upon success | |
745 | and false upon failure. */ | |
87a0ebfd ST |
746 | |
747 | static bool | |
089dacc5 | 748 | combine_reaching_defs (ext_cand *cand, const_rtx set_pat, ext_state *state) |
87a0ebfd | 749 | { |
59a0c032 | 750 | rtx_insn *def_insn; |
87a0ebfd ST |
751 | bool merge_successful = true; |
752 | int i; | |
753 | int defs_ix; | |
b57cca0b | 754 | bool outcome; |
87a0ebfd | 755 | |
9771b263 DN |
756 | state->defs_list.truncate (0); |
757 | state->copies_list.truncate (0); | |
87a0ebfd | 758 | |
b57cca0b | 759 | outcome = make_defs_and_copies_lists (cand->insn, set_pat, state); |
87a0ebfd | 760 | |
b57cca0b JJ |
761 | if (!outcome) |
762 | return false; | |
87a0ebfd | 763 | |
3c92da90 JL |
764 | /* If the destination operand of the extension is a different |
765 | register than the source operand, then additional restrictions | |
059742a4 JL |
766 | are needed. Note we have to handle cases where we have nested |
767 | extensions in the source operand. */ | |
0d732cca JL |
768 | bool copy_needed |
769 | = (REGNO (SET_DEST (PATTERN (cand->insn))) | |
770 | != REGNO (get_extended_src_reg (SET_SRC (PATTERN (cand->insn))))); | |
771 | if (copy_needed) | |
3c92da90 | 772 | { |
860dadcb JJ |
773 | /* Considering transformation of |
774 | (set (reg1) (expression)) | |
775 | ... | |
776 | (set (reg2) (any_extend (reg1))) | |
777 | ||
778 | into | |
779 | ||
780 | (set (reg2) (any_extend (expression))) | |
781 | (set (reg1) (reg2)) | |
782 | ... */ | |
783 | ||
3c92da90 JL |
784 | /* In theory we could handle more than one reaching def, it |
785 | just makes the code to update the insn stream more complex. */ | |
786 | if (state->defs_list.length () != 1) | |
787 | return false; | |
788 | ||
059742a4 JL |
789 | /* We require the candidate not already be modified. It may, |
790 | for example have been changed from a (sign_extend (reg)) | |
0d732cca | 791 | into (zero_extend (sign_extend (reg))). |
059742a4 JL |
792 | |
793 | Handling that case shouldn't be terribly difficult, but the code | |
794 | here and the code to emit copies would need auditing. Until | |
795 | we see a need, this is the safe thing to do. */ | |
3c92da90 JL |
796 | if (state->modified[INSN_UID (cand->insn)].kind != EXT_MODIFIED_NONE) |
797 | return false; | |
798 | ||
ef4bddc2 | 799 | machine_mode dst_mode = GET_MODE (SET_DEST (PATTERN (cand->insn))); |
e533e26c WD |
800 | rtx src_reg = get_extended_src_reg (SET_SRC (PATTERN (cand->insn))); |
801 | ||
802 | /* Ensure the number of hard registers of the copy match. */ | |
803 | if (HARD_REGNO_NREGS (REGNO (src_reg), dst_mode) | |
804 | != HARD_REGNO_NREGS (REGNO (src_reg), GET_MODE (src_reg))) | |
805 | return false; | |
806 | ||
3c92da90 | 807 | /* There's only one reaching def. */ |
59a0c032 | 808 | rtx_insn *def_insn = state->defs_list[0]; |
3c92da90 JL |
809 | |
810 | /* The defining statement must not have been modified either. */ | |
811 | if (state->modified[INSN_UID (def_insn)].kind != EXT_MODIFIED_NONE) | |
812 | return false; | |
813 | ||
814 | /* The defining statement and candidate insn must be in the same block. | |
815 | This is merely to keep the test for safety and updating the insn | |
7e41c852 JL |
816 | stream simple. Also ensure that within the block the candidate |
817 | follows the defining insn. */ | |
daca1a96 RS |
818 | basic_block bb = BLOCK_FOR_INSN (cand->insn); |
819 | if (bb != BLOCK_FOR_INSN (def_insn) | |
7e41c852 | 820 | || DF_INSN_LUID (def_insn) > DF_INSN_LUID (cand->insn)) |
3c92da90 JL |
821 | return false; |
822 | ||
823 | /* If there is an overlap between the destination of DEF_INSN and | |
824 | CAND->insn, then this transformation is not safe. Note we have | |
825 | to test in the widened mode. */ | |
650c4c85 JL |
826 | rtx *dest_sub_rtx = get_sub_rtx (def_insn); |
827 | if (dest_sub_rtx == NULL | |
828 | || !REG_P (SET_DEST (*dest_sub_rtx))) | |
829 | return false; | |
830 | ||
3c92da90 | 831 | rtx tmp_reg = gen_rtx_REG (GET_MODE (SET_DEST (PATTERN (cand->insn))), |
650c4c85 | 832 | REGNO (SET_DEST (*dest_sub_rtx))); |
3c92da90 JL |
833 | if (reg_overlap_mentioned_p (tmp_reg, SET_DEST (PATTERN (cand->insn)))) |
834 | return false; | |
835 | ||
836 | /* The destination register of the extension insn must not be | |
837 | used or set between the def_insn and cand->insn exclusive. */ | |
838 | if (reg_used_between_p (SET_DEST (PATTERN (cand->insn)), | |
839 | def_insn, cand->insn) | |
840 | || reg_set_between_p (SET_DEST (PATTERN (cand->insn)), | |
841 | def_insn, cand->insn)) | |
842 | return false; | |
0d732cca JL |
843 | |
844 | /* We must be able to copy between the two registers. Generate, | |
845 | recognize and verify constraints of the copy. Also fail if this | |
846 | generated more than one insn. | |
847 | ||
848 | This generates garbage since we throw away the insn when we're | |
c7ece684 JL |
849 | done, only to recreate it later if this test was successful. |
850 | ||
851 | Make sure to get the mode from the extension (cand->insn). This | |
852 | is different than in the code to emit the copy as we have not | |
853 | modified the defining insn yet. */ | |
0d732cca | 854 | start_sequence (); |
0d732cca | 855 | rtx pat = PATTERN (cand->insn); |
c7ece684 | 856 | rtx new_dst = gen_rtx_REG (GET_MODE (SET_DEST (pat)), |
e533e26c | 857 | REGNO (get_extended_src_reg (SET_SRC (pat)))); |
c7ece684 | 858 | rtx new_src = gen_rtx_REG (GET_MODE (SET_DEST (pat)), |
0d732cca JL |
859 | REGNO (SET_DEST (pat))); |
860 | emit_move_insn (new_dst, new_src); | |
861 | ||
b32d5189 | 862 | rtx_insn *insn = get_insns(); |
0d732cca JL |
863 | end_sequence (); |
864 | if (NEXT_INSN (insn)) | |
865 | return false; | |
866 | if (recog_memoized (insn) == -1) | |
867 | return false; | |
868 | extract_insn (insn); | |
daca1a96 | 869 | if (!constrain_operands (1, get_preferred_alternatives (insn, bb))) |
0d732cca | 870 | return false; |
3c92da90 JL |
871 | } |
872 | ||
873 | ||
6aae324c JJ |
874 | /* If cand->insn has been already modified, update cand->mode to a wider |
875 | mode if possible, or punt. */ | |
876 | if (state->modified[INSN_UID (cand->insn)].kind != EXT_MODIFIED_NONE) | |
877 | { | |
ef4bddc2 | 878 | machine_mode mode; |
6aae324c JJ |
879 | rtx set; |
880 | ||
881 | if (state->modified[INSN_UID (cand->insn)].kind | |
882 | != (cand->code == ZERO_EXTEND | |
883 | ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT) | |
884 | || state->modified[INSN_UID (cand->insn)].mode != cand->mode | |
885 | || (set = single_set (cand->insn)) == NULL_RTX) | |
886 | return false; | |
887 | mode = GET_MODE (SET_DEST (set)); | |
888 | gcc_assert (GET_MODE_SIZE (mode) >= GET_MODE_SIZE (cand->mode)); | |
889 | cand->mode = mode; | |
890 | } | |
891 | ||
87a0ebfd ST |
892 | merge_successful = true; |
893 | ||
894 | /* Go through the defs vector and try to merge all the definitions | |
895 | in this vector. */ | |
9771b263 DN |
896 | state->modified_list.truncate (0); |
897 | FOR_EACH_VEC_ELT (state->defs_list, defs_ix, def_insn) | |
87a0ebfd | 898 | { |
b57cca0b | 899 | if (merge_def_and_ext (cand, def_insn, state)) |
9771b263 | 900 | state->modified_list.safe_push (def_insn); |
87a0ebfd ST |
901 | else |
902 | { | |
903 | merge_successful = false; | |
904 | break; | |
905 | } | |
906 | } | |
907 | ||
908 | /* Now go through the conditional copies vector and try to merge all | |
909 | the copies in this vector. */ | |
87a0ebfd ST |
910 | if (merge_successful) |
911 | { | |
9771b263 | 912 | FOR_EACH_VEC_ELT (state->copies_list, i, def_insn) |
87a0ebfd | 913 | { |
26cd9add | 914 | if (transform_ifelse (cand, def_insn)) |
9771b263 | 915 | state->modified_list.safe_push (def_insn); |
87a0ebfd ST |
916 | else |
917 | { | |
918 | merge_successful = false; | |
919 | break; | |
920 | } | |
921 | } | |
922 | } | |
923 | ||
924 | if (merge_successful) | |
925 | { | |
282bc7b4 EB |
926 | /* Commit the changes here if possible |
927 | FIXME: It's an all-or-nothing scenario. Even if only one definition | |
928 | cannot be merged, we entirely give up. In the future, we should allow | |
929 | extensions to be partially eliminated along those paths where the | |
930 | definitions could be merged. */ | |
87a0ebfd ST |
931 | if (apply_change_group ()) |
932 | { | |
933 | if (dump_file) | |
282bc7b4 | 934 | fprintf (dump_file, "All merges were successful.\n"); |
87a0ebfd | 935 | |
9771b263 | 936 | FOR_EACH_VEC_ELT (state->modified_list, i, def_insn) |
0d732cca JL |
937 | { |
938 | ext_modified *modified = &state->modified[INSN_UID (def_insn)]; | |
939 | if (modified->kind == EXT_MODIFIED_NONE) | |
940 | modified->kind = (cand->code == ZERO_EXTEND ? EXT_MODIFIED_ZEXT | |
941 | : EXT_MODIFIED_SEXT); | |
b57cca0b | 942 | |
0d732cca JL |
943 | if (copy_needed) |
944 | modified->do_not_reextend = 1; | |
945 | } | |
87a0ebfd ST |
946 | return true; |
947 | } | |
948 | else | |
949 | { | |
0acba2b4 EB |
950 | /* Changes need not be cancelled explicitly as apply_change_group |
951 | does it. Print list of definitions in the dump_file for debug | |
26cd9add | 952 | purposes. This extension cannot be deleted. */ |
87a0ebfd ST |
953 | if (dump_file) |
954 | { | |
ca10595c EB |
955 | fprintf (dump_file, |
956 | "Merge cancelled, non-mergeable definitions:\n"); | |
9771b263 | 957 | FOR_EACH_VEC_ELT (state->modified_list, i, def_insn) |
ca10595c | 958 | print_rtl_single (dump_file, def_insn); |
87a0ebfd ST |
959 | } |
960 | } | |
961 | } | |
962 | else | |
963 | { | |
964 | /* Cancel any changes that have been made so far. */ | |
965 | cancel_changes (0); | |
966 | } | |
967 | ||
87a0ebfd ST |
968 | return false; |
969 | } | |
970 | ||
089dacc5 | 971 | /* Add an extension pattern that could be eliminated. */ |
0acba2b4 EB |
972 | |
973 | static void | |
59a0c032 | 974 | add_removable_extension (const_rtx expr, rtx_insn *insn, |
9771b263 | 975 | vec<ext_cand> *insn_list, |
68c8a824 | 976 | unsigned *def_map) |
0acba2b4 | 977 | { |
282bc7b4 | 978 | enum rtx_code code; |
ef4bddc2 | 979 | machine_mode mode; |
68c8a824 | 980 | unsigned int idx; |
0acba2b4 EB |
981 | rtx src, dest; |
982 | ||
282bc7b4 | 983 | /* We are looking for SET (REG N) (ANY_EXTEND (REG N)). */ |
0acba2b4 EB |
984 | if (GET_CODE (expr) != SET) |
985 | return; | |
986 | ||
987 | src = SET_SRC (expr); | |
282bc7b4 | 988 | code = GET_CODE (src); |
0acba2b4 | 989 | dest = SET_DEST (expr); |
282bc7b4 | 990 | mode = GET_MODE (dest); |
0acba2b4 EB |
991 | |
992 | if (REG_P (dest) | |
282bc7b4 | 993 | && (code == SIGN_EXTEND || code == ZERO_EXTEND) |
3c92da90 | 994 | && REG_P (XEXP (src, 0))) |
0acba2b4 | 995 | { |
282bc7b4 EB |
996 | struct df_link *defs, *def; |
997 | ext_cand *cand; | |
998 | ||
999 | /* First, make sure we can get all the reaching definitions. */ | |
089dacc5 | 1000 | defs = get_defs (insn, XEXP (src, 0), NULL); |
282bc7b4 | 1001 | if (!defs) |
0acba2b4 | 1002 | { |
282bc7b4 EB |
1003 | if (dump_file) |
1004 | { | |
1005 | fprintf (dump_file, "Cannot eliminate extension:\n"); | |
089dacc5 | 1006 | print_rtl_single (dump_file, insn); |
282bc7b4 EB |
1007 | fprintf (dump_file, " because of missing definition(s)\n"); |
1008 | } | |
1009 | return; | |
0acba2b4 | 1010 | } |
282bc7b4 EB |
1011 | |
1012 | /* Second, make sure the reaching definitions don't feed another and | |
1013 | different extension. FIXME: this obviously can be improved. */ | |
1014 | for (def = defs; def; def = def->next) | |
c3284718 | 1015 | if ((idx = def_map[INSN_UID (DF_REF_INSN (def->ref))]) |
860dadcb | 1016 | && idx != -1U |
9771b263 | 1017 | && (cand = &(*insn_list)[idx - 1]) |
ca3f371f | 1018 | && cand->code != code) |
282bc7b4 EB |
1019 | { |
1020 | if (dump_file) | |
1021 | { | |
1022 | fprintf (dump_file, "Cannot eliminate extension:\n"); | |
089dacc5 | 1023 | print_rtl_single (dump_file, insn); |
282bc7b4 EB |
1024 | fprintf (dump_file, " because of other extension\n"); |
1025 | } | |
1026 | return; | |
1027 | } | |
860dadcb JJ |
1028 | /* For vector mode extensions, ensure that all uses of the |
1029 | XEXP (src, 0) register are the same extension (both code | |
1030 | and to which mode), as unlike integral extensions lowpart | |
1031 | subreg of the sign/zero extended register are not equal | |
1032 | to the original register, so we have to change all uses or | |
1033 | none. */ | |
1034 | else if (VECTOR_MODE_P (GET_MODE (XEXP (src, 0)))) | |
1035 | { | |
1036 | if (idx == 0) | |
1037 | { | |
1038 | struct df_link *ref_chain, *ref_link; | |
1039 | ||
1040 | ref_chain = DF_REF_CHAIN (def->ref); | |
1041 | for (ref_link = ref_chain; ref_link; ref_link = ref_link->next) | |
1042 | { | |
1043 | if (ref_link->ref == NULL | |
1044 | || DF_REF_INSN_INFO (ref_link->ref) == NULL) | |
1045 | { | |
1046 | idx = -1U; | |
1047 | break; | |
1048 | } | |
1049 | rtx_insn *use_insn = DF_REF_INSN (ref_link->ref); | |
1050 | const_rtx use_set; | |
1051 | if (use_insn == insn || DEBUG_INSN_P (use_insn)) | |
1052 | continue; | |
1053 | if (!(use_set = single_set (use_insn)) | |
1054 | || !REG_P (SET_DEST (use_set)) | |
1055 | || GET_MODE (SET_DEST (use_set)) != GET_MODE (dest) | |
1056 | || GET_CODE (SET_SRC (use_set)) != code | |
1057 | || !rtx_equal_p (XEXP (SET_SRC (use_set), 0), | |
1058 | XEXP (src, 0))) | |
1059 | { | |
1060 | idx = -1U; | |
1061 | break; | |
1062 | } | |
1063 | } | |
1064 | if (idx == -1U) | |
1065 | def_map[INSN_UID (DF_REF_INSN (def->ref))] = idx; | |
1066 | } | |
1067 | if (idx == -1U) | |
1068 | { | |
1069 | if (dump_file) | |
1070 | { | |
1071 | fprintf (dump_file, "Cannot eliminate extension:\n"); | |
1072 | print_rtl_single (dump_file, insn); | |
1073 | fprintf (dump_file, | |
1074 | " because some vector uses aren't extension\n"); | |
1075 | } | |
1076 | return; | |
1077 | } | |
1078 | } | |
282bc7b4 EB |
1079 | |
1080 | /* Then add the candidate to the list and insert the reaching definitions | |
1081 | into the definition map. */ | |
f32682ca | 1082 | ext_cand e = {expr, code, mode, insn}; |
9771b263 DN |
1083 | insn_list->safe_push (e); |
1084 | idx = insn_list->length (); | |
282bc7b4 EB |
1085 | |
1086 | for (def = defs; def; def = def->next) | |
c3284718 | 1087 | def_map[INSN_UID (DF_REF_INSN (def->ref))] = idx; |
0acba2b4 EB |
1088 | } |
1089 | } | |
1090 | ||
26cd9add | 1091 | /* Traverse the instruction stream looking for extensions and return the |
0acba2b4 | 1092 | list of candidates. */ |
87a0ebfd | 1093 | |
9771b263 | 1094 | static vec<ext_cand> |
26cd9add | 1095 | find_removable_extensions (void) |
87a0ebfd | 1096 | { |
6e1aa848 | 1097 | vec<ext_cand> insn_list = vNULL; |
0acba2b4 | 1098 | basic_block bb; |
59a0c032 DM |
1099 | rtx_insn *insn; |
1100 | rtx set; | |
68c8a824 | 1101 | unsigned *def_map = XCNEWVEC (unsigned, max_insn_uid); |
87a0ebfd | 1102 | |
11cd3bed | 1103 | FOR_EACH_BB_FN (bb, cfun) |
0acba2b4 EB |
1104 | FOR_BB_INSNS (bb, insn) |
1105 | { | |
1106 | if (!NONDEBUG_INSN_P (insn)) | |
1107 | continue; | |
87a0ebfd | 1108 | |
089dacc5 JJ |
1109 | set = single_set (insn); |
1110 | if (set == NULL_RTX) | |
1111 | continue; | |
1112 | add_removable_extension (set, insn, &insn_list, def_map); | |
0acba2b4 EB |
1113 | } |
1114 | ||
089dacc5 | 1115 | XDELETEVEC (def_map); |
282bc7b4 | 1116 | |
089dacc5 | 1117 | return insn_list; |
87a0ebfd ST |
1118 | } |
1119 | ||
1120 | /* This is the main function that checks the insn stream for redundant | |
26cd9add | 1121 | extensions and tries to remove them if possible. */ |
87a0ebfd | 1122 | |
282bc7b4 | 1123 | static void |
26cd9add | 1124 | find_and_remove_re (void) |
87a0ebfd | 1125 | { |
282bc7b4 | 1126 | ext_cand *curr_cand; |
59a0c032 | 1127 | rtx_insn *curr_insn = NULL; |
282bc7b4 | 1128 | int num_re_opportunities = 0, num_realized = 0, i; |
9771b263 | 1129 | vec<ext_cand> reinsn_list; |
59a0c032 DM |
1130 | auto_vec<rtx_insn *> reinsn_del_list; |
1131 | auto_vec<rtx_insn *> reinsn_copy_list; | |
b57cca0b | 1132 | ext_state state; |
87a0ebfd ST |
1133 | |
1134 | /* Construct DU chain to get all reaching definitions of each | |
26cd9add | 1135 | extension instruction. */ |
7b19209f | 1136 | df_set_flags (DF_RD_PRUNE_DEAD_DEFS); |
87a0ebfd ST |
1137 | df_chain_add_problem (DF_UD_CHAIN + DF_DU_CHAIN); |
1138 | df_analyze (); | |
b57cca0b | 1139 | df_set_flags (DF_DEFER_INSN_RESCAN); |
87a0ebfd ST |
1140 | |
1141 | max_insn_uid = get_max_uid (); | |
26cd9add | 1142 | reinsn_list = find_removable_extensions (); |
9771b263 DN |
1143 | state.defs_list.create (0); |
1144 | state.copies_list.create (0); | |
1145 | state.modified_list.create (0); | |
1146 | state.work_list.create (0); | |
1147 | if (reinsn_list.is_empty ()) | |
b57cca0b JJ |
1148 | state.modified = NULL; |
1149 | else | |
1150 | state.modified = XCNEWVEC (struct ext_modified, max_insn_uid); | |
87a0ebfd | 1151 | |
9771b263 | 1152 | FOR_EACH_VEC_ELT (reinsn_list, i, curr_cand) |
87a0ebfd | 1153 | { |
26cd9add | 1154 | num_re_opportunities++; |
87a0ebfd | 1155 | |
282bc7b4 | 1156 | /* Try to combine the extension with the definition. */ |
87a0ebfd ST |
1157 | if (dump_file) |
1158 | { | |
282bc7b4 EB |
1159 | fprintf (dump_file, "Trying to eliminate extension:\n"); |
1160 | print_rtl_single (dump_file, curr_cand->insn); | |
87a0ebfd ST |
1161 | } |
1162 | ||
089dacc5 | 1163 | if (combine_reaching_defs (curr_cand, curr_cand->expr, &state)) |
87a0ebfd ST |
1164 | { |
1165 | if (dump_file) | |
282bc7b4 | 1166 | fprintf (dump_file, "Eliminated the extension.\n"); |
87a0ebfd | 1167 | num_realized++; |
059742a4 JL |
1168 | /* If the RHS of the current candidate is not (extend (reg)), then |
1169 | we do not allow the optimization of extensions where | |
1170 | the source and destination registers do not match. Thus | |
1171 | checking REG_P here is correct. */ | |
1172 | if (REG_P (XEXP (SET_SRC (PATTERN (curr_cand->insn)), 0)) | |
1173 | && (REGNO (SET_DEST (PATTERN (curr_cand->insn))) | |
1174 | != REGNO (XEXP (SET_SRC (PATTERN (curr_cand->insn)), 0)))) | |
3c92da90 JL |
1175 | { |
1176 | reinsn_copy_list.safe_push (curr_cand->insn); | |
1177 | reinsn_copy_list.safe_push (state.defs_list[0]); | |
1178 | } | |
1179 | reinsn_del_list.safe_push (curr_cand->insn); | |
b57cca0b | 1180 | state.modified[INSN_UID (curr_cand->insn)].deleted = 1; |
87a0ebfd ST |
1181 | } |
1182 | } | |
1183 | ||
3c92da90 JL |
1184 | /* The copy list contains pairs of insns which describe copies we |
1185 | need to insert into the INSN stream. | |
1186 | ||
1187 | The first insn in each pair is the extension insn, from which | |
1188 | we derive the source and destination of the copy. | |
1189 | ||
1190 | The second insn in each pair is the memory reference where the | |
1191 | extension will ultimately happen. We emit the new copy | |
1192 | immediately after this insn. | |
1193 | ||
1194 | It may first appear that the arguments for the copy are reversed. | |
1195 | Remember that the memory reference will be changed to refer to the | |
1196 | destination of the extention. So we're actually emitting a copy | |
1197 | from the new destination to the old destination. */ | |
1198 | for (unsigned int i = 0; i < reinsn_copy_list.length (); i += 2) | |
1199 | { | |
59a0c032 DM |
1200 | rtx_insn *curr_insn = reinsn_copy_list[i]; |
1201 | rtx_insn *def_insn = reinsn_copy_list[i + 1]; | |
a6a2d67b JL |
1202 | |
1203 | /* Use the mode of the destination of the defining insn | |
1204 | for the mode of the copy. This is necessary if the | |
1205 | defining insn was used to eliminate a second extension | |
1206 | that was wider than the first. */ | |
1207 | rtx sub_rtx = *get_sub_rtx (def_insn); | |
3c92da90 | 1208 | rtx pat = PATTERN (curr_insn); |
a6a2d67b | 1209 | rtx new_dst = gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx)), |
3c92da90 | 1210 | REGNO (XEXP (SET_SRC (pat), 0))); |
a6a2d67b JL |
1211 | rtx new_src = gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx)), |
1212 | REGNO (SET_DEST (pat))); | |
f7df4a84 | 1213 | rtx set = gen_rtx_SET (new_dst, new_src); |
a6a2d67b | 1214 | emit_insn_after (set, def_insn); |
3c92da90 JL |
1215 | } |
1216 | ||
26cd9add | 1217 | /* Delete all useless extensions here in one sweep. */ |
9771b263 | 1218 | FOR_EACH_VEC_ELT (reinsn_del_list, i, curr_insn) |
0acba2b4 | 1219 | delete_insn (curr_insn); |
87a0ebfd | 1220 | |
9771b263 | 1221 | reinsn_list.release (); |
9771b263 DN |
1222 | state.defs_list.release (); |
1223 | state.copies_list.release (); | |
1224 | state.modified_list.release (); | |
1225 | state.work_list.release (); | |
b57cca0b | 1226 | XDELETEVEC (state.modified); |
87a0ebfd | 1227 | |
26cd9add | 1228 | if (dump_file && num_re_opportunities > 0) |
282bc7b4 EB |
1229 | fprintf (dump_file, "Elimination opportunities = %d realized = %d\n", |
1230 | num_re_opportunities, num_realized); | |
87a0ebfd ST |
1231 | } |
1232 | ||
26cd9add | 1233 | /* Find and remove redundant extensions. */ |
87a0ebfd ST |
1234 | |
1235 | static unsigned int | |
26cd9add | 1236 | rest_of_handle_ree (void) |
87a0ebfd | 1237 | { |
26cd9add EI |
1238 | timevar_push (TV_REE); |
1239 | find_and_remove_re (); | |
1240 | timevar_pop (TV_REE); | |
87a0ebfd ST |
1241 | return 0; |
1242 | } | |
1243 | ||
27a4cd48 DM |
1244 | namespace { |
1245 | ||
1246 | const pass_data pass_data_ree = | |
87a0ebfd | 1247 | { |
27a4cd48 DM |
1248 | RTL_PASS, /* type */ |
1249 | "ree", /* name */ | |
1250 | OPTGROUP_NONE, /* optinfo_flags */ | |
27a4cd48 DM |
1251 | TV_REE, /* tv_id */ |
1252 | 0, /* properties_required */ | |
1253 | 0, /* properties_provided */ | |
1254 | 0, /* properties_destroyed */ | |
1255 | 0, /* todo_flags_start */ | |
3bea341f | 1256 | TODO_df_finish, /* todo_flags_finish */ |
87a0ebfd | 1257 | }; |
27a4cd48 DM |
1258 | |
1259 | class pass_ree : public rtl_opt_pass | |
1260 | { | |
1261 | public: | |
c3284718 RS |
1262 | pass_ree (gcc::context *ctxt) |
1263 | : rtl_opt_pass (pass_data_ree, ctxt) | |
27a4cd48 DM |
1264 | {} |
1265 | ||
1266 | /* opt_pass methods: */ | |
1a3d085c | 1267 | virtual bool gate (function *) { return (optimize > 0 && flag_ree); } |
be55bfe6 | 1268 | virtual unsigned int execute (function *) { return rest_of_handle_ree (); } |
27a4cd48 DM |
1269 | |
1270 | }; // class pass_ree | |
1271 | ||
1272 | } // anon namespace | |
1273 | ||
1274 | rtl_opt_pass * | |
1275 | make_pass_ree (gcc::context *ctxt) | |
1276 | { | |
1277 | return new pass_ree (ctxt); | |
1278 | } |