Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Richard Guenther
On Wed, May 2, 2012 at 12:04 PM, Jakub Jelinek wrote: > On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote: >> > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, >> > ok for trunk/4.7? >> >> Ok. > > Thanks. > >> > BTW, tree-ssa-loop-prefetch.c seems to have the sa

Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Jakub Jelinek
On Wed, May 02, 2012 at 11:24:13AM +0200, Richard Guenther wrote: > > Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, > > ok for trunk/4.7? > > Ok. Thanks. > > BTW, tree-ssa-loop-prefetch.c seems to have the same problem, but no idea > > how that should be handled in there..

Re: [PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-05-02 Thread Richard Guenther
On Mon, Apr 30, 2012 at 3:39 PM, Jakub Jelinek wrote: > Hi! > > compute_all_dependences in 4.7+ can fail, and cond_if_else_store_replacement > isn't prepared to handle the chrec_dont_know DDR added there in case of > failure (with NULL DDR_A/DDR_B). > > Fixed thusly, bootstrapped/regtested on x86_

[PATCH] Don't ignore compute_all_dependences failures in phiopt (PR tree-optimization/53163)

2012-04-30 Thread Jakub Jelinek
Hi! compute_all_dependences in 4.7+ can fail, and cond_if_else_store_replacement isn't prepared to handle the chrec_dont_know DDR added there in case of failure (with NULL DDR_A/DDR_B). Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.7? BTW, tree-ssa-loop-pref