> Thanks for the suggestion. This is much cleaner than to use binary parameter.
> 
> Just want to make sure I understand it correctly about the orginal hitrate:
> you want to retire the hitrate in PRED_BUILTIN_EXPECT and always use
> the one specified in the biniltin-expect-probability parameter.

Yes.
> 
> Should I use 90% as the default? It's hard to fit current value 0.9996
> in percent form.

Yes, 90% seems fine.  The original value was set quite arbitrarily and no real
performance study was made as far as I know except yours. I think users that
are sure they use expect to gueard completely cold edges may just use 100%
instead of 0.9996, so I would not worry much about the precision.

Honza
> 
> -Rong
> >
> > OK with that change.
> >
> > Honza

Reply via email to