1 /* Redundant Extension Elimination pass for the GNU compiler.
2 Copyright (C) 2010-2023 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"
230 #include "emit-rtl.h"
234 #include "tree-pass.h"
236 /* This structure represents a candidate for elimination. */
240 /* The expression. */
243 /* The kind of extension. */
246 /* The destination mode. */
249 /* The instruction where it lives. */
254 static int max_insn_uid
;
256 /* Update or remove REG_EQUAL or REG_EQUIV notes for INSN. */
259 update_reg_equal_equiv_notes (rtx_insn
*insn
, machine_mode new_mode
,
260 machine_mode old_mode
, enum rtx_code code
)
262 rtx
*loc
= ®_NOTES (insn
);
265 enum reg_note kind
= REG_NOTE_KIND (*loc
);
266 if (kind
== REG_EQUAL
|| kind
== REG_EQUIV
)
268 rtx orig_src
= XEXP (*loc
, 0);
269 /* Update equivalency constants. Recall that RTL constants are
271 if (GET_CODE (orig_src
) == CONST_INT
272 && HWI_COMPUTABLE_MODE_P (new_mode
))
274 if (INTVAL (orig_src
) >= 0 || code
== SIGN_EXTEND
)
275 /* Nothing needed. */;
278 /* Zero-extend the negative constant by masking out the
279 bits outside the source mode. */
281 = gen_int_mode (INTVAL (orig_src
)
282 & GET_MODE_MASK (old_mode
),
284 if (!validate_change (insn
, &XEXP (*loc
, 0),
285 new_const_int
, true))
288 loc
= &XEXP (*loc
, 1);
290 /* Drop all other notes, they assume a wrong mode. */
291 else if (!validate_change (insn
, loc
, XEXP (*loc
, 1), true))
295 loc
= &XEXP (*loc
, 1);
300 /* Given a insn (CURR_INSN), an extension candidate for removal (CAND)
301 and a pointer to the SET rtx (ORIG_SET) that needs to be modified,
302 this code modifies the SET rtx to a new SET rtx that extends the
303 right hand expression into a register on the left hand side. Note
304 that multiple assumptions are made about the nature of the set that
305 needs to be true for this to work and is called from merge_def_and_ext.
308 (set (reg a) (expression))
311 (set (reg a) (any_extend (expression)))
314 If the expression is a constant or another extension, then directly
315 assign it to the register. */
318 combine_set_extension (ext_cand
*cand
, rtx_insn
*curr_insn
, rtx
*orig_set
)
320 rtx orig_src
= SET_SRC (*orig_set
);
321 machine_mode orig_mode
= GET_MODE (SET_DEST (*orig_set
));
323 rtx cand_pat
= single_set (cand
->insn
);
325 /* If the extension's source/destination registers are not the same
326 then we need to change the original load to reference the destination
327 of the extension. Then we need to emit a copy from that destination
328 to the original destination of the load. */
331 = (REGNO (SET_DEST (cand_pat
)) != REGNO (XEXP (SET_SRC (cand_pat
), 0)));
333 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (cand_pat
)));
335 new_reg
= gen_rtx_REG (cand
->mode
, REGNO (SET_DEST (*orig_set
)));
337 /* Merge constants by directly moving the constant into the register under
338 some conditions. Recall that RTL constants are sign-extended. */
339 if (GET_CODE (orig_src
) == CONST_INT
340 && HWI_COMPUTABLE_MODE_P (cand
->mode
))
342 if (INTVAL (orig_src
) >= 0 || cand
->code
== SIGN_EXTEND
)
343 new_set
= gen_rtx_SET (new_reg
, orig_src
);
346 /* Zero-extend the negative constant by masking out the bits outside
349 = gen_int_mode (INTVAL (orig_src
) & GET_MODE_MASK (orig_mode
),
351 new_set
= gen_rtx_SET (new_reg
, new_const_int
);
354 else if (GET_MODE (orig_src
) == VOIDmode
)
356 /* This is mostly due to a call insn that should not be optimized. */
359 else if (GET_CODE (orig_src
) == cand
->code
)
361 /* Here is a sequence of two extensions. Try to merge them. */
363 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, XEXP (orig_src
, 0));
364 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
365 if (simplified_temp_extension
)
366 temp_extension
= simplified_temp_extension
;
367 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
369 else if (GET_CODE (orig_src
) == IF_THEN_ELSE
)
371 /* Only IF_THEN_ELSE of phi-type copies are combined. Otherwise,
372 in general, IF_THEN_ELSE should not be combined. */
377 /* This is the normal case. */
379 = gen_rtx_fmt_e (cand
->code
, cand
->mode
, orig_src
);
380 rtx simplified_temp_extension
= simplify_rtx (temp_extension
);
381 if (simplified_temp_extension
)
382 temp_extension
= simplified_temp_extension
;
383 new_set
= gen_rtx_SET (new_reg
, temp_extension
);
386 /* This change is a part of a group of changes. Hence,
387 validate_change will not try to commit the change. */
388 if (validate_change (curr_insn
, orig_set
, new_set
, true)
389 && update_reg_equal_equiv_notes (curr_insn
, cand
->mode
, orig_mode
,
395 "Tentatively merged extension with definition %s:\n",
396 (copy_needed
) ? "(copy needed)" : "");
397 print_rtl_single (dump_file
, curr_insn
);
405 /* Treat if_then_else insns, where the operands of both branches
406 are registers, as copies. For instance,
408 (set (reg:SI a) (if_then_else (cond) (reg:SI b) (reg:SI c)))
410 (set (reg:DI a) (if_then_else (cond) (reg:DI b) (reg:DI c)))
411 DEF_INSN is the if_then_else insn. */
414 transform_ifelse (ext_cand
*cand
, rtx_insn
*def_insn
)
416 rtx set_insn
= PATTERN (def_insn
);
417 rtx srcreg
, dstreg
, srcreg2
;
418 rtx map_srcreg
, map_dstreg
, map_srcreg2
;
423 gcc_assert (GET_CODE (set_insn
) == SET
);
425 cond
= XEXP (SET_SRC (set_insn
), 0);
426 dstreg
= SET_DEST (set_insn
);
427 srcreg
= XEXP (SET_SRC (set_insn
), 1);
428 srcreg2
= XEXP (SET_SRC (set_insn
), 2);
429 /* If the conditional move already has the right or wider mode,
430 there is nothing to do. */
431 if (GET_MODE_UNIT_SIZE (GET_MODE (dstreg
))
432 >= GET_MODE_UNIT_SIZE (cand
->mode
))
435 map_srcreg
= gen_rtx_REG (cand
->mode
, REGNO (srcreg
));
436 map_srcreg2
= gen_rtx_REG (cand
->mode
, REGNO (srcreg2
));
437 map_dstreg
= gen_rtx_REG (cand
->mode
, REGNO (dstreg
));
438 ifexpr
= gen_rtx_IF_THEN_ELSE (cand
->mode
, cond
, map_srcreg
, map_srcreg2
);
439 new_set
= gen_rtx_SET (map_dstreg
, ifexpr
);
441 if (validate_change (def_insn
, &PATTERN (def_insn
), new_set
, true)
442 && update_reg_equal_equiv_notes (def_insn
, cand
->mode
, GET_MODE (dstreg
),
448 "Mode of conditional move instruction extended:\n");
449 print_rtl_single (dump_file
, def_insn
);
457 /* Get all the reaching definitions of an instruction. The definitions are
458 desired for REG used in INSN. Return the definition list or NULL if a
459 definition is missing. If DEST is non-NULL, additionally push the INSN
460 of the definitions onto DEST. */
462 static struct df_link
*
463 get_defs (rtx_insn
*insn
, rtx reg
, vec
<rtx_insn
*> *dest
)
466 struct df_link
*ref_chain
, *ref_link
;
468 FOR_EACH_INSN_USE (use
, insn
)
470 if (GET_CODE (DF_REF_REG (use
)) == SUBREG
)
472 if (REGNO (DF_REF_REG (use
)) == REGNO (reg
))
476 gcc_assert (use
!= NULL
);
478 ref_chain
= DF_REF_CHAIN (use
);
480 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
482 /* Problem getting some definition for this instruction. */
483 if (ref_link
->ref
== NULL
)
485 if (DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
487 /* As global regs are assumed to be defined at each function call
488 dataflow can report a call_insn as being a definition of REG.
489 But we can't do anything with that in this pass so proceed only
490 if the instruction really sets REG in a way that can be deduced
491 from the RTL structure. */
492 if (global_regs
[REGNO (reg
)]
493 && !set_of (reg
, DF_REF_INSN (ref_link
->ref
)))
498 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
499 dest
->safe_push (DF_REF_INSN (ref_link
->ref
));
504 /* Get all the reaching uses of an instruction. The uses are desired for REG
505 set in INSN. Return use list or NULL if a use is missing or irregular. */
507 static struct df_link
*
508 get_uses (rtx_insn
*insn
, rtx reg
)
511 struct df_link
*ref_chain
, *ref_link
;
513 FOR_EACH_INSN_DEF (def
, insn
)
514 if (REGNO (DF_REF_REG (def
)) == REGNO (reg
))
517 gcc_assert (def
!= NULL
);
519 ref_chain
= DF_REF_CHAIN (def
);
521 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
523 /* Problem getting some use for this instruction. */
524 if (ref_link
->ref
== NULL
)
526 if (DF_REF_CLASS (ref_link
->ref
) != DF_REF_REGULAR
)
533 /* Return true if INSN is
534 (SET (reg REGNO (def_reg)) (if_then_else (cond) (REG x1) (REG x2)))
535 and store x1 and x2 in REG_1 and REG_2. */
538 is_cond_copy_insn (rtx_insn
*insn
, rtx
*reg1
, rtx
*reg2
)
540 rtx expr
= single_set (insn
);
543 && GET_CODE (expr
) == SET
544 && GET_CODE (SET_DEST (expr
)) == REG
545 && GET_CODE (SET_SRC (expr
)) == IF_THEN_ELSE
546 && GET_CODE (XEXP (SET_SRC (expr
), 1)) == REG
547 && GET_CODE (XEXP (SET_SRC (expr
), 2)) == REG
)
549 *reg1
= XEXP (SET_SRC (expr
), 1);
550 *reg2
= XEXP (SET_SRC (expr
), 2);
557 enum ext_modified_kind
559 /* The insn hasn't been modified by ree pass yet. */
561 /* Changed into zero extension. */
563 /* Changed into sign extension. */
569 /* Mode from which ree has zero or sign extended the destination. */
570 ENUM_BITFIELD(machine_mode
) mode
: MACHINE_MODE_BITSIZE
;
572 /* Kind of modification of the insn. */
573 ENUM_BITFIELD(ext_modified_kind
) kind
: 2;
575 unsigned int do_not_reextend
: 1;
577 /* True if the insn is scheduled to be deleted. */
578 unsigned int deleted
: 1;
581 /* Vectors used by combine_reaching_defs and its helpers. */
585 /* In order to avoid constant alloc/free, we keep these
586 4 vectors live through the entire find_and_remove_re and just
587 truncate them each time. */
588 auto_vec
<rtx_insn
*> defs_list
;
589 auto_vec
<rtx_insn
*> copies_list
;
590 auto_vec
<rtx_insn
*> modified_list
;
591 auto_vec
<rtx_insn
*> work_list
;
593 /* For instructions that have been successfully modified, this is
594 the original mode from which the insn is extending and
595 kind of extension. */
596 struct ext_modified
*modified
;
599 /* Reaching Definitions of the extended register could be conditional copies
600 or regular definitions. This function separates the two types into two
601 lists, STATE->DEFS_LIST and STATE->COPIES_LIST. This is necessary because,
602 if a reaching definition is a conditional copy, merging the extension with
603 this definition is wrong. Conditional copies are merged by transitively
604 merging their definitions. The defs_list is populated with all the reaching
605 definitions of the extension instruction (EXTEND_INSN) which must be merged
606 with an extension. The copies_list contains all the conditional moves that
607 will later be extended into a wider mode conditional move if all the merges
608 are successful. The function returns false upon failure, true upon
612 make_defs_and_copies_lists (rtx_insn
*extend_insn
, const_rtx set_pat
,
615 rtx src_reg
= XEXP (SET_SRC (set_pat
), 0);
616 bool *is_insn_visited
;
619 state
->work_list
.truncate (0);
621 /* Initialize the work list. */
622 if (!get_defs (extend_insn
, src_reg
, &state
->work_list
))
625 is_insn_visited
= XCNEWVEC (bool, max_insn_uid
);
627 /* Perform transitive closure for conditional copies. */
628 while (!state
->work_list
.is_empty ())
630 rtx_insn
*def_insn
= state
->work_list
.pop ();
633 gcc_assert (INSN_UID (def_insn
) < max_insn_uid
);
635 if (is_insn_visited
[INSN_UID (def_insn
)])
637 is_insn_visited
[INSN_UID (def_insn
)] = true;
639 if (is_cond_copy_insn (def_insn
, ®1
, ®2
))
641 /* Push it onto the copy list first. */
642 state
->copies_list
.safe_push (def_insn
);
644 /* Now perform the transitive closure. */
645 if (!get_defs (def_insn
, reg1
, &state
->work_list
)
646 || !get_defs (def_insn
, reg2
, &state
->work_list
))
653 state
->defs_list
.safe_push (def_insn
);
656 XDELETEVEC (is_insn_visited
);
661 /* If DEF_INSN has single SET expression with a register
662 destination, possibly buried inside a PARALLEL, return
663 the address of the SET expression, else return NULL.
664 This is similar to single_set, except that single_set
665 allows multiple SETs when all but one is dead. */
667 get_sub_rtx (rtx_insn
*def_insn
)
669 enum rtx_code code
= GET_CODE (PATTERN (def_insn
));
672 if (code
== PARALLEL
)
674 for (int i
= 0; i
< XVECLEN (PATTERN (def_insn
), 0); i
++)
676 rtx s_expr
= XVECEXP (PATTERN (def_insn
), 0, i
);
677 if (GET_CODE (s_expr
) != SET
)
679 if (!REG_P (SET_DEST (s_expr
)))
683 sub_rtx
= &XVECEXP (PATTERN (def_insn
), 0, i
);
686 /* PARALLEL with multiple SETs. */
691 else if (code
== SET
)
693 rtx s_expr
= PATTERN (def_insn
);
694 if (REG_P (SET_DEST (s_expr
)))
695 sub_rtx
= &PATTERN (def_insn
);
701 /* Merge the DEF_INSN with an extension. Calls combine_set_extension
702 on the SET pattern. */
705 merge_def_and_ext (ext_cand
*cand
, rtx_insn
*def_insn
, ext_state
*state
)
707 machine_mode ext_src_mode
;
710 ext_src_mode
= GET_MODE (XEXP (SET_SRC (cand
->expr
), 0));
711 sub_rtx
= get_sub_rtx (def_insn
);
716 if (GET_MODE (SET_DEST (*sub_rtx
)) == ext_src_mode
717 || ((state
->modified
[INSN_UID (def_insn
)].kind
718 == (cand
->code
== ZERO_EXTEND
719 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
))
720 && state
->modified
[INSN_UID (def_insn
)].mode
723 if (GET_MODE_UNIT_SIZE (GET_MODE (SET_DEST (*sub_rtx
)))
724 >= GET_MODE_UNIT_SIZE (cand
->mode
))
726 /* If def_insn is already scheduled to be deleted, don't attempt
728 if (state
->modified
[INSN_UID (def_insn
)].deleted
)
730 if (combine_set_extension (cand
, def_insn
, sub_rtx
))
732 if (state
->modified
[INSN_UID (def_insn
)].kind
== EXT_MODIFIED_NONE
)
733 state
->modified
[INSN_UID (def_insn
)].mode
= ext_src_mode
;
741 /* Given SRC, which should be one or more extensions of a REG, strip
742 away the extensions and return the REG. */
745 get_extended_src_reg (rtx src
)
747 while (GET_CODE (src
) == SIGN_EXTEND
|| GET_CODE (src
) == ZERO_EXTEND
)
749 gcc_assert (REG_P (src
));
753 /* This function goes through all reaching defs of the source
754 of the candidate for elimination (CAND) and tries to combine
755 the extension with the definition instruction. The changes
756 are made as a group so that even if one definition cannot be
757 merged, all reaching definitions end up not being merged.
758 When a conditional copy is encountered, merging is attempted
759 transitively on its definitions. It returns true upon success
760 and false upon failure. */
763 combine_reaching_defs (ext_cand
*cand
, const_rtx set_pat
, ext_state
*state
)
766 bool merge_successful
= true;
771 state
->defs_list
.truncate (0);
772 state
->copies_list
.truncate (0);
774 outcome
= make_defs_and_copies_lists (cand
->insn
, set_pat
, state
);
779 /* If the destination operand of the extension is a different
780 register than the source operand, then additional restrictions
781 are needed. Note we have to handle cases where we have nested
782 extensions in the source operand.
784 Candidate insns are known to be single_sets, via the test in
785 find_removable_extensions. So we continue to use single_set here
786 rather than get_sub_rtx. */
787 rtx set
= single_set (cand
->insn
);
789 = (REGNO (SET_DEST (set
)) != REGNO (get_extended_src_reg (SET_SRC (set
))));
792 /* Considering transformation of
793 (set (reg1) (expression))
795 (set (reg2) (any_extend (reg1)))
799 (set (reg2) (any_extend (expression)))
803 /* In theory we could handle more than one reaching def, it
804 just makes the code to update the insn stream more complex. */
805 if (state
->defs_list
.length () != 1)
808 /* We don't have the structure described above if there are
809 conditional moves in between the def and the candidate,
810 and we will not handle them correctly. See PR68194. */
811 if (state
->copies_list
.length () > 0)
814 /* We require the candidate not already be modified. It may,
815 for example have been changed from a (sign_extend (reg))
816 into (zero_extend (sign_extend (reg))).
818 Handling that case shouldn't be terribly difficult, but the code
819 here and the code to emit copies would need auditing. Until
820 we see a need, this is the safe thing to do. */
821 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
824 machine_mode dst_mode
= GET_MODE (SET_DEST (set
));
825 rtx src_reg
= get_extended_src_reg (SET_SRC (set
));
827 /* Ensure we can use the src_reg in dst_mode (needed for
828 the (set (reg1) (reg2)) insn mentioned above). */
829 if (!targetm
.hard_regno_mode_ok (REGNO (src_reg
), dst_mode
))
832 /* Ensure the number of hard registers of the copy match. */
833 if (hard_regno_nregs (REGNO (src_reg
), dst_mode
) != REG_NREGS (src_reg
))
836 /* There's only one reaching def. */
837 rtx_insn
*def_insn
= state
->defs_list
[0];
839 /* The defining statement must not have been modified either. */
840 if (state
->modified
[INSN_UID (def_insn
)].kind
!= EXT_MODIFIED_NONE
)
843 /* The defining statement and candidate insn must be in the same block.
844 This is merely to keep the test for safety and updating the insn
845 stream simple. Also ensure that within the block the candidate
846 follows the defining insn. */
847 basic_block bb
= BLOCK_FOR_INSN (cand
->insn
);
848 if (bb
!= BLOCK_FOR_INSN (def_insn
)
849 || DF_INSN_LUID (def_insn
) > DF_INSN_LUID (cand
->insn
))
852 /* If there is an overlap between the destination of DEF_INSN and
853 CAND->insn, then this transformation is not safe. Note we have
854 to test in the widened mode. */
855 rtx
*dest_sub_rtx
= get_sub_rtx (def_insn
);
856 if (dest_sub_rtx
== NULL
)
859 rtx tmp_reg
= gen_rtx_REG (GET_MODE (SET_DEST (set
)),
860 REGNO (SET_DEST (*dest_sub_rtx
)));
861 if (reg_overlap_mentioned_p (tmp_reg
, SET_DEST (set
)))
864 /* On RISC machines we must make sure that changing the mode of SRC_REG
865 as destination register will not affect its reaching uses, which may
866 read its value in a larger mode because DEF_INSN implicitly sets it
869 = GET_MODE_PRECISION (GET_MODE (SET_DEST (*dest_sub_rtx
)));
870 if (WORD_REGISTER_OPERATIONS
&& known_lt (prec
, BITS_PER_WORD
))
872 struct df_link
*uses
= get_uses (def_insn
, src_reg
);
876 for (df_link
*use
= uses
; use
; use
= use
->next
)
877 if (paradoxical_subreg_p (GET_MODE (*DF_REF_LOC (use
->ref
)),
878 GET_MODE (SET_DEST (*dest_sub_rtx
)))
879 && !DEBUG_INSN_P (DF_REF_INSN (use
->ref
)))
883 /* The destination register of the extension insn must not be
884 used or set between the def_insn and cand->insn exclusive. */
885 if (reg_used_between_p (SET_DEST (set
), def_insn
, cand
->insn
)
886 || reg_set_between_p (SET_DEST (set
), def_insn
, cand
->insn
))
889 /* We must be able to copy between the two registers. Generate,
890 recognize and verify constraints of the copy. Also fail if this
891 generated more than one insn.
893 This generates garbage since we throw away the insn when we're
894 done, only to recreate it later if this test was successful.
896 Make sure to get the mode from the extension (cand->insn). This
897 is different than in the code to emit the copy as we have not
898 modified the defining insn yet. */
900 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (set
)),
901 REGNO (get_extended_src_reg (SET_SRC (set
))));
902 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (set
)),
903 REGNO (SET_DEST (set
)));
904 emit_move_insn (new_dst
, new_src
);
906 rtx_insn
*insn
= get_insns ();
908 if (NEXT_INSN (insn
))
910 if (recog_memoized (insn
) == -1)
913 if (!constrain_operands (1, get_preferred_alternatives (insn
, bb
)))
916 while (REG_P (SET_SRC (*dest_sub_rtx
))
917 && (REGNO (SET_SRC (*dest_sub_rtx
)) == REGNO (SET_DEST (set
))))
919 /* Considering transformation of
920 (set (reg2) (expression))
924 (set (reg2) (any_extend (reg1)))
928 (set (reg2) (any_extend (expression)))
932 = get_defs (def_insn
, SET_SRC (*dest_sub_rtx
), NULL
);
933 if (defs
== NULL
|| defs
->next
)
936 /* There is only one reaching def. */
937 rtx_insn
*def_insn2
= DF_REF_INSN (defs
->ref
);
939 /* The defining statement must not have been modified either. */
940 if (state
->modified
[INSN_UID (def_insn2
)].kind
!= EXT_MODIFIED_NONE
)
943 /* The def_insn2 and candidate insn must be in the same
944 block and def_insn follows def_insn2. */
945 if (bb
!= BLOCK_FOR_INSN (def_insn2
)
946 || DF_INSN_LUID (def_insn2
) > DF_INSN_LUID (def_insn
))
949 rtx
*dest_sub_rtx2
= get_sub_rtx (def_insn2
);
950 if (dest_sub_rtx2
== NULL
)
953 /* On RISC machines we must make sure that changing the mode of
954 SRC_REG as destination register will not affect its reaching
955 uses, which may read its value in a larger mode because DEF_INSN
956 implicitly sets it in word mode. */
957 if (WORD_REGISTER_OPERATIONS
&& known_lt (prec
, BITS_PER_WORD
))
959 struct df_link
*uses
= get_uses (def_insn2
, SET_DEST (set
));
964 rtx dest2
= SET_DEST (*dest_sub_rtx2
);
965 for (use
= uses
; use
; use
= use
->next
)
966 if (paradoxical_subreg_p (GET_MODE (*DF_REF_LOC (use
->ref
)),
968 && !DEBUG_INSN_P (DF_REF_INSN (use
->ref
)))
974 /* The destination register of the extension insn must not be
975 used or set between the def_insn2 and def_insn exclusive.
976 Likewise for the other reg, i.e. check both reg1 and reg2
977 in the above comment. */
978 if (reg_used_between_p (SET_DEST (set
), def_insn2
, def_insn
)
979 || reg_set_between_p (SET_DEST (set
), def_insn2
, def_insn
)
980 || reg_used_between_p (src_reg
, def_insn2
, def_insn
)
981 || reg_set_between_p (src_reg
, def_insn2
, def_insn
))
984 state
->defs_list
[0] = def_insn2
;
989 /* If cand->insn has been already modified, update cand->mode to a wider
990 mode if possible, or punt. */
991 if (state
->modified
[INSN_UID (cand
->insn
)].kind
!= EXT_MODIFIED_NONE
)
995 if (state
->modified
[INSN_UID (cand
->insn
)].kind
996 != (cand
->code
== ZERO_EXTEND
997 ? EXT_MODIFIED_ZEXT
: EXT_MODIFIED_SEXT
)
998 || state
->modified
[INSN_UID (cand
->insn
)].mode
!= cand
->mode
999 || (set
== NULL_RTX
))
1001 mode
= GET_MODE (SET_DEST (set
));
1002 gcc_assert (GET_MODE_UNIT_SIZE (mode
)
1003 >= GET_MODE_UNIT_SIZE (cand
->mode
));
1007 merge_successful
= true;
1009 /* Go through the defs vector and try to merge all the definitions
1011 state
->modified_list
.truncate (0);
1012 FOR_EACH_VEC_ELT (state
->defs_list
, defs_ix
, def_insn
)
1014 if (merge_def_and_ext (cand
, def_insn
, state
))
1015 state
->modified_list
.safe_push (def_insn
);
1018 merge_successful
= false;
1023 /* Now go through the conditional copies vector and try to merge all
1024 the copies in this vector. */
1025 if (merge_successful
)
1027 FOR_EACH_VEC_ELT (state
->copies_list
, i
, def_insn
)
1029 if (transform_ifelse (cand
, def_insn
))
1030 state
->modified_list
.safe_push (def_insn
);
1033 merge_successful
= false;
1039 if (merge_successful
)
1041 /* Commit the changes here if possible
1042 FIXME: It's an all-or-nothing scenario. Even if only one definition
1043 cannot be merged, we entirely give up. In the future, we should allow
1044 extensions to be partially eliminated along those paths where the
1045 definitions could be merged. */
1046 if (apply_change_group ())
1049 fprintf (dump_file
, "All merges were successful.\n");
1051 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
1053 ext_modified
*modified
= &state
->modified
[INSN_UID (def_insn
)];
1054 if (modified
->kind
== EXT_MODIFIED_NONE
)
1055 modified
->kind
= (cand
->code
== ZERO_EXTEND
? EXT_MODIFIED_ZEXT
1056 : EXT_MODIFIED_SEXT
);
1059 modified
->do_not_reextend
= 1;
1065 /* Changes need not be cancelled explicitly as apply_change_group
1066 does it. Print list of definitions in the dump_file for debug
1067 purposes. This extension cannot be deleted. */
1071 "Merge cancelled, non-mergeable definitions:\n");
1072 FOR_EACH_VEC_ELT (state
->modified_list
, i
, def_insn
)
1073 print_rtl_single (dump_file
, def_insn
);
1079 /* Cancel any changes that have been made so far. */
1086 /* Add an extension pattern that could be eliminated. */
1089 add_removable_extension (const_rtx expr
, rtx_insn
*insn
,
1090 vec
<ext_cand
> *insn_list
,
1099 /* We are looking for SET (REG N) (ANY_EXTEND (REG N)). */
1100 if (GET_CODE (expr
) != SET
)
1103 src
= SET_SRC (expr
);
1104 code
= GET_CODE (src
);
1105 dest
= SET_DEST (expr
);
1106 mode
= GET_MODE (dest
);
1109 && (code
== SIGN_EXTEND
|| code
== ZERO_EXTEND
)
1110 && REG_P (XEXP (src
, 0)))
1112 rtx reg
= XEXP (src
, 0);
1113 struct df_link
*defs
, *def
;
1116 /* Zero-extension of an undefined value is partly defined (it's
1117 completely undefined for sign-extension, though). So if there exists
1118 a path from the entry to this zero-extension that leaves this register
1119 uninitialized, removing the extension could change the behavior of
1120 correct programs. So first, check it is not the case. */
1121 if (code
== ZERO_EXTEND
&& !bitmap_bit_p (init_regs
, REGNO (reg
)))
1125 fprintf (dump_file
, "Cannot eliminate extension:\n");
1126 print_rtl_single (dump_file
, insn
);
1127 fprintf (dump_file
, " because it can operate on uninitialized"
1133 /* Second, make sure we can get all the reaching definitions. */
1134 defs
= get_defs (insn
, reg
, NULL
);
1139 fprintf (dump_file
, "Cannot eliminate extension:\n");
1140 print_rtl_single (dump_file
, insn
);
1141 fprintf (dump_file
, " because of missing definition(s)\n");
1146 /* Third, make sure the reaching definitions don't feed another and
1147 different extension. FIXME: this obviously can be improved. */
1148 for (def
= defs
; def
; def
= def
->next
)
1149 if ((idx
= def_map
[INSN_UID (DF_REF_INSN (def
->ref
))])
1151 && (cand
= &(*insn_list
)[idx
- 1])
1152 && cand
->code
!= code
)
1156 fprintf (dump_file
, "Cannot eliminate extension:\n");
1157 print_rtl_single (dump_file
, insn
);
1158 fprintf (dump_file
, " because of other extension\n");
1162 /* For vector mode extensions, ensure that all uses of the
1163 XEXP (src, 0) register are in insn or debug insns, as unlike
1164 integral extensions lowpart subreg of the sign/zero extended
1165 register are not equal to the original register, so we have
1166 to change all uses or none and the current code isn't able
1167 to change them all at once in one transaction. */
1168 else if (VECTOR_MODE_P (GET_MODE (XEXP (src
, 0))))
1170 struct df_link
*ref_chain
, *ref_link
;
1172 ref_chain
= DF_REF_CHAIN (def
->ref
);
1173 for (ref_link
= ref_chain
; ref_link
; ref_link
= ref_link
->next
)
1175 if (ref_link
->ref
== NULL
1176 || DF_REF_INSN_INFO (ref_link
->ref
) == NULL
)
1181 rtx_insn
*use_insn
= DF_REF_INSN (ref_link
->ref
);
1182 if (use_insn
!= insn
&& !DEBUG_INSN_P (use_insn
))
1191 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1194 fprintf (dump_file
, "Cannot eliminate extension:\n");
1195 print_rtl_single (dump_file
, insn
);
1197 " because some vector uses aren't extension\n");
1203 /* Fourth, if the extended version occupies more registers than the
1204 original and the source of the extension is the same hard register
1205 as the destination of the extension, then we cannot eliminate
1206 the extension without deep analysis, so just punt.
1208 We allow this when the registers are different because the
1209 code in combine_reaching_defs will handle that case correctly. */
1210 if (hard_regno_nregs (REGNO (dest
), mode
) != REG_NREGS (reg
)
1211 && reg_overlap_mentioned_p (dest
, reg
))
1214 /* Then add the candidate to the list and insert the reaching definitions
1215 into the definition map. */
1216 ext_cand e
= {expr
, code
, mode
, insn
};
1217 insn_list
->safe_push (e
);
1218 idx
= insn_list
->length ();
1220 for (def
= defs
; def
; def
= def
->next
)
1221 def_map
[INSN_UID (DF_REF_INSN (def
->ref
))] = idx
;
1225 /* Traverse the instruction stream looking for extensions and return the
1226 list of candidates. */
1228 static vec
<ext_cand
>
1229 find_removable_extensions (void)
1231 vec
<ext_cand
> insn_list
= vNULL
;
1235 unsigned *def_map
= XCNEWVEC (unsigned, max_insn_uid
);
1236 bitmap_head init
, kill
, gen
, tmp
;
1238 bitmap_initialize (&init
, NULL
);
1239 bitmap_initialize (&kill
, NULL
);
1240 bitmap_initialize (&gen
, NULL
);
1241 bitmap_initialize (&tmp
, NULL
);
1243 FOR_EACH_BB_FN (bb
, cfun
)
1245 bitmap_copy (&init
, DF_MIR_IN (bb
));
1246 bitmap_clear (&kill
);
1247 bitmap_clear (&gen
);
1249 FOR_BB_INSNS (bb
, insn
)
1251 if (NONDEBUG_INSN_P (insn
))
1253 set
= single_set (insn
);
1254 if (set
!= NULL_RTX
)
1255 add_removable_extension (set
, insn
, &insn_list
, def_map
,
1257 df_mir_simulate_one_insn (bb
, insn
, &kill
, &gen
);
1258 bitmap_ior_and_compl (&tmp
, &gen
, &init
, &kill
);
1259 bitmap_copy (&init
, &tmp
);
1264 XDELETEVEC (def_map
);
1269 /* This is the main function that checks the insn stream for redundant
1270 extensions and tries to remove them if possible. */
1273 find_and_remove_re (void)
1275 ext_cand
*curr_cand
;
1276 rtx_insn
*curr_insn
= NULL
;
1277 int num_re_opportunities
= 0, num_realized
= 0, i
;
1278 vec
<ext_cand
> reinsn_list
;
1279 auto_vec
<rtx_insn
*> reinsn_del_list
;
1280 auto_vec
<rtx_insn
*> reinsn_copy_list
;
1282 /* Construct DU chain to get all reaching definitions of each
1283 extension instruction. */
1284 df_set_flags (DF_RD_PRUNE_DEAD_DEFS
);
1285 df_chain_add_problem (DF_UD_CHAIN
+ DF_DU_CHAIN
);
1286 df_mir_add_problem ();
1288 df_set_flags (DF_DEFER_INSN_RESCAN
);
1290 max_insn_uid
= get_max_uid ();
1291 reinsn_list
= find_removable_extensions ();
1294 if (reinsn_list
.is_empty ())
1295 state
.modified
= NULL
;
1297 state
.modified
= XCNEWVEC (struct ext_modified
, max_insn_uid
);
1299 FOR_EACH_VEC_ELT (reinsn_list
, i
, curr_cand
)
1301 num_re_opportunities
++;
1303 /* Try to combine the extension with the definition. */
1306 fprintf (dump_file
, "Trying to eliminate extension:\n");
1307 print_rtl_single (dump_file
, curr_cand
->insn
);
1310 if (combine_reaching_defs (curr_cand
, curr_cand
->expr
, &state
))
1313 fprintf (dump_file
, "Eliminated the extension.\n");
1315 /* If the RHS of the current candidate is not (extend (reg)), then
1316 we do not allow the optimization of extensions where
1317 the source and destination registers do not match. Thus
1318 checking REG_P here is correct. */
1319 rtx set
= single_set (curr_cand
->insn
);
1320 if (REG_P (XEXP (SET_SRC (set
), 0))
1321 && (REGNO (SET_DEST (set
)) != REGNO (XEXP (SET_SRC (set
), 0))))
1323 reinsn_copy_list
.safe_push (curr_cand
->insn
);
1324 reinsn_copy_list
.safe_push (state
.defs_list
[0]);
1326 reinsn_del_list
.safe_push (curr_cand
->insn
);
1327 state
.modified
[INSN_UID (curr_cand
->insn
)].deleted
= 1;
1331 /* The copy list contains pairs of insns which describe copies we
1332 need to insert into the INSN stream.
1334 The first insn in each pair is the extension insn, from which
1335 we derive the source and destination of the copy.
1337 The second insn in each pair is the memory reference where the
1338 extension will ultimately happen. We emit the new copy
1339 immediately after this insn.
1341 It may first appear that the arguments for the copy are reversed.
1342 Remember that the memory reference will be changed to refer to the
1343 destination of the extention. So we're actually emitting a copy
1344 from the new destination to the old destination. */
1345 for (unsigned int i
= 0; i
< reinsn_copy_list
.length (); i
+= 2)
1347 rtx_insn
*curr_insn
= reinsn_copy_list
[i
];
1348 rtx_insn
*def_insn
= reinsn_copy_list
[i
+ 1];
1350 /* Use the mode of the destination of the defining insn
1351 for the mode of the copy. This is necessary if the
1352 defining insn was used to eliminate a second extension
1353 that was wider than the first. */
1354 rtx sub_rtx
= *get_sub_rtx (def_insn
);
1355 rtx set
= single_set (curr_insn
);
1356 rtx new_dst
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1357 REGNO (XEXP (SET_SRC (set
), 0)));
1358 rtx new_src
= gen_rtx_REG (GET_MODE (SET_DEST (sub_rtx
)),
1359 REGNO (SET_DEST (set
)));
1360 rtx new_set
= gen_rtx_SET (new_dst
, new_src
);
1361 emit_insn_after (new_set
, def_insn
);
1364 /* Delete all useless extensions here in one sweep. */
1365 FOR_EACH_VEC_ELT (reinsn_del_list
, i
, curr_insn
)
1366 delete_insn (curr_insn
);
1368 reinsn_list
.release ();
1369 XDELETEVEC (state
.modified
);
1371 if (dump_file
&& num_re_opportunities
> 0)
1372 fprintf (dump_file
, "Elimination opportunities = %d realized = %d\n",
1373 num_re_opportunities
, num_realized
);
1376 /* Find and remove redundant extensions. */
1379 rest_of_handle_ree (void)
1381 find_and_remove_re ();
1387 const pass_data pass_data_ree
=
1389 RTL_PASS
, /* type */
1391 OPTGROUP_NONE
, /* optinfo_flags */
1393 0, /* properties_required */
1394 0, /* properties_provided */
1395 0, /* properties_destroyed */
1396 0, /* todo_flags_start */
1397 TODO_df_finish
, /* todo_flags_finish */
1400 class pass_ree
: public rtl_opt_pass
1403 pass_ree (gcc::context
*ctxt
)
1404 : rtl_opt_pass (pass_data_ree
, ctxt
)
1407 /* opt_pass methods: */
1408 bool gate (function
*) final override
{ return (optimize
> 0 && flag_ree
); }
1409 unsigned int execute (function
*) final override
1411 return rest_of_handle_ree ();
1414 }; // class pass_ree
1419 make_pass_ree (gcc::context
*ctxt
)
1421 return new pass_ree (ctxt
);