This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Cleanup and improvement of if-conversion for vectorization


On Thu, 27 May 2010, Sebastian Pop wrote:

> On Thu, May 27, 2010 at 09:36, Richard Guenther <rguenther@suse.de> wrote:
> > degenerate_phi_result () will return the degenerate value or NULL
> > if the PHI node isn't degenerate.
> 
> Fixed like this.  I will regstrap this on amd64-linux.
> Ok for trunk?

Ok with

+                   unshare_expr (cond), unshare_expr (arg_0),
+                   unshare_expr (arg_1));
+    }

there is no reason to unshare arg_0 or arg_1.  Neither

   new_stmt = gimple_build_assign (unshare_expr (PHI_RESULT (phi)), rhs);

PHI_RESULT.

So please remove those.

Thanks,
Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]