Re: [PATCH RFC][PR98736]Compute and use programing order preserved RPO in loop distribution

2021-03-23 Thread Richard Biener via Gcc-patches
On Tue, Mar 23, 2021 at 8:55 AM bin.cheng wrote: > > > > In the patch, I just duplicated and created new function > > > loop_first_rev_post_order_compute_fn. > > > I am not sure if I should change the original function > > > pre_and_rev_post_order_compute_fn > > > (maybe not at this stage)? I am

Re: [PATCH RFC][PR98736]Compute and use programing order preserved RPO in loop distribution

2021-03-23 Thread bin.cheng via Gcc-patches
> > In the patch, I just duplicated and created new function > > loop_first_rev_post_order_compute_fn. > > I am not sure if I should change the original function > > pre_and_rev_post_order_compute_fn > > (maybe not at this stage)? I am neither sure about the name, though haven't > > got a better

Re: [PATCH RFC][PR98736]Compute and use programing order preserved RPO in loop distribution

2021-03-22 Thread Richard Biener via Gcc-patches
On Mon, Mar 22, 2021 at 12:00 PM bin.cheng via Gcc-patches wrote: > > Hi, > > This is the patch for PR98736. The root cause is like: > > Use programing order preserved RPO in loop distribution. > > Tree loop distribution uses RPO to build reduced dependence graph, > it's important tha

[PATCH RFC][PR98736]Compute and use programing order preserved RPO in loop distribution

2021-03-22 Thread bin.cheng via Gcc-patches
Hi, This is the patch for PR98736. The root cause is like: Use programing order preserved RPO in loop distribution. Tree loop distribution uses RPO to build reduced dependence graph, it's important that RPO preserves the original programing order and usually it does. Howeve