[PATCH i386 AVX512] [35/n] Add vperm[it]2 insns support.

Uros Bizjak ubizjak@gmail.com
Wed Sep 10 17:17:00 GMT 2014


On Wed, Sep 10, 2014 at 3:05 PM, Kirill Yukhin <kirill.yukhin@gmail.com> wrote:
> Hello,
> Patch in the bottom extends perm[t|i] patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
>         * config/i386/sse.md
>         (define_expand "<avx512>_vpermi2var<mode>3_maskz" with VI48F
>         mode iterator): Rename from
>         "avx512f_vpermi2var<mode>3_maskz" and update mode iterator.
>         (define_expand "<avx512>_vpermi2var<mode>3_maskz" with
>         VI2_AVX512VL mode iterator): New.
>         (define_insn "<avx512>_vpermi2var<mode>3<sd_maskz_name>" with
>         VI48F): Renamefrom "avx512f_vpermi2var<mode>3<sd_maskz_name>"
>         and update mode iterator.
>         (define_insn "<avx512>_vpermi2var<mode>3<sd_maskz_name>" with
>         VI2_AVX512VL): New.
>         (define_insn "<avx512>_vpermi2var<mode>3_mask" with VI48F
>         mode iterator): Rename from "avx512f_vpermi2var<mode>3_mask"
>         and update mode iterator.
>         (define_insn "<avx512>_vpermi2var<mode>3_mask" with VI2_AVX512VL
>         mode iterator): New.
>         (define_expand "<avx512>_vpermt2var<mode>3_maskz" with VI48F
>         mode iterator): Rename from "avx512f_vpermt2var<mode>3_maskz" and
>         update mode iterator.
>         (define_expand "<avx512>_vpermt2var<mode>3_maskz" with VI2_AVX512VL
>         mode iterator): New.
>         (define_insn "<avx512>_vpermt2var<mode>3<sd_maskz_name>" with VI48F
>         mode iterator): Rename from "avx512f_vpermt2var<mode>3<sd_maskz_name>"
>         and update mode iterator.
>         (define_insn "<avx512>_vpermt2var<mode>3<sd_maskz_name>" with
>         VI2_AVX512VL mode iterator): New.
>         (define_insn "<avx512>_vpermt2var<mode>3_mask" with VI48F mode
>         iterator): Rename from "avx512f_vpermt2var<mode>3_mask" and update mode
>         iterator.
>         (define_insn "<avx512>_vpermt2var<mode>3_mask" with VI2_AVX512VL
>         mode iterator): New.

I'd personally prefer extended mode attribute names here, as described
in my previous mail.

OK.

Thanks,
Uros.



More information about the Gcc-patches mailing list