Re: [PATCH] Fix PR52881, more loop preserving fallout with RTL optimizers

2012-04-11 Thread Richard Guenther
On Tue, 10 Apr 2012, Steven Bosscher wrote: > On Tue, Apr 10, 2012 at 4:56 PM, Richard Guenther wrote: > > I'm sure more RTL optimiziation fallout will pop up - and I wonder > > if we should simply avoid modifying the CFG all over the place and > > instead do that in cleanup_cfg.  Thus, in the if

Re: [PATCH] Fix PR52881, more loop preserving fallout with RTL optimizers

2012-04-10 Thread Steven Bosscher
On Tue, Apr 10, 2012 at 4:56 PM, Richard Guenther wrote: > I'm sure more RTL optimiziation fallout will pop up - and I wonder > if we should simply avoid modifying the CFG all over the place and > instead do that in cleanup_cfg.  Thus, in the if-conversion case, > simply do the speculation and lea

[PATCH] Fix PR52881, more loop preserving fallout with RTL optimizers

2012-04-10 Thread Richard Guenther
RTL optimizers perform CFG adjustments themselves rather than relying on cleanup_cfg (as the gimple level does). This makes it necessary to handle things all-over-the-place. This case is if-conversion performing forwarder block removal (well, it calls it "speculation"), removing empty loop latch