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

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 Mansfield 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_t

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 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_

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 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) >      type        size >    

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) unit size align 32 symtab 0 alias set -1 canonical type 0x77ecb498 pre