[PATCH] D79613: [llvm][NFC] Move inlining decision-related APIs in InliningAdvisor.

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

At first I thought may be put inliner advisor and friends into InlineCost.h[cpp], but on second thought, the proposed restructuring is better: we have  inline cost analysis in InlineCost.xx, actual inliner transformation in Inliner.xx, and the inline heuristics or ML sits in InlineAdvisor.xx.

Just need to make the naming more consistent : InliningAdvisor --> InlineAdvisor


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79613