[Bug tree-optimization/38513] Only postreload will remove a no-op store

2009-03-28 Thread rguenth at gcc dot gnu dot org
--- Comment #9 from rguenth at gcc dot gnu dot org 2009-03-28 10:12 --- Fixed. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added Severity|normal

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2009-03-28 Thread rguenth at gcc dot gnu dot org
--- Comment #8 from rguenth at gcc dot gnu dot org 2009-03-28 10:11 --- Subject: Bug 38513 Author: rguenth Date: Sat Mar 28 10:11:14 2009 New Revision: 145172 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=145172 Log: 2009-03-28 Richard Guenther PR tree-optimization/

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-29 Thread pinskia at gcc dot gnu dot org
--- Comment #7 from pinskia at gcc dot gnu dot org 2008-12-29 19:39 --- One more testcase, we miss a sibcalling optimization due to this (this is originally from PR 2100): int i; int g(void) __attribute__((pure)); int f() { int t = i; int t1 = g(); i = t; return t1; } -- htt

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-29 Thread pinskia at gcc dot gnu dot org
--- Comment #6 from pinskia at gcc dot gnu dot org 2008-12-29 19:38 --- (In reply to comment #4) > The idea is simple - delete redundant stores whose RHS have the same > value-number as the LHS. Which is basically what postreload CSE does really: if (!count && reload_cse_noop_set_

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-29 Thread pinskia at gcc dot gnu dot org
--- Comment #5 from pinskia at gcc dot gnu dot org 2008-12-29 19:36 --- *** Bug 21000 has been marked as a duplicate of this bug. *** -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38513

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-29 Thread rguenth at gcc dot gnu dot org
--- Comment #4 from rguenth at gcc dot gnu dot org 2008-12-29 19:23 --- Created an attachment (id=17002) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17002&action=view) patch The idea is simple - delete redundant stores whose RHS have the same value-number as the LHS. -- htt

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-29 Thread rguenth at gcc dot gnu dot org
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-12-29 18:21 --- I have a FRE patch. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added Assigne

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-12 Thread pinskia at gcc dot gnu dot org
--- Comment #2 from pinskia at gcc dot gnu dot org 2008-12-13 02:37 --- (In reply to comment #1) > Also note if we do -Dfloat=int on a target like PPC, the stores are not > removed > because postreloadcse does not understand mode changes that well. I should say PPC64 or -Dfloat=short f

[Bug tree-optimization/38513] Only postreload will remove a no-op store

2008-12-12 Thread pinskia at gcc dot gnu dot org
--- Comment #1 from pinskia at gcc dot gnu dot org 2008-12-13 02:35 --- Also note if we do -Dfloat=int on a target like PPC, the stores are not removed because postreloadcse does not understand mode changes that well. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38513