[Bug c++/46160] New: [4.5/4.6 Regression] ICE with volatile structure and enum

[Bug c++/46160] New: [4.5/4.6 Regression] ICE with volatile structure and enum

[jsm28 at gcc dot gnu.org]

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

           Summary: [4.5/4.6 Regression] ICE with volatile structure and
                    enum
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Keywords: ice-on-valid-code
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: jsm28@gcc.gnu.org


The following C++ code

void f(int);

void
g()
{
  struct S
  {
    enum E { A } ;
  };
  S s;
  volatile S *p = &s;
  f((int) p->A);
}

results in

gcc: internal compiler error: Segmentation fault (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.

with 4.5 and 4.6, but compiles fine with 4.4.

[Bug c++/46160] [4.5/4.6 Regression] ICE with volatile structure and enum

[hjl.tools at gmail dot com]

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

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2010.10.25 02:29:50
                 CC|                            |jason at redhat dot com
     Ever Confirmed|0                           |1

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> 2010-10-25 02:29:50 UTC ---
It is caused by revision 159072:

http://gcc.gnu.org/ml/gcc-cvs/2010-05/msg00122.html

[Bug c++/46160] [4.5/4.6 Regression] ICE with volatile structure and enum

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.5.2

[Bug c++/46160] [4.5/4.6 Regression] ICE with volatile structure and enum

[jakub at gcc dot gnu.org]

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

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> 2010-11-05 15:17:17 UTC ---
Created attachment 22293
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=22293
gcc46-pr46160.patch

Untested fix.

[Bug c++/46160] [4.5/4.6 Regression] ICE with volatile structure and enum

[jakub at gcc dot gnu.org]

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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-05 19:02:52 UTC ---
Author: jakub
Date: Fri Nov  5 19:02:46 2010
New Revision: 166372

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166372
Log:
    PR c++/46160
    * cp-gimplify.c (cp_gimplify_expr): Drop volatile MEM_REFs
    on the RHS to avoid infinite recursion with gimplify_expr.

    * g++.dg/opt/empty2.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/opt/empty2.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/cp-gimplify.c
    trunk/gcc/testsuite/ChangeLog

[Bug c++/46160] [4.5 Regression] ICE with volatile structure and enum

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.5/4.6 Regression] ICE    |[4.5 Regression] ICE with
                   |with volatile structure and |volatile structure and enum
                   |enum                        |

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-05 19:11:44 UTC ---
Fixed on the trunk so far.

[Bug c++/46160] [4.5 Regression] ICE with volatile structure and enum

[jakub at gcc dot gnu.org]

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

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-11 20:39:33 UTC ---
Author: jakub
Date: Thu Nov 11 20:39:25 2010
New Revision: 166619

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=166619
Log:
2010-11-11  Jakub Jelinek  <jakub@redhat.com>

    Backport from mainline
    2010-11-05  Jakub Jelinek  <jakub@redhat.com>

    PR c++/46160
    * cp-gimplify.c (cp_gimplify_expr): Drop volatile INDIRECT_REFs
    on the RHS to avoid infinite recursion with gimplify_expr.

    * g++.dg/opt/empty2.C: New test.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/g++.dg/opt/empty2.C
Modified:
    branches/gcc-4_5-branch/gcc/cp/ChangeLog
    branches/gcc-4_5-branch/gcc/cp/cp-gimplify.c
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog

[Bug c++/46160] [4.5 Regression] ICE with volatile structure and enum

[jakub at gcc dot gnu.org]

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

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

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-11-11 22:12:44 UTC ---
Fixed.