[Bug c/52773] New: internal error: in replace_pseudos_in, at reload1.c:577

[Bug c/52773] New: internal error: in replace_pseudos_in, at reload1.c:577

[fdarkangel at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

             Bug #: 52773
           Summary: internal error: in replace_pseudos_in, at
                    reload1.c:577
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: fdarkangel@gmail.com


Created attachment 27035
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=27035
An excerpt from SGDK v0.90 that demonstrates the bug

Using m68k-elf-gcc 4.7.0 with
m68k-elf-gcc -c -m68000 -Wall -O2 -funroll-loops -fomit-frame-pointer
-fno-builtin-memset -fno-builtin-memcpy bug.c

The problem disappears without -funroll-loops parameter (or -O0).
I was able to compile the code on my host machine (x86_64, gcc 4.6.3) without
any problems.

More details on the cross compiler:
m68k-elf-gcc -v
Using built-in specs.
COLLECT_GCC=m68k-elf-gcc
COLLECT_LTO_WRAPPER=/opt/toolchains/sega/m68k-elf/libexec/gcc/m68k-elf/4.7.0/lto-wrapper
Target: m68k-elf
Configured with: ../gcc-4.7.0/configure --target=m68k-elf
--prefix=/opt/toolchains/sega/m68k-elf --without-headers --with-newlib
--enable-languages=c --disable-libssp --disable-tls --with-cpu=m68000
--disable-werror : (reconfigured) ../gcc-4.7.0/configure --target=m68k-elf
--prefix=/opt/toolchains/sega/m68k-elf --with-newlib --disable-libssp
--disable-tls --enable-threads=single --enable-languages=c,c++,objc,obj-c++
--with-cpu=m68000 --disable-werror
Thread model: single
gcc version 4.7.0 (GCC)

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

Mikael Pettersson <mikpe at it dot uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mikpe at it dot uu.se

--- Comment #1 from Mikael Pettersson <mikpe at it dot uu.se> 2012-03-29 18:11:33 UTC ---
I can reproduce the ICE with gcc 4.8-20120325, 4.7-20120324, and 4.6-20120323,
but not with gcc-4.5-20120322 or 4.4.7, all built as crosses to m68k-linux.

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

Mikael Pettersson <mikpe at it dot uu.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bernds at gcc dot gnu.org

--- Comment #2 from Mikael Pettersson <mikpe at it dot uu.se> 2012-03-29 21:41:22 UTC ---
It's caused or exposed by Bernd's widening-multiply PR29274 fix in r158633 (+
r158643 followup bugfix):
http://gcc.gnu.org/ml/gcc-cvs/2010-04/msg00739.html

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[alanh at fairlite dot co.uk]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #3 from Alan Hourihane <alanh at fairlite dot co.uk> 2013-04-08 15:52:47 UTC ---
I can't build 4.7.2 on an m68k box because of this one.

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #4 from Mikael Pettersson <mikpe at it dot uu.se> 2013-04-08 17:11:50 UTC ---
The ICE still occurs with gcc 4.6-20130405, 4.7-20130406, 4.8-20130404, and
4.9-20130407.

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[alanh at fairlite dot co.uk]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #5 from Alan Hourihane <alanh at fairlite dot co.uk> ---
any news ?

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #6 from Mikael Pettersson <mikpe at it dot uu.se> ---
Created attachment 30319
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30319&action=edit
reduced test case

Still ICEs 4.9-20130616, 4.8-20130613, and 4.7-20130615.  Needs -O1 (or higher)
-funroll-loops -m68000 (or -m68010).  With -m68020 or higher it doesn't ICE. 
Suspect a target bug, so adding Andreas to cc: list.

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpe at it dot uu.se]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #7 from Mikael Pettersson <mikpe at it dot uu.se> ---
Bernd Schmidt has posted a patch for review:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01147.html

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[fdarkangel at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #8 from fdarkangel at gmail dot com ---
Persists in gcc 4.8.2.

[Bug c/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpelinux at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #9 from Mikael Pettersson <mikpelinux at gmail dot com> ---
(In reply to fdarkangel from comment #8)
> Persists in gcc 4.8.2.

Try Bernd's patch, see link in comment #7.

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[fdarkangel at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #10 from fdarkangel at gmail dot com ---
The patch seems to be working; but it doesn't seem to be accepted. Why not?

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[mikpelinux at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #11 from Mikael Pettersson <mikpelinux at gmail dot com> ---
Probably just needs pinging.  FWIW I've had it in my 4.7 and 4.8 toolchains
(for all my targets) since last summer w/o issues.

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[law at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

--- Comment #12 from Jeffrey A. Law <law at gcc dot gnu.org> ---
Author: law
Date: Sat Jan 17 07:35:40 2015
New Revision: 219796

URL: https://gcc.gnu.org/viewcvs?rev=219796&root=gcc&view=rev
Log:
    PR rtl-optimization/52773
    * calls.c (emit_library_call_value): When pushing arguments use
    stack_pointer_rtx rather than virtual_outgoing_args_rtx in
    CALL_INSN_FUNCTION_USAGE.  Only emit one of use of the magic
    stack pointer reference into CALL_INSN_FUNCTION_USAGE.

    PR rtl-optimization/52773
    * gcc.c-torture/compile/pr52773.c: New test.

Added:
    trunk/gcc/testsuite/gcc.c-torture/compile/pr52773.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/calls.c
    trunk/gcc/testsuite/ChangeLog

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[law at redhat dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |law at redhat dot com
         Resolution|---                         |FIXED

--- Comment #13 from Jeffrey A. Law <law at redhat dot com> ---
Fixed on trunk.

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |5.0
           Keywords|                            |ice-on-valid-code

[Bug rtl-optimization/52773] internal error: in replace_pseudos_in, at reload1.c:577

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52773

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |makhaloff at gmail dot com

--- Comment #14 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
*** Bug 59553 has been marked as a duplicate of this bug. ***