On Wed, Sep 10, 2014 at 3:05 PM, Kirill Yukhin <kirill.yuk...@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.
