Re: [patch] lno branch merge -- vectorizer patch #4

[Richard Henderson <rth at redhat dot com>]

On Sun, Sep 19, 2004 at 11:22:46AM +0300, Dorit Naishlos wrote:
> > I was thinking that the DECL for the builtin might be placed in
> 
> ....where?

Oops.  targetm.vectorize.builtin_mask_for_load.

> so with this approach the vectorizer would assume that ALIGN_INDIRECT_REF
> is always supported, and the expander will always generate the AND
> sequence.

Yes.

> We may need to add something later in the compiler to get rid of
> that sequence if it's not needed (in combine?)

No.  If the target supports ALIGN_INDIRECT_REF directly, then it
must support AND as an addressing mode.  See the beginning of
alpha_legitimate_address_p, for example.

> so in expand_expr_real_1, MISALIGNED_INDIRECT_REF would be treated exactly
> like a regular INDIRECT_REF except we would also have:
>              if (code == MISALIGNED_INDIRECT_REF
>                  && !targetm.vectorize.misaligned_mem_ok)
>                abort ();
> right?

Yes.


r~