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.pe...@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