Hi, On 11/15, Richard Biener wrote: > On Thu, Nov 14, 2019 at 10:35 PM Giuliano Belinassi > <giuliano.belina...@usp.br> wrote: > > > > Previously, the suggested patch removed all tree-loop-distributions.c global > > variables moving them into a struct and passing it aroung across the > > functions. > > This patch address this problem by using C++ classes instead, avoiding > > passing > > the struct as argument since it will be accessible from this pointer. > > This patch is OK if it passes bootstrap and regtest.
Took me while to check if it did not trigger any fail. Anyway, I just commited it to trunk. > > Thanks, > Richard. > > > gcc/ChangeLog > > 2019-11-14 Giuliano Belinassi <giuliano.belina...@usp.br> > > > > * cfgloop.c (get_loop_body_in_custom_order): New. > > * cfgloop.h (get_loop_body_in_custom_order): New prototype. > > * tree-loop-distribution.c (class loop_distribution): New. > > (bb_top_order_cmp): Remove. > > (bb_top_order_cmp_r): New. > > (create_rdg_vertices): Move into class loop_distribution. > > (stmts_from_loop): Same as above. > > (update_for_merge): Same as above. > > (partition_merge_into): Same as above. > > (get_data_dependence): Same as above. > > (data_dep_in_cycle_p): Same as above. > > (update_type_for_merge): Same as above. > > (build_rdg_partition_for-vertex): Same as above. > > (classify_builtin_ldst): Same as above. > > (classify_partition): Same as above. > > (share_memory_accesses): Same as above. > > (rdg_build_partitions): Same as above. > > (pg_add_dependence_edges): Same as above. > > (build_partition_graph): Same as above. > > (merge_dep_scc_partitions): Same as above. > > (break_alias_scc_partitions): Same as above. > > (finalize_partitions): Same as above. > > (distribute_loop): Same as above. > > (bb_top_order_init): New method > > (bb_top_order_destroy): New method. > > (get_bb_top_order_index_size): New method. > > (get_bb_top_order_index_index): New method. > > (get_bb_top_order_index_index): New method. > > (loop_distribution::execute): New method. > > (pass_loop_distribution::execute): Instantiate loop_distribution. Thank you, Giuliano.
