[PATCH] D79138: [Inlining] Teach shouldBeDeferred to take the total cost into account

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 30 14:32:59 PDT 2020


davidxl added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:346
   bool InliningPreventsSomeOuterInline = false;
+  unsigned SecondaryUsers = 0;
   for (User *U : Caller->users()) {
----------------
NumCallerUsers may be more explicit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79138





More information about the llvm-commits mailing list