On Tue, Feb 18, 2020 at 4:24 PM Uros Bizjak <ubiz...@gmail.com> wrote: > > > > 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. >
Will do. -- BR, Hongtao