This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH i386 AVX512] [63.1/n] Add vpshufb, perm autogen (except for v64qi).
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Ilya Tocar <tocarip dot intel at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Kirill Yukhin <kirill dot yukhin at gmail dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 20 Oct 2014 18:08:30 +0200
- Subject: Re: [PATCH i386 AVX512] [63.1/n] Add vpshufb, perm autogen (except for v64qi).
- Authentication-results: sourceware.org; auth=none
- References: <20141006125527 dot GC13369 at msticlxl57 dot ims dot intel dot com> <20141006141035 dot GZ1986 at tucnak dot redhat dot com> <20141009121523 dot GB81768 at msticlxl7 dot ims dot intel dot com> <20141009185105 dot GM1986 at tucnak dot redhat dot com> <20141010154719 dot GA121201 at msticlxl7 dot ims dot intel dot com> <CAFULd4bGehNdNGtZOt+UE50ShHxbL-Xxfv4rNhwPD1sByDjyag at mail dot gmail dot com> <20141016102316 dot GA30455 at msticlxl7 dot ims dot intel dot com> <20141020151927 dot GA99413 at msticlxl7 dot ims dot intel dot com>
On Mon, Oct 20, 2014 at 5:19 PM, Ilya Tocar <tocarip.intel@gmail.com> wrote:
>> >
>> > The patch is OK with the above improvement.
>> >
>> >
>>
>> Will commit version below, if no objections in 24 hours.
>>
>>
> Sorry,
> I've missed palignr, which should also have v64qi version,
> and lost return in expand_vec_perm_palignr case
> (this caused avx512f-vec-unpack test failures).
> Patch below fixes it. Ok for trunk?
>
> 2014-10-20 Ilya Tocar <ilya.tocar@intel.com>
>
> * config/i386/i386.c (expand_vec_perm_1): Fix
> expand_vec_perm_palignr case.
> * config/i386/sse.md (<ssse3_avx2>_palignr<mode>_mask): Use
> VI1_AVX512.
OK.
Thanks,
Uros.