[Bug c/18867] New: [4.0 Regression] ICE on invalid switch quantity

[Bug c/18867] New: [4.0 Regression] ICE on invalid switch quantity

[belyshev at lubercy dot com]

current mainline crashes on this invalid code:
----------------------------------------------------------------------------------------
void f()
{
  float x;
  switch (x) {case 0: break;};
}
----------------------------------------------------------------------------------------

bug.c: In function `f':
bug.c:4: error: switch quantity not an integer
bug.c:4: internal compiler error: Segmentation fault

: Search converges between 2004-07-20-trunk (#491) and 2004-07-21-trunk (#492).

-- 
           Summary: [4.0 Regression] ICE on invalid switch quantity
           Product: gcc
           Version: 4.0.0
            Status: UNCONFIRMED
          Keywords: ice-on-invalid-code
          Severity: normal
          Priority: P2
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: belyshev at lubercy dot com
                CC: gcc-bugs at gcc dot gnu dot org


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[belyshev at lubercy dot com]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |4.0.0
      Known to work|                            |3.4.4
   Target Milestone|---                         |4.0.0


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[steven at gcc dot gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |steven at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2004-12-07 13:05 -------
Test this:

Index: c-common.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/c-common.c,v
retrieving revision 1.589
diff -u -p -r1.589 c-common.c
--- c-common.c  29 Nov 2004 23:11:23 -0000      1.589
+++ c-common.c  7 Dec 2004 13:04:57 -0000
@@ -1467,7 +1467,7 @@ check_case_bounds (tree type, tree orig_
   tree case_high = case_high_p ? *case_high_p : case_low;
  
   /* If there was a problem with the original type, do nothing.  */
-  if (orig_type == error_mark_node)
+  if (! INTEGRAL_TYPE_P (orig_type))
     return true;
  
   min_value = TYPE_MIN_VALUE (orig_type);


-- 


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-12-08 00:12 -------
Subject: Bug 18867

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	steven@gcc.gnu.org	2004-12-08 00:12:06

Modified files:
	gcc            : c-typeck.c ChangeLog 
	gcc/testsuite  : ChangeLog 
Added files:
	gcc/testsuite/gcc.dg/noncompile: 20041207.c 

Log message:
	PR c/18867
	* c-typeck.c (c_start_case): Set orig_type to error_mark_node
	when the type of the controlling expression is not a valid type.
	
	testsuite/
	* gcc.dg/noncompile/20041207.c: New test.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/c-typeck.c.diff?cvsroot=gcc&r1=1.402&r2=1.403
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.6738&r2=2.6739
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/noncompile/20041207.c.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.4719&r2=1.4720



-- 


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[steven at gcc dot gnu dot org]

------- Additional Comments From steven at gcc dot gnu dot org  2004-12-08 00:15 -------
. 

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


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

[Bug c/18867] [4.0 Regression] ICE on invalid switch quantity

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2004-12-08 07:41 -------
Subject: Bug 18867

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	hp@gcc.gnu.org	2004-12-08 07:41:32

Modified files:
	gcc/testsuite  : ChangeLog 
	gcc/testsuite/gcc.dg/noncompile: 20041207.c 

Log message:
	PR c/18867
	* gcc.dg/noncompile/20041207.c: Fix typo in dg directive.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.4720&r2=1.4721
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/testsuite/gcc.dg/noncompile/20041207.c.diff?cvsroot=gcc&r1=1.1&r2=1.2



-- 


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