On Tue, Dec 27, 2016 at 2:47 PM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Tue, Dec 27, 2016 at 2:40 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-12-27 16:35 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> Hello! >>> >>>> this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently >>>> added in Instruction Set Extensions >>>> (https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf). >>> >>> @@ -265,6 +268,9 @@ >>> (define_mode_iterator VF_512 >>> [V16SF V8DF]) >>> >>> +(define_mode_iterator VI_AVX512F >>> + [V16SI V8DI]) >>> >>> Please name this iterator VI_512. >> >> But there are already VI_512 :) >> >> ;; All 512bit vector integer modes >> (define_mode_iterator VI_512 >> [(V64QI "TARGET_AVX512BW") >> (V32HI "TARGET_AVX512BW") >> V16SI V8DI]) > > Eh, this one is duplicate of VI_AVX512BW and should be removed.
Actually, you should use existing VI48_512 mode iterator. Uros.
