Re: [PATCH, ARM] use vmov.i64 to load 0 into FP reg if neon enabled

2016-05-07 Thread Jim Wilson
On Fri, May 6, 2016 at 7:29 AM, Kyrill Tkachov wrote: > Since you're modifying the both the ARM and Thumb2 pattern > can you please do two bootstrap and tests, one with --with-mode=arm > and one with --with-mode=thumb. > Ok after adding the assert mentioned above, the arm/thumb testing and fixing

Re: [PATCH, ARM] use vmov.i64 to load 0 into FP reg if neon enabled

2016-05-06 Thread Kyrill Tkachov
Hi Jim, On 05/05/16 22:37, Jim Wilson wrote: For this simple testcase double sub (void) { return 0.0; } Without the attached patch, an ARM compiler with neon support enabled, gives vldr.64 d0, .L2 With the attached patch, an ARM compiler with neon enabled, gives vmov.i64 d0, #0@

[PATCH, ARM] use vmov.i64 to load 0 into FP reg if neon enabled

2016-05-05 Thread Jim Wilson
For this simple testcase double sub (void) { return 0.0; } Without the attached patch, an ARM compiler with neon support enabled, gives vldr.64 d0, .L2 With the attached patch, an ARM compiler with neon enabled, gives vmov.i64 d0, #0@ float which is faster and smaller, as there is no