[Bug middle-end/51339] New: [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[Bug middle-end/51339] New: [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[zsojka at seznam dot cz]

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

             Bug #: 51339
           Summary: [4.6/4.7 Regression] ICE: in convert_move, at
                    expr.c:326 with -fopenmp and parallel for
    Classification: Unclassified
           Product: gcc
           Version: 4.7.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: middle-end
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz
              Host: x86_64-pc-linux-gnu
            Target: x86_64-pc-linux-gnu


Created attachment 25941
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25941
reduced testcase (from libgomp.c/pr26943-4.c)

Compiler output:
$ gcc -fopenmp testcase.c 
testcase.c: In function 'foo._omp_fn.0':
testcase.c:7:9: internal compiler error: in convert_move, at expr.c:330
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

Tested revisions:
r181770 - crash
4.6 r180325 - crash
4.5 r180325 - crash
4.4 r180325 - crash

[Bug middle-end/51339] [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.6.3

[Bug c/51339] [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011-12-05
                 CC|                            |jakub at gcc dot gnu.org
          Component|middle-end                  |c
     Ever Confirmed|0                           |1

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-05 15:44:15 UTC ---
Reduced testcase:

char g[] = "g";

void
foo (void)
{
#pragma omp parallel sections firstprivate (g) lastprivate (g)
  {
  #pragma omp section
    g[0] = 'h';
  }
}

With char g[2] this works.  Seems a C FE bug.

[Bug c/51339] [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-05 16:25:33 UTC ---
Created attachment 25993
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25993
gcc47-pr51339.patch

Untested fix.

[Bug c/51339] [4.6/4.7 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-05 20:49:42 UTC ---
Author: jakub
Date: Mon Dec  5 20:49:38 2011
New Revision: 182026

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182026
Log:
    PR c/51339
    * c-decl.c (c_finish_incomplete_decl, finish_decl): Call
    relayout_decl instead of layout_decl.

    * gcc.dg/gomp/pr51339.c: New test.

Added:
    trunk/gcc/testsuite/gcc.dg/gomp/pr51339.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-decl.c
    trunk/gcc/testsuite/ChangeLog

[Bug c/51339] [4.6 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.6/4.7 Regression] ICE:   |[4.6 Regression] ICE: in
                   |in convert_move, at         |convert_move, at expr.c:326
                   |expr.c:326 with -fopenmp    |with -fopenmp and parallel
                   |and parallel for            |for
      Known to fail|4.7.0                       |

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-05 21:02:29 UTC ---
Fixed on the trunk so far.

[Bug c/51339] [4.6 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-08 13:37:48 UTC ---
Author: jakub
Date: Thu Dec  8 13:37:44 2011
New Revision: 182113

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=182113
Log:
    Backport from mainline
    2011-12-05  Jakub Jelinek  <jakub@redhat.com>

    PR c/51339
    * c-decl.c (c_finish_incomplete_decl, finish_decl): Call
    relayout_decl instead of layout_decl.

    * gcc.dg/gomp/pr51339.c: New test.

Added:
    branches/gcc-4_6-branch/gcc/testsuite/gcc.dg/gomp/pr51339.c
Modified:
    branches/gcc-4_6-branch/gcc/ChangeLog
    branches/gcc-4_6-branch/gcc/c-decl.c
    branches/gcc-4_6-branch/gcc/testsuite/ChangeLog

[Bug c/51339] [4.6 Regression] ICE: in convert_move, at expr.c:326 with -fopenmp and parallel for

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-12-08 13:41:28 UTC ---
Fixed.