[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[pinskia at gcc dot gnu dot org]

--- Comment #7 from pinskia at gcc dot gnu dot org  2007-02-23 00:10 ---
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||FIXED


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

[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[sandra at codesourcery dot com]

--- Comment #6 from sandra at codesourcery dot com  2007-02-19 04:15 ---
I looked around and found another switch that was missing case tcc_vl_exp as
well.  Revised patch has been regtested and committed; see

http://gcc.gnu.org/ml/gcc-patches/2007-02/msg01584.html


-- 


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

[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[sandra at gcc dot gnu dot org]

--- Comment #5 from sandra at gcc dot gnu dot org  2007-02-19 04:13 ---
Subject: Bug 30833

Author: sandra
Date: Mon Feb 19 04:12:49 2007
New Revision: 122122

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=122122
Log:
2007-02-18  Sandra Loosemore  <[EMAIL PROTECTED]>

PR middle-end/30833
* tree-dump.c (dequeue_and_dump): Add tcc_vl_exp case missed
during CALL_EXPR representation conversion.
* tree-ssa-propagate.c (set_rhs): Likewise.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-dump.c
trunk/gcc/tree-ssa-propagate.c


-- 


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

[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[pcarlini at suse dot de]

--- Comment #4 from pcarlini at suse dot de  2007-02-17 19:08 ---
I would venture to say you can commit it as obvious (after proper
regtesting)...


-- 


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

[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[sandra at codesourcery dot com]

--- Comment #3 from sandra at codesourcery dot com  2007-02-17 18:51 ---
Created an attachment (id=13061)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13061&action=view)
patch to tree-dump.c

I've verified that this patch gets rid of the ICE.


-- 


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

[Bug middle-end/30833] [4.3 Regression] ICE with -fdump-tree-all

[pinskia at gcc dot gnu dot org]

--- Comment #2 from pinskia at gcc dot gnu dot org  2007-02-17 17:49 ---
I am 99% sure this was caused by the CALL_EXPR change as tcc_vl_exp is not
handled in that switch.


-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||pinskia at gcc dot gnu dot
   ||org, sandra at codesourcery
   ||dot com
  Component|c++ |middle-end
Summary|ICE with -fdump-tree-all|[4.3 Regression] ICE with -
   ||fdump-tree-all
   Target Milestone|--- |4.3.0


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