[Bug c/31710] New: ICE in in set_value_range, at tree-vrp.c:278

[Bug c/31710] New: ICE in in set_value_range, at tree-vrp.c:278

[marcus at jet dot franken dot de]

/home/marcus/projects/gcc/BIN/bin/gcc    -O2   -c variant.i
variant.i: In function 'f':
variant.i:7: internal compiler error: in set_value_range, at tree-vrp.c:278


-- 
           Summary: ICE in in set_value_range, at tree-vrp.c:278
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: marcus at jet dot franken dot de
 GCC build triplet: x86_64-unknown-linux-gnu
  GCC host triplet: x86_64-unknown-linux-gnu
GCC target triplet: x86_64-unknown-linux-gnu


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

[Bug c/31710] ICE in in set_value_range, at tree-vrp.c:278

[marcus at jet dot franken dot de]

------- Comment #1 from marcus at jet dot franken dot de  2007-04-26 08:46 -------
Created an attachment (id=13446)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13446&action=view)
variant.i

gcc -c -O2 variant.i


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[pinskia at gcc dot gnu dot org]

------- Comment #2 from pinskia at gcc dot gnu dot org  2007-04-26 09:05 -------
So the range in an semi-old (3 days) compiler, we get is:
D.1602_2: VARYING
D.1602_3: [-32768, -32768]  EQUIVALENCES: { D.1602_2 D.1602_8 } (2 elements)
D.1603_4: [-INF(OVF), -32768]  EQUIVALENCES: { } (0 elements)
D.1602_8: [-32768, -32768]  EQUIVALENCES: { D.1602_2 } (1 elements)

Well obviously the recent patch to tree-vrp.c (which was also applied to the
4.2 branch) caused this failure as we are checking for overflow in some cases. 
I always said the patch should never have been applied to the 4.2 branch
without a lot more testing.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ian at gcc dot gnu dot org
           Severity|normal                      |blocker
  GCC build triplet|x86_64-unknown-linux-gnu    |
   GCC host triplet|x86_64-unknown-linux-gnu    |
 GCC target triplet|x86_64-unknown-linux-gnu    |
           Keywords|                            |ice-checking, ice-on-valid-
                   |                            |code
            Summary|ICE in in set_value_range,  |[4.2/4.3 Regression] ICE in
                   |at tree-vrp.c:278           |in set_value_range, at tree-
                   |                            |vrp.c:278
   Target Milestone|---                         |4.2.0


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[ian at airs dot com]

------- Comment #3 from ian at airs dot com  2007-04-27 04:52 -------
Created an attachment (id=13453)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13453&action=view)
Proposed patch

This is an interesting test case.  It turns out that VRP has always
consistently assumed that TREE_TYPE (TYPE_MIN_VALUE (t)) == t.  The assumption
turns up in == comparisons against TYPE_MIN_VALUE (and TYPE_MAX_VALUE).  Those
== comparisons are normally valid because we hash integer constants so that
they are all the same.  The hash includes the type.  So when TREE_TYPE
(TYPE_MIN_VALUE (t)) != t, the equality will actually never hold, and VRP will
do the wrong thing.

This patch simply ensures that the equality holds.  I'm testing it now.


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[ian at gcc dot gnu dot org]

------- Comment #4 from ian at gcc dot gnu dot org  2007-04-27 21:47 -------
Subject: Bug 31710

Author: ian
Date: Fri Apr 27 21:47:24 2007
New Revision: 124237

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124237
Log:
./:
        PR middle-end/31710
        * tree.c (build_distinct_type_copy): If TYPE_MIN_VALUE or
        TYPE_MAX_VALUE exist, convert them to the new type.
testsuite/:
        PR middle-end/31710
        * gcc.c-torture/compile/pr31710.c: New test.

Added:
    trunk/gcc/testsuite/gcc.c-torture/compile/pr31710.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/tree.c


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[ian at gcc dot gnu dot org]

------- Comment #5 from ian at gcc dot gnu dot org  2007-04-27 21:49 -------
Subject: Bug 31710

Author: ian
Date: Fri Apr 27 21:48:55 2007
New Revision: 124238

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=124238
Log:
./:
        PR middle-end/31710
        * tree.c (build_distinct_type_copy): If TYPE_MIN_VALUE or
        TYPE_MAX_VALUE exist, convert them to the new type.
testsuite/:
        PR middle-end/31710
        * gcc.c-torture/compile/pr31710.c: New test.

Added:
    branches/gcc-4_2-branch/gcc/testsuite/gcc.c-torture/compile/pr31710.c
Modified:
    branches/gcc-4_2-branch/gcc/ChangeLog
    branches/gcc-4_2-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_2-branch/gcc/tree.c


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[ian at airs dot com]

------- Comment #6 from ian at airs dot com  2007-04-27 21:50 -------
Fixed on mainline and 4.2 branch.


-- 

ian at airs dot com changed:

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


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2007-04-28 22:47 -------
*** Bug 31740 has been marked as a duplicate of this bug. ***


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mstein at phenix dot
                   |                            |rootshell dot be


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[mstein at phenix dot rootshell dot be]

------- Comment #8 from mstein at phenix dot rootshell dot be  2007-04-29 20:19 -------
(In reply to comment #6)
> Fixed on mainline and 4.2 branch.

Hi Ian,
I still get this bug on the mainline(124261) for the avr,frv,h8300,m32c,m68hc11
and mips.


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[mstein at phenix dot rootshell dot be]

------- Comment #9 from mstein at phenix dot rootshell dot be  2007-05-01 17:59 -------
Created an attachment (id=13469)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13469&action=view)
from src/sim/frv

This attachment triggers a ICE in set_value_range, at tree-vrp.c:278
with a host gcc from today (trunk).


-- 


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

[Bug middle-end/31710] [4.2/4.3 Regression] ICE in in set_value_range, at tree-vrp.c:278

[ian at airs dot com]

------- Comment #10 from ian at airs dot com  2007-05-01 18:58 -------
Please let me know if this is still a problem after revision 124334.


-- 


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