[Bug c++/39060] New: [4.4 regression] ICE with lots of invalid member functions

[Bug c++/39060] New: [4.4 regression] ICE with lots of invalid member functions

[reichelt at gcc dot gnu dot org]

The following invalid testcase triggers an ICE on the trunk:

============================================================
struct A
{
  A(void* i=);
  A(void* i=);
  A(void* i=);

  void operator+ (void* i=);

  virtual void foo1(=);
  void foo2(=);
  void foo3(=);
  void foo4(=);
  void foo5(=);
};

A::A (void* i=) {}
============================================================

bug.cc:4: error: 'A::A(void*)' cannot be overloaded
bug.cc:3: error: with 'A::A(void*)'
[...lots of errors removed...]
bug.cc:16: error: default argument given for parameter 1 of 'A::A(void*)'
bug.cc:3: error: after previous specification in 'A::A(void*)'
bug.cc:16: internal compiler error: canonical types differ for identical types
void (A::)(void*) and void (A::)(void*)
Please submit a full bug report, [etc.]

The testcase is a little fragile, it looks like some memory access is not OK.


-- 
           Summary: [4.4 regression] ICE with lots of invalid member
                    functions
           Product: gcc
           Version: 4.4.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=39060

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[reichelt at gcc dot gnu dot org]

-- 

reichelt at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.4.0


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P5


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[paolo dot carlini at oracle dot com]

------- Comment #1 from paolo dot carlini at oracle dot com  2009-02-07 11:35 -------
Today I can't reproduce it. Do you?


-- 


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[reichelt at gcc dot gnu dot org]

------- Comment #2 from reichelt at gcc dot gnu dot org  2009-02-09 22:58 -------
I can still reproduce it with trunk from 2009-02-07 (updated after your
comment).

As mentioned before the testcase is a little fragile. Some weeks ago I had a
much larger testcase (about 150 lines) which I couldn't reduce any further. To
me this looks like some fishy memory access.


-- 

reichelt at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   GCC host triplet|                            |i686-pc-linux-gnu
 GCC target triplet|                            |i686-pc-linux-gnu


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[paolo dot carlini at oracle dot com]

------- Comment #3 from paolo dot carlini at oracle dot com  2009-02-10 00:08 -------
Ah, ok, it's i686...


-- 


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[bangerth at gmail dot com]

------- Comment #4 from bangerth at gmail dot com  2009-02-12 23:21 -------
Confirmed.


-- 

bangerth at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bangerth at gmail dot com
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-02-12 23:21:52
               date|                            |


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[hjl dot tools at gmail dot com]

------- Comment #5 from hjl dot tools at gmail dot com  2009-02-14 23:38 -------
This latent bug is triggered by the part of revision 143157:

---
Index: i386.c
===================================================================
--- i386.c      (revision 143154)
+++ i386.c      (working copy)
@@ -22183,6 +22183,11 @@ ix86_init_mmx_sse_builtins (void)
                                V8SI_type_node, V4SI_type_node,
                                integer_type_node,
                                NULL_TREE);
+  tree pv4di_type_node = build_pointer_type (V4DI_type_node);
+  tree void_ftype_pv4di_v4di
+    = build_function_type_list (void_type_node,
+                               pv4di_type_node, V4DI_type_node,
+                               NULL_TREE);
   tree v8sf_ftype_v8sf_v4sf_int
     = build_function_type_list (V8SF_type_node,
                                V8SF_type_node, V4SF_type_node,
---

This patch:

Index: cp/decl.c
===================================================================
--- cp/decl.c   (revision 143157)
+++ cp/decl.c   (working copy)
@@ -1549,6 +1549,7 @@ duplicate_decls (tree newdecl, tree oldd
                    error ("after previous specification in %q+#D",
                                 olddecl);
                  }
+               return error_mark_node;
              }
        }
     }

works for me.


-- 


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[hjl dot tools at gmail dot com]

------- Comment #6 from hjl dot tools at gmail dot com  2009-02-23 02:36 -------
Created an attachment (id=17346)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17346&action=view)
A patch


-- 


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[hjl dot tools at gmail dot com]

------- Comment #7 from hjl dot tools at gmail dot com  2009-03-07 19:44 -------
A patch is posted at

http://gcc.gnu.org/ml/gcc-patches/2009-03/msg00432.html


-- 

hjl dot tools at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jason at redhat dot com
                URL|                            |http://gcc.gnu.org/ml/gcc-
                   |                            |patches/2009-
                   |                            |03/msg00432.html


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[hjl at gcc dot gnu dot org]

------- Comment #8 from hjl at gcc dot gnu dot org  2009-03-08 17:29 -------
Subject: Bug 39060

Author: hjl
Date: Sun Mar  8 17:29:12 2009
New Revision: 144710

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=144710
Log:
gcc/cp/

2009-03-08  H.J. Lu  <hongjiu.lu@intel.com>

        PR c++/39060
        * parser.c (cp_parser_late_parsing_default_args): Continue
        the loop when cp_parser_assignment_expression returns
        error_mark_node.

gcc/testsuite/

2009-03-08  H.J. Lu  <hongjiu.lu@intel.com>

        PR c++/39060
        * g++.dg/other/new1.C: Adjusted.
        * g++.dg/parse/crash40.C: Likewise.
        * g++.dg/parse/defarg12.C: Likewise.
        * g++.dg/template/error15.C: Likewise.

        * g++.dg/other/pr39060.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/other/pr39060.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/parser.c
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/other/new1.C
    trunk/gcc/testsuite/g++.dg/parse/crash40.C
    trunk/gcc/testsuite/g++.dg/parse/defarg12.C
    trunk/gcc/testsuite/g++.dg/template/error15.C


-- 


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[hjl dot tools at gmail dot com]

------- Comment #9 from hjl dot tools at gmail dot com  2009-03-08 17:34 -------
Fixed.


-- 

hjl dot tools at gmail dot com changed:

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


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

[Bug c++/39060] [4.4 regression] ICE with lots of invalid member functions

[pinskia at gcc dot gnu.org]

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

--- Comment #10 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
>The testcase is a little fragile, it looks like some memory access is not OK.

Looks like it is still not fixed in the end:
https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604748.html

The patch which hit the problem just added unrelated builtins even.