[PATCH] D138602: [WIP] Alwaysinliner time explosion with new pass manager

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

It seems that the explosion can happen when the always-inline callee has multiple call edges within the nontrivial SCC. Should the check be more refined?



================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:906
+      //   void D() { B(); B(); B(); B(); }
+      // Note that we are only checking mutual-recursion here. Self-recurision
+      // is handled by inline cost analyzer.
----------------
typo : recursion


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138602