[Bug target/86820] [8 regression] gcc-8 miscompiles binutils on m68k

2018-08-02 Thread sch...@linux-m68k.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

Andreas Schwab  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from Andreas Schwab  ---
Fixed.

[Bug target/86820] [8 regression] gcc-8 miscompiles binutils on m68k

2018-08-02 Thread schwab at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

--- Comment #5 from Andreas Schwab  ---
Author: schwab
Date: Thu Aug  2 17:23:13 2018
New Revision: 263269

URL: https://gcc.gnu.org/viewcvs?rev=263269=gcc=rev
Log:
Fix invalid cc_status after [const_][us]mulsi3_highpart

Backport from mainline
PR target/86820
* config/m68k/m68k.md (umulsi3_highpart+1, const_umulsi3_highpart)
(smulsi3_highpart+1, const_smulsi3_highpart): Add CC_STATUS_INIT.

testsuite/:
PR target/86820
* gcc.target/m68k/mulsi_highpart.c: New test.

Added:
branches/gcc-8-branch/gcc/testsuite/gcc.target/m68k/mulsi_highpart.c
Modified:
branches/gcc-8-branch/gcc/ChangeLog
branches/gcc-8-branch/gcc/config/m68k/m68k.md
branches/gcc-8-branch/gcc/testsuite/ChangeLog

[Bug target/86820] [8 regression] gcc-8 miscompiles binutils on m68k

2018-08-02 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P4
  Known to work||9.0

[Bug target/86820] [8 regression] gcc-8 miscompiles binutils on m68k

2018-08-01 Thread glaubitz at physik dot fu-berlin.de 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

--- Comment #4 from John Paul Adrian Glaubitz  ---
Yes, I can confirm that the patch from comment #2 fixes the problem for me.

(sid-m68k-sbuild)root@epyc:/binutils/binutils-gdb# ls -l /tmp/json2yaml 
-rwxr-xr-x 1 root root 43823968 Aug  2 01:53 /tmp/json2yaml
(sid-m68k-sbuild)root@epyc:/binutils/binutils-gdb# ./binutils/strip-new
/tmp/json2yaml 
(sid-m68k-sbuild)root@epyc:/binutils/binutils-gdb# ls -l /tmp/json2yaml 
-rwxr-xr-x 1 root root 31372868 Aug  2 01:54 /tmp/json2yaml
(sid-m68k-sbuild)root@epyc:/binutils/binutils-gdb#

Andreas, can you backport the patch to the gcc-8 branch unless that has already
happened?

[Bug target/86820] [8 regression] gcc-8 miscompiles binutils on m68k

2018-08-01 Thread pinskia at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

Andrew Pinski  changed:

   What|Removed |Added

   Target Milestone|--- |8.3
Summary|[8, regression] gcc-8   |[8 regression] gcc-8
   |miscompiles binutils on |miscompiles binutils on
   |m68k|m68k

[Bug target/86820] [8, regression] gcc-8 miscompiles binutils on m68k

2018-08-01 Thread glaubitz at physik dot fu-berlin.de 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

--- Comment #3 from John Paul Adrian Glaubitz  ---
(In reply to Andreas Schwab from comment #2)
> Does this help
> ?

Thanks. Rebuilding gcc-8 with the patch applied now, will report back ASAP.

[Bug target/86820] [8, regression] gcc-8 miscompiles binutils on m68k

2018-08-01 Thread sch...@linux-m68k.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

--- Comment #2 from Andreas Schwab  ---
Does this help ?

[Bug target/86820] [8, regression] gcc-8 miscompiles binutils on m68k

2018-08-01 Thread glaubitz at physik dot fu-berlin.de 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86820

--- Comment #1 from John Paul Adrian Glaubitz  ---
Related (invalid) binutils bug report:
https://sourceware.org/bugzilla/show_bug.cgi?id=23471