Re: [PATCH][AArch64 array_mode 7/8] Combine the expanders using VSTRUCT:nregs

2015-09-15 Thread Alan Lawrence
On 15/09/15 10:43, James Greenhalgh wrote: > > It is convenient that this falls out, but likely surprising for nregs. > Please add a comment to nregs explaining the dual use of nregs to represent > both the number of Q registers used for the type, and the number of elements > touched by the structu

Re: [PATCH][AArch64 array_mode 7/8] Combine the expanders using VSTRUCT:nregs

2015-09-15 Thread James Greenhalgh
On Tue, Sep 15, 2015 at 10:14:42AM +0100, Alan Lawrence wrote: > The previous patches leave ld[234]_lane, st[234]_lane, and ld[234]r expanders > all nearly identical, so we can easily parameterize across the number of > lanes and combine them. > > For the ld_lane pattern, I switched from the VCONQ

[PATCH][AArch64 array_mode 7/8] Combine the expanders using VSTRUCT:nregs

2015-09-15 Thread Alan Lawrence
The previous patches leave ld[234]_lane, st[234]_lane, and ld[234]r expanders all nearly identical, so we can easily parameterize across the number of lanes and combine them. For the ld_lane pattern, I switched from the VCONQ attribute to just using the MODE attribute, this is identical for all

[PATCH][AArch64 array_mode 7/8] Combine the expanders using VSTRUCT:nregs

2015-08-26 Thread Alan Lawrence
The previous patches leave ld[234]_lane, st[234]_lane, and ld[234]r expanders all nearly identical, so we can easily parameterize across the number of lanes and combine them. For the ld_lane pattern, I switched from the VCONQ attribute to just using the MODE attribute, this is identical for all