[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-28 Thread amodra at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

--- Comment #12 from Alan Modra  ---
Author: amodra
Date: Sat Sep 28 07:12:14 2019
New Revision: 276236

URL: https://gcc.gnu.org/viewcvs?rev=276236&root=gcc&view=rev
Log:
Fix endian issue in pr91656 testcases

PR testsuite/91676
PR rtl-optimization/91656
* gcc.dg/torture/pr91656-1.c: Correct for big and pdp endian.
* gcc.dg/torture/pr91656-2.c: Likewise.
* gcc.dg/torture/pr91656-3.c: Likewise.

Modified:
trunk/gcc/testsuite/ChangeLog
trunk/gcc/testsuite/gcc.dg/torture/pr91656-1.c
trunk/gcc/testsuite/gcc.dg/torture/pr91656-2.c
trunk/gcc/testsuite/gcc.dg/torture/pr91656-3.c

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-09 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Richard Biener  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution|--- |FIXED

--- Comment #11 from Richard Biener  ---
We have PR91676 tracking that.

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-08 Thread segher at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Segher Boessenkool  changed:

   What|Removed |Added

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

--- Comment #10 from Segher Boessenkool  ---
The new testcases pr91656-[12].c fail on all BE systems as written (the
memmove copies the MSB, not the LSB as intended).

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-05 Thread janus at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

janus at gcc dot gnu.org changed:

   What|Removed |Added

 CC||janus at gcc dot gnu.org

--- Comment #9 from janus at gcc dot gnu.org ---
*** Bug 91671 has been marked as a duplicate of this bug. ***

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-05 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

--- Comment #6 from Richard Biener  ---
Author: rguenth
Date: Thu Sep  5 13:23:28 2019
New Revision: 275406

URL: https://gcc.gnu.org/viewcvs?rev=275406&root=gcc&view=rev
Log:
2019-09-05  Richard Biener  

PR rtl-optimization/91656
* postreload-gcse.c (record_last_mem_set_info): Revert addition
of early out.

* gcc.dg/torture/pr91656-1.c: New testcase.
* gcc.dg/torture/pr91656-2.c: Likewise.
* gcc.dg/torture/pr91656-3.c: Likewise.

Added:
trunk/gcc/testsuite/gcc.dg/torture/pr91656-1.c
trunk/gcc/testsuite/gcc.dg/torture/pr91656-2.c
trunk/gcc/testsuite/gcc.dg/torture/pr91656-3.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/postreload-gcse.c
trunk/gcc/testsuite/ChangeLog

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-05 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Richard Biener  changed:

   What|Removed |Added

 CC||seurer at gcc dot gnu.org
 CC||kargl at gcc dot gnu.org

--- Comment #7 from Richard Biener  ---
*** Bug 91657 has been marked as a duplicate of this bug. ***

--- Comment #8 from Richard Biener  ---
*** Bug 91659 has been marked as a duplicate of this bug. ***

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-05 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Richard Biener  changed:

   What|Removed |Added

 CC||seurer at gcc dot gnu.org
 CC||kargl at gcc dot gnu.org

--- Comment #7 from Richard Biener  ---
*** Bug 91657 has been marked as a duplicate of this bug. ***

--- Comment #8 from Richard Biener  ---
*** Bug 91659 has been marked as a duplicate of this bug. ***

[Bug rtl-optimization/91656] [10 Regression] wrong code with -fgcse-after-reload

2019-09-05 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91656

Richard Biener  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Richard Biener  ---
Fixed.