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
bits)); Kind Regards, Tamar From: gcc-patches-ow...@gcc.gnu.org on behalf of Tamar Christina Sent: Tuesday, January 17, 2017 2:50:19 PM To: GCC Patches; James Greenhalgh; Marcus Shawcroft; Richard Earnshaw Cc: nd Subject: [PATCH][GCC][Aarch64] Add vectoriz

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

2017-01-17 Thread Tamar Christina
Hi All, This patch vectorizes the copysign builtin for AArch64 similar to how it is done for Arm. AArch64 now generates: ... .L4: ldr q1, [x6, x3] add w4, w4, 1 ldr q0, [x5, x3] cmp w4, w7 bif v1.16b, v2.16b, v3.16b fmulv0.2