On Tue, Feb 18, 2020 at 7:00 PM Hongtao Liu <crazy...@gmail.com> wrote: > > 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
Done, also port to gcc-8 and gcc-9. -- BR, Hongtao
