[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-05-09 Thread dorit at il dot ibm dot com


--- Comment #8 from dorit at il dot ibm dot com  2007-05-09 07:14 ---
 So I guess this should be handled somewhere else. I'll open a new
 missed-optimization PR instead (not against PRE this time). thanks.

This is now PR31873


-- 


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-04-17 Thread dorit at il dot ibm dot com


--- Comment #5 from dorit at il dot ibm dot com  2007-04-17 07:22 ---
 Doing cast motion actually causes about 25 *more* failures in the vectorizer
 testsuite.
 I'm closing this as won't fix since it seems there was no other reason to do
 this.

can you please send me the patch so that I could look at this failures before
you close this PR?


-- 


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-04-17 Thread dorit at il dot ibm dot com


--- Comment #6 from dorit at il dot ibm dot com  2007-04-17 07:38 ---
 can you please send me the patch so that I could look at this failures before
 you close this PR?

I'm going over my inbox top down, so I just saw that you had laready sent the
patch... so I will look into it. (thanks!)


-- 


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-04-17 Thread dorit at il dot ibm dot com


--- Comment #7 from dorit at il dot ibm dot com  2007-04-17 19:31 ---
 so I will look into it. 

(for reference: http://gcc.gnu.org/ml/gcc-patches/2007-04/msg01103.html).

So I guess this should be handled somewhere else. I'll open a new
missed-optimization PR instead (not against PRE this time). thanks.


-- 


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-04-16 Thread dberlin at gcc dot gnu dot org


--- Comment #4 from dberlin at gcc dot gnu dot org  2007-04-16 19:14 ---
Doing cast motion actually causes about 25 *more* failures in the vectorizer
testsuite.
I'm closing this as won't fix since it seems there was no other reason to do
this.


-- 

dberlin at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||WONTFIX


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2007-04-15 Thread dberlin at gcc dot gnu dot org


--- Comment #3 from dberlin at gcc dot gnu dot org  2007-04-16 03:12 ---
PRE actually would perform the optimization on these two cases, but it sees the
cast as an induction variable.

Found partial redundancy for expression (short unsigned int) VH.29 (VH.37)
Skipping insertion of phi for partial redundancy: Looks like an induction
variable

I will change it to decide that only binary expressions are induction
variables, for the moment.


-- 

dberlin at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |dberlin at gcc dot gnu dot
   |dot org |org
 Status|NEW |ASSIGNED
   Last reconfirmed|2006-01-17 11:47:14 |2007-04-16 03:12:10
   date||


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2006-01-17 Thread rguenth at gcc dot gnu dot org


--- Comment #1 from rguenth at gcc dot gnu dot org  2006-01-17 11:47 ---
Confirmed.  LIM could do it also, and it looks like its a full redundancy even.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2006-01-17 11:47:14
   date||


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



[Bug tree-optimization/25809] missed PRE optimization - move invariant casts out of loops

2006-01-17 Thread dberlin at dberlin dot org


--- Comment #2 from dberlin at gcc dot gnu dot org  2006-01-17 14:27 ---
Subject: Re:  missed PRE optimization - move
 invariant casts out of loops

rguenth at gcc dot gnu dot org wrote:
 --- Comment #1 from rguenth at gcc dot gnu dot org  2006-01-17 11:47 
 ---
 Confirmed.  LIM could do it also, and it looks like its a full redundancy 
 even.
 
 
Well, PRE certainly can't sink the one assignment out the bottom, but it
should be able to move the one to the top :)


-- 


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