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

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin added a comment.

In D79613#2027187 <https://reviews.llvm.org/D79613#2027187>, @davidxl wrote:

> 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


Done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79613