On Tue, Jun 13, 2017 at 12:06 PM, Richard Biener <richard.guent...@gmail.com> wrote: > On Mon, Jun 12, 2017 at 7:02 PM, Bin Cheng <bin.ch...@arm.com> wrote: >> Hi, >> This simple patch computes and preserves loop nest vector for whole >> distribution >> life time. The loop nest will be used multiple times in on-demand data >> dependence >> computation. >> >> Bootstrap and test on x86_64 and AArch64. Is it OK? > > Don't like it too much but I guess we can see if refactoring it back > to pass down > loop_nest can work. The global data is partly to avoid patch conflicts when separating patches, otherwise several parameters are needed for quite number of functions. We can introduce a global distribution data and only pass it to various functions.
Thanks, bin > > Ok. > > Thanks, > Richard. > >> Thanks, >> bin >> 2017-06-07 Bin Cheng <bin.ch...@arm.com> >> >> * tree-loop-distribution.c (loop_nest): New global var. >> (build_rdg): Use loop directly, rather than loop nest. >> (pg_add_dependence_edges): Remove loop nest parameter. Use global >> variable directly. >> (distribute_loop): Compute global variable loop nest. Update use.
