1 /* Redundant Extension Elimination pass for the GNU compiler.
2 Copyright (C) 2010-2015 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"
229 #include "hard-reg-set.h"
231 #include "function.h"
232 #include "dominance.h"
235 #include "basic-block.h"
236 #include "insn-config.h"
241 #include "emit-rtl.h"
245 #include "insn-attr.h"
247 #include "diagnostic-core.h"
249 #include "insn-codes.h"
251 #include "rtlhooks-def.h"
253 #include "tree-pass.h"
257 /* This structure represents a candidate for elimination. */
259 typedef struct ext_cand
261 /* The expression. */
264 /* The kind of extension. */
267 /* The destination mode. */
270 /* The instruction where it lives. */
275 static int max_insn_uid
;
277 /* Update or remove REG_EQUAL or REG_EQUIV notes for INSN. */
280 update_reg_equal_equiv_notes (rtx_insn
*insn
, machine_mode new_mode
,
281 machine_mode old_mode
, enum rtx_code code
)
283 rtx
*loc
= ®_NOTES (insn
);
286 enum reg_note kind
= REG_NOTE_KIND (*loc
);
287 if (kind
== REG_EQUAL
|| kind
== REG_EQUIV
)
289 rtx orig_src
= XEXP (*loc
, 0);
290 /* Update equivalency constants. Recall that RTL constants are
292 if (GET_CODE (orig_src
) == CONST_INT
293 && HOST_BITS_PER_WIDE_INT
>= GET_MODE_BITSIZE (new_mode
))
295 if (INTVAL (orig_src
) >= 0 || code
== SIGN_EXTEND
)
296 /* Nothing needed. */;
299 /* Zero-extend the negative constant by masking out the
300 bits outside the source mode. */
302 = gen_int_mode (INTVAL (orig_src
)
303 & GET_MODE_MASK (old_mode
),
305 if (!validate_change (insn
, &XEXP (*loc
, 0),
306 new_const_int
, true))
309 loc
= &XEXP (*loc
, 1);
311 /* Drop all other notes, they assume a wrong mode. */
312 else if (!validate_change (insn
, loc
, XEXP (*loc
, 1), true))
316 loc
= &XEXP (*loc
, 1);
321 /* Given a insn (CURR_INSN), an extension candidate for removal (CAND)
322 and a pointer to the SET rtx (ORIG_SET) that needs to be modified,
323 this code modifies the SET rtx to a new SET rtx that extends the
324 right hand expression into a register on the left hand side. Note
325 that multiple assumptions are made about the nature of the set that
326 needs to be true for this to work and is called from merge_def_and_ext.
329 (set (reg a) (expression))
332 (set (reg a) (any_extend (expression)))
335 If the expression is a constant or another extension, then directly
336 assign it to the register. */
339 combine_set_extension (ext_cand
*cand
, rtx_insn
*curr_insn
, rtx
*orig_set
)
341 rtx orig_src
= SET_SRC (*orig_set
);
342 machine_mode orig_mode
= GET_MODE (SET_DEST (*orig_set
));
344 rtx cand_pat
= PATTERN (cand
->insn
);
346 /* If the extension's source/destination registers are not the same
347 then we need to change the original load to reference the destination
348 of the extension. Then we need to emit a copy from that destination
349 to the original destination of the load. */
352 = (REGNO (SET_DEST (cand_pat
)) != REGNO (XEXP (SET_SRC (cand_pat
), 0)));
354 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (cand_pat
)));
356 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (*orig_set
)));
359 /* Rethinking test. Temporarily disabled. */
360 /* We're going to be widening the result of DEF_INSN, ensure that doing so
361 doesn't change the number of hard registers needed for the result. */
362 if (HARD_REGNO_NREGS (REGNO (new_reg
), cand
->mode
)
363 != HARD_REGNO_NREGS (REGNO (SET_DEST (*orig_set
)),
364 GET_MODE (SET_DEST (*orig_set
))))
368 /* Merge constants by directly moving the constant into the register under
369 some conditions. Recall that RTL constants are sign-extended. */
370 if (GET_CODE (orig_src
) == CONST_INT
371 && HOST_BITS_PER_WIDE_INT
>= GET_MODE_BITSIZE (cand
->mode
))
373 if (INTVAL (orig_src
) >= 0 || cand
->code
== SIGN_EXTEND
)
374 new_set
= gen_rtx_SET (new_reg
, orig_src
);
377 /* Zero-extend the negative constant by masking out the bits outside
380 = gen_int_mode (INTVAL (orig_src
) & GET_MODE_MASK (orig_mode
),
382 new_set
= gen_rtx_SET (new_reg
, new_const_int
);
385 else if (GET_MODE (orig_src
) == VOIDmode
)
387 /* This is mostly due to a call insn that should not be optimized. */
390 else if (GET_CODE (orig_src
) == cand
->code
)
392 /* Here is a sequence of two extensions. Try to merge them. */
394 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, XEXP (orig_src
, 0));
395 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
396 if (simplified_temp_extension
)
397 temp_extension
= simplified_temp_extension
;
398 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
400 else if (GET_CODE (orig_src
) == IF_THEN_ELSE
)
402 /* Only IF_THEN_ELSE of phi-type copies are combined. Otherwise,
403 in general, IF_THEN_ELSE should not be combined. */
408 /* This is the normal case. */
410 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, orig_src
);
411 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
412 if (simplified_temp_extension
)
413 temp_extension
= simplified_temp_extension
;
414 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
417 /* This change is a part of a group of changes. Hence,
418 validate_change will not try to commit the change. */
419 if (validate_change (curr_insn
, orig_set
, new_set
, true)
420 && update_reg_equal_equiv_notes (curr_insn
, cand
->mode
, orig_mode
,
426 "Tentatively merged extension with definition %s:\n",
427 (copy_needed
) ? "(copy needed)" : "");
428 print_rtl_single (dump_file
, curr_insn
);
436 /* Treat if_then_else insns, where the operands of both branches
437 are registers, as copies. For instance,
439 (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c)))
441 (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c)))
442 DEF_INSN is the if_then_else insn. */
445 transform_ifelse (ext_cand
*cand
, rtx_insn
*def_insn
)
447 rtx set_insn
= PATTERN (def_insn
);
448 rtx srcreg
, dstreg
, srcreg2
;
449 rtx map_srcreg
, map_dstreg
, map_srcreg2
;
454 gcc_assert (GET_CODE (set_insn
) == SET
);
456 cond
= XEXP (SET_SRC (set_insn
), 0);
457 dstreg
= SET_DEST (set_insn
);
458 srcreg
= XEXP (SET_SRC (set_insn
), 1);
459 srcreg2
= XEXP (SET_SRC (set_insn
), 2);
460 /* If the conditional move already has the right or wider mode,
461 there is nothing to do. */
462 if (GET_MODE_SIZE (GET_MODE (dstreg
)) >= GET_MODE_SIZE (cand
->mode
))
465 map_srcreg
= gen_rtx_REG (cand
->mode
, REGNO (srcreg
));
466 map_srcreg2
= gen_rtx_REG (cand
->mode
, REGNO (srcreg2
));
467 map_dstreg
= gen_rtx_REG (cand
->mode
, REGNO (dstreg
));
468 ifexpr
= gen_rtx_IF_THEN_ELSE (cand
->mode
, cond
, map_srcreg
, map_srcreg2
);
469 new_set
= gen_rtx_SET (map_dstreg
, ifexpr
);
471 if (validate_change (def_insn
, &PATTERN (def_insn
), new_set
, true)
472 && update_reg_equal_equiv_notes (def_insn
, cand
->mode
, GET_MODE (dstreg
),
478 "Mode of conditional move instruction extended:\n");
479 print_rtl_single (dump_file
, def_insn
);
487 /* Get all the reaching definitions of an instruction. The definitions are
488 desired for REG used in INSN. Return the definition list or NULL if a
489 definition is missing. If DEST is non-NULL, additionally push the INSN
490 of the definitions onto DEST. */
492 static struct df_link
*
493 get_defs (rtx_insn
*insn
, rtx reg
, vec
<rtx_insn
*> *dest
)
496 struct df_link
*ref_chain
, *ref_link
;
498 FOR_EACH_INSN_USE (use
, insn
)
500 if (GET_CODE (DF_REF_REG (use
)) == SUBREG
)
502 if (REGNO (DF_REF_REG (use
)) == REGNO (reg
))
506 gcc_assert (use
!= NULL
);
508 ref_chain
= DF_REF_CHAIN (use
);
510 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
512 /* Problem getting some definition for this instruction. */
513 if (ref_link
->ref
== NULL
)
515 if (DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
520 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
521 dest
->safe_push (DF_REF_INSN (ref_link
->ref
));
526 /* Return true if INSN is
527 (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2)))
528 and store x1 and x2 in REG_1 and REG_2. */
531 is_cond_copy_insn (rtx_insn
*insn
, rtx
*reg1
, rtx
*reg2
)
533 rtx expr
= single_set (insn
);
536 && GET_CODE (expr
) == SET
537 && GET_CODE (SET_DEST (expr
)) == REG
538 && GET_CODE (SET_SRC (expr
)) == IF_THEN_ELSE
539 && GET_CODE (XEXP (SET_SRC (expr
), 1)) == REG
540 && GET_CODE (XEXP (SET_SRC (expr
), 2)) == REG
)
542 *reg1
= XEXP (SET_SRC (expr
), 1);
543 *reg2
= XEXP (SET_SRC (expr
), 2);
550 enum ext_modified_kind
552 /* The insn hasn't been modified by ree pass yet. */
554 /* Changed into zero extension. */
556 /* Changed into sign extension. */
560 struct ATTRIBUTE_PACKED ext_modified
562 /* Mode from which ree has zero or sign extended the destination. */
563 ENUM_BITFIELD(machine_mode
) mode
: 8;
565 /* Kind of modification of the insn. */
566 ENUM_BITFIELD(ext_modified_kind
) kind
: 2;
568 unsigned int do_not_reextend
: 1;
570 /* True if the insn is scheduled to be deleted. */
571 unsigned int deleted
: 1;
574 /* Vectors used by combine_reaching_defs and its helpers. */
575 typedef struct ext_state
577 /* In order to avoid constant alloc/free, we keep these
578 4 vectors live through the entire find_and_remove_re and just
579 truncate them each time. */
580 vec
<rtx_insn
*> defs_list
;
581 vec
<rtx_insn
*> copies_list
;
582 vec
<rtx_insn
*> modified_list
;
583 vec
<rtx_insn
*> work_list
;
585 /* For instructions that have been successfully modified, this is
586 the original mode from which the insn is extending and
587 kind of extension. */
588 struct ext_modified
*modified
;
591 /* Reaching Definitions of the extended register could be conditional copies
592 or regular definitions. This function separates the two types into two
593 lists, STATE->DEFS_LIST and STATE->COPIES_LIST. This is necessary because,
594 if a reaching definition is a conditional copy, merging the extension with
595 this definition is wrong. Conditional copies are merged by transitively
596 merging their definitions. The defs_list is populated with all the reaching
597 definitions of the extension instruction (EXTEND_INSN) which must be merged
598 with an extension. The copies_list contains all the conditional moves that
599 will later be extended into a wider mode conditional move if all the merges
600 are successful. The function returns false upon failure, true upon
604 make_defs_and_copies_lists (rtx_insn
*extend_insn
, const_rtx set_pat
,
607 rtx src_reg
= XEXP (SET_SRC (set_pat
), 0);
608 bool *is_insn_visited
;
611 state
->work_list
.truncate (0);
613 /* Initialize the work list. */
614 if (!get_defs (extend_insn
, src_reg
, &state
->work_list
))
617 is_insn_visited
= XCNEWVEC (bool, max_insn_uid
);
619 /* Perform transitive closure for conditional copies. */
620 while (!state
->work_list
.is_empty ())
622 rtx_insn
*def_insn
= state
->work_list
.pop ();
625 gcc_assert (INSN_UID (def_insn
) < max_insn_uid
);
627 if (is_insn_visited
[INSN_UID (def_insn
)])
629 is_insn_visited
[INSN_UID (def_insn
)] = true;
631 if (is_cond_copy_insn (def_insn
, ®1
, ®2
))
633 /* Push it onto the copy list first. */
634 state
->copies_list
.safe_push (def_insn
);
636 /* Now perform the transitive closure. */
637 if (!get_defs (def_insn
, reg1
, &state
->work_list
)
638 || !get_defs (def_insn
, reg2
, &state
->work_list
))
645 state
->defs_list
.safe_push (def_insn
);
648 XDELETEVEC (is_insn_visited
);
653 /* If DEF_INSN has single SET expression, possibly buried inside
654 a PARALLEL, return the address of the SET expression, else
655 return NULL. This is similar to single_set, except that
656 single_set allows multiple SETs when all but one is dead. */
658 get_sub_rtx (rtx_insn
*def_insn
)
660 enum rtx_code code
= GET_CODE (PATTERN (def_insn
));
663 if (code
== PARALLEL
)
665 for (int i
= 0; i
< XVECLEN (PATTERN (def_insn
), 0); i
++)
667 rtx s_expr
= XVECEXP (PATTERN (def_insn
), 0, i
);
668 if (GET_CODE (s_expr
) != SET
)
672 sub_rtx
= &XVECEXP (PATTERN (def_insn
), 0, i
);
675 /* PARALLEL with multiple SETs. */
680 else if (code
== SET
)
681 sub_rtx
= &PATTERN (def_insn
);
684 /* It is not a PARALLEL or a SET, what could it be ? */
688 gcc_assert (sub_rtx
!= NULL
);
692 /* Merge the DEF_INSN with an extension. Calls combine_set_extension
693 on the SET pattern. */
696 merge_def_and_ext (ext_cand
*cand
, rtx_insn
*def_insn
, ext_state
*state
)
698 machine_mode ext_src_mode
;
701 ext_src_mode
= GET_MODE (XEXP (SET_SRC (cand
->expr
), 0));
702 sub_rtx
= get_sub_rtx (def_insn
);
707 if (REG_P (SET_DEST (*sub_rtx
))
708 && (GET_MODE (SET_DEST (*sub_rtx
)) == ext_src_mode
709 || ((state
->modified
[INSN_UID (def_insn
)].kind
710 == (cand
->code
== ZERO_EXTEND
711 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
))
712 && state
->modified
[INSN_UID (def_insn
)].mode
715 if (GET_MODE_SIZE (GET_MODE (SET_DEST (*sub_rtx
)))
716 >= GET_MODE_SIZE (cand
->mode
))
718 /* If def_insn is already scheduled to be deleted, don't attempt
720 if (state
->modified
[INSN_UID (def_insn
)].deleted
)
722 if (combine_set_extension (cand
, def_insn
, sub_rtx
))
724 if (state
->modified
[INSN_UID (def_insn
)].kind
== EXT_MODIFIED_NONE
)
725 state
->modified
[INSN_UID (def_insn
)].mode
= ext_src_mode
;
733 /* Given SRC, which should be one or more extensions of a REG, strip
734 away the extensions and return the REG. */
737 get_extended_src_reg (rtx src
)
739 while (GET_CODE (src
) == SIGN_EXTEND
|| GET_CODE (src
) == ZERO_EXTEND
)
741 gcc_assert (REG_P (src
));
745 /* This function goes through all reaching defs of the source
746 of the candidate for elimination (CAND) and tries to combine
747 the extension with the definition instruction. The changes
748 are made as a group so that even if one definition cannot be
749 merged, all reaching definitions end up not being merged.
750 When a conditional copy is encountered, merging is attempted
751 transitively on its definitions. It returns true upon success
752 and false upon failure. */
755 combine_reaching_defs (ext_cand
*cand
, const_rtx set_pat
, ext_state
*state
)
758 bool merge_successful
= true;
763 state
->defs_list
.truncate (0);
764 state
->copies_list
.truncate (0);
766 outcome
= make_defs_and_copies_lists (cand
->insn
, set_pat
, state
);
771 /* If the destination operand of the extension is a different
772 register than the source operand, then additional restrictions
773 are needed. Note we have to handle cases where we have nested
774 extensions in the source operand. */
776 = (REGNO (SET_DEST (PATTERN (cand
->insn
)))
777 != REGNO (get_extended_src_reg (SET_SRC (PATTERN (cand
->insn
)))));
780 /* Considering transformation of
781 (set (reg1) (expression))
783 (set (reg2) (any_extend (reg1)))
787 (set (reg2) (any_extend (expression)))
791 /* In theory we could handle more than one reaching def, it
792 just makes the code to update the insn stream more complex. */
793 if (state
->defs_list
.length () != 1)
796 /* We require the candidate not already be modified. It may,
797 for example have been changed from a (sign_extend (reg))
798 into (zero_extend (sign_extend (reg))).
800 Handling that case shouldn't be terribly difficult, but the code
801 here and the code to emit copies would need auditing. Until
802 we see a need, this is the safe thing to do. */
803 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
806 machine_mode dst_mode
= GET_MODE (SET_DEST (PATTERN (cand
->insn
)));
807 rtx src_reg
= get_extended_src_reg (SET_SRC (PATTERN (cand
->insn
)));
809 /* Ensure the number of hard registers of the copy match. */
810 if (HARD_REGNO_NREGS (REGNO (src_reg
), dst_mode
)
811 != HARD_REGNO_NREGS (REGNO (src_reg
), GET_MODE (src_reg
)))
814 /* There's only one reaching def. */
815 rtx_insn
*def_insn
= state
->defs_list
[0];
817 /* The defining statement must not have been modified either. */
818 if (state
->modified
[INSN_UID (def_insn
)].kind
!= EXT_MODIFIED_NONE
)
821 /* The defining statement and candidate insn must be in the same block.
822 This is merely to keep the test for safety and updating the insn
823 stream simple. Also ensure that within the block the candidate
824 follows the defining insn. */
825 basic_block bb
= BLOCK_FOR_INSN (cand
->insn
);
826 if (bb
!= BLOCK_FOR_INSN (def_insn
)
827 || DF_INSN_LUID (def_insn
) > DF_INSN_LUID (cand
->insn
))
830 /* If there is an overlap between the destination of DEF_INSN and
831 CAND->insn, then this transformation is not safe. Note we have
832 to test in the widened mode. */
833 rtx
*dest_sub_rtx
= get_sub_rtx (def_insn
);
834 if (dest_sub_rtx
== NULL
835 || !REG_P (SET_DEST (*dest_sub_rtx
)))
838 rtx tmp_reg
= gen_rtx_REG (GET_MODE (SET_DEST (PATTERN (cand
->insn
))),
839 REGNO (SET_DEST (*dest_sub_rtx
)));
840 if (reg_overlap_mentioned_p (tmp_reg
, SET_DEST (PATTERN (cand
->insn
))))
843 /* The destination register of the extension insn must not be
844 used or set between the def_insn and cand->insn exclusive. */
845 if (reg_used_between_p (SET_DEST (PATTERN (cand
->insn
)),
846 def_insn
, cand
->insn
)
847 || reg_set_between_p (SET_DEST (PATTERN (cand
->insn
)),
848 def_insn
, cand
->insn
))
851 /* We must be able to copy between the two registers. Generate,
852 recognize and verify constraints of the copy. Also fail if this
853 generated more than one insn.
855 This generates garbage since we throw away the insn when we're
856 done, only to recreate it later if this test was successful.
858 Make sure to get the mode from the extension (cand->insn). This
859 is different than in the code to emit the copy as we have not
860 modified the defining insn yet. */
862 rtx pat
= PATTERN (cand
->insn
);
863 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (pat
)),
864 REGNO (get_extended_src_reg (SET_SRC (pat
))));
865 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (pat
)),
866 REGNO (SET_DEST (pat
)));
867 emit_move_insn (new_dst
, new_src
);
869 rtx_insn
*insn
= get_insns();
871 if (NEXT_INSN (insn
))
873 if (recog_memoized (insn
) == -1)
876 if (!constrain_operands (1, get_preferred_alternatives (insn
, bb
)))
881 /* If cand->insn has been already modified, update cand->mode to a wider
882 mode if possible, or punt. */
883 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
888 if (state
->modified
[INSN_UID (cand
->insn
)].kind
889 != (cand
->code
== ZERO_EXTEND
890 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
)
891 || state
->modified
[INSN_UID (cand
->insn
)].mode
!= cand
->mode
892 || (set
= single_set (cand
->insn
)) == NULL_RTX
)
894 mode
= GET_MODE (SET_DEST (set
));
895 gcc_assert (GET_MODE_SIZE (mode
) >= GET_MODE_SIZE (cand
->mode
));
899 merge_successful
= true;
901 /* Go through the defs vector and try to merge all the definitions
903 state
->modified_list
.truncate (0);
904 FOR_EACH_VEC_ELT (state
->defs_list
, defs_ix
, def_insn
)
906 if (merge_def_and_ext (cand
, def_insn
, state
))
907 state
->modified_list
.safe_push (def_insn
);
910 merge_successful
= false;
915 /* Now go through the conditional copies vector and try to merge all
916 the copies in this vector. */
917 if (merge_successful
)
919 FOR_EACH_VEC_ELT (state
->copies_list
, i
, def_insn
)
921 if (transform_ifelse (cand
, def_insn
))
922 state
->modified_list
.safe_push (def_insn
);
925 merge_successful
= false;
931 if (merge_successful
)
933 /* Commit the changes here if possible
934 FIXME: It's an all-or-nothing scenario. Even if only one definition
935 cannot be merged, we entirely give up. In the future, we should allow
936 extensions to be partially eliminated along those paths where the
937 definitions could be merged. */
938 if (apply_change_group ())
941 fprintf (dump_file
, "All merges were successful.\n");
943 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
945 ext_modified
*modified
= &state
->modified
[INSN_UID (def_insn
)];
946 if (modified
->kind
== EXT_MODIFIED_NONE
)
947 modified
->kind
= (cand
->code
== ZERO_EXTEND
? EXT_MODIFIED_ZEXT
948 : EXT_MODIFIED_SEXT
);
951 modified
->do_not_reextend
= 1;
957 /* Changes need not be cancelled explicitly as apply_change_group
958 does it. Print list of definitions in the dump_file for debug
959 purposes. This extension cannot be deleted. */
963 "Merge cancelled, non-mergeable definitions:\n");
964 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
965 print_rtl_single (dump_file
, def_insn
);
971 /* Cancel any changes that have been made so far. */
978 /* Add an extension pattern that could be eliminated. */
981 add_removable_extension (const_rtx expr
, rtx_insn
*insn
,
982 vec
<ext_cand
> *insn_list
,
990 /* We are looking for SET (REG N) (ANY_EXTEND (REG N)). */
991 if (GET_CODE (expr
) != SET
)
994 src
= SET_SRC (expr
);
995 code
= GET_CODE (src
);
996 dest
= SET_DEST (expr
);
997 mode
= GET_MODE (dest
);
1000 && (code
== SIGN_EXTEND
|| code
== ZERO_EXTEND
)
1001 && REG_P (XEXP (src
, 0)))
1003 struct df_link
*defs
, *def
;
1006 /* First, make sure we can get all the reaching definitions. */
1007 defs
= get_defs (insn
, XEXP (src
, 0), NULL
);
1012 fprintf (dump_file
, "Cannot eliminate extension:\n");
1013 print_rtl_single (dump_file
, insn
);
1014 fprintf (dump_file
, " because of missing definition(s)\n");
1019 /* Second, make sure the reaching definitions don't feed another and
1020 different extension. FIXME: this obviously can be improved. */
1021 for (def
= defs
; def
; def
= def
->next
)
1022 if ((idx
= def_map
[INSN_UID (DF_REF_INSN (def
->ref
))])
1024 && (cand
= &(*insn_list
)[idx
- 1])
1025 && cand
->code
!= code
)
1029 fprintf (dump_file
, "Cannot eliminate extension:\n");
1030 print_rtl_single (dump_file
, insn
);
1031 fprintf (dump_file
, " because of other extension\n");
1035 /* For vector mode extensions, ensure that all uses of the
1036 XEXP (src, 0) register are the same extension (both code
1037 and to which mode), as unlike integral extensions lowpart
1038 subreg of the sign/zero extended register are not equal
1039 to the original register, so we have to change all uses or
1041 else if (VECTOR_MODE_P (GET_MODE (XEXP (src
, 0))))
1045 struct df_link
*ref_chain
, *ref_link
;
1047 ref_chain
= DF_REF_CHAIN (def
->ref
);
1048 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
1050 if (ref_link
->ref
== NULL
1051 || DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
1056 rtx_insn
*use_insn
= DF_REF_INSN (ref_link
->ref
);
1058 if (use_insn
== insn
|| DEBUG_INSN_P (use_insn
))
1060 if (!(use_set
= single_set (use_insn
))
1061 || !REG_P (SET_DEST (use_set
))
1062 || GET_MODE (SET_DEST (use_set
)) != GET_MODE (dest
)
1063 || GET_CODE (SET_SRC (use_set
)) != code
1064 || !rtx_equal_p (XEXP (SET_SRC (use_set
), 0),
1072 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1078 fprintf (dump_file
, "Cannot eliminate extension:\n");
1079 print_rtl_single (dump_file
, insn
);
1081 " because some vector uses aren't extension\n");
1087 /* Then add the candidate to the list and insert the reaching definitions
1088 into the definition map. */
1089 ext_cand e
= {expr
, code
, mode
, insn
};
1090 insn_list
->safe_push (e
);
1091 idx
= insn_list
->length ();
1093 for (def
= defs
; def
; def
= def
->next
)
1094 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1098 /* Traverse the instruction stream looking for extensions and return the
1099 list of candidates. */
1101 static vec
<ext_cand
>
1102 find_removable_extensions (void)
1104 vec
<ext_cand
> insn_list
= vNULL
;
1108 unsigned *def_map
= XCNEWVEC (unsigned, max_insn_uid
);
1110 FOR_EACH_BB_FN (bb
, cfun
)
1111 FOR_BB_INSNS (bb
, insn
)
1113 if (!NONDEBUG_INSN_P (insn
))
1116 set
= single_set (insn
);
1117 if (set
== NULL_RTX
)
1119 add_removable_extension (set
, insn
, &insn_list
, def_map
);
1122 XDELETEVEC (def_map
);
1127 /* This is the main function that checks the insn stream for redundant
1128 extensions and tries to remove them if possible. */
1131 find_and_remove_re (void)
1133 ext_cand
*curr_cand
;
1134 rtx_insn
*curr_insn
= NULL
;
1135 int num_re_opportunities
= 0, num_realized
= 0, i
;
1136 vec
<ext_cand
> reinsn_list
;
1137 auto_vec
<rtx_insn
*> reinsn_del_list
;
1138 auto_vec
<rtx_insn
*> reinsn_copy_list
;
1141 /* Construct DU chain to get all reaching definitions of each
1142 extension instruction. */
1143 df_set_flags (DF_RD_PRUNE_DEAD_DEFS
);
1144 df_chain_add_problem (DF_UD_CHAIN
+ DF_DU_CHAIN
);
1146 df_set_flags (DF_DEFER_INSN_RESCAN
);
1148 max_insn_uid
= get_max_uid ();
1149 reinsn_list
= find_removable_extensions ();
1150 state
.defs_list
.create (0);
1151 state
.copies_list
.create (0);
1152 state
.modified_list
.create (0);
1153 state
.work_list
.create (0);
1154 if (reinsn_list
.is_empty ())
1155 state
.modified
= NULL
;
1157 state
.modified
= XCNEWVEC (struct ext_modified
, max_insn_uid
);
1159 FOR_EACH_VEC_ELT (reinsn_list
, i
, curr_cand
)
1161 num_re_opportunities
++;
1163 /* Try to combine the extension with the definition. */
1166 fprintf (dump_file
, "Trying to eliminate extension:\n");
1167 print_rtl_single (dump_file
, curr_cand
->insn
);
1170 if (combine_reaching_defs (curr_cand
, curr_cand
->expr
, &state
))
1173 fprintf (dump_file
, "Eliminated the extension.\n");
1175 /* If the RHS of the current candidate is not (extend (reg)), then
1176 we do not allow the optimization of extensions where
1177 the source and destination registers do not match. Thus
1178 checking REG_P here is correct. */
1179 if (REG_P (XEXP (SET_SRC (PATTERN (curr_cand
->insn
)), 0))
1180 && (REGNO (SET_DEST (PATTERN (curr_cand
->insn
)))
1181 != REGNO (XEXP (SET_SRC (PATTERN (curr_cand
->insn
)), 0))))
1183 reinsn_copy_list
.safe_push (curr_cand
->insn
);
1184 reinsn_copy_list
.safe_push (state
.defs_list
[0]);
1186 reinsn_del_list
.safe_push (curr_cand
->insn
);
1187 state
.modified
[INSN_UID (curr_cand
->insn
)].deleted
= 1;
1191 /* The copy list contains pairs of insns which describe copies we
1192 need to insert into the INSN stream.
1194 The first insn in each pair is the extension insn, from which
1195 we derive the source and destination of the copy.
1197 The second insn in each pair is the memory reference where the
1198 extension will ultimately happen. We emit the new copy
1199 immediately after this insn.
1201 It may first appear that the arguments for the copy are reversed.
1202 Remember that the memory reference will be changed to refer to the
1203 destination of the extention. So we're actually emitting a copy
1204 from the new destination to the old destination. */
1205 for (unsigned int i
= 0; i
< reinsn_copy_list
.length (); i
+= 2)
1207 rtx_insn
*curr_insn
= reinsn_copy_list
[i
];
1208 rtx_insn
*def_insn
= reinsn_copy_list
[i
+ 1];
1210 /* Use the mode of the destination of the defining insn
1211 for the mode of the copy. This is necessary if the
1212 defining insn was used to eliminate a second extension
1213 that was wider than the first. */
1214 rtx sub_rtx
= *get_sub_rtx (def_insn
);
1215 rtx pat
= PATTERN (curr_insn
);
1216 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1217 REGNO (XEXP (SET_SRC (pat
), 0)));
1218 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1219 REGNO (SET_DEST (pat
)));
1220 rtx set
= gen_rtx_SET (new_dst
, new_src
);
1221 emit_insn_after (set
, def_insn
);
1224 /* Delete all useless extensions here in one sweep. */
1225 FOR_EACH_VEC_ELT (reinsn_del_list
, i
, curr_insn
)
1226 delete_insn (curr_insn
);
1228 reinsn_list
.release ();
1229 state
.defs_list
.release ();
1230 state
.copies_list
.release ();
1231 state
.modified_list
.release ();
1232 state
.work_list
.release ();
1233 XDELETEVEC (state
.modified
);
1235 if (dump_file
&& num_re_opportunities
> 0)
1236 fprintf (dump_file
, "Elimination opportunities = %d realized = %d\n",
1237 num_re_opportunities
, num_realized
);
1240 /* Find and remove redundant extensions. */
1243 rest_of_handle_ree (void)
1245 timevar_push (TV_REE
);
1246 find_and_remove_re ();
1247 timevar_pop (TV_REE
);
1253 const pass_data pass_data_ree
=
1255 RTL_PASS
, /* type */
1257 OPTGROUP_NONE
, /* optinfo_flags */
1259 0, /* properties_required */
1260 0, /* properties_provided */
1261 0, /* properties_destroyed */
1262 0, /* todo_flags_start */
1263 TODO_df_finish
, /* todo_flags_finish */
1266 class pass_ree
: public rtl_opt_pass
1269 pass_ree (gcc::context
*ctxt
)
1270 : rtl_opt_pass (pass_data_ree
, ctxt
)
1273 /* opt_pass methods: */
1274 virtual bool gate (function
*) { return (optimize
> 0 && flag_ree
); }
1275 virtual unsigned int execute (function
*) { return rest_of_handle_ree (); }
1277 }; // class pass_ree
1282 make_pass_ree (gcc::context
*ctxt
)
1284 return new pass_ree (ctxt
);