[Bug objc/28050] New: [4.1/4.2 regression] ICE on invalid initializer

[Bug objc/28050] New: [4.1/4.2 regression] ICE on invalid initializer

[reichelt at gcc dot gnu dot org]

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

======================
int i = [0];
======================

bug.m:1: error: expected ':' before ']' token
bug.m:1: internal compiler error: Segmentation fault
Please submit a full bug report, [etc.]

This also happens with the objc++ frontend.


-- 
           Summary: [4.1/4.2 regression] ICE on invalid initializer
           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=28050

[Bug objc/28050] [4.1/4.2 regression] ICE on invalid initializer

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
   Target Milestone|---                         |4.1.2


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

[Bug objc/28050] [4.1/4.2 regression] ICE on invalid initializer

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-06-17 02:24 -------
Confirmed:
#0  0x000000000040c9ff in objc_build_message_expr (mess=0x2aaaaaf3b900)
    at /home/pinskia/src/gcc-4.1/gcc/gcc/objc/objc-act.c:6178
#1  0x0000000000472de9 in c_parser_postfix_expression (parser=0x2aaaaae03410)
    at /home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:5172
#2  0x0000000000471a31 in c_parser_unary_expression (parser=0x2aaaaae03410) at
/home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:4622
#3  0x0000000000471350 in c_parser_cast_expression (parser=0x2aaaaae03410,
after=0x0)
    at /home/pinskia/src/gcc-4.1/gcc/gcc/c-parser.c:4498


-- 

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:24:59
               date|                            |


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

[Bug objc/28050] [4.1/4.2 regression] ICE on invalid initializer

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

[Bug objc/28050] [4.1/4.2/4.3 regression] ICE on invalid initializer

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

[Bug objc/28050] [4.1/4.2/4.3 regression] ICE on invalid initializer

[steven at gcc dot gnu dot org]

------- Comment #2 from steven at gcc dot gnu dot org  2008-01-12 22:03 -------
Created an attachment (id=14932)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14932&action=view)
Robustify C/objC parser

Tested (objc only) on cygwin.


-- 


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

[Bug objc/28050] [4.2/4.3/4.4 regression] ICE on invalid initializer

[jsm28 at gcc dot gnu dot org]

------- Comment #3 from jsm28 at gcc dot gnu dot org  2008-07-04 21:24 -------
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 invalid initializer  |on invalid initializer
   Target Milestone|4.1.3                       |4.2.5


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[jsm28 at gcc dot gnu dot org]

------- Comment #4 from jsm28 at gcc dot gnu dot org  2009-03-31 19:38 -------
Closing 4.2 branch.  The C part of the attached patch is OK.


-- 

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 invalid initializer  |on invalid initializer
   Target Milestone|4.2.5                       |4.3.4


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[ayers at gcc dot gnu dot org]

------- Comment #5 from ayers at gcc dot gnu dot org  2009-03-31 22:22 -------
Technically this doesn't ICE anymore, yet I've been told that an "confused by
earlier errors, bailing out" is equivalent... so I'll reconfirm this.

The patch from looks sane to me.  Has there been an official RFA for this?

I could post it together with the test case if you like?


-- 

ayers at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |4.3.3 4.4.0 4.5.0
   Last reconfirmed|2006-10-21 21:32:55         |2009-03-31 22:22:36
               date|                            |


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[pinskia at gcc dot gnu dot org]

------- Comment #6 from pinskia at gcc dot gnu dot org  2009-03-31 22:43 -------
(In reply to comment #5)
> Technically this doesn't ICE anymore, yet I've been told that an "confused by
> earlier errors, bailing out" is equivalent... so I'll reconfirm this.

That is an ICE if you have checking enabled.


-- 


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2009-06-21 23:18 -------
*** Bug 40507 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |debian-gcc at lists dot
                   |                            |debian dot org


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[steven at gcc dot gnu dot org]

------- Comment #8 from steven at gcc dot gnu dot org  2009-06-22 08:12 -------
Re. Comment #5:
I have no plans to sumbit this patch. But do feel free to foster-parent it ;-)


-- 


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

[Bug objc/28050] [4.3/4.4/4.5 regression] ICE on invalid initializer

[doko at gcc dot gnu dot org]

------- Comment #9 from doko at gcc dot gnu dot org  2009-06-22 15:18 -------
Subject: Bug 28050

Author: doko
Date: Mon Jun 22 15:18:38 2009
New Revision: 148804

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148804
Log:
gcc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>

        PR objc/28050
        * c-parser.c (c_parser_objc_message_args): Return error_mark_node
        instead of NULL if a parser error occurs.

gcc/objc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>

        PR objc/28050
        * objc-act.c (objc_build_message_args): Return if ARGS is the
        error_mark_node.

gcc/testsuite/
2009-06-22  Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * pr28050.m: New testcase.

Added:
    trunk/gcc/testsuite/objc.dg/pr28050.m
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-parser.c
    trunk/gcc/objc/ChangeLog
    trunk/gcc/objc/objc-act.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug objc/28050] [4.3/4.4 regression] ICE on invalid initializer

[steven at gcc dot gnu dot org]

------- Comment #10 from steven at gcc dot gnu dot org  2009-06-22 16:14 -------
Note that we usually add the name of the committer to the ChangeLog too, like
so:

2009-06-22  Steven Bosscher  <...>
            Matthias Klose <...>

etc.

But thanks for handling the patch. Fixed on trunk.


-- 

steven at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |doko at gcc dot gnu dot org
      Known to fail|4.3.3 4.4.0 4.5.0           |4.3.3 4.4.0
            Summary|[4.3/4.4/4.5 regression] ICE|[4.3/4.4 regression] ICE on
                   |on invalid initializer      |invalid initializer


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

[Bug objc/28050] [4.3/4.4 regression] ICE on invalid initializer

[doko at gcc dot gnu dot org]

------- Comment #11 from doko at gcc dot gnu dot org  2009-06-22 22:21 -------
Subject: Bug 28050

Author: doko
Date: Mon Jun 22 22:21:17 2009
New Revision: 148819

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148819
Log:
gcc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>
            Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * c-parser.c (c_parser_objc_message_args): Return error_mark_node
        instead of NULL if a parser error occurs.

gcc/objc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>
            Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * objc-act.c (objc_build_message_args): Return if ARGS is the
        error_mark_node.

gcc/testsuite/
2009-06-22  Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * pr28050.m: New testcase.

Added:
    branches/gcc-4_3-branch/gcc/testsuite/objc.dg/pr28050.m
      - copied unchanged from r148804, trunk/gcc/testsuite/objc.dg/pr28050.m
Modified:
    branches/gcc-4_3-branch/gcc/ChangeLog
    branches/gcc-4_3-branch/gcc/c-parser.c
    branches/gcc-4_3-branch/gcc/objc/ChangeLog
    branches/gcc-4_3-branch/gcc/objc/objc-act.c
    branches/gcc-4_3-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug objc/28050] [4.3/4.4 regression] ICE on invalid initializer

[doko at gcc dot gnu dot org]

------- Comment #12 from doko at gcc dot gnu dot org  2009-06-22 22:25 -------
Subject: Bug 28050

Author: doko
Date: Mon Jun 22 22:25:38 2009
New Revision: 148820

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148820
Log:
gcc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>
            Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * c-parser.c (c_parser_objc_message_args): Return error_mark_node
        instead of NULL if a parser error occurs.

gcc/objc/
2009-06-22  Steven Bosscher  <steven@gcc.gnu.org>
            Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * objc-act.c (objc_build_message_args): Return if ARGS is the
        error_mark_node.

gcc/testsuite/
2009-06-22  Matthias Klose  <doko@ubuntu.com>

        PR objc/28050
        * pr28050.m: New testcase.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/objc.dg/pr28050.m
      - copied unchanged from r148804, trunk/gcc/testsuite/objc.dg/pr28050.m
Modified:
    branches/gcc-4_4-branch/gcc/ChangeLog
    branches/gcc-4_4-branch/gcc/c-parser.c
    branches/gcc-4_4-branch/gcc/objc/ChangeLog
    branches/gcc-4_4-branch/gcc/objc/objc-act.c
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug objc++/28050] ICE on invalid initializer

[reichelt at gcc dot gnu dot org]

------- Comment #13 from reichelt at gcc dot gnu dot org  2009-07-18 10:54 -------
The objc++ isn't fixed yet, but this is not a regression.


-- 

reichelt at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|objc                        |objc++
            Summary|[4.3/4.4 regression] ICE on |ICE on invalid initializer
                   |invalid initializer         |
   Target Milestone|4.3.4                       |---


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