RE: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-27 Thread Tamar Christina
redhat.com > Subject: Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)] > > On 26 November 2017 at 13:56, Christophe Lyon > wrote: > > On 24 November 2017 at 20:38, Christophe Lyon > > wrote: > >> On 24 November 2017 at 19:05, Tamar Christina >

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-27 Thread Kyrill Tkachov
Hi Christophe, On 26/11/17 20:01, Christophe Lyon wrote: On 26 November 2017 at 13:56, Christophe Lyon wrote: On 24 November 2017 at 20:38, Christophe Lyon wrote: On 24 November 2017 at 19:05, Tamar Christina wrote: Hi Christophe, After your commit, I have these reports: http://people.li

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-26 Thread Christophe Lyon
On 26 November 2017 at 13:56, Christophe Lyon wrote: > On 24 November 2017 at 20:38, Christophe Lyon > wrote: >> On 24 November 2017 at 19:05, Tamar Christina >> wrote: >>> Hi Christophe, >>> After your commit, I have these reports: http://people.linaro.org/~christophe.lyon/cross

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-26 Thread Christophe Lyon
On 24 November 2017 at 20:38, Christophe Lyon wrote: > On 24 November 2017 at 19:05, Tamar Christina wrote: >> Hi Christophe, >> >>> >>> After your commit, I have these reports: >>> http://people.linaro.org/~christophe.lyon/cross- >>> validation/gcc/trunk/255064/report-build-info.html >>> >>> Aft

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-24 Thread Christophe Lyon
On 24 November 2017 at 19:05, Tamar Christina wrote: > Hi Christophe, > >> >> After your commit, I have these reports: >> http://people.linaro.org/~christophe.lyon/cross- >> validation/gcc/trunk/255064/report-build-info.html >> >> After my commit, I have these reports: >> http://people.linaro.org/

RE: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-24 Thread Tamar Christina
Hi Christophe, > > After your commit, I have these reports: > http://people.linaro.org/~christophe.lyon/cross- > validation/gcc/trunk/255064/report-build-info.html > > After my commit, I have these reports: > http://people.linaro.org/~christophe.lyon/cross- > validation/gcc/trunk/255126/report-b

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-24 Thread Christophe Lyon
On 24 November 2017 at 11:31, Tamar Christina wrote: >> > >> Not sure if Kyrill actually meant to comment about the three lines above, but >> they have a bug: >> #if should be before #pragma GCC push_options. >> >> Indeed, after this patch was committed (r255064), I've noticed many >> regressions,

RE: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-24 Thread Tamar Christina
> > > Not sure if Kyrill actually meant to comment about the three lines above, but > they have a bug: > #if should be before #pragma GCC push_options. > > Indeed, after this patch was committed (r255064), I've noticed many > regressions, for instance > p64_p128 is now unsupported. This is because

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-24 Thread Kyrill Tkachov
Hi Christophe, On 23/11/17 23:26, Christophe Lyon wrote: On 22 November 2017 at 12:26, Kyrill Tkachov wrote: Hi Tamar, On 06/11/17 16:53, Tamar Christina wrote: Hi All, This patch adds the NEON intrinsics for Dot product. Dot product is available from ARMv8.2-a and onwards. Regtested on

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-23 Thread Christophe Lyon
On 22 November 2017 at 12:26, Kyrill Tkachov wrote: > Hi Tamar, > > On 06/11/17 16:53, Tamar Christina wrote: >> >> Hi All, >> >> This patch adds the NEON intrinsics for Dot product. >> >> Dot product is available from ARMv8.2-a and onwards. >> >> Regtested on arm-none-eabi, armeb-none-eabi, >> a

Re: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-22 Thread Kyrill Tkachov
Hi Tamar, On 06/11/17 16:53, Tamar Christina wrote: Hi All, This patch adds the NEON intrinsics for Dot product. Dot product is available from ARMv8.2-a and onwards. Regtested on arm-none-eabi, armeb-none-eabi, aarch64-none-elf and aarch64_be-none-elf with no issues found. Ok for trunk? gcc

RE: [PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-21 Thread Tamar Christina
Ping > -Original Message- > From: Tamar Christina [mailto:tamar.christ...@arm.com] > Sent: Monday, November 6, 2017 16:54 > To: gcc-patches@gcc.gnu.org > Cc: nd ; Ramana Radhakrishnan > ; Richard Earnshaw > ; ni...@redhat.com; Kyrylo Tkachov > > Subject: [P

[PATCH][GCC][ARM] Dot Product NEON intrinsics [Patch (3/8)]

2017-11-06 Thread Tamar Christina
Hi All, This patch adds the NEON intrinsics for Dot product. Dot product is available from ARMv8.2-a and onwards. Regtested on arm-none-eabi, armeb-none-eabi, aarch64-none-elf and aarch64_be-none-elf with no issues found. Ok for trunk? gcc/ 2017-11-06 Tamar Christina * config/aarch