Patchwork Fix vectorizer fallout

login
register
mail settings
Submitter Richard Guenther
Date March 19, 2013, 3:21 p.m.
Message ID <alpine.LNX.2.00.1303191620490.3543@zhemvz.fhfr.qr>
Download mbox | patch
Permalink /patch/229096/
State New
Headers show

Comments

Richard Guenther - March 19, 2013, 3:21 p.m.
This fixes fallout reported for "Fix ???s in find_uses_to_rename".

Bootstrapped and tested on x86_64-unknown-linux-gnu, scheduled for
commit.

Richard.

2013-03-19  Richard Biener  <rguenther@suse.de>

	PR tree-optimization/
	* tree-vect-loop-manip.c (slpeel_can_duplicate_loop_p): Do not
	check whether an SSA update is needed.
Jakub Jelinek - March 19, 2013, 3:42 p.m.
On Tue, Mar 19, 2013 at 04:21:46PM +0100, Richard Biener wrote:
> 
> This fixes fallout reported for "Fix ???s in find_uses_to_rename".
> 
> Bootstrapped and tested on x86_64-unknown-linux-gnu, scheduled for
> commit.
> 
> Richard.
> 
> 2013-03-19  Richard Biener  <rguenther@suse.de>
> 
> 	PR tree-optimization/

Missing PR number?
Are you just waiting for bugzilla to come up to fill it in?

> 	* tree-vect-loop-manip.c (slpeel_can_duplicate_loop_p): Do not
> 	check whether an SSA update is needed.
> 
> Index: gcc/tree-vect-loop-manip.c
> ===================================================================
> --- gcc/tree-vect-loop-manip.c	(revision 196791)
> +++ gcc/tree-vect-loop-manip.c	(working copy)
> @@ -848,9 +848,6 @@ slpeel_can_duplicate_loop_p (const struc
>    gimple orig_cond = get_loop_exit_condition (loop);
>    gimple_stmt_iterator loop_exit_gsi = gsi_last_bb (exit_e->src);
>  
> -  if (need_ssa_update_p (cfun))
> -    return false;
> -
>    if (loop->inner
>        /* All loops have an outer scope; the only case loop->outer is NULL is for
>           the function itself.  */

	Jakub
Richard Guenther - March 19, 2013, 3:44 p.m.
On Tue, 19 Mar 2013, Jakub Jelinek wrote:

> On Tue, Mar 19, 2013 at 04:21:46PM +0100, Richard Biener wrote:
> > 
> > This fixes fallout reported for "Fix ???s in find_uses_to_rename".
> > 
> > Bootstrapped and tested on x86_64-unknown-linux-gnu, scheduled for
> > commit.
> > 
> > Richard.
> > 
> > 2013-03-19  Richard Biener  <rguenther@suse.de>
> > 
> > 	PR tree-optimization/
> 
> Missing PR number?
> Are you just waiting for bugzilla to come up to fill it in?

Yes.

Richard.

Patch

Index: gcc/tree-vect-loop-manip.c
===================================================================
--- gcc/tree-vect-loop-manip.c	(revision 196791)
+++ gcc/tree-vect-loop-manip.c	(working copy)
@@ -848,9 +848,6 @@  slpeel_can_duplicate_loop_p (const struc
   gimple orig_cond = get_loop_exit_condition (loop);
   gimple_stmt_iterator loop_exit_gsi = gsi_last_bb (exit_e->src);
 
-  if (need_ssa_update_p (cfun))
-    return false;
-
   if (loop->inner
       /* All loops have an outer scope; the only case loop->outer is NULL is for
          the function itself.  */