[PATCH] D79215: [llvm][NFC] Inliner: simplify inlining decision logic

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 3 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:419
+
+class CostBasedInliningDecision final {
+public:
----------------
davidxl wrote:
> Why final? There is no inheritance and virtual function, or do you plan to create a hierarchy later?
Just to reinforce this is meant to be a simple wrapper. Of course, we can change any of this later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79215