[PATCH] D94982: [NPM][Inliner] Factor ImportedFunctionStats in the InlineAdvisor
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 19 16:55:06 PST 2021
aeubanks added inline comments.
================
Comment at: llvm/include/llvm/Analysis/InlineAdvisor.h:170
+ Module &M;
FunctionAnalysisManager &FAM;
----------------
mtrofin wrote:
> aeubanks wrote:
> > is this used anywhere except the constructor?
> MLInlineAdvisor, which had this member as a protected, and now is using this instead.
I tried removing the variable from MLInlineAdvisor and it still compiled.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D94982/new/
https://reviews.llvm.org/D94982
More information about the llvm-commits
mailing list