Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread James Greenhalgh
On Thu, Jan 19, 2017 at 06:05:52PM +, Tamar Christina wrote: > Hi James, > > I have corrected the testsuite changes and attached is the new file and > changelog. > > Ok for trunk? > > Tamar > > Hi All, > > This patch vectorizes the copysign builtin for Aarch64 > similar to how it is done

Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread Tamar Christina
Sent: Thursday, January 19, 2017 4:58:09 PM To: Tamar Christina Cc: GCC Patches; Marcus Shawcroft; Richard Earnshaw; nd Subject: Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign. On Thu, Jan 19, 2017 at 03:55:36PM +, Tamar Christina wrote: > > It seems I can drop even more

Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread James Greenhalgh
On Thu, Jan 19, 2017 at 03:55:36PM +, Tamar Christina wrote: > > It seems I can drop even more: The AArch64 parts of this look fine to me, and based on benchmarking of the patch they are low risk for high reward (and other targets have had a vectorized copysign for a while without issue). Ho

Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread Tamar Christina
PM To: GCC Patches; James Greenhalgh; Marcus Shawcroft; Richard Earnshaw Cc: nd Subject: Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign. Hi All, It seems the entry in config/aarch64/aarch64-builtins.c isn't needed, as such I've simplified the patch and the changelog. Ok

Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread Tamar Christina
t; Richard Earnshaw Cc: nd Subject: Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign. Hi All, This is a slight modification of the earlier patch (Using a different constant in the mask creation.) < + HOST_WIDE_I

Re: [PATCH][GCC][Aarch64] Add vectorize patten for copysign.

2017-01-19 Thread Tamar Christina
Hi All, This is a slight modification of the earlier patch (Using a different constant in the mask creation.) < + HOST_WIDE_INT_M1 << bits)); --- > + HOST_WIDE_INT_M1U << > bits)); Kind Regards, T