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

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 1 17:14:25 PDT 2020


davidxl added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:419
+
+class CostBasedInliningDecision final {
+public:
----------------
Why final? There is no inheritance and virtual function, or do you plan to create a hierarchy later?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79215





More information about the llvm-commits mailing list