This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Evgeny Stupachenko <evstupac at gmail dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>, Richard Henderson <rth at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <rguenther at suse dot de>
- Date: Thu, 2 Oct 2014 08:30:42 +0200
- Subject: Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Authentication-results: sourceware.org; auth=none
- References: <CAOvf_xyNC1mRGNrM1kU_nNz_tO6_M4T8wox75D+zndhY5=TVAQ at mail dot gmail dot com> <CAFULd4bfOLW2kOmSndwK=LdNbUwHR1Ogds+5_AZ7j=tH=zu12w at mail dot gmail dot com> <20141001103514 dot GO1986 at tucnak dot redhat dot com> <20141001113815 dot GQ1986 at tucnak dot redhat dot com> <CAFULd4b_T0XByAhGew-wL6D-udF6oPwuw=v6NPYdupAn9JtzXA at mail dot gmail dot com> <20141001121715 dot GR1986 at tucnak dot redhat dot com> <CAFULd4Yo0VVJ_Z6dkc1VMpFmO6BNkbaddieXgOX7uTQTrUL11A at mail dot gmail dot com> <20141001125618 dot GT1986 at tucnak dot redhat dot com> <CAFULd4YfZsZ_T1zPK_dYh4ytPRSxFJ4WkWPK-5J7BB5sZazo+Q at mail dot gmail dot com> <20141001141215 dot GU1986 at tucnak dot redhat dot com> <20141001204358 dot GW1986 at tucnak dot redhat dot com>
On Wed, Oct 1, 2014 at 10:43 PM, Jakub Jelinek <jakub@redhat.com> wrote:
> > For PR62128, IMHO the right fix is attached. Note, this is again covered
> > in vshuf-*.c tests (test 22 in both vshuf-v32*.c and vshuf-v16*.c).
> > With that attached patch, pr62128.c (aka test_22 in vshuf-v32qi.c), changes:
> > - vpshufb .LC0(%rip), %ymm0, %ymm1
> > - vpshufb .LC1(%rip), %ymm0, %ymm0
> > - vpermq $78, %ymm1, %ymm1
> > - vpor %ymm1, %ymm0, %ymm0
> > + vpermq $78, %ymm0, %ymm1
> > + vpalignr $1, %ymm0, %ymm1, %ymm0
> > ret
> >
> > 2014-10-01 Jakub Jelinek <jakub@redhat.com>
> >
> > PR target/62128
> > * config/i386/i386.c (expand_vec_perm_1): Try expand_vec_perm_palignr
> > if it expands to a single insn only.
> > (expand_vec_perm_palignr): Add SINGLE_INSN_ONLY_P argument. If true,
> > fail unless in_order is true. Add forward declaration.
> > (expand_vec_perm_vperm2f128): Fix up comment about which permutation
> > is useful for one_operand_p.
> > (ix86_expand_vec_perm_const_1): Adjust expand_vec_perm_palignr caller.
>
> Now bootstrapped/regtested on x86_64-linux and i686-linux (and additionally
> tested also with --target_board=unix/-mavx2), ok for trunk?
>
OK.
Thanks,
Uros.
- References:
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.
- Re: [PATCH 1/2, x86] Add palignr support for AVX2.