diff options
| author | H.J. Lu <hongjiu.lu@intel.com> | 2019-04-21 18:23:00 +0000 |
|---|---|---|
| committer | H.J. Lu <hjl@gcc.gnu.org> | 2019-04-21 11:23:00 -0700 |
| commit | f94302e90b09bc30d5ba357d4f84aa37f7a75ad1 (patch) | |
| tree | 9ddd83f023197d5d210e4a65188cb15843813408 /gcc/lra-spills.c | |
| parent | 6397d8df302d964740dca942445165d4c2a6c0aa (diff) | |
| download | gcc-f94302e90b09bc30d5ba357d4f84aa37f7a75ad1.zip gcc-f94302e90b09bc30d5ba357d4f84aa37f7a75ad1.tar.gz gcc-f94302e90b09bc30d5ba357d4f84aa37f7a75ad1.tar.bz2 | |
LRA: Revert "Remove useless move insns"
Useless move insn removal was added to LRA just to avoid wasting CPU
cycles on such insn processing afterwards. Such insns are removed
anyway later in the pass pipeline. The CPU time savings are tiny but
the removal creates too many problems including PR target/90178.
Vladimir pre-approved the patch to remove the code:
https://gcc.gnu.org/ml/gcc-patches/2019-04/msg00834.html
gcc/
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.
gcc/testsuite/
PR target/90178
* gcc.target/i386/pr90178.c: New test.
From-SVN: r270484
Diffstat (limited to 'gcc/lra-spills.c')
| -rw-r--r-- | gcc/lra-spills.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/gcc/lra-spills.c b/gcc/lra-spills.c index c19b76a..18db79e7 100644 --- a/gcc/lra-spills.c +++ b/gcc/lra-spills.c @@ -740,7 +740,6 @@ 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++) @@ -819,19 +818,5 @@ 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); - } } } |