Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-19 20:55 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: > On 19 Jan 19:42, Andrew Senkevich wrote: >> 2017-01-19 13:39 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: >> > Hi Andrew, >> > On 18 Jan 15:45, Andrew Senkevich wrote: >> &g

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-20 20:08 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: > Hi, > On 20 Jan 14:46, Uros Bizjak wrote: >> On Fri, Jan 20, 2017 at 2:32 PM, Andrew Senkevich >> <andrew.n.senkev...@gmail.com> wrote: >> >> > here is intrinsics for ktest{b,w,d,q}

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 13:58 GMT+01:00 Kirill Yukhin <kirill.yuk...@gmail.com>: > On 10 Jan 15:00, Andrew Senkevich wrote: >> 2017-01-10 13:31 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >> > On Tue, Jan 10, 2017 at 11:21 AM, Andrew Senkevich >> > <andrew.n.senkev..

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-17 16:51 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: > On Tue, Jan 17, 2017 at 04:03:08PM +0300, Andrew Senkevich wrote: >> > I've played a bit w/ SDE. And looks like operands are not early clobber: >> > TID0: INS 0x004003ee AVX512VEX km

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-17 15:30 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: > Hi Anrey, > On 17 Jan 14:04, Andrew Senkevich wrote: >> 2017-01-17 1:55 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: >> > On Tue, Jan 17, 2017 at 01:30:11AM +0300, Andrew Senkevich wrote: >>

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-17 1:55 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: > On Tue, Jan 17, 2017 at 01:30:11AM +0300, Andrew Senkevich wrote: >> here is one more part of intrinsics for k-mask registers shifts: > > The software developer manuals describe KSHIFT{L,R}* like: > KSHIFTLW >

Re: [PATCH] Add AVX512 k-mask intrinsics

Hi, here is one more part of intrinsics for k-mask registers shifts: gcc/ * config/i386/avx512bwintrin.h: Add k-mask registers shift intrinsics. * config/i386/avx512dqintrin.h: Ditto. * config/i386/avx512fintrin.h: Ditto. * config/i386/i386-builtin-types.def: Add new types. *

Re: [PATCH] Add AVX512 k-mask intrinsics

2017-01-19 13:39 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: > Hi Andrew, > On 18 Jan 15:45, Andrew Senkevich wrote: >> 2017-01-17 16:51 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: >> > On Tue, Jan 17, 2017 at 04:03:08PM +0300, Andrew Senkevich wrote:

Re: [PATCH] Remove x86 pcommit instruction

2017-02-25 18:10 GMT+01:00 Gerald Pfeifer <ger...@pfeifer.com>: > On Wed, 7 Dec 2016, Andrew Senkevich wrote: >>>> But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? >>> There are corresponding documents at [1] and [2], please add a "GCC >&

Re: [PATCH] Remove x86 pcommit instruction

Thank you, it is fixed in patch below. Ok for trunk? Is it subject for backport for 5.* and 6.* releases? 2016-10-05 Andrew Senkevich <andrew.senkev...@intel.com> gcc/ * common/config/i386/i386-common.c (OPTION_MASK_ISA_PCOMMIT_UNSET, OPTION_MASK_ISA_PCO

Re: [PATCH] Remove x86 pcommit instruction

> (Please also provide an entry for Release Changes, since this is > user-facing change. Also for release branches.) What is BKM for doing that for 7.1 release? File a PR and mention it in ChangeLog? -- WBR, Andrew

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 18:06 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >>> -mpcommit >>> -Target Report Mask(ISA_PCOMMIT) Var(ix86_isa_flags) Save >>> -Support PCOMMIT ins

[PATCH] Remove x86 pcommit instruction

Hi, this patch removes PCOMMIT instruction since it was deprecated, please visit https://software.intel.com/en-us/blogs/2016/09/12/deprecate-pcommit-instruction for details. Regtested on x86_64. Is it Ok for trunk? 2016-10-03 Andrew Senkevich <andrew.senkev...@intel.com>

[PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

Hi, this patch enabled AVX512_4FMAPS and AVX512_4VNNIW instructions. It requires additional patch for register allocator from Vladimir Makarov to be committed before. gcc/ * common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX5124FMAPS_SET,

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-10 19:36 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: > On Thu, Nov 10, 2016 at 07:27:00PM +0300, Andrew Senkevich wrote: >> Hi, >> >> this patch enabled AVX512_4FMAPS and AVX512_4VNNIW instructions. >> >> It requires additional patch for regi

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-10 20:14 GMT+03:00 Vladimir N Makarov <vmaka...@redhat.com>: > > > On 11/10/2016 11:27 AM, Andrew Senkevich wrote: >> >> Hi, >> >> this patch enabled AVX512_4FMAPS and AVX512_4VNNIW instructions. >> >> It requires additional pa

Re: [PATCH] Remove x86 pcommit instruction

> That said - why do we need to remove pcommit support on the branches at all? I think it is better to have old branches updated because pcommit will not be implemented in hardware. -- WBR, Andrew

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-11-11 18:26 GMT+03:00 Marc Glisse <marc.gli...@inria.fr>: > On Fri, 11 Nov 2016, Andrew Senkevich wrote: > >> +extern __inline __mmask32 >> +__attribute__ ((__gnu_inline__, __always_inline__, __artificial__)) >> +_kand_mask32 (__mmask32 __A, __mmask32 __B)

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-11 14:16 GMT+03:00 Uros Bizjak : > The x86 part of the patch is OK with the above changes and additional > target attribute test for flags2 ISA features.. Fixed according your comments, I will followup with additional tests soon. -- WBR, Andrew

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-11 14:29 GMT+03:00 Jakub Jelinek <ja...@redhat.com>: > Hi! > > I've noticed preexisting: > > On Thu, Nov 10, 2016 at 07:27:00PM +0300, Andrew Senkevich wrote: > >> --- a/gcc/config/i386/i386-modes.def >> +++ b/gcc/config/i386/i386-modes.def >>

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-21 20:12 GMT+03:00 Martin Sebor : > On 11/20/2016 11:16 AM, Uros Bizjak wrote: >> >> On Sat, Nov 19, 2016 at 7:52 PM, Uros Bizjak wrote: >>> >>> On Sat, Nov 19, 2016 at 6:24 PM, Jakub Jelinek wrote: On Sat, Nov 19, 2016 at

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-15 17:56 GMT+03:00 Jeff Law <l...@redhat.com>: > On 11/15/2016 05:55 AM, Andrew Senkevich wrote: >> >> 2016-11-11 14:16 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> >>> --- a/gcc/genmodes.c >>> +++ b/gcc/genmodes.c >>>

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-16 19:21 GMT+03:00 Bernd Schmidt <bschm...@redhat.com>: > On 11/15/2016 05:31 PM, Andrew Senkevich wrote: >> >> 2016-11-15 17:56 GMT+03:00 Jeff Law <l...@redhat.com>: >>> >>> On 11/15/2016 05:55 AM, Andrew Senkevich wrote: >>>&g

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-19 13:17 GMT+03:00 Uros Bizjak : > On Sat, Nov 19, 2016 at 9:05 AM, Jakub Jelinek wrote: >> On Fri, Nov 18, 2016 at 09:30:06PM +0100, Jakub Jelinek wrote: >>> On Fri, Nov 18, 2016 at 08:41:01PM +0100, Jakub Jelinek wrote: >>> > I'm seeing lots of ICEs

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-11-11 20:56 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Fri, Nov 11, 2016 at 6:50 PM, Uros Bizjak <ubiz...@gmail.com> wrote: >> On Fri, Nov 11, 2016 at 6:38 PM, Andrew Senkevich >> <andrew.n.senkev...@gmail.com> wrote: >>> 2016-11-11 17:3

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-11-11 17:34 GMT+03:00 Uros Bizjak : > Some quick remarks: > > +(define_insn "kmovb" > + [(set (match_operand:QI 0 "nonimmediate_operand" "=k,k") > + (unspec:QI > + [(match_operand:QI 1 "nonimmediate_operand" "r,km")] > + UNSPEC_KMOV))] > + "!(MEM_P (operands[0]) &&

[PATCH] Follow-up patch on enabling new AVX512 instructions

Hi, this is follow-up with tests for new __target__ attributes and __builtin_cpu_supports update. gcc/ * config/i386/i386.c (processor_features): Add F_AVX5124VNNIW, F_AVX5124FMAPS. (isa_names_table): Handle new features. libgcc/ * config/i386/cpuinfo.c

Re: [PATCH] Enable Intel AVX512_4FMAPS and AVX512_4VNNIW instructions

2016-11-11 14:16 GMT+03:00 Uros Bizjak : > --- a/gcc/genmodes.c > +++ b/gcc/genmodes.c > --- a/gcc/init-regs.c > +++ b/gcc/init-regs.c > --- a/gcc/machmode.h > +++ b/gcc/machmode.h > > These are middle-end changes, you will need a separate review for these. Who could review

Re: [PATCH] Remove x86 pcommit instruction

2016-10-06 1:07 GMT+03:00 H.J. Lu <hjl.to...@gmail.com>: > On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-10-05 18:06 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senk

Re: [PATCH] Remove x86 pcommit instruction

2016-10-11 20:09 GMT+03:00 H.J. Lu <hjl.to...@gmail.com>: > On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-10-06 1:07 GMT+03:00 H.J. Lu <hjl.to...@gmail.com>: >>> On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senk

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-12-15 19:51 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Thu, Dec 15, 2016 at 2:31 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-12-14 22:55 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> On Wed, Dec 14, 2016 at 8:04 PM

Re: [PATCH] Remove x86 pcommit instruction

2016-12-07 15:34 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Wed, Dec 7, 2016 at 1:25 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: > >> But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? > > There are corresponding documents a

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 12:23 GMT+03:00 Richard Biener <richard.guent...@gmail.com>: > On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu <hjl.to...@gmail.com> wrote: >> On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich >> <andrew.n.senkev...@gmail.com> wrote: >>> 2016-10-11 20

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-12-02 21:31 GMT+03:00 Uros Bizjak : . . . . . >> >> I split this patch after last updates in md files, here is the first >> part which doesn't change md files. >> Regtested on x86_64-linux-gnu. Is this part ok? > > There is no point to scan for kmovX insn in e.g.: > > +/*

Re: [PATCH] Enable SGX intrinsics

On Fri, Dec 30, 2016 at 03:37:14PM +0100, Uros Bizjak wrote: >> As suggested in [1], you should write multi-line enums like: >> >> enum foo >> { >> a = ... >> b = ... >> } > > Sure. Plus it depends on if users of the APIs should just write the operands > on their own as numbers, or as

Re: [PATCH] Fix PR78306

2016-11-30 11:52 GMT+03:00 Richard Biener : > On Tue, 29 Nov 2016, Jeff Law wrote: > >> On 11/29/2016 12:47 AM, Richard Biener wrote: >> > > Balaji added this check explicitly. There should be tests in the >> > > testsuite >> > > (spawnee_inline, spawner_inline) which exercise

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-12-02 21:31 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Fri, Dec 2, 2016 at 6:44 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-11-11 22:14 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> On Fri, Nov 11, 2016 at 7:23 PM

Re: [PATCH] Add AVX512 k-mask intrinsics

2016-11-11 22:14 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Fri, Nov 11, 2016 at 7:23 PM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2016-11-11 20:56 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: >>> On Fri, Nov 11, 2016 at 6:50 PM

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 13:04 GMT+03:00 Kirill Yukhin : > Hi, > In addition to Uroš's inputs: >> diff --git a/gcc/config/i386/avx512vpopcntdqintrin.h >> b/gcc/config/i386/avx512vpopcntdqintrin.h >> new file mode 100644 >> index 000..28305f6 >> --- /dev/null >> +++

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2017-01-10 13:31 GMT+03:00 Uros Bizjak <ubiz...@gmail.com>: > On Tue, Jan 10, 2017 at 11:21 AM, Andrew Senkevich > <andrew.n.senkev...@gmail.com> wrote: >> 2017-01-10 13:04 GMT+03:00 Kirill Yukhin <kirill.yuk...@gmail.com>: >>> Hi, >>> In additio

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2016-12-27 16:35 GMT+03:00 Uros Bizjak : > Hello! > >> this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently >> added in Instruction Set Extensions >>

[PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

Hi, 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). gcc/ * common/config/i386/i386-common.c