[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #15 from rguenth at gcc dot gnu dot org  2005-11-09 18:02 
---
Fixed.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #14 from rguenth at gcc dot gnu dot org  2005-11-09 18:01 
---
Subject: Bug 24716

Author: rguenth
Date: Wed Nov  9 18:00:59 2005
New Revision: 106700

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=106700
Log:
2005-11-09  Richard Guenther  <[EMAIL PROTECTED]>

PR tree-optimization/24716
* tree-scalar-evolution.c (analyze_evolution_in_loop): Use
t_bool to track results from follow_ssa_edge.

* gcc.c-torture/execute/pr24716.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.c-torture/execute/pr24716.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-scalar-evolution.c


-- 


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[pinskia at gcc dot gnu dot org]

--- Comment #13 from pinskia at gcc dot gnu dot org  2005-11-09 15:07 
---
(In reply to comment #10)
> Or more definitely, store copyprop.
s/store/scev/


-- 


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #12 from rguenth at gcc dot gnu dot org  2005-11-09 14:01 
---
Created an attachment (id=10185)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10185&action=view)
patch

this is what I'm going to test.


-- 


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #11 from rguenth at gcc dot gnu dot org  2005-11-09 13:59 
---
Doh, I have a fix.  What a stupid error in analyze_evolution_in_loop.


-- 

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|NEW |ASSIGNED
   Last reconfirmed|2005-11-09 08:20:55 |2005-11-09 13:59:19
   date||


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #10 from rguenth at gcc dot gnu dot org  2005-11-09 13:31 
---
Or more definitely, store copyprop.

  # BLOCK 1 freq:122  
  # PRED: 0 [100.0%]  (fallthru,exec) 31 [100.0%]  (fallthru,exec)
  # jD.1285_18 = PHI <0(0), 1(31)>; 
  # pD.1281_7 = PHI ;
  # kD.1280_5 = PHI ;
  # WD.1277_1 = PHI ;
:;
  if (0) goto ; else goto ;
  # SUCC: 2 [50.0%]  (true,exec) 6 [50.0%]  (false,exec)

...

  # BLOCK 6 freq:122 
  # PRED: 1 [50.0%]  (false,exec) 5 [100.0%]  (fallthru,exec)
  # jD.1285_20 = PHI ;
  # pdestD.1284_11 = PHI <0(1), 1(5)>;
:;
  if (jD.1285_20 == 1) goto ; else goto ;
  # SUCC: 33 [10.0%]  (loop_exit,true,exec) 7 [90.0%]  (false,exec)

...

  # BLOCK 33 freq:12
  # PRED: 6 [10.0%]  (loop_exit,true,exec)
  # pdestD.1284_23 = PHI ;
:;
  return pdestD.1284_23; 


cfg_cleanup will propagate through the PHIs.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||rakdver at gcc dot gnu dot
   ||org


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[rguenth at gcc dot gnu dot org]

--- Comment #9 from rguenth at gcc dot gnu dot org  2005-11-09 13:14 ---
It's IVCANONs fault, pr24716.c.t76.ivcanon:

...

  # pdest_23 = PHI <0(1)>;
:;
  return pdest_23;

}


-- 


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||law at gcc dot gnu dot org
   Severity|normal  |critical
  GCC build triplet|powerpc-apple-darwin8.2.0   |
   GCC host triplet|powerpc-apple-darwin8.2.0   |
 GCC target triplet|powerpc-apple-darwin8.2.0   |


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

[Bug tree-optimization/24716] [4.1 Regression] Wrong code generated when optimising

[bonzini at gcc dot gnu dot org]

--- Comment #8 from bonzini at gcc dot gnu dot org  2005-11-09 08:22 ---
dom3 is at fault


-- 

bonzini at gcc dot gnu dot org changed:

   What|Removed |Added

  Component|middle-end  |tree-optimization


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