[PATCH] D70147: Repeat DevirtSCCRepeatedPass if any inlining happens.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 15:29:12 PST 2019


davidxl added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:1089
       DidInline = true;
+      UR.DidInline = true;
       InlinedCallees.insert(&Callee);
----------------
Alternatively, if compile time is a concern,  the inline analysis 'shouldInline' can return a flag indicating if the callee has any indirect calls. If it is true and if the site is inlined,  set this flag.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70147





More information about the llvm-commits mailing list