diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2019-04-29 16:18:55 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2019-04-29 16:18:55 +0200 |
| commit | 1008df90cca4a4f49f2a898115f8756fafff3187 (patch) | |
| tree | 65b74e56f1a118c9e0f26cb15c7ca45a6794cce3 /gcc/lra-spills.c | |
| parent | 4e29671cb258f486e498a0697f88861ff351de50 (diff) | |
| download | gcc-1008df90cca4a4f49f2a898115f8756fafff3187.zip gcc-1008df90cca4a4f49f2a898115f8756fafff3187.tar.gz gcc-1008df90cca4a4f49f2a898115f8756fafff3187.tar.bz2 | |
re PR rtl-optimization/90257 (8% degradation on cpu2006 403.gcc starting with r270484)
PR rtl-optimization/90257
* cfgrtl.c (flow_active_insn_p): Return true for USE of a function
return value.
Revert the revert:
2019-04-21 H.J. Lu <hongjiu.lu@intel.com>
PR target/90178
Revert:
2018-11-21 Uros Bizjak <ubizjak@gmail.com>
Revert the revert:
2013-10-26 Vladimir Makarov <vmakarov@redhat.com>
Revert:
2013-10-25 Vladimir Makarov <vmakarov@redhat.com>
* lra-spills.c (lra_final_code_change): Remove useless move insns.
From-SVN: r270653
Diffstat (limited to 'gcc/lra-spills.c')
| -rw-r--r-- | gcc/lra-spills.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/gcc/lra-spills.c b/gcc/lra-spills.c index 18db79e7..c19b76a 100644 --- a/gcc/lra-spills.c +++ b/gcc/lra-spills.c @@ -740,6 +740,7 @@ lra_final_code_change (void) int i, hard_regno; basic_block bb; rtx_insn *insn, *curr; + rtx set; int max_regno = max_reg_num (); for (i = FIRST_PSEUDO_REGISTER; i < max_regno; i++) @@ -818,5 +819,19 @@ lra_final_code_change (void) } if (insn_change_p) lra_update_operator_dups (id); + + if ((set = single_set (insn)) != NULL + && REG_P (SET_SRC (set)) && REG_P (SET_DEST (set)) + && REGNO (SET_SRC (set)) == REGNO (SET_DEST (set))) + { + /* Remove an useless move insn. IRA can generate move + insns involving pseudos. It is better remove them + earlier to speed up compiler a bit. It is also + better to do it here as they might not pass final RTL + check in LRA, (e.g. insn moving a control register + into itself). */ + lra_invalidate_insn_data (insn); + delete_insn (insn); + } } } |