[PATCH] D71733: [NFC][InlineCost] Factor cost modeling out of CallAnalyzer traversal.
Easwaran Raman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 8 16:05:46 PST 2020
eraman added a comment.
LGTM
================
Comment at: llvm/lib/Analysis/InlineCost.cpp:450
+ InlineConstants::IndirectCallThreshold;
+ InlineCostCallAnalyzer CA(TTI, GetAssumptionCache, GetBFI, PSI, ORE, *F,
+ Call, IndirectCallParams, false);
----------------
Please add a FIXME here saying we should be using the same derived class instance
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71733/new/
https://reviews.llvm.org/D71733
More information about the llvm-commits
mailing list