Re: [PATCH] Remove SLP_TREE_VEC_STMTS in favor of SLP_TREE_VEC_DEFS

[Richard Sandiford via Gcc-patches]

Richard Biener <rguent...@suse.de> writes:
> The following unifies SLP_TREE_VEC_STMTS into SLP_TREE_VEC_DEFS
> which can handle all cases we need.
>
> Bootstrap & regtest running on x86_64-unknown-linux-gnu.

Nice!  Just curious...

> @@ -149,6 +147,20 @@ _slp_tree::~_slp_tree ()
>      free (failed);
>  }
>  
> +/* Push the single SSA definition in DEF to the vector of vector defs.  */
> +
> +void
> +_slp_tree::push_vec_def (gimple *def)
> +{
> +  if (gphi *phi = dyn_cast <gphi *> (def))
> +    vec_defs.quick_push (gimple_phi_result (phi));
> +  else
> +    {
> +      def_operand_p defop = single_ssa_def_operand (def, SSA_OP_ALL_DEFS);
> +      vec_defs.quick_push (get_def_from_ptr (defop));
> +    }
> +}

...what does this handle that gimple_get_lhs wouldn't?  asms?

Thanks,
Richard