Re: [PATCH][gimple-linterchange] Rewrite compute_access_stride

2017-12-01 Thread Bin.Cheng
On Fri, Dec 1, 2017 at 2:26 PM, Richard Biener wrote: > On Fri, 1 Dec 2017, Bin.Cheng wrote: > >> On Fri, Dec 1, 2017 at 12:31 PM, Richard Biener wrote: >> > >> > This is the access stride computation change. Apart from the >> > stride extraction I adjusted the cost model to handle non-constant

Re: [PATCH][gimple-linterchange] Rewrite compute_access_stride

2017-12-01 Thread Richard Biener
On Fri, 1 Dec 2017, Bin.Cheng wrote: > On Fri, Dec 1, 2017 at 12:31 PM, Richard Biener wrote: > > > > This is the access stride computation change. Apart from the > > stride extraction I adjusted the cost model to handle non-constant > > strides by checking if either is a multiple of the other a

Re: [PATCH][gimple-linterchange] Rewrite compute_access_stride

2017-12-01 Thread Bin.Cheng
On Fri, Dec 1, 2017 at 12:31 PM, Richard Biener wrote: > > This is the access stride computation change. Apart from the > stride extraction I adjusted the cost model to handle non-constant > strides by checking if either is a multiple of the other and > simply fail interchanging if it's the wrong

[PATCH][gimple-linterchange] Rewrite compute_access_stride

2017-12-01 Thread Richard Biener
This is the access stride computation change. Apart from the stride extraction I adjusted the cost model to handle non-constant strides by checking if either is a multiple of the other and simply fail interchanging if it's the wrong way around for one ref or if the simple method using multiple_of