On Thu, Sep 11, 2014 at 3:00 PM, Kirill Yukhin <kirill.yuk...@gmail.com> wrote:
> Patch in the bottom extends integer max/min patterns. > Also, it seems, like rounding variant was generated > for maxmin patterns. Bug fixed. > > Bootstrapped. > AVX-512* tests on top of patch-set all pass > under simulator. > > Is it ok for trunk? > > gcc/ > * config/i386/sse.md (VI128_256): Delete. > (define_mode_iterator VI124_256): New. > (define_mode_iterator VI124_256_AVX512F_AVX512BW): Ditto. > (define_expand "<code><mode>3<mask_name><round_name>"): Delete. > (define_expand "<code><VI124_256_AVX512F_AVX512BW:mode>3"): New. > (define_insn "*avx2_<code><VI124_256:mode>3"): Rename from > "*avx2_<code><mode>3<mask_name><round_name>" and update mode iterator. > (define_expand "<code><VI48_AVX512VL:mode>3_mask"): New. > (define_insn "*avx512bw_<code><VI48_AVX512VL:mode>3<mask_name>"): > Ditto. > (define_insn "<mask_codefor><code><mode>3<mask_name>"): Update mode > iterator. > (define_expand "<code><mode:VI8_AVX2>3"): Update pettern generation "<code><VI8_AVX2:mode>3" > in presence of AVX-512. The patch is OK. (BTW: Sometimes "svn diff -x -upw" comes handy to exclude whitespace changes. I don't know the equivalent option for git, though.) Thanks, Uros.
