[PATCH] D76248: Fix a bug in the inliner that causes subsequent double inlining

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:721
+            // callsite will be reconsidered here.
+            for (auto &II : CallSites)
+              assert(II.first != CallSite(Ptr));
----------------
This has quadratic behavior and can cause excessive compile time increase for some cases with assertion on. Use a DenseSet to avoid that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76248