[PATCH] D79042: [llvm] Add interface to drive inlining decision using ML model

[David Li via Phabricator via llvm-commits]

davidxl accepted this revision.
davidxl added a comment.
This revision is now accepted and ready to land.

LGTM (assuming all pending comments are addressed).



================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:851
 
-      auto OIC = shouldInline(*CB, GetInlineCost, ORE);
+      auto PendingRecord = Advisor.getAdvice(*CB, FAM);
       // Check whether we want to inline this callsite.
----------------
PendingRecord -- IA or Advise 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79042/new/

https://reviews.llvm.org/D79042