This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH i386 AVX512] [26/n] Support 512-bit/masked interleave.


On Wed, Aug 27, 2014 at 2:47 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello,
> Patch in the bottom extends interleaves toward support
> of AVX-512.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
>         * config/i386/sse.md
>         (define_insn "avx512bw_interleave_highv64qi<mask_name>"): New.
>         (define_insn "avx2_interleave_highv32qi<mask_name>"): Add masking.
>         (define_insn "vec_interleave_highv16qi<mask_name>"): Ditto.
>         (define_insn "avx2_interleave_lowv32qi<mask_name>"): Ditto.
>         (define_insn "vec_interleave_lowv16qi<mask_name>"): Ditto.
>         (define_insn "avx2_interleave_highv16hi<mask_name>"): Ditto.
>         (define_insn "vec_interleave_highv8hi<mask_name>"): Ditto.
>         (define_insn "avx2_interleave_lowv16hi<mask_name>"): Ditto.
>         (define_insn "vec_interleave_lowv8hi<mask_name>"): Ditto.
>         (define_insn "avx2_interleave_highv8si<mask_name>"): Ditto.
>         (define_insn "vec_interleave_highv4si<mask_name>"): Ditto.
>         (define_insn "avx2_interleave_lowv8si<mask_name>"): Ditto.
>         (define_insn "vec_interleave_lowv4si<mask_name>"): Ditto.
>         (define_insn "vec_interleave_highv16qi<mask_name>"): New.
>         (define_insn "avx512bw_interleave_highv32hi<mask_name>"): Ditto.
>         (define_insn "<mask_codefor>avx512bw_interleave_lowv32hi<mask_name>"): Ditto..

I didn't check all the selectors in detail, but the patch LGTM.

So, OK.

Thanks,
Uros.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]