On Sun, Dec 2, 2012, at 02:42 AM, Lawrence Crowl wrote: > Change graphite-related hash tables from htab_t to hash_table: > > graphite-clast-to-gimple.c ivs_params::newivs_index > graphite-clast-to-gimple.c ivs_params::params_index > graphite-clast-to-gimple.c print_generated_program::params_index > graphite-clast-to-gimple.c gloog::newivs_index > graphite-clast-to-gimple.c gloog::params_index > graphite.c graphite_transform_loops::bb_pbb_mapping > sese.c copy_bb_and_scalar_dependences::rename_map > > Move hash table declarations to a new graphite-htab.h, because they > are used in few places. > > Remove unused: > > htab_t scop::original_pddrs > SCOP_ORIGINAL_PDDRS > > Remove unused: > > insert_loop_close_phis > insert_guard_phis > debug_ivtype_map > ivtype_map_elt_info > new_ivtype_map_elt > > > Tested on x86-64. > > Okay for branch?
Hi Lawrance, from the point of graphite, the update looks very good. Thanks lot for getting the new hash table implementation in. Also, thanks for the dead code removal. Assuming the test suite passes and this is just a no-behavior change intended commit, I am fine with this. No idea, who needs to sign off branch commits, though. Cheers Tobi
