On 05/05/2014 09:49 AM, Evgeny Stupachenko wrote:
> @@ -42946,6 +42948,10 @@ expand_vec_perm_1 (struct expand_vec_perm_d *d)
> if (expand_vec_perm_pshufb (d))
> return true;
>
> + /* Try the AVX2 vpshufb. */
> + if (expand_vec_perm_vpshufb2_vpermq (d))
> + return true;
Why is this here? It doesn't expand to 1 insn, which is
what expand_vec_perm_1 is intended to check.
It's already called from ix86_expand_vec_perm_const_1.
If things need to be shuffled around in that function,
then that's the right place to do so.
It's also clearly unrelated to "palignr", so has no
business at all within this patch.
> - min = nelt, max = 0;
> + min = 2 * nelt, max = 0;
This change to min is wrong for this patch. It probably
belonged in your 2/2 patch.
> + shift1 = GEN_INT ((min - nelt / 2) *
> + GET_MODE_BITSIZE (GET_MODE_INNER (d->vmode)));
Coding convention sez
shift1 = GEN_INT ((min - nelt / 2)
* GET_MODE_BITSIZE (GET_MODE_INNER (d->vmode)));
r~
