Skip to content

Commit

Permalink
[ModuleInliner] Remove a write-only variable (NFC)
Browse files Browse the repository at this point in the history
InlinedCallees is a remnant from the CGSCC inliner.  We don't use it
in the module inliner.
  • Loading branch information
kazutakahirata committed Sep 16, 2022
1 parent 95c3e9c commit 513717d
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions llvm/lib/Transforms/IPO/ModuleInliner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -189,11 +189,6 @@ PreservedAnalyses ModuleInlinerPass::run(Module &M,
// index into the InlineHistory vector.
SmallVector<std::pair<Function *, int>, 16> InlineHistory;

// Track a set vector of inlined callees so that we can augment the caller
// with all of their edges in the call graph before pruning out the ones that
// got simplified away.
SmallSetVector<Function *, 4> InlinedCallees;

// Track the dead functions to delete once finished with inlining calls. We
// defer deleting these to make it easier to handle the call graph updates.
SmallVector<Function *, 4> DeadFunctions;
Expand Down Expand Up @@ -251,7 +246,6 @@ PreservedAnalyses ModuleInlinerPass::run(Module &M,
}

DidInline = true;
InlinedCallees.insert(&Callee);
++NumInlined;

LLVM_DEBUG(dbgs() << " Size after inlining: " << F.getInstructionCount()
Expand Down Expand Up @@ -314,8 +308,6 @@ PreservedAnalyses ModuleInlinerPass::run(Module &M,
if (!DidInline)
continue;
Changed = true;

InlinedCallees.clear();
}

// Now that we've finished inlining all of the calls across this module,
Expand Down

0 comments on commit 513717d

Please sign in to comment.