[Bug rtl-optimization/57100] New: [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[Bug rtl-optimization/57100] New: [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[zsojka at seznam dot cz]

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

             Bug #: 57100
           Summary: [4.8/4.9 Regression] ICE: in
                    pre_and_rev_post_order_compute, at cfganal.c:869 with
                    -fdump-rtl-pro_and_epilogue-graph
    Classification: Unclassified
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: rtl-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz


Created attachment 29960
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29960
reduced testcase

Compiler output:
$ gcc -O2 -fdump-rtl-pro_and_epilogue-graph testcase.c 
testcase.c: In function 'bar':
testcase.c:13:1: internal compiler error: in pre_and_rev_post_order_compute, at
cfganal.c:869
 }
 ^
0x68fc9a pre_and_rev_post_order_compute(int*, int*, bool)
        /mnt/svn/gcc-trunk/gcc/cfganal.c:869
0x110379c draw_cfg_nodes_no_loops
        /mnt/svn/gcc-trunk/gcc/graph.c:184
0x110379c draw_cfg_nodes
        /mnt/svn/gcc-trunk/gcc/graph.c:266
0x110379c print_graph_cfg(char const*, function*)
        /mnt/svn/gcc-trunk/gcc/graph.c:307
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.

$ gcc -v                                               
Using built-in specs.
COLLECT_GCC=/mnt/svn/gcc-trunk/binary-latest/bin/gcc
COLLECT_LTO_WRAPPER=/mnt/svn/gcc-trunk/binary-198362-lto-fortran-checking-yes-rtl-df/libexec/gcc/x86_64-unknown-linux-gnu/4.9.0/lto-wrapper
Target: x86_64-unknown-linux-gnu
Configured with: /mnt/svn/gcc-trunk//configure --enable-checking=yes,rtl,df
--enable-languages=c,c++,lto,fortran
--prefix=/mnt/svn/gcc-trunk/binary-198362-lto-fortran-checking-yes-rtl-df/
--without-cloog --without-ppl
Thread model: posix
gcc version 4.9.0 20130427 (experimental) (GCC) 

Tested revisions:
r198362 - crash
4.8 r198018 - crash
4.7 r198018 - OK

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2013-04-29
                 CC|                            |rguenth at gcc dot gnu.org,
                   |                            |stevenb.gcc at gmail dot
                   |                            |com
   Target Milestone|---                         |4.8.1
     Ever Confirmed|0                           |1

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> 2013-04-29 08:12:18 UTC ---
Well, you cannot reliably dump graphs at each point in the pass pipeline
in case there are unreachable blocks in the CFG ...

I also routinely hit ICEs in the graph dumping when dumping broken loop
trees (for debugging).

I think we eventually want a custom CFG collecting routine in graph.c
that is not prone to such issues (but just gives up and appends the
rest of the BBs unsorted).

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.8.1                       |4.8.2

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.8.1 has been released.

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.8.2                       |4.8.3

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.8.2 has been released.

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
I have a patch.

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[rguenth at gcc dot gnu.org]

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

--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
Author: rguenth
Date: Wed Oct 30 14:22:01 2013
New Revision: 204211

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

    PR middle-end/57100
    * basic-block.h (pre_and_rev_post_order_compute_fn): New function.
    * cfganal.c (pre_and_rev_post_order_compute_fn): New function
    as worker for ...
    (pre_and_rev_post_order_compute): ... which now wraps it.
    * graph.c (draw_cfg_nodes_no_loops): Use
    pre_and_rev_post_order_compute_fn to avoid ICEing and dependence
    on cfun.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/basic-block.h
    trunk/gcc/cfganal.c
    trunk/gcc/graph.c

[Bug rtl-optimization/57100] [4.8/4.9 Regression] ICE: in pre_and_rev_post_order_compute, at cfganal.c:869 with -fdump-rtl-pro_and_epilogue-graph

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
      Known to work|                            |4.9.0
         Resolution|---                         |FIXED
   Target Milestone|4.8.3                       |4.9.0
      Known to fail|4.8.1, 4.9.0                |4.8.2

--- Comment #6 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed for 4.9, not backporting.