On Thu, Feb 13, 2020 at 9:39 AM Uros Bizjak <ubiz...@gmail.com> wrote:
> > Changelog > > gcc/ > > * config/i386/avx512vbmi2intrin.h > > (_mm512_[,mask_,maskz_]shrdi_epi16, > > _mm512_[,mask_,maskz_]shrdi_epi32, > > _m512_[,mask_,maskz_]shrdi_epi64, > > _mm512_[,mask_,maskz_]shldi_epi16, > > _mm512_[,mask_,maskz_]shldi_epi32, > > _m512_[,mask_,maskz_]shldi_epi64): Fix typo of lacking a > > closing parenthesis. > > * config/i386/avx512vbmi2vlintrin.h > > (_mm256_[,mask_,maskz_]shrdi_epi16, > > _mm256_[,mask_,maskz_]shrdi_epi32, > > _m256_[,mask_,maskz_]shrdi_epi64, > > _mm_[,mask_,maskz_]shrdi_epi16, > > _mm_[,mask_,maskz_]shrdi_epi32, > > _mm_[,mask_,maskz_]shrdi_epi64, > > _mm256_[,mask_,maskz_]shldi_epi16, > > _mm256_[,mask_,maskz_]shldi_epi32, > > _m256_[,mask_,maskz_]shldi_epi64, > > _mm_[,mask_,maskz_]shldi_epi16, > > _mm_[,mask_,maskz_]shldi_epi32, > > _mm_[,mask_,maskz_]shldi_epi64): Ditto. > > > > gcc/testsuite/ > > * gcc.target/i386/avx512vbmi2-vpshld-1.c: New test. > > * gcc.target/i386/avx512vbmi2-vpshld-O0-1.c: Ditto. > > * gcc.target/i386/avx512vbmi2-vpshrd-1.c: Ditto. > > * gcc.target/i386/avx512vbmi2-vpshrd-O0-1.c: Ditto. > > * gcc.target/i386/avx512vl-vpshld-O0-1.c: Ditto. > > * gcc.target/i386/avx512vl-vpshrd-O0-1.c: Ditto. > > This is obvious patch, so OK for mainline and backports. > > Can you please also add missing -mavx512vbmi2 to c++ testcases g++.dg/other/i386-2.C and i386-3.C? Patch is pre-approved if tested appropriately. Thanks, Uros.