> I think the approach is reasonable though it might still have > interesting effects on > optimization for very small growth. So for further experimenting it > would be nice
Test it on SPEC CPU with FDO enabled? > to have a separate PARAM_EARLY_FDO_INLINING_INSNS or maybe simply > adjust the PARAM_EARLY_INLINING_INSNS default accordingly when FDO is > enabled? Setting PARAM_EARLY_INLINING_INSNS to 0 when FDO is enabled should be equivalent to my patch. Regards, Yuan, Pengfei > I'll let Honza also double-check the condition detecting FDO (it looks > like we should > have some abstraction for that). > > Thanks, > Richard.
