Hello,
On 11 Oct 10:30, Richard Henderson wrote:
> On 10/09/2013 03:29 AM, Kirill Yukhin wrote:
> > +(define_insn "avx512f_vec_dup_mem"
> > + [(set (match_operand:VI48F_512 0 "register_operand" "=x")
> > + (vec_duplicate:VI48F_512
> > + (match_operand: 1 "nonimmediate_operand" "xm")))]
> > +
On 10/09/2013 03:29 AM, Kirill Yukhin wrote:
> +(define_insn "avx512f_vec_dup_mem"
> + [(set (match_operand:VI48F_512 0 "register_operand" "=x")
> + (vec_duplicate:VI48F_512
> + (match_operand: 1 "nonimmediate_operand" "xm")))]
> + "TARGET_AVX512F"
> + "vbroadcast\t{%1, %0|%0, %1}"
> +
Hello,
> This patch is still far too large.
>
> I think you should split it up based on every single mode iterator that
> you need to add or change.
Here's 15th subpatch. It introduces VI48F_512 iterator.
Is it Ok?
Testing:
1. Bootstrap pass.
2. make check shows no regressions.
3. Spec 20
