[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 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 -

[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 #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 -

[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 +

[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  0x0040c33f in build_method_decl (code=167, ret_type=0x2af3b8d0,
selector=0x0, add_args=0x2af3b8a0, 
ellipsis=0 '\0') at /home/pinskia/src/gcc-4.1/gcc/gcc/objc/objc-act.c:5952
#1  0x00402fff in objc_build_method_signature (rettype=0x0,
selector=0x0, optparms=0x2af3b8a0, ellipsis=0 '\0')
at /home/pinskia/src/gcc-4.1/gcc/gcc/objc/objc-act.c:731
#2  0x004740fd in c_parser_objc_method_decl (parser=0x2ae03410) at
/home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:5900
#3  0x00473d2f in c_parser_objc_method_definition
(parser=0x2ae03410)
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|-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