[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-04-21 Thread hjl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

--- Comment #8 from hjl at gcc dot gnu.org  ---
Author: hjl
Date: Sun Apr 21 18:23:00 2019
New Revision: 270484

URL: https://gcc.gnu.org/viewcvs?rev=270484&root=gcc&view=rev
Log:
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  

Revert the revert:
2013-10-26  Vladimir Makarov  

Revert:
2013-10-25  Vladimir Makarov  

* lra-spills.c (lra_final_code_change): Remove useless move insns.

gcc/testsuite/

PR target/90178
* gcc.target/i386/pr90178.c: New test.

Added:
trunk/gcc/testsuite/gcc.target/i386/pr90178.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/lra-spills.c
trunk/gcc/testsuite/ChangeLog

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-04-21 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

H.J. Lu  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Target Milestone|--- |9.0

--- Comment #9 from H.J. Lu  ---
Fixed.

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-04-26 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Jakub Jelinek  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 CC||jakub at gcc dot gnu.org
 Resolution|FIXED   |---

--- Comment #10 from Jakub Jelinek  ---
The patch has been reverted (and is not the right fix anyway).

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-04-29 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

--- Comment #11 from Jakub Jelinek  ---
Author: jakub
Date: Mon Apr 29 14:18:55 2019
New Revision: 270653

URL: https://gcc.gnu.org/viewcvs?rev=270653&root=gcc&view=rev
Log:
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  

PR target/90178
Revert:
2018-11-21  Uros Bizjak  

Revert the revert:
2013-10-26  Vladimir Makarov  

Revert:
2013-10-25  Vladimir Makarov  

* lra-spills.c (lra_final_code_change): Remove useless move insns.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/cfgrtl.c
trunk/gcc/lra-spills.c

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-04-29 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178
Bug 90178 depends on bug 90257, which changed state.

Bug 90257 Summary: [10 Regression] 8% degradation on cpu2006 403.gcc starting 
with r270484
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90257

   What|Removed |Added

 Status|WAITING |RESOLVED
 Resolution|--- |FIXED

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-05-03 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|9.0 |9.2

--- Comment #12 from Jakub Jelinek  ---
GCC 9.1 has been released.

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2021-04-30 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P2
  Known to work||10.1.0, 8.4.1
 Target||x86_64-*-*

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2019-08-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|9.2 |9.3

--- Comment #13 from Jakub Jelinek  ---
GCC 9.2 has been released.

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2020-03-12 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|9.3 |9.4

--- Comment #14 from Jakub Jelinek  ---
GCC 9.3.0 has been released, adjusting target milestone.

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2022-05-27 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|9.5 |10.0
 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED
  Known to fail||9.5.0

--- Comment #16 from Richard Biener  ---
Fixed for GCC 10.

[Bug target/90178] [9 Regression] Missed optimization: duplicated terminal basic block with -mavx

2021-06-01 Thread rguenth at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90178

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|9.4 |9.5

--- Comment #15 from Richard Biener  ---
GCC 9.4 is being released, retargeting bugs to GCC 9.5.