[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