This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][i386,AVX] Fix PR84783 - backport missing permutexvar to GCC7
- From: Kirill Yukhin <kirill dot yukhin at gmail dot com>
- To: jakub at redhat dot com, "Peryt, Sebastian" <sebastian dot peryt at intel dot com>
- Cc: "'gcc-patches at gcc dot gnu dot org'" <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 23 Mar 2018 08:48:46 +0300
- Subject: Re: [PATCH][i386,AVX] Fix PR84783 - backport missing permutexvar to GCC7
- References: <17623B198193D741876BD81A6E3AE5AD3C6156D5@irsmsx111.ger.corp.intel.com>
Hello Sebastian!
On 22 мар 13:01, Peryt, Sebastian wrote:
> Hi,
>
> This patch adds missing permutexvar intrinsics for backporting to GCC 7 to resolve PR84783.
>
> 2018-03-22 Sebastian Peryt <sebastian.peryt@intel.com>
>
> gcc:
> PR84783
> * config/i386/avx512vlintrin.h (_mm256_permutexvar_epi64)
> (_mm256_permutexvar_epi32, _mm256_permutex_epi64): New intrinsics.
>
> gcc/testsuite:
> PR84783
>
> * gcc.target/i386/avx512vl-vpermd-1.c (_mm256_permutexvar_epi32):
> Test new intrinsic.
> * gcc.target/i386/avx512vl-vpermq-imm-1.c (_mm256_permutex_epi64):
> Ditto.
> * gcc.target/i386/avx512vl-vpermq-var-1.c (_mm256_permutexvar_epi64):
> Ditto.
> * gcc.target/i386/avx512f-vpermd-2.c: Do not check for AVX512F_LEN.
> * gcc.target/i386/avx512f-vpermq-imm-2.c: Ditto.
> * gcc.target/i386/avx512f-vpermq-var-2.c: Ditto.
>
> Is it ok for merge?
Your patch is pretty much simple and is OK to me.
However, since you're aiming to GCC 7, I'd like to here GM's OK here as well.
--
Thanks, K
>
> Thanks,
> Sebastian