[Bug c/47963] New: [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[Bug c/47963] New: [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[zsojka at seznam dot cz]

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

           Summary: [4.5/4.6 Regression] ICE: tree check: expected tree
                    that contains 'decl common' structure, have
                    'integer_cst' in is_global_var, at
                    tree-flow-inline.h:599 on invalid code with -fopenmp
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: minor
          Priority: P3
         Component: c
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: zsojka@seznam.cz


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

Compiler output:
$ gcc -fopenmp testcase.c
testcase.c: In function 'foo':
testcase.c:3:7: error: size of array 'A' has non-integer type
testcase.c:3:7: error: size of array 'A' has non-integer type
testcase.c:4:9: internal compiler error: tree check: expected tree that
contains 'decl common' structure, have 'integer_cst' in is_global_var, at
tree-flow-inline.h:599
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

Tested revisions:
r170594 - crash
4.5 r170013 - crash
4.4 r170013 - OK

[Bug c/47963] [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011.03.03 09:29:45
         AssignedTo|unassigned at gcc dot       |jakub at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-03-03 09:29:45 UTC ---
Created attachment 23521
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=23521
gcc46-pr47963.patch

Untested fix.

[Bug c/47963] [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.5.3

[Bug c/47963] [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2

[Bug c/47963] [4.5/4.6 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-03-03 16:09:59 UTC ---
Author: jakub
Date: Thu Mar  3 16:09:55 2011
New Revision: 170655

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=170655
Log:
    PR c/47963
    * gimplify.c (omp_add_variable): Only call omp_notice_variable
    on TYPE_SIZE_UNIT if it is a DECL.

    * gcc.dg/gomp/pr47963.c: New test.
    * g++.dg/gomp/pr47963.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/gomp/pr47963.C
    trunk/gcc/testsuite/gcc.dg/gomp/pr47963.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/gimplify.c
    trunk/gcc/testsuite/ChangeLog

[Bug c/47963] [4.5 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.6.0
            Summary|[4.5/4.6 Regression] ICE:   |[4.5 Regression] ICE: tree
                   |tree check: expected tree   |check: expected tree that
                   |that contains 'decl common' |contains 'decl common'
                   |structure, have             |structure, have
                   |'integer_cst' in            |'integer_cst' in
                   |is_global_var, at           |is_global_var, at
                   |tree-flow-inline.h:599 on   |tree-flow-inline.h:599 on
                   |invalid code with -fopenmp  |invalid code with -fopenmp
      Known to fail|4.6.0                       |

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-03-03 16:15:40 UTC ---
Fixed on the trunk so far.

[Bug c/47963] [4.5 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-04-07 18:27:24 UTC ---
Author: jakub
Date: Thu Apr  7 18:27:20 2011
New Revision: 172115

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

    PR c/47963
    * gimplify.c (omp_add_variable): Only call omp_notice_variable
    on TYPE_SIZE_UNIT if it is a DECL.

    * gcc.dg/gomp/pr47963.c: New test.
    * g++.dg/gomp/pr47963.C: New test.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/g++.dg/gomp/pr47963.C
    branches/gcc-4_5-branch/gcc/testsuite/gcc.dg/gomp/pr47963.c
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/gimplify.c
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog

[Bug c/47963] [4.5 Regression] ICE: tree check: expected tree that contains 'decl common' structure, have 'integer_cst' in is_global_var, at tree-flow-inline.h:599 on invalid code with -fopenmp

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2011-04-07 18:40:58 UTC ---
Fixed.