Re: [PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-09-02 Thread Marcus Shawcroft
On 18 August 2014 17:50, Alan Lawrence wrote: > Well, you're right that it could be. So I presented the wrong justification. > > Clearly we would benefit from some better cost infrastructure here, ideally > that is expressive, taken into account at all appropriate stages of the > compiler, and tun

Re: [PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-08-18 Thread Alan Lawrence
Well, you're right that it could be. So I presented the wrong justification. Clearly we would benefit from some better cost infrastructure here, ideally that is expressive, taken into account at all appropriate stages of the compiler, and tunable per core. I imagine that steps (patches) towards

Re: [PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-08-13 Thread Vladimir Makarov
On 2014-08-13, 4:36 AM, James Greenhalgh wrote: On Tue, Aug 12, 2014 at 04:53:38PM +0100, pins...@gmail.com wrote: On Aug 12, 2014, at 7:40 AM, Alan Lawrence wrote: (It is no more expensive.) Yes on some processors it could be. Haven't we been here before [1]? This disparaging mechanis

Re: [PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-08-13 Thread James Greenhalgh
On Tue, Aug 12, 2014 at 04:53:38PM +0100, pins...@gmail.com wrote: > > > > On Aug 12, 2014, at 7:40 AM, Alan Lawrence wrote: > > > > (It is no more expensive.) > > Yes on some processors it could be. Haven't we been here before [1]? This disparaging mechanism is still not going to give what

Re: [PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-08-12 Thread pinskia
> On Aug 12, 2014, at 7:40 AM, Alan Lawrence wrote: > > (It is no more expensive.) Yes on some processors it could be. Thanks, Andrew > > gcc/ChangeLog: > >* config/aarch64/aarch64.md (subdi3, adddi3_aarch64): Don't penalize >SIMD reg variant. > diff --git a/gcc/config/aarch64/aa

[PATCH AArch64 1/3] Don't disparage add/sub in SIMD registers

2014-08-12 Thread Alan Lawrence
(It is no more expensive.) gcc/ChangeLog: * config/aarch64/aarch64.md (subdi3, adddi3_aarch64): Don't penalize SIMD reg variant.diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index f8eb305140e7b0aed006b33f1724a90939e48316..0a8ca4bcc7941f912c8d42200b332