[Bug rtl-optimization/59133] New: [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[Bug rtl-optimization/59133] New: [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[ysrumyan at gmail dot com]

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

            Bug ID: 59133
           Summary: [4.9 regression] ICE after r204219 on SPEC2006
                    435.gromacs.
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: rtl-optimization
          Assignee: unassigned at gcc dot gnu.org
          Reporter: ysrumyan at gmail dot com

I attached a simple reproducer to compile with -march=core-avx2 -c -O3
-ffast-math options (-ffast-math is essential) and got the following ICE:

t.c: In function 'bar2':
t.c:39:1: internal compiler error: Max. number of generated reload insns per
insn is achieved (90)

 }
 ^
0x87548b lra_constraints(bool)
        /export/users/mstester/stability/svn/trunk/gcc/lra-constraints.c:3955
0x866c3c lra(_IO_FILE*)
        /export/users/mstester/stability/svn/trunk/gcc/lra.c:2303
0x829606 do_reload
        /export/users/mstester/stability/svn/trunk/gcc/ira.c:5407
0x829606 rest_of_handle_reload
        /export/users/mstester/stability/svn/trunk/gcc/ira.c:5536
0x829606 execute
        /export/users/mstester/stability/svn/trunk/gcc/ira.c:5565
Please submit a full bug report,

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[ysrumyan at gmail dot com]

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

--- Comment #1 from Yuri Rumyantsev <ysrumyan at gmail dot com> ---
Created attachment 31219
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=31219&action=edit
test-case to reproduce

Need to be compiled with -m32 -march=core-avx2  -O3 -ffast-math options to
reproduce ICE.

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[ysrumyan at gmail dot com]

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

--- Comment #2 from Yuri Rumyantsev <ysrumyan at gmail dot com> ---
It is worth noting that -m32 option is also essential for reproducing.

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vmakarov at gcc dot gnu.org
   Target Milestone|---                         |4.9.0

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[vmakarov at gcc dot gnu.org]

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

--- Comment #3 from Vladimir Makarov <vmakarov at gcc dot gnu.org> ---
I've started to work on it.  The problem is in that LRA has not enough code to
deal with creation of pseudos out of its scope.  I guess the fix will be ready
next week on Wednesday as the patch will need a lot of testing.

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[vmakarov at gcc dot gnu.org]

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

--- Comment #4 from Vladimir Makarov <vmakarov at gcc dot gnu.org> ---
Author: vmakarov
Date: Wed Nov 20 20:32:57 2013
New Revision: 205141

URL: http://gcc.gnu.org/viewcvs?rev=205141&root=gcc&view=rev
Log:
2013-11-20  Vladimir Makarov  <vmakarov@redhat.com>

    PR rtl-optimization/59133
    * lra.c (expand_reg_data): Add new argument.  Set up ALL_REGS for
    new pseudos.
    (lra_create_new_reg_with_unique_value): Pass new argument value.
    (lra_emit_add, lra_emit_move): Ditto.
    * lra-constraints.c (in_class_p): Add check for move for a new
    insn.
    (change_class): Rename to lra_change_class.  Move to lra-int.h.
    (get_reload_reg, narrow_reload_pseudo_class): Adjust calls of
    change_class.
    (process_addr_reg, process_addr): Ditto.
    (curr_insn_transform): Ditto.  Add check on old pseudo for
    optional reload.
    * lra-int.h (lra_get_regno_hard_regno): Move below.
    (lra_change_class): Renamed change_class from lra.c.

2013-11-20  Vladimir Makarov  <vmakarov@redhat.com>

    PR rtl-optimization/59133
    * gcc.target/i386/pr59133.c: New.


Added:
    trunk/gcc/testsuite/gcc.target/i386/pr59133.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/lra-constraints.c
    trunk/gcc/lra-int.h
    trunk/gcc/lra.c
    trunk/gcc/testsuite/ChangeLog

[Bug rtl-optimization/59133] [4.9 regression] ICE after r204219 on SPEC2006 435.gromacs.

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed I assume.