[PATCH] D99302: [InlineCost] Make cost-benefit decision explicit
David Li via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 24 14:59:40 PDT 2021
davidxl added inline comments.
================
Comment at: llvm/lib/Analysis/InlineCost.cpp:2618
// Check if there was a reason to force inlining or no inlining.
+ if (ShouldInline.isSuccess() && CA.wasDecidedByCostBenefit())
+ return InlineCost::getAlways("benefit over cost");
----------------
Should this be folded into line 2622?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99302/new/
https://reviews.llvm.org/D99302
More information about the llvm-commits
mailing list