RE: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-08-24 Thread Kyrylo Tkachov via Gcc-patches
> -Original Message- > From: Prathamesh Kulkarni > Sent: 24 August 2021 09:02 > To: gcc Patches ; Kyrylo Tkachov > > Subject: Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics > > On Fri, 13 Aug 2021 at 16:40, Prathamesh Kulkarni > wrote: &g

Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-08-24 Thread Prathamesh Kulkarni via Gcc-patches
On Fri, 13 Aug 2021 at 16:40, Prathamesh Kulkarni wrote: > > On Thu, 5 Aug 2021 at 15:44, Prathamesh Kulkarni > wrote: > > > > On Mon, 12 Jul 2021 at 15:24, Prathamesh Kulkarni > > wrote: > > > > > > On Mon, 12 Jul 2021 at 15:23, Prathamesh Kulkarni > > > wrote: > > > > > > > > On Mon, 5 Jul 20

Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-08-13 Thread Prathamesh Kulkarni via Gcc-patches
On Thu, 5 Aug 2021 at 15:44, Prathamesh Kulkarni wrote: > > On Mon, 12 Jul 2021 at 15:24, Prathamesh Kulkarni > wrote: > > > > On Mon, 12 Jul 2021 at 15:23, Prathamesh Kulkarni > > wrote: > > > > > > On Mon, 5 Jul 2021 at 14:47, Prathamesh Kulkarni > > > wrote: > > > > > > > > Hi, > > > > This

Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-08-05 Thread Prathamesh Kulkarni via Gcc-patches
On Mon, 12 Jul 2021 at 15:24, Prathamesh Kulkarni wrote: > > On Mon, 12 Jul 2021 at 15:23, Prathamesh Kulkarni > wrote: > > > > On Mon, 5 Jul 2021 at 14:47, Prathamesh Kulkarni > > wrote: > > > > > > Hi, > > > This patch replaces builtins with __a * __b for signed variants of > > > vmul_n intrin

Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-07-12 Thread Prathamesh Kulkarni via Gcc-patches
On Mon, 12 Jul 2021 at 15:23, Prathamesh Kulkarni wrote: > > On Mon, 5 Jul 2021 at 14:47, Prathamesh Kulkarni > wrote: > > > > Hi, > > This patch replaces builtins with __a * __b for signed variants of > > vmul_n intrinsics. > > As discussed earlier, the patch has issue if __a * __b overflows, an

Re: [ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-07-12 Thread Prathamesh Kulkarni via Gcc-patches
On Mon, 5 Jul 2021 at 14:47, Prathamesh Kulkarni wrote: > > Hi, > This patch replaces builtins with __a * __b for signed variants of > vmul_n intrinsics. > As discussed earlier, the patch has issue if __a * __b overflows, and > whether we wish to leave > that as UB. ping https://gcc.gnu.org/git/g

[ARM] PR66791: Replace builtins for signed vmul_n intrinsics

2021-07-05 Thread Prathamesh Kulkarni via Gcc-patches
Hi, This patch replaces builtins with __a * __b for signed variants of vmul_n intrinsics. As discussed earlier, the patch has issue if __a * __b overflows, and whether we wish to leave that as UB. Thanks, Prathamesh diff --git a/gcc/config/arm/arm_neon.h b/gcc/config/arm/arm_neon.h index 41b596b5f