1 /* Redundant Extension Elimination pass for the GNU compiler.
2 Copyright (C) 2010-2016 Free Software Foundation, Inc.
3 Contributed by Ilya Enkovich (ilya.enkovich@intel.com)
5 Based on the Redundant Zero-extension elimination pass contributed by
6 Sriraman Tallam (tmsriram@google.com) and Silvius Rus (rus@google.com).
8 This file is part of GCC.
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
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
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/>. */
25 /* Problem Description :
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.
36 How does this pass work ?
37 --------------------------
39 This pass is run after register allocation. Hence, all registers that
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.
45 Now, this pass tries to eliminate this instruction by merging the
46 extension with the definitions of register x. For instance, if
47 one of the definitions of register x was :
48 (set (reg:SI x) (plus:SI (reg:SI z1) (reg:SI z2))),
49 followed by extension :
50 (set (reg:DI x) (zero_extend:DI (reg:SI x)))
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,
54 the extension is effectively eliminated.
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.
62 However, there are cases where the definition instruction cannot be
63 merged with an extension. Examples are CALL instructions. In such
64 cases, the original extension is not redundant and this pass does
67 Handling conditional moves :
68 ----------------------------
70 Architectures like x86-64 support conditional moves whose semantics for
71 extension differ from the other instructions. For instance, the
72 instruction *cmov ebx, eax*
73 zero-extends eax onto rax only when the move from ebx to eax happens.
74 Otherwise, eax may not be zero-extended. Consider conditional moves as
75 RTL instructions of the form
76 (set (reg:SI x) (if_then_else (cond) (reg:SI y) (reg:SI z))).
77 This pass tries to merge an extension with a conditional move by
78 actually merging the definitions of y and z with an extension and then
79 converting the conditional move into :
80 (set (reg:DI x) (if_then_else (cond) (reg:DI y) (reg:DI z))).
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
84 another conditional copy.
86 Motivating Example I :
89 **********************************************
102 **********************************************
106 400315: b8 4e 00 00 00 mov $0x4e,%eax
107 40031a: 0f af f8 imul %eax,%edi
108 40031d: 89 ff mov %edi,%edi - useless extension
109 40031f: 8b 04 bd 60 19 40 00 mov 0x401960(,%rdi,4),%eax
112 400330: ba 2d 00 00 00 mov $0x2d,%edx
113 400335: 0f af fa imul %edx,%edi
114 400338: 89 ff mov %edi,%edi - useless extension
115 40033a: 8b 04 bd 60 19 40 00 mov 0x401960(,%rdi,4),%eax
118 $ gcc -O2 -free bad_code.c
120 400315: 6b ff 4e imul $0x4e,%edi,%edi
121 400318: 8b 04 bd 40 19 40 00 mov 0x401940(,%rdi,4),%eax
123 400320: 6b ff 2d imul $0x2d,%edi,%edi
124 400323: 8b 04 bd 40 19 40 00 mov 0x401940(,%rdi,4),%eax
127 Motivating Example II :
128 ---------------------
130 Here is an example with a conditional move.
133 **********************************************
135 unsigned long long foo(unsigned x , unsigned y)
142 return (unsigned long long)(z);
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
152 40036d: 89 c0 mov %eax,%eax - useless extension
155 $ gcc -O2 -free bad_code.c
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
165 Motivating Example III :
166 ---------------------
168 Here is an example with a type cast.
171 **********************************************
173 void test(int size, unsigned char *in, unsigned char *out)
176 unsigned char xr, xg, xy=0;
178 for (i = 0; i < size; i++) {
181 xy = (unsigned char) ((19595*xr + 38470*xg) >> 16);
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
191 1b: 0f b6 c9 movzbl %cl,%ecx - useless extension
192 1e: 0f b6 c0 movzbl %al,%eax - useless extension
193 21: 69 c9 8b 4c 00 00 imul $0x4c8b,%ecx,%ecx
194 27: 69 c0 46 96 00 00 imul $0x9646,%eax,%eax
196 $ gcc -O2 -free bad_code.c
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
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%.
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
220 #include "coretypes.h"
226 #include "memmodel.h"
229 #include "emit-rtl.h"
233 #include "tree-pass.h"
235 /* This structure represents a candidate for elimination. */
239 /* The expression. */
242 /* The kind of extension. */
245 /* The destination mode. */
248 /* The instruction where it lives. */
253 static int max_insn_uid
;
255 /* Update or remove REG_EQUAL or REG_EQUIV notes for INSN. */
258 update_reg_equal_equiv_notes (rtx_insn
*insn
, machine_mode new_mode
,
259 machine_mode old_mode
, enum rtx_code code
)
261 rtx
*loc
= ®_NOTES (insn
);
264 enum reg_note kind
= REG_NOTE_KIND (*loc
);
265 if (kind
== REG_EQUAL
|| kind
== REG_EQUIV
)
267 rtx orig_src
= XEXP (*loc
, 0);
268 /* Update equivalency constants. Recall that RTL constants are
270 if (GET_CODE (orig_src
) == CONST_INT
271 && HOST_BITS_PER_WIDE_INT
>= GET_MODE_BITSIZE (new_mode
))
273 if (INTVAL (orig_src
) >= 0 || code
== SIGN_EXTEND
)
274 /* Nothing needed. */;
277 /* Zero-extend the negative constant by masking out the
278 bits outside the source mode. */
280 = gen_int_mode (INTVAL (orig_src
)
281 & GET_MODE_MASK (old_mode
),
283 if (!validate_change (insn
, &XEXP (*loc
, 0),
284 new_const_int
, true))
287 loc
= &XEXP (*loc
, 1);
289 /* Drop all other notes, they assume a wrong mode. */
290 else if (!validate_change (insn
, loc
, XEXP (*loc
, 1), true))
294 loc
= &XEXP (*loc
, 1);
299 /* Given a insn (CURR_INSN), an extension candidate for removal (CAND)
300 and a pointer to the SET rtx (ORIG_SET) that needs to be modified,
301 this code modifies the SET rtx to a new SET rtx that extends the
302 right hand expression into a register on the left hand side. Note
303 that multiple assumptions are made about the nature of the set that
304 needs to be true for this to work and is called from merge_def_and_ext.
307 (set (reg a) (expression))
310 (set (reg a) (any_extend (expression)))
313 If the expression is a constant or another extension, then directly
314 assign it to the register. */
317 combine_set_extension (ext_cand
*cand
, rtx_insn
*curr_insn
, rtx
*orig_set
)
319 rtx orig_src
= SET_SRC (*orig_set
);
320 machine_mode orig_mode
= GET_MODE (SET_DEST (*orig_set
));
322 rtx cand_pat
= PATTERN (cand
->insn
);
324 /* If the extension's source/destination registers are not the same
325 then we need to change the original load to reference the destination
326 of the extension. Then we need to emit a copy from that destination
327 to the original destination of the load. */
330 = (REGNO (SET_DEST (cand_pat
)) != REGNO (XEXP (SET_SRC (cand_pat
), 0)));
332 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (cand_pat
)));
334 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (*orig_set
)));
336 /* Merge constants by directly moving the constant into the register under
337 some conditions. Recall that RTL constants are sign-extended. */
338 if (GET_CODE (orig_src
) == CONST_INT
339 && HOST_BITS_PER_WIDE_INT
>= GET_MODE_BITSIZE (cand
->mode
))
341 if (INTVAL (orig_src
) >= 0 || cand
->code
== SIGN_EXTEND
)
342 new_set
= gen_rtx_SET (new_reg
, orig_src
);
345 /* Zero-extend the negative constant by masking out the bits outside
348 = gen_int_mode (INTVAL (orig_src
) & GET_MODE_MASK (orig_mode
),
350 new_set
= gen_rtx_SET (new_reg
, new_const_int
);
353 else if (GET_MODE (orig_src
) == VOIDmode
)
355 /* This is mostly due to a call insn that should not be optimized. */
358 else if (GET_CODE (orig_src
) == cand
->code
)
360 /* Here is a sequence of two extensions. Try to merge them. */
362 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, XEXP (orig_src
, 0));
363 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
364 if (simplified_temp_extension
)
365 temp_extension
= simplified_temp_extension
;
366 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
368 else if (GET_CODE (orig_src
) == IF_THEN_ELSE
)
370 /* Only IF_THEN_ELSE of phi-type copies are combined. Otherwise,
371 in general, IF_THEN_ELSE should not be combined. */
376 /* This is the normal case. */
378 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, orig_src
);
379 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
380 if (simplified_temp_extension
)
381 temp_extension
= simplified_temp_extension
;
382 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
385 /* This change is a part of a group of changes. Hence,
386 validate_change will not try to commit the change. */
387 if (validate_change (curr_insn
, orig_set
, new_set
, true)
388 && update_reg_equal_equiv_notes (curr_insn
, cand
->mode
, orig_mode
,
394 "Tentatively merged extension with definition %s:\n",
395 (copy_needed
) ? "(copy needed)" : "");
396 print_rtl_single (dump_file
, curr_insn
);
404 /* Treat if_then_else insns, where the operands of both branches
405 are registers, as copies. For instance,
407 (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c)))
409 (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c)))
410 DEF_INSN is the if_then_else insn. */
413 transform_ifelse (ext_cand
*cand
, rtx_insn
*def_insn
)
415 rtx set_insn
= PATTERN (def_insn
);
416 rtx srcreg
, dstreg
, srcreg2
;
417 rtx map_srcreg
, map_dstreg
, map_srcreg2
;
422 gcc_assert (GET_CODE (set_insn
) == SET
);
424 cond
= XEXP (SET_SRC (set_insn
), 0);
425 dstreg
= SET_DEST (set_insn
);
426 srcreg
= XEXP (SET_SRC (set_insn
), 1);
427 srcreg2
= XEXP (SET_SRC (set_insn
), 2);
428 /* If the conditional move already has the right or wider mode,
429 there is nothing to do. */
430 if (GET_MODE_SIZE (GET_MODE (dstreg
)) >= GET_MODE_SIZE (cand
->mode
))
433 map_srcreg
= gen_rtx_REG (cand
->mode
, REGNO (srcreg
));
434 map_srcreg2
= gen_rtx_REG (cand
->mode
, REGNO (srcreg2
));
435 map_dstreg
= gen_rtx_REG (cand
->mode
, REGNO (dstreg
));
436 ifexpr
= gen_rtx_IF_THEN_ELSE (cand
->mode
, cond
, map_srcreg
, map_srcreg2
);
437 new_set
= gen_rtx_SET (map_dstreg
, ifexpr
);
439 if (validate_change (def_insn
, &PATTERN (def_insn
), new_set
, true)
440 && update_reg_equal_equiv_notes (def_insn
, cand
->mode
, GET_MODE (dstreg
),
446 "Mode of conditional move instruction extended:\n");
447 print_rtl_single (dump_file
, def_insn
);
455 /* Get all the reaching definitions of an instruction. The definitions are
456 desired for REG used in INSN. Return the definition list or NULL if a
457 definition is missing. If DEST is non-NULL, additionally push the INSN
458 of the definitions onto DEST. */
460 static struct df_link
*
461 get_defs (rtx_insn
*insn
, rtx reg
, vec
<rtx_insn
*> *dest
)
464 struct df_link
*ref_chain
, *ref_link
;
466 FOR_EACH_INSN_USE (use
, insn
)
468 if (GET_CODE (DF_REF_REG (use
)) == SUBREG
)
470 if (REGNO (DF_REF_REG (use
)) == REGNO (reg
))
474 gcc_assert (use
!= NULL
);
476 ref_chain
= DF_REF_CHAIN (use
);
478 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
480 /* Problem getting some definition for this instruction. */
481 if (ref_link
->ref
== NULL
)
483 if (DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
485 /* As global regs are assumed to be defined at each function call
486 dataflow can report a call_insn as being a definition of REG.
487 But we can't do anything with that in this pass so proceed only
488 if the instruction really sets REG in a way that can be deduced
489 from the RTL structure. */
490 if (global_regs
[REGNO (reg
)]
491 && !set_of (reg
, DF_REF_INSN (ref_link
->ref
)))
496 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
497 dest
->safe_push (DF_REF_INSN (ref_link
->ref
));
502 /* Return true if INSN is
503 (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2)))
504 and store x1 and x2 in REG_1 and REG_2. */
507 is_cond_copy_insn (rtx_insn
*insn
, rtx
*reg1
, rtx
*reg2
)
509 rtx expr
= single_set (insn
);
512 && GET_CODE (expr
) == SET
513 && GET_CODE (SET_DEST (expr
)) == REG
514 && GET_CODE (SET_SRC (expr
)) == IF_THEN_ELSE
515 && GET_CODE (XEXP (SET_SRC (expr
), 1)) == REG
516 && GET_CODE (XEXP (SET_SRC (expr
), 2)) == REG
)
518 *reg1
= XEXP (SET_SRC (expr
), 1);
519 *reg2
= XEXP (SET_SRC (expr
), 2);
526 enum ext_modified_kind
528 /* The insn hasn't been modified by ree pass yet. */
530 /* Changed into zero extension. */
532 /* Changed into sign extension. */
536 struct ATTRIBUTE_PACKED ext_modified
538 /* Mode from which ree has zero or sign extended the destination. */
539 ENUM_BITFIELD(machine_mode
) mode
: 8;
541 /* Kind of modification of the insn. */
542 ENUM_BITFIELD(ext_modified_kind
) kind
: 2;
544 unsigned int do_not_reextend
: 1;
546 /* True if the insn is scheduled to be deleted. */
547 unsigned int deleted
: 1;
550 /* Vectors used by combine_reaching_defs and its helpers. */
553 /* In order to avoid constant alloc/free, we keep these
554 4 vectors live through the entire find_and_remove_re and just
555 truncate them each time. */
556 auto_vec
<rtx_insn
*> defs_list
;
557 auto_vec
<rtx_insn
*> copies_list
;
558 auto_vec
<rtx_insn
*> modified_list
;
559 auto_vec
<rtx_insn
*> work_list
;
561 /* For instructions that have been successfully modified, this is
562 the original mode from which the insn is extending and
563 kind of extension. */
564 struct ext_modified
*modified
;
567 /* Reaching Definitions of the extended register could be conditional copies
568 or regular definitions. This function separates the two types into two
569 lists, STATE->DEFS_LIST and STATE->COPIES_LIST. This is necessary because,
570 if a reaching definition is a conditional copy, merging the extension with
571 this definition is wrong. Conditional copies are merged by transitively
572 merging their definitions. The defs_list is populated with all the reaching
573 definitions of the extension instruction (EXTEND_INSN) which must be merged
574 with an extension. The copies_list contains all the conditional moves that
575 will later be extended into a wider mode conditional move if all the merges
576 are successful. The function returns false upon failure, true upon
580 make_defs_and_copies_lists (rtx_insn
*extend_insn
, const_rtx set_pat
,
583 rtx src_reg
= XEXP (SET_SRC (set_pat
), 0);
584 bool *is_insn_visited
;
587 state
->work_list
.truncate (0);
589 /* Initialize the work list. */
590 if (!get_defs (extend_insn
, src_reg
, &state
->work_list
))
593 is_insn_visited
= XCNEWVEC (bool, max_insn_uid
);
595 /* Perform transitive closure for conditional copies. */
596 while (!state
->work_list
.is_empty ())
598 rtx_insn
*def_insn
= state
->work_list
.pop ();
601 gcc_assert (INSN_UID (def_insn
) < max_insn_uid
);
603 if (is_insn_visited
[INSN_UID (def_insn
)])
605 is_insn_visited
[INSN_UID (def_insn
)] = true;
607 if (is_cond_copy_insn (def_insn
, ®1
, ®2
))
609 /* Push it onto the copy list first. */
610 state
->copies_list
.safe_push (def_insn
);
612 /* Now perform the transitive closure. */
613 if (!get_defs (def_insn
, reg1
, &state
->work_list
)
614 || !get_defs (def_insn
, reg2
, &state
->work_list
))
621 state
->defs_list
.safe_push (def_insn
);
624 XDELETEVEC (is_insn_visited
);
629 /* If DEF_INSN has single SET expression, possibly buried inside
630 a PARALLEL, return the address of the SET expression, else
631 return NULL. This is similar to single_set, except that
632 single_set allows multiple SETs when all but one is dead. */
634 get_sub_rtx (rtx_insn
*def_insn
)
636 enum rtx_code code
= GET_CODE (PATTERN (def_insn
));
639 if (code
== PARALLEL
)
641 for (int i
= 0; i
< XVECLEN (PATTERN (def_insn
), 0); i
++)
643 rtx s_expr
= XVECEXP (PATTERN (def_insn
), 0, i
);
644 if (GET_CODE (s_expr
) != SET
)
648 sub_rtx
= &XVECEXP (PATTERN (def_insn
), 0, i
);
651 /* PARALLEL with multiple SETs. */
656 else if (code
== SET
)
657 sub_rtx
= &PATTERN (def_insn
);
660 /* It is not a PARALLEL or a SET, what could it be ? */
664 gcc_assert (sub_rtx
!= NULL
);
668 /* Merge the DEF_INSN with an extension. Calls combine_set_extension
669 on the SET pattern. */
672 merge_def_and_ext (ext_cand
*cand
, rtx_insn
*def_insn
, ext_state
*state
)
674 machine_mode ext_src_mode
;
677 ext_src_mode
= GET_MODE (XEXP (SET_SRC (cand
->expr
), 0));
678 sub_rtx
= get_sub_rtx (def_insn
);
683 if (REG_P (SET_DEST (*sub_rtx
))
684 && (GET_MODE (SET_DEST (*sub_rtx
)) == ext_src_mode
685 || ((state
->modified
[INSN_UID (def_insn
)].kind
686 == (cand
->code
== ZERO_EXTEND
687 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
))
688 && state
->modified
[INSN_UID (def_insn
)].mode
691 if (GET_MODE_SIZE (GET_MODE (SET_DEST (*sub_rtx
)))
692 >= GET_MODE_SIZE (cand
->mode
))
694 /* If def_insn is already scheduled to be deleted, don't attempt
696 if (state
->modified
[INSN_UID (def_insn
)].deleted
)
698 if (combine_set_extension (cand
, def_insn
, sub_rtx
))
700 if (state
->modified
[INSN_UID (def_insn
)].kind
== EXT_MODIFIED_NONE
)
701 state
->modified
[INSN_UID (def_insn
)].mode
= ext_src_mode
;
709 /* Given SRC, which should be one or more extensions of a REG, strip
710 away the extensions and return the REG. */
713 get_extended_src_reg (rtx src
)
715 while (GET_CODE (src
) == SIGN_EXTEND
|| GET_CODE (src
) == ZERO_EXTEND
)
717 gcc_assert (REG_P (src
));
721 /* This function goes through all reaching defs of the source
722 of the candidate for elimination (CAND) and tries to combine
723 the extension with the definition instruction. The changes
724 are made as a group so that even if one definition cannot be
725 merged, all reaching definitions end up not being merged.
726 When a conditional copy is encountered, merging is attempted
727 transitively on its definitions. It returns true upon success
728 and false upon failure. */
731 combine_reaching_defs (ext_cand
*cand
, const_rtx set_pat
, ext_state
*state
)
734 bool merge_successful
= true;
739 state
->defs_list
.truncate (0);
740 state
->copies_list
.truncate (0);
742 outcome
= make_defs_and_copies_lists (cand
->insn
, set_pat
, state
);
747 /* If the destination operand of the extension is a different
748 register than the source operand, then additional restrictions
749 are needed. Note we have to handle cases where we have nested
750 extensions in the source operand. */
752 = (REGNO (SET_DEST (PATTERN (cand
->insn
)))
753 != REGNO (get_extended_src_reg (SET_SRC (PATTERN (cand
->insn
)))));
756 /* Considering transformation of
757 (set (reg1) (expression))
759 (set (reg2) (any_extend (reg1)))
763 (set (reg2) (any_extend (expression)))
767 /* In theory we could handle more than one reaching def, it
768 just makes the code to update the insn stream more complex. */
769 if (state
->defs_list
.length () != 1)
772 /* We don't have the structure described above if there are
773 conditional moves in between the def and the candidate,
774 and we will not handle them correctly. See PR68194. */
775 if (state
->copies_list
.length () > 0)
778 /* We require the candidate not already be modified. It may,
779 for example have been changed from a (sign_extend (reg))
780 into (zero_extend (sign_extend (reg))).
782 Handling that case shouldn't be terribly difficult, but the code
783 here and the code to emit copies would need auditing. Until
784 we see a need, this is the safe thing to do. */
785 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
788 machine_mode dst_mode
= GET_MODE (SET_DEST (PATTERN (cand
->insn
)));
789 rtx src_reg
= get_extended_src_reg (SET_SRC (PATTERN (cand
->insn
)));
791 /* Ensure we can use the src_reg in dst_mode (needed for
792 the (set (reg1) (reg2)) insn mentioned above). */
793 if (!HARD_REGNO_MODE_OK (REGNO (src_reg
), dst_mode
))
796 /* Ensure the number of hard registers of the copy match. */
797 if (HARD_REGNO_NREGS (REGNO (src_reg
), dst_mode
)
798 != HARD_REGNO_NREGS (REGNO (src_reg
), GET_MODE (src_reg
)))
801 /* There's only one reaching def. */
802 rtx_insn
*def_insn
= state
->defs_list
[0];
804 /* The defining statement must not have been modified either. */
805 if (state
->modified
[INSN_UID (def_insn
)].kind
!= EXT_MODIFIED_NONE
)
808 /* The defining statement and candidate insn must be in the same block.
809 This is merely to keep the test for safety and updating the insn
810 stream simple. Also ensure that within the block the candidate
811 follows the defining insn. */
812 basic_block bb
= BLOCK_FOR_INSN (cand
->insn
);
813 if (bb
!= BLOCK_FOR_INSN (def_insn
)
814 || DF_INSN_LUID (def_insn
) > DF_INSN_LUID (cand
->insn
))
817 /* If there is an overlap between the destination of DEF_INSN and
818 CAND->insn, then this transformation is not safe. Note we have
819 to test in the widened mode. */
820 rtx
*dest_sub_rtx
= get_sub_rtx (def_insn
);
821 if (dest_sub_rtx
== NULL
822 || !REG_P (SET_DEST (*dest_sub_rtx
)))
825 rtx tmp_reg
= gen_rtx_REG (GET_MODE (SET_DEST (PATTERN (cand
->insn
))),
826 REGNO (SET_DEST (*dest_sub_rtx
)));
827 if (reg_overlap_mentioned_p (tmp_reg
, SET_DEST (PATTERN (cand
->insn
))))
830 /* The destination register of the extension insn must not be
831 used or set between the def_insn and cand->insn exclusive. */
832 if (reg_used_between_p (SET_DEST (PATTERN (cand
->insn
)),
833 def_insn
, cand
->insn
)
834 || reg_set_between_p (SET_DEST (PATTERN (cand
->insn
)),
835 def_insn
, cand
->insn
))
838 /* We must be able to copy between the two registers. Generate,
839 recognize and verify constraints of the copy. Also fail if this
840 generated more than one insn.
842 This generates garbage since we throw away the insn when we're
843 done, only to recreate it later if this test was successful.
845 Make sure to get the mode from the extension (cand->insn). This
846 is different than in the code to emit the copy as we have not
847 modified the defining insn yet. */
849 rtx pat
= PATTERN (cand
->insn
);
850 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (pat
)),
851 REGNO (get_extended_src_reg (SET_SRC (pat
))));
852 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (pat
)),
853 REGNO (SET_DEST (pat
)));
854 emit_move_insn (new_dst
, new_src
);
856 rtx_insn
*insn
= get_insns();
858 if (NEXT_INSN (insn
))
860 if (recog_memoized (insn
) == -1)
863 if (!constrain_operands (1, get_preferred_alternatives (insn
, bb
)))
868 /* If cand->insn has been already modified, update cand->mode to a wider
869 mode if possible, or punt. */
870 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
875 if (state
->modified
[INSN_UID (cand
->insn
)].kind
876 != (cand
->code
== ZERO_EXTEND
877 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
)
878 || state
->modified
[INSN_UID (cand
->insn
)].mode
!= cand
->mode
879 || (set
= single_set (cand
->insn
)) == NULL_RTX
)
881 mode
= GET_MODE (SET_DEST (set
));
882 gcc_assert (GET_MODE_SIZE (mode
) >= GET_MODE_SIZE (cand
->mode
));
886 merge_successful
= true;
888 /* Go through the defs vector and try to merge all the definitions
890 state
->modified_list
.truncate (0);
891 FOR_EACH_VEC_ELT (state
->defs_list
, defs_ix
, def_insn
)
893 if (merge_def_and_ext (cand
, def_insn
, state
))
894 state
->modified_list
.safe_push (def_insn
);
897 merge_successful
= false;
902 /* Now go through the conditional copies vector and try to merge all
903 the copies in this vector. */
904 if (merge_successful
)
906 FOR_EACH_VEC_ELT (state
->copies_list
, i
, def_insn
)
908 if (transform_ifelse (cand
, def_insn
))
909 state
->modified_list
.safe_push (def_insn
);
912 merge_successful
= false;
918 if (merge_successful
)
920 /* Commit the changes here if possible
921 FIXME: It's an all-or-nothing scenario. Even if only one definition
922 cannot be merged, we entirely give up. In the future, we should allow
923 extensions to be partially eliminated along those paths where the
924 definitions could be merged. */
925 if (apply_change_group ())
928 fprintf (dump_file
, "All merges were successful.\n");
930 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
932 ext_modified
*modified
= &state
->modified
[INSN_UID (def_insn
)];
933 if (modified
->kind
== EXT_MODIFIED_NONE
)
934 modified
->kind
= (cand
->code
== ZERO_EXTEND
? EXT_MODIFIED_ZEXT
935 : EXT_MODIFIED_SEXT
);
938 modified
->do_not_reextend
= 1;
944 /* Changes need not be cancelled explicitly as apply_change_group
945 does it. Print list of definitions in the dump_file for debug
946 purposes. This extension cannot be deleted. */
950 "Merge cancelled, non-mergeable definitions:\n");
951 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
952 print_rtl_single (dump_file
, def_insn
);
958 /* Cancel any changes that have been made so far. */
965 /* Add an extension pattern that could be eliminated. */
968 add_removable_extension (const_rtx expr
, rtx_insn
*insn
,
969 vec
<ext_cand
> *insn_list
,
978 /* We are looking for SET (REG N) (ANY_EXTEND (REG N)). */
979 if (GET_CODE (expr
) != SET
)
982 src
= SET_SRC (expr
);
983 code
= GET_CODE (src
);
984 dest
= SET_DEST (expr
);
985 mode
= GET_MODE (dest
);
988 && (code
== SIGN_EXTEND
|| code
== ZERO_EXTEND
)
989 && REG_P (XEXP (src
, 0)))
991 rtx reg
= XEXP (src
, 0);
992 struct df_link
*defs
, *def
;
995 /* Zero-extension of an undefined value is partly defined (it's
996 completely undefined for sign-extension, though). So if there exists
997 a path from the entry to this zero-extension that leaves this register
998 uninitialized, removing the extension could change the behavior of
999 correct programs. So first, check it is not the case. */
1000 if (code
== ZERO_EXTEND
&& !bitmap_bit_p (init_regs
, REGNO (reg
)))
1004 fprintf (dump_file
, "Cannot eliminate extension:\n");
1005 print_rtl_single (dump_file
, insn
);
1006 fprintf (dump_file
, " because it can operate on uninitialized"
1012 /* Second, make sure we can get all the reaching definitions. */
1013 defs
= get_defs (insn
, reg
, NULL
);
1018 fprintf (dump_file
, "Cannot eliminate extension:\n");
1019 print_rtl_single (dump_file
, insn
);
1020 fprintf (dump_file
, " because of missing definition(s)\n");
1025 /* Third, make sure the reaching definitions don't feed another and
1026 different extension. FIXME: this obviously can be improved. */
1027 for (def
= defs
; def
; def
= def
->next
)
1028 if ((idx
= def_map
[INSN_UID (DF_REF_INSN (def
->ref
))])
1030 && (cand
= &(*insn_list
)[idx
- 1])
1031 && cand
->code
!= code
)
1035 fprintf (dump_file
, "Cannot eliminate extension:\n");
1036 print_rtl_single (dump_file
, insn
);
1037 fprintf (dump_file
, " because of other extension\n");
1041 /* For vector mode extensions, ensure that all uses of the
1042 XEXP (src, 0) register are in insn or debug insns, as unlike
1043 integral extensions lowpart subreg of the sign/zero extended
1044 register are not equal to the original register, so we have
1045 to change all uses or none and the current code isn't able
1046 to change them all at once in one transaction. */
1047 else if (VECTOR_MODE_P (GET_MODE (XEXP (src
, 0))))
1051 struct df_link
*ref_chain
, *ref_link
;
1053 ref_chain
= DF_REF_CHAIN (def
->ref
);
1054 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
1056 if (ref_link
->ref
== NULL
1057 || DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
1062 rtx_insn
*use_insn
= DF_REF_INSN (ref_link
->ref
);
1063 if (use_insn
!= insn
&& !DEBUG_INSN_P (use_insn
))
1070 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1076 fprintf (dump_file
, "Cannot eliminate extension:\n");
1077 print_rtl_single (dump_file
, insn
);
1079 " because some vector uses aren't extension\n");
1085 /* Fourth, if the extended version occupies more registers than the
1086 original and the source of the extension is the same hard register
1087 as the destination of the extension, then we can not eliminate
1088 the extension without deep analysis, so just punt.
1090 We allow this when the registers are different because the
1091 code in combine_reaching_defs will handle that case correctly. */
1092 if ((HARD_REGNO_NREGS (REGNO (dest
), mode
)
1093 != HARD_REGNO_NREGS (REGNO (reg
), GET_MODE (reg
)))
1094 && reg_overlap_mentioned_p (dest
, reg
))
1097 /* Then add the candidate to the list and insert the reaching definitions
1098 into the definition map. */
1099 ext_cand e
= {expr
, code
, mode
, insn
};
1100 insn_list
->safe_push (e
);
1101 idx
= insn_list
->length ();
1103 for (def
= defs
; def
; def
= def
->next
)
1104 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1108 /* Traverse the instruction stream looking for extensions and return the
1109 list of candidates. */
1111 static vec
<ext_cand
>
1112 find_removable_extensions (void)
1114 vec
<ext_cand
> insn_list
= vNULL
;
1118 unsigned *def_map
= XCNEWVEC (unsigned, max_insn_uid
);
1119 bitmap_head init
, kill
, gen
, tmp
;
1121 bitmap_initialize (&init
, NULL
);
1122 bitmap_initialize (&kill
, NULL
);
1123 bitmap_initialize (&gen
, NULL
);
1124 bitmap_initialize (&tmp
, NULL
);
1126 FOR_EACH_BB_FN (bb
, cfun
)
1128 bitmap_copy (&init
, DF_MIR_IN (bb
));
1129 bitmap_clear (&kill
);
1130 bitmap_clear (&gen
);
1132 FOR_BB_INSNS (bb
, insn
)
1134 if (NONDEBUG_INSN_P (insn
))
1136 set
= single_set (insn
);
1137 if (set
!= NULL_RTX
)
1138 add_removable_extension (set
, insn
, &insn_list
, def_map
,
1140 df_mir_simulate_one_insn (bb
, insn
, &kill
, &gen
);
1141 bitmap_ior_and_compl (&tmp
, &gen
, &init
, &kill
);
1142 bitmap_copy (&init
, &tmp
);
1147 XDELETEVEC (def_map
);
1152 /* This is the main function that checks the insn stream for redundant
1153 extensions and tries to remove them if possible. */
1156 find_and_remove_re (void)
1158 ext_cand
*curr_cand
;
1159 rtx_insn
*curr_insn
= NULL
;
1160 int num_re_opportunities
= 0, num_realized
= 0, i
;
1161 vec
<ext_cand
> reinsn_list
;
1162 auto_vec
<rtx_insn
*> reinsn_del_list
;
1163 auto_vec
<rtx_insn
*> reinsn_copy_list
;
1165 /* Construct DU chain to get all reaching definitions of each
1166 extension instruction. */
1167 df_set_flags (DF_RD_PRUNE_DEAD_DEFS
);
1168 df_chain_add_problem (DF_UD_CHAIN
+ DF_DU_CHAIN
);
1169 df_mir_add_problem ();
1171 df_set_flags (DF_DEFER_INSN_RESCAN
);
1173 max_insn_uid
= get_max_uid ();
1174 reinsn_list
= find_removable_extensions ();
1177 if (reinsn_list
.is_empty ())
1178 state
.modified
= NULL
;
1180 state
.modified
= XCNEWVEC (struct ext_modified
, max_insn_uid
);
1182 FOR_EACH_VEC_ELT (reinsn_list
, i
, curr_cand
)
1184 num_re_opportunities
++;
1186 /* Try to combine the extension with the definition. */
1189 fprintf (dump_file
, "Trying to eliminate extension:\n");
1190 print_rtl_single (dump_file
, curr_cand
->insn
);
1193 if (combine_reaching_defs (curr_cand
, curr_cand
->expr
, &state
))
1196 fprintf (dump_file
, "Eliminated the extension.\n");
1198 /* If the RHS of the current candidate is not (extend (reg)), then
1199 we do not allow the optimization of extensions where
1200 the source and destination registers do not match. Thus
1201 checking REG_P here is correct. */
1202 if (REG_P (XEXP (SET_SRC (PATTERN (curr_cand
->insn
)), 0))
1203 && (REGNO (SET_DEST (PATTERN (curr_cand
->insn
)))
1204 != REGNO (XEXP (SET_SRC (PATTERN (curr_cand
->insn
)), 0))))
1206 reinsn_copy_list
.safe_push (curr_cand
->insn
);
1207 reinsn_copy_list
.safe_push (state
.defs_list
[0]);
1209 reinsn_del_list
.safe_push (curr_cand
->insn
);
1210 state
.modified
[INSN_UID (curr_cand
->insn
)].deleted
= 1;
1214 /* The copy list contains pairs of insns which describe copies we
1215 need to insert into the INSN stream.
1217 The first insn in each pair is the extension insn, from which
1218 we derive the source and destination of the copy.
1220 The second insn in each pair is the memory reference where the
1221 extension will ultimately happen. We emit the new copy
1222 immediately after this insn.
1224 It may first appear that the arguments for the copy are reversed.
1225 Remember that the memory reference will be changed to refer to the
1226 destination of the extention. So we're actually emitting a copy
1227 from the new destination to the old destination. */
1228 for (unsigned int i
= 0; i
< reinsn_copy_list
.length (); i
+= 2)
1230 rtx_insn
*curr_insn
= reinsn_copy_list
[i
];
1231 rtx_insn
*def_insn
= reinsn_copy_list
[i
+ 1];
1233 /* Use the mode of the destination of the defining insn
1234 for the mode of the copy. This is necessary if the
1235 defining insn was used to eliminate a second extension
1236 that was wider than the first. */
1237 rtx sub_rtx
= *get_sub_rtx (def_insn
);
1238 rtx pat
= PATTERN (curr_insn
);
1239 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1240 REGNO (XEXP (SET_SRC (pat
), 0)));
1241 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1242 REGNO (SET_DEST (pat
)));
1243 rtx set
= gen_rtx_SET (new_dst
, new_src
);
1244 emit_insn_after (set
, def_insn
);
1247 /* Delete all useless extensions here in one sweep. */
1248 FOR_EACH_VEC_ELT (reinsn_del_list
, i
, curr_insn
)
1249 delete_insn (curr_insn
);
1251 reinsn_list
.release ();
1252 XDELETEVEC (state
.modified
);
1254 if (dump_file
&& num_re_opportunities
> 0)
1255 fprintf (dump_file
, "Elimination opportunities = %d realized = %d\n",
1256 num_re_opportunities
, num_realized
);
1259 /* Find and remove redundant extensions. */
1262 rest_of_handle_ree (void)
1264 find_and_remove_re ();
1270 const pass_data pass_data_ree
=
1272 RTL_PASS
, /* type */
1274 OPTGROUP_NONE
, /* optinfo_flags */
1276 0, /* properties_required */
1277 0, /* properties_provided */
1278 0, /* properties_destroyed */
1279 0, /* todo_flags_start */
1280 TODO_df_finish
, /* todo_flags_finish */
1283 class pass_ree
: public rtl_opt_pass
1286 pass_ree (gcc::context
*ctxt
)
1287 : rtl_opt_pass (pass_data_ree
, ctxt
)
1290 /* opt_pass methods: */
1291 virtual bool gate (function
*) { return (optimize
> 0 && flag_ree
); }
1292 virtual unsigned int execute (function
*) { return rest_of_handle_ree (); }
1294 }; // class pass_ree
1299 make_pass_ree (gcc::context
*ctxt
)
1301 return new pass_ree (ctxt
);