https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69652

--- Comment #7 from rguenther at suse dot de <rguenther at suse dot de> ---
On Fri, 5 Feb 2016, jakub at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69652
> 
> --- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Well, MASK_STORE you don't want to handle in that loop, that is a store.
> MASK_LOAD is an exception, so IMHO you should just check for is_gimple_assign
> || MASK_LOAD.  Allowing move of arbitrary other stmts looks dangerous.

Why?  We track "dangerous" by having a VDEF.  That's good enough IMHO.

> Another question is what to do with gimple_clobber_p.  Those should have a
> vdef, so we handle them likely conservatively, which might be good enough for
> now.  Or does vectorization just remove them already?

It does.

Reply via email to