[Bug middle-end/54782] New: [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[Bug middle-end/54782] New: [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[zsojka at seznam dot cz]

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

             Bug #: 54782
           Summary: [4.8 Regression] ICE: in change_scope, at final.c:1543
                    with -O -ffast-math -ftree-parallelize-loops=2 -g
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz


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

Compiler output:
$ gcc -O -ffast-math -ftree-parallelize-loops=2 -g testcase.c         
testcase.c: In function 'foo._loopfn.0':
testcase.c:12:3: internal compiler error: in change_scope, at final.c:1543
   for (i = 0; i < s->n; i++)
   ^
0x7980ea change_scope
        /mnt/svn/gcc-trunk/gcc/final.c:1543
0x79d68c reemit_insn_block_notes
        /mnt/svn/gcc-trunk/gcc/final.c:1613
0x79d68c final_start_function(rtx_def*, _IO_FILE*, int)
        /mnt/svn/gcc-trunk/gcc/final.c:1670
0x7a0585 rest_of_handle_final
        /mnt/svn/gcc-trunk/gcc/final.c:4291
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.

Tested revisions:
r191953 - crash
r191586 - crash
4.7 r191640 - OK

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[mpolacek at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-10-03
                 CC|                            |dehao at google dot com,
                   |                            |mpolacek at gcc dot gnu.org
     Ever Confirmed|0                           |1

--- Comment #1 from Marek Polacek <mpolacek at gcc dot gnu.org> 2012-10-03 07:58:48 UTC ---
Started with http://gcc.gnu.org/viewcvs?view=revision&amp;revision=191494

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[dehao at google dot com]

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

--- Comment #2 from Dehao Chen <dehao at google dot com> 2012-10-03 16:47:23 UTC ---
Thanks for reporting this problem. The root cause has been identified that
phi_arg_location is not correctly updated in move_block_to_fn. A patch is on
the way.

BTW, how can I assign this patch to me? Looks like there is no such a button in
my view.

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-10-03 16:54:47 UTC ---
You need to log into bugzilla using your gcc.gnu.org account.

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[dehao at gcc dot gnu.org]

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

--- Comment #4 from dehao at gcc dot gnu.org 2012-10-03 20:34:30 UTC ---
Author: dehao
Date: Wed Oct  3 20:34:26 2012
New Revision: 192049

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192049
Log:
2012-10-03  Dehao Chen  <dehao@google.com>

gcc:
    PR middle-end/54782
    * tree-cfg.c (move_block_to_fn): Update lexical block for phi_args.

testsuite:
    PR middle-end/54782
    * gcc.dg/pr54782.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/pr54782.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree-cfg.c

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[dehao at gcc dot gnu.org]

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

dehao at gcc dot gnu.org changed:

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

--- Comment #5 from dehao at gcc dot gnu.org 2012-10-03 20:35:42 UTC ---
Fixed by http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192049

[Bug middle-end/54782] [4.8 Regression] ICE: in change_scope, at final.c:1543 with -O -ffast-math -ftree-parallelize-loops=2 -g

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.8.0