[patch] Fix PR 49087 (was Re: Fix crash in vect_is_slp_reduction)

2011-05-22 Thread Ira Rosen
No, we shouldn't arrive with a NULL use_stmt here. I think a proper fix will be to fail if there are no uses. I'll prepare a patch on Sunday. Here is the patch. It bails out if LHS has no uses. Bootstrapped and tested on powerpc64-suse-linux. Committed. Ira ChangeLog: PR

Fix crash in vect_is_slp_reduction

2011-05-20 Thread Ryan Mansfield
There is a crash in vect_is_slp_reduction where use_stmt doesn't get initialized in the FOR_EACH_IMM_USE_FAST loop. 1718 FOR_EACH_IMM_USE_FAST (use_p, imm_iter, lhs) (gdb) call debug_tree(lhs) ssa_name 0x76e3a5a0 type integer_type 0x77f8c7e0 int32_t sizes-gimplified public

Re: Fix crash in vect_is_slp_reduction

2011-05-20 Thread Richard Guenther
On Fri, May 20, 2011 at 4:06 PM, Ryan Mansfield rmansfi...@qnx.com wrote: There is a crash in vect_is_slp_reduction where use_stmt doesn't get initialized in the FOR_EACH_IMM_USE_FAST loop. 1718          FOR_EACH_IMM_USE_FAST (use_p, imm_iter, lhs) (gdb) call debug_tree(lhs)  ssa_name

Re: Fix crash in vect_is_slp_reduction

2011-05-20 Thread Ira Rosen
gcc-patches-ow...@gcc.gnu.org wrote on 20/05/2011 05:17:47 PM: On Fri, May 20, 2011 at 4:06 PM, Ryan Mansfield rmansfi...@qnx.com wrote: There is a crash in vect_is_slp_reduction where use_stmt doesn't get initialized in the FOR_EACH_IMM_USE_FAST loop. 1718          

Re: Fix crash in vect_is_slp_reduction

2011-05-20 Thread Ryan Mansfield
On 11-05-20 10:17 AM, Richard Guenther wrote: On Fri, May 20, 2011 at 4:06 PM, Ryan Mansfieldrmansfi...@qnx.com wrote: There is a crash in vect_is_slp_reduction where use_stmt doesn't get initialized in the FOR_EACH_IMM_USE_FAST loop. 1718 FOR_EACH_IMM_USE_FAST (use_p, imm_iter, lhs)