On Wed, Oct 31, 2018 at 7:31 AM bin.cheng <bin.ch...@linux.alibaba.com> wrote: > > Hi, > This is a simple patch setting profile_count with according precision > category in > AutoFDO, as well as setting profile_count for exit basic block. > > Bootstrap and test on x86_64 in patch set. Is it OK?
OK. It might look cleaner (and cheaper) to add a profile-quality argument to from_gcov_type instead of calling .afdo() on the result. Your call. Richard. > Thanks, > bin > 2018-10-31 Bin Cheng <bin.ch...@linux.alibaba.com> > > * auto-profile.c (afdo_calculate_branch_prob): Convert profile_count > to afdo precision quality. > (afdo_callsite_hot_enough_for_early_inline): Ditto. > (afdo_annotate_cfg): Set profile_count for exit basic block.