[Bug c++/59645] New: [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[Bug c++/59645] New: [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[reichelt at gcc dot gnu.org]

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

            Bug ID: 59645
           Summary: [4.7/4.8/4.9 Regression] ICE with covariant return and
                    volatile
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: reichelt at gcc dot gnu.org

The following valid code snippet triggers an ICE since GCC 4.6.0:

===========================================================
struct A { virtual ~A(); };
struct B { virtual ~B(); };
struct C : A, B {};

struct X
{
  virtual B* foo(volatile int);
};

struct Y : X
{
  virtual C* foo(volatile int);
};

C* Y::foo(volatile int) { return 0; }
===========================================================


bug.cc: In member function 'virtual C* Y::_ZTch0_h8_N1Y3fooEi(int)':
bug.cc:15:37: error: invalid argument to gimple call
 C* Y::foo(volatile int) { return 0; }
                                     ^
D.2311
retval.0_3 ={v} Y::foo (this_2(D), D.2311);
bug.cc:15:37: internal compiler error: verify_gimple failed
0xba3116 verify_gimple_in_cfg(function*)
        ../../gcc/gcc/tree-cfg.c:4854
0xacafe2 execute_function_todo
        ../../gcc/gcc/passes.c:1850
0xacb933 execute_todo
        ../../gcc/gcc/passes.c:1884
Please submit a full bug report, [etc.]

[Bug c++/59645] [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.7.4

[Bug c++/59645] [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[mpolacek at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-01-09
                 CC|                            |mpolacek at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Confirmed.

[Bug c++/59645] [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

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

[Bug c++/59645] [4.7/4.8/4.9 Regression] ICE with covariant return and volatile

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> ---
Author: jason
Date: Thu Jan 30 14:18:15 2014
New Revision: 207301

URL: http://gcc.gnu.org/viewcvs?rev=207301&root=gcc&view=rev
Log:
    PR c++/59645
    * cgraphunit.c (expand_thunk): Copy volatile arg to a temporary.

Added:
    trunk/gcc/testsuite/g++.dg/inherit/covariant21.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cgraphunit.c

[Bug c++/59645] [4.7/4.8 Regression] ICE with covariant return and volatile

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |4.9.0
            Summary|[4.7/4.8/4.9 Regression]    |[4.7/4.8 Regression] ICE
                   |ICE with covariant return   |with covariant return and
                   |and volatile                |volatile
      Known to fail|4.9.0                       |

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> ---
Fixed for 4.9 so far.

[Bug c++/59645] [4.7/4.8 Regression] ICE with covariant return and volatile

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59645

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.7.4                       |4.8.4

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
The 4.7 branch is being closed, moving target milestone to 4.8.4.

[Bug c++/59645] [4.8 Regression] ICE with covariant return and volatile

[jason at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59645

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|4.8.4                       |4.9.0

--- Comment #5 from Jason Merrill <jason at gcc dot gnu.org> ---
Closing.  If anyone cares about fixing this in 4.8, we can backport it.