Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-10-27 Thread Kyrill Tkachov
Hi Wilco, On 16/10/17 12:29, Wilco Dijkstra wrote: ping Kyrill Tkachov wrote: On 14/12/16 16:37, Wilco Dijkstra wrote: Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied only to

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-10-16 Thread Wilco Dijkstra
ping   Kyrill Tkachov wrote: > On 14/12/16 16:37, Wilco Dijkstra wrote: > > > Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid > > unnecessary duplication and repeating bugs like PR78439 due to changes being > > applied only to one of the duplicates. > > When this was

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-06-27 Thread Wilco Dijkstra
  ping     Richard Earnshaw (lists) wrote: >  (define_insn "*movdi_vfp" > -  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,r,w,w, Uv") > +  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,!r,w,w, Uv") > Why have you introduced a no-reloads

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-06-13 Thread Wilco Dijkstra
ping   Richard Earnshaw (lists) wrote: >  (define_insn "*movdi_vfp" > -  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,r,w,w, Uv") > +  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,!r,w,w, Uv") > Why have you introduced a no-reloads block

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-05-05 Thread Wilco Dijkstra
Richard Earnshaw (lists) wrote: >  (define_insn "*movdi_vfp" > -  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,r,w,w, Uv") > +  [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,!r,w,w, Uv") > Why have you introduced a no-reloads block on the 9th

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-05-05 Thread Richard Earnshaw (lists)
On 29/11/16 11:05, Wilco Dijkstra wrote: > Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid > unnecessary duplication and repeating bugs like PR78439 due to changes being > applied only to one of the duplicates. > > Bootstrap OK for ARM and Thumb-2 gnueabihf targets. OK f

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-04-20 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8   Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied only

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-02-23 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8     Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-02-02 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8   Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2017-01-17 Thread Wilco Dijkstra
Wilco Dijkstra wrote: > Ramana Radhakrishnan wrote: >> On Wed, Dec 14, 2016 at 5:43 PM, Wilco Dijkstra >> wrote: > > > > Yes, the reason to split the pattern was to introduce the '!' to > > > discourage Neon->int moves on Cortex-A8 > (https://patches.linaro.org/patch/541/). I am not removing the

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-19 Thread Wilco Dijkstra
Ramana Radhakrishnan wrote: > On Wed, Dec 14, 2016 at 5:43 PM, Wilco Dijkstra > wrote: > > Yes, the reason to split the pattern was to introduce the '!' to discourage > > Neon->int moves on Cortex-A8 (https://patches.linaro.org/patch/541/). I am > > not removing the optimization for Cortex-A8,

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-19 Thread Ramana Radhakrishnan
On Wed, Dec 14, 2016 at 5:43 PM, Wilco Dijkstra wrote: > Kyrill Tkachov wrote: >> On 14/12/16 16:37, Wilco Dijkstra wrote: >> >> > Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid >> > unnecessary duplication and repeating bugs like PR78439 due to changes >> > being >> >

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-14 Thread Wilco Dijkstra
Kyrill Tkachov wrote: > On 14/12/16 16:37, Wilco Dijkstra wrote: > > > Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid > > unnecessary duplication and repeating bugs like PR78439 due to changes being > > applied only to one of the duplicates. > > When this was brought up

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-14 Thread Kyrill Tkachov
Hi Wilco, On 14/12/16 16:37, Wilco Dijkstra wrote: ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8 Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-14 Thread Wilco Dijkstra
  ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8     Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being

Re: [PATCH][ARM] Remove movdi_vfp_cortexa8

2016-12-06 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 29 November 2016 11:05 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Remove movdi_vfp_cortexa8   Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied only

[PATCH][ARM] Remove movdi_vfp_cortexa8

2016-11-29 Thread Wilco Dijkstra
Merge the movdi_vfp_cortexa8 pattern into movdi_vfp and remove it to avoid unnecessary duplication and repeating bugs like PR78439 due to changes being applied only to one of the duplicates. Bootstrap OK for ARM and Thumb-2 gnueabihf targets. OK for commit? ChangeLog: 2016-11-29 Wilco Dijkstra