Re: [PATCH][ARM] Cleanup DImode shifts

2019-08-22 Thread Kyrill Tkachov
Hi Wilco, On 7/31/19 5:25 PM, Wilco Dijkstra wrote: ping Like the logical operations, expand all shifts early rather than only  sometimes.  The Neon shift expansions are never emitted (not even with  -fneon-for-64bits), so they are not useful.  So all the late expansions  and Neon shift patter

Re: [PATCH][ARM] Cleanup DImode shifts

2019-08-19 Thread Wilco Dijkstra
  ping      Like the logical operations, expand all shifts early rather than only  sometimes.  The Neon shift expansions are never emitted (not even with  -fneon-for-64bits), so they are not useful.  So all the late expansions  and Neon shift patterns can be removed, and shifts are mor

Re: [PATCH][ARM] Cleanup DImode shifts

2019-07-31 Thread Wilco Dijkstra
ping   Like the logical operations, expand all shifts early rather than only sometimes.  The Neon shift expansions are never emitted (not even with -fneon-for-64bits), so they are not useful.  So all the late expansions and Neon shift patterns can be removed, and shifts are more optimized a

Re: [PATCH][ARM] Cleanup DImode shifts

2019-07-22 Thread Wilco Dijkstra
Hi Ramana, > Thanks for this patch set - What I'm missing in this is any analysis as > to what's the impact on code generation for neon intrinsics that use > uint64_t ? Especially things like v_u64 ? Well things like this continue to work exactly like before: uint64x1_t f20(uint64x1_t x, uint6

Re: [PATCH][ARM] Cleanup DImode shifts

2019-07-22 Thread Ramana Radhakrishnan
On 22/07/2019 17:16, Wilco Dijkstra wrote: Like the logical operations, expand all shifts early rather than only sometimes. The Neon shift expansions are never emitted (not even with -fneon-for-64bits), so they are not useful. So all the late expansions and Neon shift patterns can be removed, a

[PATCH][ARM] Cleanup DImode shifts

2019-07-22 Thread Wilco Dijkstra
Like the logical operations, expand all shifts early rather than only sometimes. The Neon shift expansions are never emitted (not even with -fneon-for-64bits), so they are not useful. So all the late expansions and Neon shift patterns can be removed, and shifts are more optimized as a result. Si