[PATCH] D98213: [InlineCost] Enable the cost benefit analysis on FDO
David Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 8 15:12:41 PST 2021
davidxl added inline comments.
================
Comment at: llvm/lib/Analysis/InlineCost.cpp:683
+ // Honor the explicit request from the user.
+ if (!InlineEnableCostBenefitAnalysis)
+ return false;
----------------
Why not change the default of this option to true, then there is no need to check explicit occurences?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98213/new/
https://reviews.llvm.org/D98213
More information about the llvm-commits
mailing list