This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH i386 AVX512] [75/n] Update vec_init.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 9 Oct 2014 17:53:38 +0200
- Subject: Re: [PATCH i386 AVX512] [75/n] Update vec_init.
- Authentication-results: sourceware.org; auth=none
- References: <20141009121324 dot GL25028 at msticlxl57 dot ims dot intel dot com>
On Thu, Oct 9, 2014 at 2:13 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello,
> This patch extends vec_init-related routines/patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_vector_init_duplicate): Handle V64QI and V32HI modes,
> update V8HI, V16QI, V32QI modes handling.
> (ix86_expand_vector_init_general): Handle V64QI and V32HI modes.
> * config/i386/sse.md
> (define_mode_iterator VI48F_512): Rename to ...
> (define_mode_iterator VI48F_I12_AVX512BW): ... this. Extend to
> AVX-512BW modes.
> (define_expand "vec_init<mode>"): Use VI48F_I12_AVX512BW.
LGTM, but I'd like to ask Jakub for his opinion on vec_init stuff.
Uros.