]>
Commit | Line | Data |
---|---|---|
26cd9add | 1 | /* Redundant Extension Elimination pass for the GNU compiler. |
d1e082c2 | 2 | Copyright (C) 2010-2013 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" | |
221 | #include "tm.h" | |
222 | #include "rtl.h" | |
223 | #include "tree.h" | |
224 | #include "tm_p.h" | |
225 | #include "flags.h" | |
226 | #include "regs.h" | |
227 | #include "hard-reg-set.h" | |
228 | #include "basic-block.h" | |
229 | #include "insn-config.h" | |
230 | #include "function.h" | |
231 | #include "expr.h" | |
232 | #include "insn-attr.h" | |
233 | #include "recog.h" | |
718f9c0f | 234 | #include "diagnostic-core.h" |
87a0ebfd | 235 | #include "target.h" |
87a0ebfd ST |
236 | #include "optabs.h" |
237 | #include "insn-codes.h" | |
238 | #include "rtlhooks-def.h" | |
87a0ebfd | 239 | #include "params.h" |
87a0ebfd ST |
240 | #include "tree-pass.h" |
241 | #include "df.h" | |
242 | #include "cgraph.h" | |
243 | ||
282bc7b4 | 244 | /* This structure represents a candidate for elimination. */ |
87a0ebfd | 245 | |
8a1239ac | 246 | typedef struct ext_cand |
87a0ebfd | 247 | { |
282bc7b4 EB |
248 | /* The expression. */ |
249 | const_rtx expr; | |
87a0ebfd | 250 | |
282bc7b4 EB |
251 | /* The kind of extension. */ |
252 | enum rtx_code code; | |
26cd9add | 253 | |
282bc7b4 EB |
254 | /* The destination mode. */ |
255 | enum machine_mode mode; | |
256 | ||
257 | /* The instruction where it lives. */ | |
26cd9add | 258 | rtx insn; |
282bc7b4 | 259 | } ext_cand; |
26cd9add | 260 | |
26cd9add | 261 | |
87a0ebfd ST |
262 | static int max_insn_uid; |
263 | ||
26cd9add EI |
264 | /* Given a insn (CURR_INSN), an extension candidate for removal (CAND) |
265 | and a pointer to the SET rtx (ORIG_SET) that needs to be modified, | |
266 | this code modifies the SET rtx to a new SET rtx that extends the | |
267 | right hand expression into a register on the left hand side. Note | |
268 | that multiple assumptions are made about the nature of the set that | |
269 | needs to be true for this to work and is called from merge_def_and_ext. | |
87a0ebfd ST |
270 | |
271 | Original : | |
26cd9add | 272 | (set (reg a) (expression)) |
87a0ebfd ST |
273 | |
274 | Transform : | |
282bc7b4 | 275 | (set (reg a) (any_extend (expression))) |
87a0ebfd ST |
276 | |
277 | Special Cases : | |
282bc7b4 | 278 | If the expression is a constant or another extension, then directly |
26cd9add | 279 | assign it to the register. */ |
87a0ebfd ST |
280 | |
281 | static bool | |
282bc7b4 | 282 | combine_set_extension (ext_cand *cand, rtx curr_insn, rtx *orig_set) |
87a0ebfd | 283 | { |
282bc7b4 EB |
284 | rtx orig_src = SET_SRC (*orig_set); |
285 | rtx new_reg = gen_rtx_REG (cand->mode, REGNO (SET_DEST (*orig_set))); | |
286 | rtx new_set; | |
87a0ebfd | 287 | |
282bc7b4 EB |
288 | /* Merge constants by directly moving the constant into the register under |
289 | some conditions. Recall that RTL constants are sign-extended. */ | |
26cd9add | 290 | if (GET_CODE (orig_src) == CONST_INT |
282bc7b4 | 291 | && HOST_BITS_PER_WIDE_INT >= GET_MODE_BITSIZE (cand->mode)) |
26cd9add | 292 | { |
282bc7b4 EB |
293 | if (INTVAL (orig_src) >= 0 || cand->code == SIGN_EXTEND) |
294 | new_set = gen_rtx_SET (VOIDmode, new_reg, orig_src); | |
87a0ebfd | 295 | else |
26cd9add EI |
296 | { |
297 | /* Zero-extend the negative constant by masking out the bits outside | |
298 | the source mode. */ | |
299 | enum machine_mode src_mode = GET_MODE (SET_DEST (*orig_set)); | |
282bc7b4 | 300 | rtx new_const_int |
26cd9add | 301 | = GEN_INT (INTVAL (orig_src) & GET_MODE_MASK (src_mode)); |
282bc7b4 | 302 | new_set = gen_rtx_SET (VOIDmode, new_reg, new_const_int); |
26cd9add EI |
303 | } |
304 | } | |
305 | else if (GET_MODE (orig_src) == VOIDmode) | |
306 | { | |
282bc7b4 | 307 | /* This is mostly due to a call insn that should not be optimized. */ |
26cd9add | 308 | return false; |
87a0ebfd | 309 | } |
282bc7b4 | 310 | else if (GET_CODE (orig_src) == cand->code) |
87a0ebfd | 311 | { |
282bc7b4 EB |
312 | /* Here is a sequence of two extensions. Try to merge them. */ |
313 | rtx temp_extension | |
314 | = gen_rtx_fmt_e (cand->code, cand->mode, XEXP (orig_src, 0)); | |
315 | rtx simplified_temp_extension = simplify_rtx (temp_extension); | |
87a0ebfd ST |
316 | if (simplified_temp_extension) |
317 | temp_extension = simplified_temp_extension; | |
282bc7b4 | 318 | new_set = gen_rtx_SET (VOIDmode, new_reg, temp_extension); |
87a0ebfd ST |
319 | } |
320 | else if (GET_CODE (orig_src) == IF_THEN_ELSE) | |
321 | { | |
26cd9add | 322 | /* Only IF_THEN_ELSE of phi-type copies are combined. Otherwise, |
87a0ebfd | 323 | in general, IF_THEN_ELSE should not be combined. */ |
87a0ebfd ST |
324 | return false; |
325 | } | |
326 | else | |
327 | { | |
282bc7b4 EB |
328 | /* This is the normal case. */ |
329 | rtx temp_extension | |
330 | = gen_rtx_fmt_e (cand->code, cand->mode, orig_src); | |
331 | rtx simplified_temp_extension = simplify_rtx (temp_extension); | |
87a0ebfd ST |
332 | if (simplified_temp_extension) |
333 | temp_extension = simplified_temp_extension; | |
282bc7b4 | 334 | new_set = gen_rtx_SET (VOIDmode, new_reg, temp_extension); |
87a0ebfd ST |
335 | } |
336 | ||
26cd9add | 337 | /* This change is a part of a group of changes. Hence, |
87a0ebfd | 338 | validate_change will not try to commit the change. */ |
87a0ebfd ST |
339 | if (validate_change (curr_insn, orig_set, new_set, true)) |
340 | { | |
341 | if (dump_file) | |
342 | { | |
ca10595c EB |
343 | fprintf (dump_file, |
344 | "Tentatively merged extension with definition:\n"); | |
87a0ebfd ST |
345 | print_rtl_single (dump_file, curr_insn); |
346 | } | |
347 | return true; | |
348 | } | |
282bc7b4 | 349 | |
87a0ebfd ST |
350 | return false; |
351 | } | |
352 | ||
87a0ebfd | 353 | /* Treat if_then_else insns, where the operands of both branches |
26cd9add | 354 | are registers, as copies. For instance, |
87a0ebfd ST |
355 | Original : |
356 | (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c))) | |
357 | Transformed : | |
358 | (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c))) | |
359 | DEF_INSN is the if_then_else insn. */ | |
360 | ||
361 | static bool | |
282bc7b4 | 362 | transform_ifelse (ext_cand *cand, rtx def_insn) |
87a0ebfd ST |
363 | { |
364 | rtx set_insn = PATTERN (def_insn); | |
365 | rtx srcreg, dstreg, srcreg2; | |
366 | rtx map_srcreg, map_dstreg, map_srcreg2; | |
367 | rtx ifexpr; | |
368 | rtx cond; | |
369 | rtx new_set; | |
370 | ||
371 | gcc_assert (GET_CODE (set_insn) == SET); | |
282bc7b4 | 372 | |
87a0ebfd ST |
373 | cond = XEXP (SET_SRC (set_insn), 0); |
374 | dstreg = SET_DEST (set_insn); | |
375 | srcreg = XEXP (SET_SRC (set_insn), 1); | |
376 | srcreg2 = XEXP (SET_SRC (set_insn), 2); | |
b57cca0b JJ |
377 | /* If the conditional move already has the right or wider mode, |
378 | there is nothing to do. */ | |
379 | if (GET_MODE_SIZE (GET_MODE (dstreg)) >= GET_MODE_SIZE (cand->mode)) | |
380 | return true; | |
381 | ||
282bc7b4 EB |
382 | map_srcreg = gen_rtx_REG (cand->mode, REGNO (srcreg)); |
383 | map_srcreg2 = gen_rtx_REG (cand->mode, REGNO (srcreg2)); | |
384 | map_dstreg = gen_rtx_REG (cand->mode, REGNO (dstreg)); | |
385 | ifexpr = gen_rtx_IF_THEN_ELSE (cand->mode, cond, map_srcreg, map_srcreg2); | |
87a0ebfd ST |
386 | new_set = gen_rtx_SET (VOIDmode, map_dstreg, ifexpr); |
387 | ||
388 | if (validate_change (def_insn, &PATTERN (def_insn), new_set, true)) | |
389 | { | |
390 | if (dump_file) | |
391 | { | |
282bc7b4 EB |
392 | fprintf (dump_file, |
393 | "Mode of conditional move instruction extended:\n"); | |
87a0ebfd ST |
394 | print_rtl_single (dump_file, def_insn); |
395 | } | |
396 | return true; | |
397 | } | |
282bc7b4 EB |
398 | |
399 | return false; | |
87a0ebfd ST |
400 | } |
401 | ||
282bc7b4 EB |
402 | /* Get all the reaching definitions of an instruction. The definitions are |
403 | desired for REG used in INSN. Return the definition list or NULL if a | |
404 | definition is missing. If DEST is non-NULL, additionally push the INSN | |
405 | of the definitions onto DEST. */ | |
87a0ebfd | 406 | |
282bc7b4 | 407 | static struct df_link * |
9771b263 | 408 | get_defs (rtx insn, rtx reg, vec<rtx> *dest) |
87a0ebfd | 409 | { |
ca10595c | 410 | df_ref reg_info, *uses; |
282bc7b4 | 411 | struct df_link *ref_chain, *ref_link; |
87a0ebfd | 412 | |
87a0ebfd ST |
413 | reg_info = NULL; |
414 | ||
ca10595c | 415 | for (uses = DF_INSN_USES (insn); *uses; uses++) |
87a0ebfd | 416 | { |
ca10595c | 417 | reg_info = *uses; |
87a0ebfd | 418 | if (GET_CODE (DF_REF_REG (reg_info)) == SUBREG) |
282bc7b4 EB |
419 | return NULL; |
420 | if (REGNO (DF_REF_REG (reg_info)) == REGNO (reg)) | |
87a0ebfd | 421 | break; |
87a0ebfd ST |
422 | } |
423 | ||
ca10595c | 424 | gcc_assert (reg_info != NULL && uses != NULL); |
87a0ebfd | 425 | |
282bc7b4 EB |
426 | ref_chain = DF_REF_CHAIN (reg_info); |
427 | ||
428 | for (ref_link = ref_chain; ref_link; ref_link = ref_link->next) | |
87a0ebfd ST |
429 | { |
430 | /* Problem getting some definition for this instruction. */ | |
282bc7b4 EB |
431 | if (ref_link->ref == NULL) |
432 | return NULL; | |
433 | if (DF_REF_INSN_INFO (ref_link->ref) == NULL) | |
434 | return NULL; | |
87a0ebfd ST |
435 | } |
436 | ||
282bc7b4 EB |
437 | if (dest) |
438 | for (ref_link = ref_chain; ref_link; ref_link = ref_link->next) | |
9771b263 | 439 | dest->safe_push (DF_REF_INSN (ref_link->ref)); |
87a0ebfd | 440 | |
282bc7b4 | 441 | return ref_chain; |
87a0ebfd ST |
442 | } |
443 | ||
282bc7b4 EB |
444 | /* Return true if INSN is |
445 | (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2))) | |
446 | and store x1 and x2 in REG_1 and REG_2. */ | |
87a0ebfd | 447 | |
282bc7b4 EB |
448 | static bool |
449 | is_cond_copy_insn (rtx insn, rtx *reg1, rtx *reg2) | |
87a0ebfd | 450 | { |
282bc7b4 | 451 | rtx expr = single_set (insn); |
87a0ebfd | 452 | |
282bc7b4 EB |
453 | if (expr != NULL_RTX |
454 | && GET_CODE (expr) == SET | |
87a0ebfd | 455 | && GET_CODE (SET_DEST (expr)) == REG |
87a0ebfd ST |
456 | && GET_CODE (SET_SRC (expr)) == IF_THEN_ELSE |
457 | && GET_CODE (XEXP (SET_SRC (expr), 1)) == REG | |
26cd9add | 458 | && GET_CODE (XEXP (SET_SRC (expr), 2)) == REG) |
87a0ebfd | 459 | { |
282bc7b4 EB |
460 | *reg1 = XEXP (SET_SRC (expr), 1); |
461 | *reg2 = XEXP (SET_SRC (expr), 2); | |
462 | return true; | |
87a0ebfd ST |
463 | } |
464 | ||
282bc7b4 | 465 | return false; |
87a0ebfd ST |
466 | } |
467 | ||
b57cca0b JJ |
468 | enum ext_modified_kind |
469 | { | |
470 | /* The insn hasn't been modified by ree pass yet. */ | |
471 | EXT_MODIFIED_NONE, | |
472 | /* Changed into zero extension. */ | |
473 | EXT_MODIFIED_ZEXT, | |
474 | /* Changed into sign extension. */ | |
475 | EXT_MODIFIED_SEXT | |
476 | }; | |
477 | ||
925e30ff | 478 | struct ATTRIBUTE_PACKED ext_modified |
b57cca0b JJ |
479 | { |
480 | /* Mode from which ree has zero or sign extended the destination. */ | |
481 | ENUM_BITFIELD(machine_mode) mode : 8; | |
482 | ||
483 | /* Kind of modification of the insn. */ | |
484 | ENUM_BITFIELD(ext_modified_kind) kind : 2; | |
485 | ||
486 | /* True if the insn is scheduled to be deleted. */ | |
487 | unsigned int deleted : 1; | |
488 | }; | |
489 | ||
490 | /* Vectors used by combine_reaching_defs and its helpers. */ | |
491 | typedef struct ext_state | |
492 | { | |
9771b263 | 493 | /* In order to avoid constant alloc/free, we keep these |
b57cca0b | 494 | 4 vectors live through the entire find_and_remove_re and just |
9771b263 DN |
495 | truncate them each time. */ |
496 | vec<rtx> defs_list; | |
497 | vec<rtx> copies_list; | |
498 | vec<rtx> modified_list; | |
499 | vec<rtx> work_list; | |
b57cca0b JJ |
500 | |
501 | /* For instructions that have been successfully modified, this is | |
502 | the original mode from which the insn is extending and | |
503 | kind of extension. */ | |
504 | struct ext_modified *modified; | |
505 | } ext_state; | |
506 | ||
26cd9add EI |
507 | /* Reaching Definitions of the extended register could be conditional copies |
508 | or regular definitions. This function separates the two types into two | |
b57cca0b JJ |
509 | lists, STATE->DEFS_LIST and STATE->COPIES_LIST. This is necessary because, |
510 | if a reaching definition is a conditional copy, merging the extension with | |
511 | this definition is wrong. Conditional copies are merged by transitively | |
512 | merging their definitions. The defs_list is populated with all the reaching | |
513 | definitions of the extension instruction (EXTEND_INSN) which must be merged | |
514 | with an extension. The copies_list contains all the conditional moves that | |
515 | will later be extended into a wider mode conditional move if all the merges | |
516 | are successful. The function returns false upon failure, true upon | |
517 | success. */ | |
518 | ||
519 | static bool | |
089dacc5 | 520 | make_defs_and_copies_lists (rtx extend_insn, const_rtx set_pat, |
b57cca0b | 521 | ext_state *state) |
87a0ebfd | 522 | { |
282bc7b4 | 523 | rtx src_reg = XEXP (SET_SRC (set_pat), 0); |
87a0ebfd | 524 | bool *is_insn_visited; |
b57cca0b JJ |
525 | bool ret = true; |
526 | ||
9771b263 | 527 | state->work_list.truncate (0); |
87a0ebfd | 528 | |
282bc7b4 | 529 | /* Initialize the work list. */ |
b57cca0b JJ |
530 | if (!get_defs (extend_insn, src_reg, &state->work_list)) |
531 | gcc_unreachable (); | |
87a0ebfd | 532 | |
282bc7b4 | 533 | is_insn_visited = XCNEWVEC (bool, max_insn_uid); |
87a0ebfd ST |
534 | |
535 | /* Perform transitive closure for conditional copies. */ | |
9771b263 | 536 | while (!state->work_list.is_empty ()) |
87a0ebfd | 537 | { |
9771b263 | 538 | rtx def_insn = state->work_list.pop (); |
282bc7b4 EB |
539 | rtx reg1, reg2; |
540 | ||
87a0ebfd ST |
541 | gcc_assert (INSN_UID (def_insn) < max_insn_uid); |
542 | ||
543 | if (is_insn_visited[INSN_UID (def_insn)]) | |
282bc7b4 | 544 | continue; |
87a0ebfd | 545 | is_insn_visited[INSN_UID (def_insn)] = true; |
87a0ebfd | 546 | |
282bc7b4 EB |
547 | if (is_cond_copy_insn (def_insn, ®1, ®2)) |
548 | { | |
549 | /* Push it onto the copy list first. */ | |
9771b263 | 550 | state->copies_list.safe_push (def_insn); |
282bc7b4 EB |
551 | |
552 | /* Now perform the transitive closure. */ | |
b57cca0b JJ |
553 | if (!get_defs (def_insn, reg1, &state->work_list) |
554 | || !get_defs (def_insn, reg2, &state->work_list)) | |
282bc7b4 | 555 | { |
b57cca0b | 556 | ret = false; |
282bc7b4 EB |
557 | break; |
558 | } | |
87a0ebfd ST |
559 | } |
560 | else | |
9771b263 | 561 | state->defs_list.safe_push (def_insn); |
87a0ebfd ST |
562 | } |
563 | ||
87a0ebfd | 564 | XDELETEVEC (is_insn_visited); |
282bc7b4 EB |
565 | |
566 | return ret; | |
87a0ebfd ST |
567 | } |
568 | ||
282bc7b4 | 569 | /* Merge the DEF_INSN with an extension. Calls combine_set_extension |
87a0ebfd ST |
570 | on the SET pattern. */ |
571 | ||
572 | static bool | |
b57cca0b | 573 | merge_def_and_ext (ext_cand *cand, rtx def_insn, ext_state *state) |
87a0ebfd | 574 | { |
26cd9add | 575 | enum machine_mode ext_src_mode; |
87a0ebfd | 576 | enum rtx_code code; |
87a0ebfd ST |
577 | rtx *sub_rtx; |
578 | rtx s_expr; | |
579 | int i; | |
580 | ||
26cd9add | 581 | ext_src_mode = GET_MODE (XEXP (SET_SRC (cand->expr), 0)); |
87a0ebfd ST |
582 | code = GET_CODE (PATTERN (def_insn)); |
583 | sub_rtx = NULL; | |
584 | ||
585 | if (code == PARALLEL) | |
586 | { | |
587 | for (i = 0; i < XVECLEN (PATTERN (def_insn), 0); i++) | |
588 | { | |
589 | s_expr = XVECEXP (PATTERN (def_insn), 0, i); | |
590 | if (GET_CODE (s_expr) != SET) | |
591 | continue; | |
592 | ||
593 | if (sub_rtx == NULL) | |
594 | sub_rtx = &XVECEXP (PATTERN (def_insn), 0, i); | |
595 | else | |
596 | { | |
597 | /* PARALLEL with multiple SETs. */ | |
598 | return false; | |
599 | } | |
600 | } | |
601 | } | |
602 | else if (code == SET) | |
603 | sub_rtx = &PATTERN (def_insn); | |
604 | else | |
605 | { | |
606 | /* It is not a PARALLEL or a SET, what could it be ? */ | |
607 | return false; | |
608 | } | |
609 | ||
610 | gcc_assert (sub_rtx != NULL); | |
611 | ||
b57cca0b JJ |
612 | if (REG_P (SET_DEST (*sub_rtx)) |
613 | && (GET_MODE (SET_DEST (*sub_rtx)) == ext_src_mode | |
614 | || ((state->modified[INSN_UID (def_insn)].kind | |
615 | == (cand->code == ZERO_EXTEND | |
616 | ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT)) | |
617 | && state->modified[INSN_UID (def_insn)].mode | |
618 | == ext_src_mode))) | |
87a0ebfd | 619 | { |
b57cca0b JJ |
620 | if (GET_MODE_SIZE (GET_MODE (SET_DEST (*sub_rtx))) |
621 | >= GET_MODE_SIZE (cand->mode)) | |
622 | return true; | |
623 | /* If def_insn is already scheduled to be deleted, don't attempt | |
624 | to modify it. */ | |
625 | if (state->modified[INSN_UID (def_insn)].deleted) | |
626 | return false; | |
627 | if (combine_set_extension (cand, def_insn, sub_rtx)) | |
628 | { | |
629 | if (state->modified[INSN_UID (def_insn)].kind == EXT_MODIFIED_NONE) | |
630 | state->modified[INSN_UID (def_insn)].mode = ext_src_mode; | |
631 | return true; | |
632 | } | |
87a0ebfd | 633 | } |
26cd9add EI |
634 | |
635 | return false; | |
87a0ebfd ST |
636 | } |
637 | ||
638 | /* This function goes through all reaching defs of the source | |
26cd9add EI |
639 | of the candidate for elimination (CAND) and tries to combine |
640 | the extension with the definition instruction. The changes | |
641 | are made as a group so that even if one definition cannot be | |
642 | merged, all reaching definitions end up not being merged. | |
643 | When a conditional copy is encountered, merging is attempted | |
644 | transitively on its definitions. It returns true upon success | |
645 | and false upon failure. */ | |
87a0ebfd ST |
646 | |
647 | static bool | |
089dacc5 | 648 | combine_reaching_defs (ext_cand *cand, const_rtx set_pat, ext_state *state) |
87a0ebfd ST |
649 | { |
650 | rtx def_insn; | |
651 | bool merge_successful = true; | |
652 | int i; | |
653 | int defs_ix; | |
b57cca0b | 654 | bool outcome; |
87a0ebfd | 655 | |
9771b263 DN |
656 | state->defs_list.truncate (0); |
657 | state->copies_list.truncate (0); | |
87a0ebfd | 658 | |
b57cca0b | 659 | outcome = make_defs_and_copies_lists (cand->insn, set_pat, state); |
87a0ebfd | 660 | |
b57cca0b JJ |
661 | if (!outcome) |
662 | return false; | |
87a0ebfd | 663 | |
6aae324c JJ |
664 | /* If cand->insn has been already modified, update cand->mode to a wider |
665 | mode if possible, or punt. */ | |
666 | if (state->modified[INSN_UID (cand->insn)].kind != EXT_MODIFIED_NONE) | |
667 | { | |
668 | enum machine_mode mode; | |
669 | rtx set; | |
670 | ||
671 | if (state->modified[INSN_UID (cand->insn)].kind | |
672 | != (cand->code == ZERO_EXTEND | |
673 | ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT) | |
674 | || state->modified[INSN_UID (cand->insn)].mode != cand->mode | |
675 | || (set = single_set (cand->insn)) == NULL_RTX) | |
676 | return false; | |
677 | mode = GET_MODE (SET_DEST (set)); | |
678 | gcc_assert (GET_MODE_SIZE (mode) >= GET_MODE_SIZE (cand->mode)); | |
679 | cand->mode = mode; | |
680 | } | |
681 | ||
87a0ebfd ST |
682 | merge_successful = true; |
683 | ||
684 | /* Go through the defs vector and try to merge all the definitions | |
685 | in this vector. */ | |
9771b263 DN |
686 | state->modified_list.truncate (0); |
687 | FOR_EACH_VEC_ELT (state->defs_list, defs_ix, def_insn) | |
87a0ebfd | 688 | { |
b57cca0b | 689 | if (merge_def_and_ext (cand, def_insn, state)) |
9771b263 | 690 | state->modified_list.safe_push (def_insn); |
87a0ebfd ST |
691 | else |
692 | { | |
693 | merge_successful = false; | |
694 | break; | |
695 | } | |
696 | } | |
697 | ||
698 | /* Now go through the conditional copies vector and try to merge all | |
699 | the copies in this vector. */ | |
87a0ebfd ST |
700 | if (merge_successful) |
701 | { | |
9771b263 | 702 | FOR_EACH_VEC_ELT (state->copies_list, i, def_insn) |
87a0ebfd | 703 | { |
26cd9add | 704 | if (transform_ifelse (cand, def_insn)) |
9771b263 | 705 | state->modified_list.safe_push (def_insn); |
87a0ebfd ST |
706 | else |
707 | { | |
708 | merge_successful = false; | |
709 | break; | |
710 | } | |
711 | } | |
712 | } | |
713 | ||
714 | if (merge_successful) | |
715 | { | |
282bc7b4 EB |
716 | /* Commit the changes here if possible |
717 | FIXME: It's an all-or-nothing scenario. Even if only one definition | |
718 | cannot be merged, we entirely give up. In the future, we should allow | |
719 | extensions to be partially eliminated along those paths where the | |
720 | definitions could be merged. */ | |
87a0ebfd ST |
721 | if (apply_change_group ()) |
722 | { | |
723 | if (dump_file) | |
282bc7b4 | 724 | fprintf (dump_file, "All merges were successful.\n"); |
87a0ebfd | 725 | |
9771b263 | 726 | FOR_EACH_VEC_ELT (state->modified_list, i, def_insn) |
b57cca0b JJ |
727 | if (state->modified[INSN_UID (def_insn)].kind == EXT_MODIFIED_NONE) |
728 | state->modified[INSN_UID (def_insn)].kind | |
729 | = (cand->code == ZERO_EXTEND | |
730 | ? EXT_MODIFIED_ZEXT : EXT_MODIFIED_SEXT); | |
731 | ||
87a0ebfd ST |
732 | return true; |
733 | } | |
734 | else | |
735 | { | |
0acba2b4 EB |
736 | /* Changes need not be cancelled explicitly as apply_change_group |
737 | does it. Print list of definitions in the dump_file for debug | |
26cd9add | 738 | purposes. This extension cannot be deleted. */ |
87a0ebfd ST |
739 | if (dump_file) |
740 | { | |
ca10595c EB |
741 | fprintf (dump_file, |
742 | "Merge cancelled, non-mergeable definitions:\n"); | |
9771b263 | 743 | FOR_EACH_VEC_ELT (state->modified_list, i, def_insn) |
ca10595c | 744 | print_rtl_single (dump_file, def_insn); |
87a0ebfd ST |
745 | } |
746 | } | |
747 | } | |
748 | else | |
749 | { | |
750 | /* Cancel any changes that have been made so far. */ | |
751 | cancel_changes (0); | |
752 | } | |
753 | ||
87a0ebfd ST |
754 | return false; |
755 | } | |
756 | ||
089dacc5 | 757 | /* Add an extension pattern that could be eliminated. */ |
0acba2b4 EB |
758 | |
759 | static void | |
089dacc5 | 760 | add_removable_extension (const_rtx expr, rtx insn, |
9771b263 | 761 | vec<ext_cand> *insn_list, |
68c8a824 | 762 | unsigned *def_map) |
0acba2b4 | 763 | { |
282bc7b4 EB |
764 | enum rtx_code code; |
765 | enum machine_mode mode; | |
68c8a824 | 766 | unsigned int idx; |
0acba2b4 EB |
767 | rtx src, dest; |
768 | ||
282bc7b4 | 769 | /* We are looking for SET (REG N) (ANY_EXTEND (REG N)). */ |
0acba2b4 EB |
770 | if (GET_CODE (expr) != SET) |
771 | return; | |
772 | ||
773 | src = SET_SRC (expr); | |
282bc7b4 | 774 | code = GET_CODE (src); |
0acba2b4 | 775 | dest = SET_DEST (expr); |
282bc7b4 | 776 | mode = GET_MODE (dest); |
0acba2b4 EB |
777 | |
778 | if (REG_P (dest) | |
282bc7b4 | 779 | && (code == SIGN_EXTEND || code == ZERO_EXTEND) |
0acba2b4 | 780 | && REG_P (XEXP (src, 0)) |
0acba2b4 EB |
781 | && REGNO (dest) == REGNO (XEXP (src, 0))) |
782 | { | |
282bc7b4 EB |
783 | struct df_link *defs, *def; |
784 | ext_cand *cand; | |
785 | ||
786 | /* First, make sure we can get all the reaching definitions. */ | |
089dacc5 | 787 | defs = get_defs (insn, XEXP (src, 0), NULL); |
282bc7b4 | 788 | if (!defs) |
0acba2b4 | 789 | { |
282bc7b4 EB |
790 | if (dump_file) |
791 | { | |
792 | fprintf (dump_file, "Cannot eliminate extension:\n"); | |
089dacc5 | 793 | print_rtl_single (dump_file, insn); |
282bc7b4 EB |
794 | fprintf (dump_file, " because of missing definition(s)\n"); |
795 | } | |
796 | return; | |
0acba2b4 | 797 | } |
282bc7b4 EB |
798 | |
799 | /* Second, make sure the reaching definitions don't feed another and | |
800 | different extension. FIXME: this obviously can be improved. */ | |
801 | for (def = defs; def; def = def->next) | |
68c8a824 | 802 | if ((idx = def_map[INSN_UID(DF_REF_INSN (def->ref))]) |
9771b263 | 803 | && (cand = &(*insn_list)[idx - 1]) |
ca3f371f | 804 | && cand->code != code) |
282bc7b4 EB |
805 | { |
806 | if (dump_file) | |
807 | { | |
808 | fprintf (dump_file, "Cannot eliminate extension:\n"); | |
089dacc5 | 809 | print_rtl_single (dump_file, insn); |
282bc7b4 EB |
810 | fprintf (dump_file, " because of other extension\n"); |
811 | } | |
812 | return; | |
813 | } | |
814 | ||
815 | /* Then add the candidate to the list and insert the reaching definitions | |
816 | into the definition map. */ | |
f32682ca | 817 | ext_cand e = {expr, code, mode, insn}; |
9771b263 DN |
818 | insn_list->safe_push (e); |
819 | idx = insn_list->length (); | |
282bc7b4 EB |
820 | |
821 | for (def = defs; def; def = def->next) | |
68c8a824 | 822 | def_map[INSN_UID(DF_REF_INSN (def->ref))] = idx; |
0acba2b4 EB |
823 | } |
824 | } | |
825 | ||
26cd9add | 826 | /* Traverse the instruction stream looking for extensions and return the |
0acba2b4 | 827 | list of candidates. */ |
87a0ebfd | 828 | |
9771b263 | 829 | static vec<ext_cand> |
26cd9add | 830 | find_removable_extensions (void) |
87a0ebfd | 831 | { |
6e1aa848 | 832 | vec<ext_cand> insn_list = vNULL; |
0acba2b4 | 833 | basic_block bb; |
089dacc5 | 834 | rtx insn, set; |
68c8a824 | 835 | unsigned *def_map = XCNEWVEC (unsigned, max_insn_uid); |
87a0ebfd | 836 | |
0acba2b4 EB |
837 | FOR_EACH_BB (bb) |
838 | FOR_BB_INSNS (bb, insn) | |
839 | { | |
840 | if (!NONDEBUG_INSN_P (insn)) | |
841 | continue; | |
87a0ebfd | 842 | |
089dacc5 JJ |
843 | set = single_set (insn); |
844 | if (set == NULL_RTX) | |
845 | continue; | |
846 | add_removable_extension (set, insn, &insn_list, def_map); | |
0acba2b4 EB |
847 | } |
848 | ||
089dacc5 | 849 | XDELETEVEC (def_map); |
282bc7b4 | 850 | |
089dacc5 | 851 | return insn_list; |
87a0ebfd ST |
852 | } |
853 | ||
854 | /* This is the main function that checks the insn stream for redundant | |
26cd9add | 855 | extensions and tries to remove them if possible. */ |
87a0ebfd | 856 | |
282bc7b4 | 857 | static void |
26cd9add | 858 | find_and_remove_re (void) |
87a0ebfd | 859 | { |
282bc7b4 | 860 | ext_cand *curr_cand; |
87a0ebfd | 861 | rtx curr_insn = NULL_RTX; |
282bc7b4 | 862 | int num_re_opportunities = 0, num_realized = 0, i; |
9771b263 DN |
863 | vec<ext_cand> reinsn_list; |
864 | vec<rtx> reinsn_del_list; | |
b57cca0b | 865 | ext_state state; |
87a0ebfd ST |
866 | |
867 | /* Construct DU chain to get all reaching definitions of each | |
26cd9add | 868 | extension instruction. */ |
7b19209f | 869 | df_set_flags (DF_RD_PRUNE_DEAD_DEFS); |
87a0ebfd ST |
870 | df_chain_add_problem (DF_UD_CHAIN + DF_DU_CHAIN); |
871 | df_analyze (); | |
b57cca0b | 872 | df_set_flags (DF_DEFER_INSN_RESCAN); |
87a0ebfd ST |
873 | |
874 | max_insn_uid = get_max_uid (); | |
9771b263 | 875 | reinsn_del_list.create (0); |
26cd9add | 876 | reinsn_list = find_removable_extensions (); |
9771b263 DN |
877 | state.defs_list.create (0); |
878 | state.copies_list.create (0); | |
879 | state.modified_list.create (0); | |
880 | state.work_list.create (0); | |
881 | if (reinsn_list.is_empty ()) | |
b57cca0b JJ |
882 | state.modified = NULL; |
883 | else | |
884 | state.modified = XCNEWVEC (struct ext_modified, max_insn_uid); | |
87a0ebfd | 885 | |
9771b263 | 886 | FOR_EACH_VEC_ELT (reinsn_list, i, curr_cand) |
87a0ebfd | 887 | { |
26cd9add | 888 | num_re_opportunities++; |
87a0ebfd | 889 | |
282bc7b4 | 890 | /* Try to combine the extension with the definition. */ |
87a0ebfd ST |
891 | if (dump_file) |
892 | { | |
282bc7b4 EB |
893 | fprintf (dump_file, "Trying to eliminate extension:\n"); |
894 | print_rtl_single (dump_file, curr_cand->insn); | |
87a0ebfd ST |
895 | } |
896 | ||
089dacc5 | 897 | if (combine_reaching_defs (curr_cand, curr_cand->expr, &state)) |
87a0ebfd ST |
898 | { |
899 | if (dump_file) | |
282bc7b4 | 900 | fprintf (dump_file, "Eliminated the extension.\n"); |
87a0ebfd | 901 | num_realized++; |
9771b263 | 902 | reinsn_del_list.safe_push (curr_cand->insn); |
b57cca0b | 903 | state.modified[INSN_UID (curr_cand->insn)].deleted = 1; |
87a0ebfd ST |
904 | } |
905 | } | |
906 | ||
26cd9add | 907 | /* Delete all useless extensions here in one sweep. */ |
9771b263 | 908 | FOR_EACH_VEC_ELT (reinsn_del_list, i, curr_insn) |
0acba2b4 | 909 | delete_insn (curr_insn); |
87a0ebfd | 910 | |
9771b263 DN |
911 | reinsn_list.release (); |
912 | reinsn_del_list.release (); | |
913 | state.defs_list.release (); | |
914 | state.copies_list.release (); | |
915 | state.modified_list.release (); | |
916 | state.work_list.release (); | |
b57cca0b | 917 | XDELETEVEC (state.modified); |
87a0ebfd | 918 | |
26cd9add | 919 | if (dump_file && num_re_opportunities > 0) |
282bc7b4 EB |
920 | fprintf (dump_file, "Elimination opportunities = %d realized = %d\n", |
921 | num_re_opportunities, num_realized); | |
87a0ebfd ST |
922 | } |
923 | ||
26cd9add | 924 | /* Find and remove redundant extensions. */ |
87a0ebfd ST |
925 | |
926 | static unsigned int | |
26cd9add | 927 | rest_of_handle_ree (void) |
87a0ebfd | 928 | { |
26cd9add EI |
929 | timevar_push (TV_REE); |
930 | find_and_remove_re (); | |
931 | timevar_pop (TV_REE); | |
87a0ebfd ST |
932 | return 0; |
933 | } | |
934 | ||
26cd9add | 935 | /* Run REE pass when flag_ree is set at optimization level > 0. */ |
87a0ebfd ST |
936 | |
937 | static bool | |
26cd9add | 938 | gate_handle_ree (void) |
87a0ebfd | 939 | { |
26cd9add | 940 | return (optimize > 0 && flag_ree); |
87a0ebfd ST |
941 | } |
942 | ||
26cd9add | 943 | struct rtl_opt_pass pass_ree = |
87a0ebfd ST |
944 | { |
945 | { | |
946 | RTL_PASS, | |
26cd9add | 947 | "ree", /* name */ |
2b4e6bf1 | 948 | OPTGROUP_NONE, /* optinfo_flags */ |
26cd9add EI |
949 | gate_handle_ree, /* gate */ |
950 | rest_of_handle_ree, /* execute */ | |
87a0ebfd ST |
951 | NULL, /* sub */ |
952 | NULL, /* next */ | |
953 | 0, /* static_pass_number */ | |
26cd9add | 954 | TV_REE, /* tv_id */ |
87a0ebfd ST |
955 | 0, /* properties_required */ |
956 | 0, /* properties_provided */ | |
957 | 0, /* properties_destroyed */ | |
958 | 0, /* todo_flags_start */ | |
7bca81dc | 959 | TODO_df_finish |
7bca81dc | 960 | | TODO_verify_rtl_sharing, /* todo_flags_finish */ |
87a0ebfd ST |
961 | } |
962 | }; |