[Bug middle-end/58463] New: ICE with -fdump-tree-all-all in vector indexed access

[Bug middle-end/58463] New: ICE with -fdump-tree-all-all in vector indexed access

[paulo@matos-sorge.com]

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

            Bug ID: 58463
           Summary: ICE with -fdump-tree-all-all in vector indexed access
           Product: gcc
           Version: 4.8.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
          Assignee: unassigned at gcc dot gnu.org
          Reporter: paulo@matos-sorge.com

Created attachment 30855
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30855&action=edit
Code example to reproduce ICE

GCC tries to access varmap through get_varinfo in tree-ssa-structalias.c, but
it fails with an ICE.

I have attached a reduced code example to reproduce the problem.

The command line and output with 4.8.1 is:
$ ~/work/tmp/gcc-4.8.1/gcc/cc1 -fpreprocessed -fdump-tree-all-all -O2
core_list_join.i 
core_list_join.i:4:1: warning: no semicolon at end of struct or union [enabled
by default]
 }
 ^
 fn1
Analyzing compilation unit
Performing interprocedural optimizations
 <*free_lang_data> <visibility> <early_local_cleanups>
core_list_join.i: In function 'fn1':
core_list_join.i:9:1: internal compiler error: in operator[], at vec.h:815
 }
 ^
0xbfd572 vec<variable_info*, va_heap, vl_embed>::operator[](unsigned int)
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/vec.h:815
0xbfcea0 vec<variable_info*, va_heap, vl_ptr>::operator[](unsigned int)
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/vec.h:1244
0xbeb5ba get_varinfo
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/tree-ssa-structalias.c:319
0xbefa59 perform_var_substitution
       
/home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/tree-ssa-structalias.c:2244
0xbfb279 solve_constraints
       
/home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/tree-ssa-structalias.c:6569
0xbfb679 compute_points_to_sets
       
/home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/tree-ssa-structalias.c:6665
0xbfbdca compute_may_aliases()
       
/home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/tree-ssa-structalias.c:6814
0x967b9e execute_function_todo
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/passes.c:1941
0x966f88 do_per_function
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/passes.c:1701
0x967d64 execute_todo
        /home/pmatos/work/gcc-releases/gcc-4.8.1/gcc/passes.c:1996
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <http://gcc.gnu.org/bugs.html> for instructions.

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[mpolacek at gcc dot gnu.org]

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

Marek Polacek <mpolacek at gcc dot gnu.org> changed:

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

--- Comment #1 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
I think this has been already fixed.

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-09-20
     Ever confirmed|0                           |1

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Can you check whether backporting

Author: rguenth
Date: Wed Mar 27 15:10:50 2013
New Revision: 197165

URL: http://gcc.gnu.org/viewcvs?rev=197165&root=gcc&view=rev
Log:
2013-03-27  Richard Biener  <rguenther@suse.de>

    PR tree-optimization/56716
    * tree-ssa-structalias.c (perform_var_substitution): Adjust
    dumping for ref nodes.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/tree-ssa-structalias.c

fixes this isssue?

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[paulo@matos-sorge.com]

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

--- Comment #3 from Paulo J. Matos <paulo@matos-sorge.com> ---
Thanks Richard, will check.

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[paulo@matos-sorge.com]

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

--- Comment #4 from Paulo J. Matos <paulo@matos-sorge.com> ---
Backporting fixes the problem. Can we go ahead and backport to 4.8?

Can we add the testcase to the testsuite as well please?

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[rguenther at suse dot de]

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

--- Comment #5 from rguenther at suse dot de <rguenther at suse dot de> ---
On Fri, 20 Sep 2013, paulo@matos-sorge.com wrote:

> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58463
> 
> --- Comment #4 from Paulo J. Matos <paulo@matos-sorge.com> ---
> Backporting fixes the problem. Can we go ahead and backport to 4.8?

Feel free to do the backport - I don't have time for it right now.

Richard.

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[paulo@matos-sorge.com]

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

--- Comment #6 from Paulo J. Matos <paulo@matos-sorge.com> ---
(In reply to rguenther@suse.de from comment #5)
> On Fri, 20 Sep 2013, paulo@matos-sorge.com wrote:
> 
> > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58463
> > 
> > --- Comment #4 from Paulo J. Matos <paulo@matos-sorge.com> ---
> > Backporting fixes the problem. Can we go ahead and backport to 4.8?
> 
> Feel free to do the backport - I don't have time for it right now.
> 
> Richard.

Sure, no problem. If by doing the backport you mean sending the relevant
patches to gcc-patches then I will do that today.

Thanks.

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[pmatos at gcc dot gnu.org]

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

--- Comment #8 from pmatos at gcc dot gnu.org ---
Author: pmatos
Date: Fri Sep 27 14:54:43 2013
New Revision: 202976

URL: http://gcc.gnu.org/viewcvs?rev=202976&root=gcc&view=rev
Log:
    PR middle-end/58463
        * gcc.dg/pr58463.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/pr58463.c
Modified:
    trunk/gcc/ChangeLog

[Bug tree-optimization/58463] ICE with -fdump-tree-all-all in vector indexed access

[pmatos at gcc dot gnu.org]

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

--- Comment #9 from pmatos at gcc dot gnu.org ---
Author: pmatos
Date: Fri Sep 27 16:30:15 2013
New Revision: 202978

URL: http://gcc.gnu.org/viewcvs?rev=202978&root=gcc&view=rev
Log:
        Backport from mainline.

        2013-09-27  Paulo Matos  <pmatos@broadcom.com>
        PR middle-end/58463
        * gcc.dg/pr58463.c: New test.

Added:
    branches/gcc-4_8-branch/gcc/testsuite/gcc.dg/pr58463.c
Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[pmatos at gcc dot gnu.org]

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

--- Comment #10 from pmatos at gcc dot gnu.org ---
Author: pmatos
Date: Fri Sep 27 16:44:39 2013
New Revision: 202979

URL: http://gcc.gnu.org/viewcvs?rev=202979&root=gcc&view=rev
Log:
        Backport from mainline.

         PR middle-end/58463
         2013-03-27  Richard Biener  <rguenther@suse.de>

         PR tree-optimization/56716
         * tree-ssa-structalias.c (perform_var_substitution): Adjust
         dumping for ref nodes.

Modified:
    branches/gcc-4_8-branch/gcc/ChangeLog
    branches/gcc-4_8-branch/gcc/tree-ssa-structalias.c

[Bug middle-end/58463] ICE with -fdump-tree-all-all in vector indexed access

[paulo@matos-sorge.com]

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

Paulo J. Matos <paulo@matos-sorge.com> changed:

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

--- Comment #11 from Paulo J. Matos <paulo@matos-sorge.com> ---
Backported Richard's patch to branch 4.8 under r202979.
Will mark as fixed.