Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-08 Thread James Greenhalgh
On Wed, Sep 02, 2015 at 02:00:23PM +0100, Kyrill Tkachov wrote: > > On 01/09/15 11:40, Kyrill Tkachov wrote: > > Hi James, > > > > On 01/09/15 10:25, James Greenhalgh wrote: > >> On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: > >> > >> Some comments below. > > Thanks, I'll incorpo

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-02 Thread Kyrill Tkachov
On 01/09/15 11:40, Kyrill Tkachov wrote: Hi James, On 01/09/15 10:25, James Greenhalgh wrote: On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. Thanks, I'll incorporate them, with one clarification inline. And here's the updated patch. Thanks, Kyrill 20

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-01 Thread Kyrill Tkachov
Hi James, On 01/09/15 10:25, James Greenhalgh wrote: On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. Thanks, I'll incorporate them, with one clarification inline. diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index 1394ed7..c

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-01 Thread James Greenhalgh
On Thu, Aug 13, 2015 at 01:36:50PM +0100, Kyrill Tkachov wrote: Some comments below. > diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c > index 1394ed7..c8bd8d2 100644 > --- a/gcc/config/aarch64/aarch64.c > +++ b/gcc/config/aarch64/aarch64.c > @@ -6652,6 +6652,25 @@ cost_p

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-09-01 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 20/08/15 09:24, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00710.html Thanks, Kyrill On 03/08/15 14:01, James Greenhalgh wrote: On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: Hi all, This patch implements an aarch64-sp

RE: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-08-20 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00710.html Thanks, Kyrill On 03/08/15 14:01, James Greenhalgh wrote: > On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: >> Hi all, >> >> This patch implements an aarch64-specific expansion of the signed modulo by a power of 2. >>

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-08-13 Thread Kyrill Tkachov
On 03/08/15 14:01, James Greenhalgh wrote: On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: Hi all, This patch implements an aarch64-specific expansion of the signed modulo by a power of 2. The proposed sequence makes use of the conditional negate instruction CSNEG. For a power

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-08-03 Thread James Greenhalgh
On Fri, Jul 24, 2015 at 11:55:33AM +0100, Kyrill Tkachov wrote: > Hi all, > > This patch implements an aarch64-specific expansion of the signed modulo by a > power of 2. > The proposed sequence makes use of the conditional negate instruction CSNEG. > For a power of N, x % N can be calculated with

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-07-31 Thread Andrew Pinski
On Fri, Jul 24, 2015 at 3:55 AM, Kyrill Tkachov wrote: > Hi all, > > This patch implements an aarch64-specific expansion of the signed modulo by > a power of 2. > The proposed sequence makes use of the conditional negate instruction CSNEG. > For a power of N, x % N can be calculated with: > negs

Re: [PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-07-31 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02036.html Thanks, Kyrill On 24/07/15 11:55, Kyrill Tkachov wrote: Hi all, This patch implements an aarch64-specific expansion of the signed modulo by a power of 2. The proposed sequence makes use of the conditional negate instruction CSNEG.

[PATCH][AArch64][1/3] Expand signed mod by power of 2 using CSNEG

2015-07-24 Thread Kyrill Tkachov
Hi all, This patch implements an aarch64-specific expansion of the signed modulo by a power of 2. The proposed sequence makes use of the conditional negate instruction CSNEG. For a power of N, x % N can be calculated with: negs x1, x0 andx0, x0, #(N - 1) andx1, x1, #(N - 1) csneg x0,