[Bug tree-optimization/38458] copy-propagation doesn't handle cycles

2009-03-28 Thread rguenth at gcc dot gnu dot org 


--- Comment #5 from rguenth at gcc dot gnu dot org  2009-03-28 12:55 ---
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

   Severity|normal  |enhancement
 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.5.0


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

[Bug tree-optimization/38458] copy-propagation doesn't handle cycles

2009-03-28 Thread rguenth at gcc dot gnu dot org 


--- Comment #4 from rguenth at gcc dot gnu dot org  2009-03-28 12:54 ---
Subject: Bug 38458

Author: rguenth
Date: Sat Mar 28 12:54:14 2009
New Revision: 145185

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=145185
Log:
2009-03-28  Richard Guenther  rguent...@suse.de

PR tree-optimization/38458
* tree-ssa-copy.c (copy_prop_visit_phi_node): For the first
argument use the arguments copy-of value.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/tree-ssa-copy.c


-- 


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

[Bug tree-optimization/38458] copy-propagation doesn't handle cycles

2008-12-10 Thread rguenth at gcc dot gnu dot org 


--- Comment #3 from rguenth at gcc dot gnu dot org  2008-12-10 17:53 ---
Subject: Bug 38458

Author: rguenth
Date: Wed Dec 10 17:51:52 2008
New Revision: 142654

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142654
Log:
2008-12-10  Richard Guenther  [EMAIL PROTECTED]

PR tree-optimization/38458
* tree-ssa-copy.c (copy_prop_visit_phi_node): For the first
argument use the arguments copy-of value.

Modified:
branches/alias-improvements/gcc/ChangeLog.alias
branches/alias-improvements/gcc/tree-ssa-copy.c


-- 


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

[Bug tree-optimization/38458] copy-propagation doesn't handle cycles

2008-12-09 Thread dnovillo at google dot com 


--- Comment #1 from dnovillo at google dot com  2008-12-09 20:22 ---
Subject: Re:  New: copy-propagation doesn't 
handle cycles

On Tue, Dec 9, 2008 at 14:53, rguenth at gcc dot gnu dot org
[EMAIL PROTECTED] wrote:
{
 - phi_val.value = arg;
 + phi_val.value = arg_val-value;
  continue;
}

This looks OK.

 -  rhs_val = get_copy_of_val (rhs);
 +  rhs_val = get_last_copy_of (rhs);

We don't want to propagate using get_last_copy_of here.  The reason
now escapes me, but it should be documented in the code.  It was
related to phi-cycles, but it's been a long time.


Diego.


-- 


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

[Bug tree-optimization/38458] copy-propagation doesn't handle cycles

2008-12-09 Thread rguenth at gcc dot gnu dot org 


--- Comment #2 from rguenth at gcc dot gnu dot org  2008-12-09 20:45 ---
Thanks.  Mine then.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
   |dot org |org
 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2008-12-09 20:45:20
   date||


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