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 ja...@redhat.com 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,

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... I

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 ja...@redhat.com 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

[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,