[Bug objc/28049] New: [4.1/4.2 regression] ICE on single +

[Bug objc/28049] New: [4.1/4.2 regression] ICE on single +

[reichelt at gcc dot gnu dot org]

The following invalid code snippet causes an ICE since GCC 4.1.0:

======================
+
======================

bug.m:1: error: expected ':' at end of input
bug.m:1: internal compiler error: Segmentation fault
Please submit a full bug report, [etc.]


-- 
           Summary: [4.1/4.2 regression] ICE on single +
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Keywords: ice-on-invalid-code, error-recovery, monitored
          Severity: normal
          Priority: P3
         Component: objc
        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=28049

[Bug objc/28049] [4.1/4.2 regression] ICE on single +

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |minor
   Target Milestone|---                         |4.1.2


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-06-17 02:27 -------
A single minus have the same ICE and backtrace:
#0  0x000000000040c33f in build_method_decl (code=167, ret_type=0x2aaaaaf3b8d0,
selector=0x0, add_args=0x2aaaaaf3b8a0, 
    ellipsis=0 '\0') at /home/pinskia/src/gcc-4.1/gcc/gcc/objc/objc-act.c:5952
#1  0x0000000000402fff in objc_build_method_signature (rettype=0x0,
selector=0x0, optparms=0x2aaaaaf3b8a0, ellipsis=0 '\0')
    at /home/pinskia/src/gcc-4.1/gcc/gcc/objc/objc-act.c:731
#2  0x00000000004740fd in c_parser_objc_method_decl (parser=0x2aaaaae03410) at
/home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:5900
#3  0x0000000000473d2f in c_parser_objc_method_definition
(parser=0x2aaaaae03410)
    at /home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:5720


-- 

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-06-17 02:27:29
               date|                            |
            Summary|[4.1/4.2 regression] ICE on |[4.1/4.2 regression] ICE on
                   |single +                    |single + or -


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

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


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2006-08-10 08:19 -------
Mark, is there a reason why this is a P2 when this only affects the Objective-C
front-end?


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mmitchel at gcc dot gnu dot
                   |                            |org


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-08-17 04:51 -------
I have a fix for this, it was casued by the new C parser.


-- 

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-06-17 02:27:29         |2006-08-17 04:51:58
               date|                            |


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2006-08-17 05:32 -------
Here is the patch which I am testing:
Index: c-parser.c
===================================================================
--- c-parser.c  (revision 116204)
+++ c-parser.c  (working copy)
@@ -6149,7 +6149,10 @@ c_parser_objc_method_decl (c_parser *par
          parms = chainon (parms,
                           build_tree_list (NULL_TREE, grokparm (parm)));
        }
-      sel = list;
+      if (list)
+       sel = list;
+      else
+       sel = error_mark_node;
     }
   return objc_build_method_signature (type, sel, parms, ellipsis);
 }


-- 


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2006-08-18 02:23 -------
Actually I am going to implement the fix better, we really should be even
calling  c_parser_objc_method_decl at the toplevel of parsing.  I think it was
implemented this way because the old bison based parser did that.

Very much related to PR 23710.
The only issue now is that the correct way of fixing this will almost certain
require a copyright assignment ...... which is a problem.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |23710
              nThis|                            |


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

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2006-08-18 02:28 -------
Actually no I am not going to fix this.  Because it means I have to do a
partical rewrite on how c_parser_external_declaration is done for objc and I am
not going to do it.


-- 

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=28049

[Bug objc/28049] [4.1/4.2 regression] ICE on single + or -

[mmitchel at gcc dot gnu dot org]

------- Comment #7 from mmitchel at gcc dot gnu dot org  2006-10-16 23:38 -------
In rsponse to Comment #2, no this PR should not be P2.  Downgraded to P5.


-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P2                          |P5


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

[Bug objc/28049] [4.1/4.2/4.3 regression] ICE on single + or -

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.1.2                       |4.1.3


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

[Bug objc/28049] [4.2/4.3/4.4 regression] ICE on single + or -

[jsm28 at gcc dot gnu dot org]

------- Comment #8 from jsm28 at gcc dot gnu dot org  2008-07-04 21:23 -------
Closing 4.1 branch.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.1/4.2/4.3/4.4 regression]|[4.2/4.3/4.4 regression] ICE
                   |ICE on single + or -        |on single + or -
   Target Milestone|4.1.3                       |4.2.5


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

[Bug objc/28049] [4.3/4.4/4.5 regression] ICE on single + or -

[jsm28 at gcc dot gnu dot org]

------- Comment #9 from jsm28 at gcc dot gnu dot org  2009-03-31 19:37 -------
Closing 4.2 branch.


-- 

jsm28 at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.2/4.3/4.4/4.5 regression]|[4.3/4.4/4.5 regression] ICE
                   |ICE on single + or -        |on single + or -
   Target Milestone|4.2.5                       |4.3.4


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

[Bug objc/28049] [4.3/4.4/4.5 regression] ICE on single + or -

[rguenth at gcc dot gnu dot org]

------- Comment #10 from rguenth at gcc dot gnu dot org  2009-08-04 12:27 -------
GCC 4.3.4 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.4                       |4.3.5


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

[Bug objc/28049] [4.3/4.4/4.5/4.6 regression] ICE on single + or -

[rguenth at gcc dot gnu dot org]

------- Comment #11 from rguenth at gcc dot gnu dot org  2010-05-22 18:11 -------
GCC 4.3.5 is being released, adjusting target milestone.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.3.5                       |4.3.6


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