Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-09-13 Thread Wilco Dijkstra
Andrew Pinski wrote: > > Note this caused a few testsuite failures: Confirmed. The diffs show the new sequence is always better. I've committed this: Update vmov_n_1.c now we are generating better code for dup: ldr s0, [x0] dup v0.2s, v0.s[0] ret Instead of:

Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-09-12 Thread Andrew Pinski
On Tue, Sep 12, 2017 at 9:10 AM, James Greenhalgh wrote: > On Wed, Jul 26, 2017 at 02:46:14PM +0100, Wilco Dijkstra wrote: >> Remove the remaining uses of '*' from the movsi/di/ti patterns. >> Using '*' in alternatives is typically incorrect at it tells the register >>

Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-09-12 Thread James Greenhalgh
On Wed, Jul 26, 2017 at 02:46:14PM +0100, Wilco Dijkstra wrote: > Remove the remaining uses of '*' from the movsi/di/ti patterns. > Using '*' in alternatives is typically incorrect at it tells the register > allocator to ignore those alternatives. So remove these from all the > integer move

Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-09-09 Thread Andrew Pinski
On Wed, Jul 26, 2017 at 6:46 AM, Wilco Dijkstra wrote: > Remove the remaining uses of '*' from the movsi/di/ti patterns. > Using '*' in alternatives is typically incorrect at it tells the register > allocator to ignore those alternatives. So remove these from all the >

Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-08-15 Thread Wilco Dijkstra
  ping From: Wilco Dijkstra Sent: 26 July 2017 14:46 To: GCC Patches; James Greenhalgh Cc: nd Subject: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns     Remove the remaining uses of '*' from the movsi/di/ti patterns. Using '*' in alternatives is typically incorrect at it tells

Re: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-08-01 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 26 July 2017 14:46 To: GCC Patches; James Greenhalgh Cc: nd Subject: [PATCH][AArch64] Remove '*' from movsi/di/ti patterns   Remove the remaining uses of '*' from the movsi/di/ti patterns. Using '*' in alternatives is typically incorrect at it tells

[PATCH][AArch64] Remove '*' from movsi/di/ti patterns

2017-07-26 Thread Wilco Dijkstra
Remove the remaining uses of '*' from the movsi/di/ti patterns. Using '*' in alternatives is typically incorrect at it tells the register allocator to ignore those alternatives. So remove these from all the integer move patterns. This removes unnecessary int to float moves, for example