[Bug c++/30303] New: [4.2/4.3 regression] ICE with invalid constructor definition

[Bug c++/30303] New: [4.2/4.3 regression] ICE with invalid constructor definition

[reichelt at gcc dot gnu dot org]

The following invalid code snippet triggers an ICE on mainline and 4.2 branch:

==================================
class A
{
  int i;
};

A::A() { A(); }
==================================

bug.cc:6: error: definition of implicitly-declared 'A::A()'
bug.cc:6: error: declaration of 'A::A()' throws different exceptions
bug.cc:2: error: from previous declaration 'A::A() throw ()'
bug.cc: In constructor 'A::A()':
bug.cc:6: internal compiler error: in finish_function, at cp/decl.c:11162
Please submit a full bug report, [etc.]


-- 
           Summary: [4.2/4.3 regression] ICE with invalid constructor
                    definition
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Keywords: ice-on-invalid-code, error-recovery, monitored
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: reichelt at gcc dot gnu dot org


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[reichelt at gcc dot gnu dot org]

-- 

reichelt at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.2.0


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-12-26 22:22 -------
Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-12-26 22:22:23
               date|                            |


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2006-12-27 03:35 -------
Mine for the new year:
Index: decl.c
===================================================================
--- decl.c      (revision 120211)
+++ decl.c      (working copy)
@@ -6216,7 +6219,10 @@
               XXX Isn't this done in start_function, too?  */
            revert_static_member_fn (decl);
          if (DECL_ARTIFICIAL (old_decl))
-           error ("definition of implicitly-declared %qD", old_decl);
+           {
+             error ("definition of implicitly-declared %qD", old_decl);
+             return error_mark_node;
+           }

          /* Since we've smashed OLD_DECL to its
             DECL_TEMPLATE_RESULT, we must do the same to DECL.  */


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-12-26 22:22:23         |2006-12-27 03:35:47
               date|                            |


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-12-27 03:36 -------
*** Bug 30304 has been marked as a duplicate of this bug. ***


-- 


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2007-01-02 02:10 -------
I am testing a slightly different patch which is closer to what the rest of the
function does, in that return NULL_TREE instead of error_mark_node.
This fixes the problem still and also removes the error "throws different
exceptions" which seems like a good idea.


-- 


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[dayas2003 at yahoo dot com]

------- Comment #5 from dayas2003 at yahoo dot com  2007-03-25 15:07 -------
I am unsing the following version:
arm-2006q1-6-arm-none-linux-gnueabi-i686-pc-linux-gnu.tar

I get the same error as said above. I am not sure whether to modify the Code [
there are many files; so painful solution :-( ] . Is there a way I can compile
the above without touching the C-Files ?
Which version of gcc should I use ?
Can any one suggest me a solution.

Thanks and Regards,
Dayananda
dayas2003@yahoo.com
+91 98452 59517


-- 

dayas2003 at yahoo dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dayas2003 at yahoo dot com


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[mmitchel at gcc dot gnu dot org]

------- Comment #6 from mmitchel at gcc dot gnu dot org  2007-05-14 22:27 -------
Will not be fixed in 4.2.0; retargeting at 4.2.1.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.2.0                       |4.2.1


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.2.1                       |4.2.2


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2007-08-20 06:18 -------
testing the patch right now


-- 


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pcarlini at suse dot de]

------- Comment #8 from pcarlini at suse dot de  2007-09-23 23:02 -------
Any news? ;)


-- 

pcarlini at suse dot de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pcarlini at suse dot de


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pcarlini at suse dot de]

------- Comment #9 from pcarlini at suse dot de  2007-09-25 23:54 -------
*** Bug 33556 has been marked as a duplicate of this bug. ***


-- 

pcarlini at suse dot de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jojelino at gmail dot com


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[mmitchel at gcc dot gnu dot org]

------- Comment #10 from mmitchel at gcc dot gnu dot org  2007-10-09 19:22 -------
Change target milestone to 4.2.3, as 4.2.2 has been released.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.2.2                       |4.2.3


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #11 from pinskia at gcc dot gnu dot org  2007-10-13 19:44 -------
Testing this patch again to make sure it still works.


-- 


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

[Bug c++/30303] [4.2/4.3 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #12 from pinskia at gcc dot gnu dot org  2007-10-14 18:15 -------
Subject: Bug 30303

Author: pinskia
Date: Sun Oct 14 18:15:35 2007
New Revision: 129298

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129298
Log:
2007-10-14  Andrew Pinski  <pinskia@gmail.com>

        PR c++/30303
        * decl.c (grokfndecl): Return NULL after the "definition of
        implicitly-declared" error happened.

2007-10-14  Andrew Pinski  <pinskia@gmail.com>

        PR c++/30303
        * g++.dg/other/ctor1.C: New test.
        * g++.dg/other/ctor2.C: New test.
        * g++.dg/other/dtor1.C: New test.


Added:
    trunk/gcc/testsuite/g++.dg/other/ctor1.C
    trunk/gcc/testsuite/g++.dg/other/ctor2.C
    trunk/gcc/testsuite/g++.dg/other/dtor1.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/decl.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/30303] [4.2 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #13 from pinskia at gcc dot gnu dot org  2007-10-14 18:16 -------
Fixed on the trunk.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.2/4.3 regression] ICE    |[4.2 regression] ICE with
                   |with invalid constructor    |invalid constructor
                   |definition                  |definition


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

[Bug c++/30303] [4.2 regression] ICE with invalid constructor definition

[pinskia at gcc dot gnu dot org]

------- Comment #14 from pinskia at gcc dot gnu dot org  2008-01-03 17:44 -------
I am not going to be able to get this done for 4.2.x so unassigning.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|pinskia at gcc dot gnu dot  |unassigned at gcc dot gnu
                   |org                         |dot org
             Status|ASSIGNED                    |NEW


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

[Bug c++/30303] [4.2 regression] ICE with invalid constructor definition

[jsm28 at gcc dot gnu dot org]

------- Comment #15 from jsm28 at gcc dot gnu dot org  2008-02-01 16:53 -------
4.2.3 is being released now, changing milestones of open bugs to 4.2.4.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.2.3                       |4.2.4


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

[Bug c++/30303] [4.2 regression] ICE with invalid constructor definition

[reichelt at gcc dot gnu dot org]

------- Comment #16 from reichelt at gcc dot gnu dot org  2008-02-13 07:31 -------
The ICE disappeared on the 4.2 branch. Probably due to

2008-02-12  Jason Merrill  <jason@redhat.com>

        PR c++/33916
        * Revert:
        2006-10-17  Mark Mitchell  <mark@codesourcery.com>
        PR c++/29039
        * typeck2.c (build_functional_cast): Don't zero-initialize
        non-PODs; instead, call their constructors.


-- 

reichelt at gcc dot gnu dot org changed:

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


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