[llvm] [NFC][AlwaysInliner] Reduce AlwaysInliner memory consumption. (PR #96958)

Daniil Fukalov via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 1 13:58:25 PDT 2024


================
@@ -37,86 +37,74 @@ bool AlwaysInlineImpl(
     function_ref<BlockFrequencyInfo &(Function &)> GetBFI) {
   SmallSetVector<CallBase *, 16> Calls;
   bool Changed = false;
-  SmallVector<Function *, 16> InlinedFunctions;
-  for (Function &F : M) {
-    // When callee coroutine function is inlined into caller coroutine function
-    // before coro-split pass,
-    // coro-early pass can not handle this quiet well.
-    // So we won't inline the coroutine function if it have not been unsplited
+  SmallVector<Function *, 16> InlinedComdatFunctions;
+
+  for (Function &F : make_early_inc_range(M)) {
----------------
dfukalov wrote:

I've checked a number of places `make_early_inc_range()` used - found no comments about its functionality. It seems its usage instead of default iterator is quite obvious, what do you think?

Updated the comment below about `InlinedComdatFunctions` usage.

https://github.com/llvm/llvm-project/pull/96958


More information about the llvm-commits mailing list