Re: PR81635: Use chrecs to help find related data refs

2017-08-24 Thread Richard Sandiford
Richard Biener writes: > @@ -787,14 +821,14 @@ canonicalize_base_object_address (tree a > > bool > dr_analyze_innermost (innermost_loop_behavior *drb, tree ref, > - struct loop *loop) > + gimple *stmt, struct loop *loop) > { >

Re: PR81635: Use chrecs to help find related data refs

2017-08-24 Thread Richard Biener
On Tue, Aug 22, 2017 at 4:19 PM, Richard Sandiford wrote: > Richard Biener writes: >> On Fri, Aug 18, 2017 at 12:30 PM, Richard Biener >> wrote: >>> On Thu, Aug 17, 2017 at 2:24 PM, Bin.Cheng

Re: PR81635: Use chrecs to help find related data refs

2017-08-22 Thread Richard Sandiford
Richard Biener writes: > On Fri, Aug 18, 2017 at 12:30 PM, Richard Biener > wrote: >> On Thu, Aug 17, 2017 at 2:24 PM, Bin.Cheng wrote: >>> On Thu, Aug 17, 2017 at 12:35 PM, Richard Sandiford >>>

Re: PR81635: Use chrecs to help find related data refs

2017-08-18 Thread Richard Biener
On Fri, Aug 18, 2017 at 12:30 PM, Richard Biener wrote: > On Thu, Aug 17, 2017 at 2:24 PM, Bin.Cheng wrote: >> On Thu, Aug 17, 2017 at 12:35 PM, Richard Sandiford >> wrote: >>> "Bin.Cheng"

Re: PR81635: Use chrecs to help find related data refs

2017-08-18 Thread Richard Biener
On Thu, Aug 17, 2017 at 2:24 PM, Bin.Cheng wrote: > On Thu, Aug 17, 2017 at 12:35 PM, Richard Sandiford > wrote: >> "Bin.Cheng" writes: >>> On Wed, Aug 16, 2017 at 6:50 PM, Richard Sandiford >>>

Re: PR81635: Use chrecs to help find related data refs

2017-08-17 Thread Bin.Cheng
On Thu, Aug 17, 2017 at 12:35 PM, Richard Sandiford wrote: > "Bin.Cheng" writes: >> On Wed, Aug 16, 2017 at 6:50 PM, Richard Sandiford >> wrote: >>> "Bin.Cheng" writes: On Wed, Aug

Re: PR81635: Use chrecs to help find related data refs

2017-08-17 Thread Richard Sandiford
"Bin.Cheng" writes: > On Wed, Aug 16, 2017 at 6:50 PM, Richard Sandiford > wrote: >> "Bin.Cheng" writes: >>> On Wed, Aug 16, 2017 at 5:00 PM, Richard Sandiford >>> wrote: "Bin.Cheng"

Re: PR81635: Use chrecs to help find related data refs

2017-08-17 Thread Bin.Cheng
On Wed, Aug 16, 2017 at 6:50 PM, Richard Sandiford wrote: > "Bin.Cheng" writes: >> On Wed, Aug 16, 2017 at 5:00 PM, Richard Sandiford >> wrote: >>> "Bin.Cheng" writes: On Wed, Aug 16,

Re: PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Richard Sandiford
"Bin.Cheng" writes: > On Wed, Aug 16, 2017 at 5:00 PM, Richard Sandiford > wrote: >> "Bin.Cheng" writes: >>> On Wed, Aug 16, 2017 at 2:38 PM, Richard Sandiford >>> wrote: The first

Re: PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Bin.Cheng
On Wed, Aug 16, 2017 at 5:00 PM, Richard Sandiford wrote: > "Bin.Cheng" writes: >> On Wed, Aug 16, 2017 at 2:38 PM, Richard Sandiford >> wrote: >>> The first loop in the testcase regressed after my recent changes

Re: PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Richard Sandiford
"Bin.Cheng" writes: > On Wed, Aug 16, 2017 at 2:38 PM, Richard Sandiford > wrote: >> The first loop in the testcase regressed after my recent changes to >> dr_analyze_innermost. Previously we would treat "i" as an iv even >> for bb analysis

Re: PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Bin.Cheng
On Wed, Aug 16, 2017 at 2:38 PM, Richard Sandiford wrote: > The first loop in the testcase regressed after my recent changes to > dr_analyze_innermost. Previously we would treat "i" as an iv even > for bb analysis and end up with: > >DR_BASE_ADDRESS: p or q >

Re: PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Richard Sandiford
Richard Sandiford writes: > The first loop in the testcase regressed after my recent changes to > dr_analyze_innermost. Previously we would treat "i" as an iv even > for bb analysis and end up with: > >DR_BASE_ADDRESS: p or q >DR_OFFSET: 0 >DR_INIT: 0 or

PR81635: Use chrecs to help find related data refs

2017-08-16 Thread Richard Sandiford
The first loop in the testcase regressed after my recent changes to dr_analyze_innermost. Previously we would treat "i" as an iv even for bb analysis and end up with: DR_BASE_ADDRESS: p or q DR_OFFSET: 0 DR_INIT: 0 or 4 DR_STEP: 16 We now always keep the step as 0 instead, so for an