On Fri, May 4, 2018 at 5:21 PM, Bin Cheng <bin.ch...@arm.com> wrote: > Hi, > This is the second patch computing available/clobber registers for register > classes. > It's the same as the original patch posted > @https://gcc.gnu.org/ml/gcc-patches/2017-05/msg01022.html > > Bootstrap and test on x86_64 and AArch64 ongoing. Any comments? Same as previous one. The original version of this patch was approved by Jeff @https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01810.html Jeff, any new comments or the approval is valid?
Thanks, bin > > Thanks, > bin > 2017-04-27 Bin Cheng <bin.ch...@arm.com> > > * cfgloop.h (struct target_cfgloop): Change x_target_avail_regs and > x_target_clobbered_regs into array fields. > (init_avail_clobber_regs): New declaration. > * cfgloopanal.c (memmodel.h, ira.h): Include header files. > (init_set_costs): Remove computation for old x_target_avail_regs and > x_target_clobbered_regs fields. > (init_avail_clobber_regs): New function. > (estimate_reg_pressure_cost): Update the uses. > * toplev.c (cfgloop.h): Update comment why the header file is needed. > (backend_init_target): Call init_avail_clobber_regs. > * tree-predcom.c (memmodel.h, ira.h): Include header files. > (MAX_DISTANCE): Update the use. > * tree-ssa-loop-ivopts.c (AVAILABLE_REGS, CLOBBERED_REGS): New marco. > (ivopts_estimate_reg_pressure, determine_set_costs): Update the uses.
