blob: 29e53e78d73b4990318206f83e29ce6bbc8bbe67 [file] [log] [blame]
Chris Lattner7d30a6c2004-06-20 04:11:48 +00001//===- Inliner.cpp - Code common to all inliners --------------------------===//
Misha Brukmanb1c93172005-04-21 23:48:37 +00002//
John Criswell482202a2003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
Chris Lattnerf3ebc3f2007-12-29 20:36:04 +00005// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
Misha Brukmanb1c93172005-04-21 23:48:37 +00007//
John Criswell482202a2003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattnerd075cc22003-08-31 19:10:30 +00009//
Chris Lattner6754b822004-05-23 21:22:17 +000010// This file implements the mechanics required to implement inlining without
11// missing any calls and updating the call graph. The decisions of which calls
12// are profitable to inline are implemented elsewhere.
Chris Lattnerd075cc22003-08-31 19:10:30 +000013//
14//===----------------------------------------------------------------------===//
15
Chandler Carruthed0881b2012-12-03 16:50:05 +000016#include "llvm/ADT/SmallPtrSet.h"
17#include "llvm/ADT/Statistic.h"
Hal Finkel0c083022014-09-01 09:01:39 +000018#include "llvm/Analysis/AliasAnalysis.h"
Chandler Carruth66b31302015-01-04 12:03:27 +000019#include "llvm/Analysis/AssumptionCache.h"
Chandler Carruth7b560d42015-09-09 17:55:00 +000020#include "llvm/Analysis/BasicAliasAnalysis.h"
Chris Lattnerd075cc22003-08-31 19:10:30 +000021#include "llvm/Analysis/CallGraph.h"
Dan Gohman4552e3c2009-10-13 18:30:07 +000022#include "llvm/Analysis/InlineCost.h"
Easwaran Raman71069cf2016-06-09 22:23:21 +000023#include "llvm/Analysis/ProfileSummaryInfo.h"
Benjamin Kramer799003b2015-03-23 19:32:43 +000024#include "llvm/Analysis/TargetLibraryInfo.h"
Chandler Carruth219b89b2014-03-04 11:01:28 +000025#include "llvm/IR/CallSite.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000026#include "llvm/IR/DataLayout.h"
Diego Novilloa9298b22014-04-08 16:42:34 +000027#include "llvm/IR/DiagnosticInfo.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000028#include "llvm/IR/Instructions.h"
29#include "llvm/IR/IntrinsicInst.h"
30#include "llvm/IR/Module.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000031#include "llvm/Support/Debug.h"
Daniel Dunbar0dd5e1e2009-07-25 00:23:56 +000032#include "llvm/Support/raw_ostream.h"
Mehdi Aminib550cb12016-04-18 09:17:29 +000033#include "llvm/Transforms/IPO/InlinerPass.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000034#include "llvm/Transforms/Utils/Cloning.h"
35#include "llvm/Transforms/Utils/Local.h"
Chris Lattnera82f1312003-11-21 21:45:31 +000036using namespace llvm;
Brian Gaeke960707c2003-11-11 22:41:34 +000037
Chandler Carruth964daaa2014-04-22 02:55:47 +000038#define DEBUG_TYPE "inline"
39
Chris Lattner1631bcb2006-12-19 22:09:18 +000040STATISTIC(NumInlined, "Number of functions inlined");
Chris Lattnerb49a6222010-05-01 17:19:38 +000041STATISTIC(NumCallsDeleted, "Number of call sites deleted, not inlined");
Chris Lattner1631bcb2006-12-19 22:09:18 +000042STATISTIC(NumDeleted, "Number of functions deleted because all callers found");
Chris Lattnerd3374e82009-08-27 06:29:33 +000043STATISTIC(NumMergedAllocas, "Number of allocas merged together");
Chris Lattner1631bcb2006-12-19 22:09:18 +000044
Benjamin Kramerbde91762012-06-02 10:20:22 +000045// This weirdly named statistic tracks the number of times that, when attempting
Chandler Carruth7ae90d42012-04-11 10:15:10 +000046// to inline a function A into B, we analyze the callers of B in order to see
47// if those would be more profitable and blocked inline steps.
48STATISTIC(NumCallerCallersAnalyzed, "Number of caller-callers analyzed");
49
Piotr Padlewski84abc742016-07-29 00:27:16 +000050namespace {
51enum class InlinerFunctionImportStatsOpts {
52 No = 0,
53 Basic = 1,
54 Verbose = 2,
55};
56
57cl::opt<InlinerFunctionImportStatsOpts> InlinerFunctionImportStats(
58 "inliner-function-import-stats",
59 cl::init(InlinerFunctionImportStatsOpts::No),
60 cl::values(clEnumValN(InlinerFunctionImportStatsOpts::Basic, "basic",
61 "basic statistics"),
62 clEnumValN(InlinerFunctionImportStatsOpts::Verbose, "verbose",
63 "printing of statistics for each inlined function"),
64 clEnumValEnd),
65 cl::Hidden, cl::desc("Enable inliner stats for imported functions"));
66} // namespace
67
Easwaran Ramanb1bd3982016-03-08 00:36:35 +000068Inliner::Inliner(char &ID) : CallGraphSCCPass(ID), InsertLifetime(true) {}
Chris Lattnerd075cc22003-08-31 19:10:30 +000069
Easwaran Ramanf4bb2f02016-01-14 23:16:29 +000070Inliner::Inliner(char &ID, bool InsertLifetime)
Easwaran Ramanb1bd3982016-03-08 00:36:35 +000071 : CallGraphSCCPass(ID), InsertLifetime(InsertLifetime) {}
Chris Lattner22ad7ab2008-01-12 06:49:13 +000072
Sanjay Patelf1b0db12015-03-10 16:42:24 +000073/// For this class, we declare that we require and preserve the call graph.
74/// If the derived class implements this method, it should
Chris Lattnerf94bed32007-01-30 23:28:39 +000075/// always explicitly call the implementation here.
Chandler Carruthe40e60e2012-12-27 11:17:15 +000076void Inliner::getAnalysisUsage(AnalysisUsage &AU) const {
Chandler Carruth66b31302015-01-04 12:03:27 +000077 AU.addRequired<AssumptionCacheTracker>();
Easwaran Raman71069cf2016-06-09 22:23:21 +000078 AU.addRequired<ProfileSummaryInfoWrapperPass>();
Chandler Carruth7b560d42015-09-09 17:55:00 +000079 AU.addRequired<TargetLibraryInfoWrapperPass>();
Chandler Carruth12884f72016-03-02 15:56:53 +000080 getAAResultsAnalysisUsage(AU);
Chandler Carruthe40e60e2012-12-27 11:17:15 +000081 CallGraphSCCPass::getAnalysisUsage(AU);
Chris Lattnerf94bed32007-01-30 23:28:39 +000082}
83
Chris Lattnerd3374e82009-08-27 06:29:33 +000084
Chris Lattner229907c2011-07-18 04:54:35 +000085typedef DenseMap<ArrayType*, std::vector<AllocaInst*> >
Chris Lattnerd3374e82009-08-27 06:29:33 +000086InlinedArrayAllocasTy;
87
Sanjay Patelf1b0db12015-03-10 16:42:24 +000088/// If it is possible to inline the specified call site,
Chris Lattnerd3374e82009-08-27 06:29:33 +000089/// do so and update the CallGraph for this operation.
90///
91/// This function also does some basic book-keeping to update the IR. The
Chris Lattner0e890182009-08-28 04:48:54 +000092/// InlinedArrayAllocas map keeps track of any allocas that are already
Sanjay Patelf1b0db12015-03-10 16:42:24 +000093/// available from other functions inlined into the caller. If we are able to
Chris Lattner0e890182009-08-28 04:48:54 +000094/// inline this call site we attempt to reuse already available allocas or add
95/// any new allocas to the set if not possible.
Piotr Padlewski84abc742016-07-29 00:27:16 +000096static bool InlineCallIfPossible(
97 CallSite CS, InlineFunctionInfo &IFI,
98 InlinedArrayAllocasTy &InlinedArrayAllocas, int InlineHistory,
99 bool InsertLifetime, std::function<AAResults &(Function &)> &AARGetter,
100 ImportedFunctionsInliningStatistics &ImportedFunctionsStats) {
Chris Lattner6754b822004-05-23 21:22:17 +0000101 Function *Callee = CS.getCalledFunction();
Bill Wendlingf5260d22008-11-21 00:09:21 +0000102 Function *Caller = CS.getCaller();
103
Sean Silvaab6a6832016-07-23 04:22:50 +0000104 AAResults &AAR = AARGetter(*Callee);
Chandler Carruth7b560d42015-09-09 17:55:00 +0000105
Chris Lattnerd3374e82009-08-27 06:29:33 +0000106 // Try to inline the function. Get the list of static allocas that were
107 // inlined.
Chandler Carruth7b560d42015-09-09 17:55:00 +0000108 if (!InlineFunction(CS, IFI, &AAR, InsertLifetime))
Chris Lattnerb9d0a962009-08-27 04:32:07 +0000109 return false;
Misha Brukmanb1c93172005-04-21 23:48:37 +0000110
Piotr Padlewski84abc742016-07-29 00:27:16 +0000111 if (InlinerFunctionImportStats != InlinerFunctionImportStatsOpts::No)
112 ImportedFunctionsStats.recordInline(*Caller, *Callee);
113
Akira Hatanaka1cb242e2015-12-22 23:57:37 +0000114 AttributeFuncs::mergeAttributesForInlining(*Caller, *Callee);
Bill Wendling26c6a3e2008-11-21 00:06:32 +0000115
Chris Lattnerd3374e82009-08-27 06:29:33 +0000116 // Look at all of the allocas that we inlined through this call site. If we
117 // have already inlined other allocas through other calls into this function,
118 // then we know that they have disjoint lifetimes and that we can merge them.
119 //
120 // There are many heuristics possible for merging these allocas, and the
121 // different options have different tradeoffs. One thing that we *really*
122 // don't want to hurt is SRoA: once inlining happens, often allocas are no
123 // longer address taken and so they can be promoted.
124 //
125 // Our "solution" for that is to only merge allocas whose outermost type is an
126 // array type. These are usually not promoted because someone is using a
127 // variable index into them. These are also often the most important ones to
128 // merge.
129 //
130 // A better solution would be to have real memory lifetime markers in the IR
131 // and not have the inliner do any merging of allocas at all. This would
132 // allow the backend to do proper stack slot coloring of all allocas that
133 // *actually make it to the backend*, which is really what we want.
134 //
135 // Because we don't have this information, we do this simple and useful hack.
136 //
137 SmallPtrSet<AllocaInst*, 16> UsedAllocas;
138
Chris Lattnerfb212de2010-12-06 07:52:42 +0000139 // When processing our SCC, check to see if CS was inlined from some other
140 // call site. For example, if we're processing "A" in this code:
141 // A() { B() }
142 // B() { x = alloca ... C() }
143 // C() { y = alloca ... }
144 // Assume that C was not inlined into B initially, and so we're processing A
145 // and decide to inline B into A. Doing this makes an alloca available for
146 // reuse and makes a callsite (C) available for inlining. When we process
147 // the C call site we don't want to do any alloca merging between X and Y
148 // because their scopes are not disjoint. We could make this smarter by
149 // keeping track of the inline history for each alloca in the
150 // InlinedArrayAllocas but this isn't likely to be a significant win.
151 if (InlineHistory != -1) // Only do merging for top-level call sites in SCC.
152 return true;
153
Chris Lattnerd3374e82009-08-27 06:29:33 +0000154 // Loop over all the allocas we have so far and see if they can be merged with
155 // a previously inlined alloca. If not, remember that we had it.
Chris Lattner4ba01ec2010-04-22 23:07:58 +0000156 for (unsigned AllocaNo = 0, e = IFI.StaticAllocas.size();
Chris Lattnerd3374e82009-08-27 06:29:33 +0000157 AllocaNo != e; ++AllocaNo) {
Chris Lattner4ba01ec2010-04-22 23:07:58 +0000158 AllocaInst *AI = IFI.StaticAllocas[AllocaNo];
Chris Lattnerd3374e82009-08-27 06:29:33 +0000159
160 // Don't bother trying to merge array allocations (they will usually be
161 // canonicalized to be an allocation *of* an array), or allocations whose
162 // type is not itself an array (because we're afraid of pessimizing SRoA).
Chris Lattner229907c2011-07-18 04:54:35 +0000163 ArrayType *ATy = dyn_cast<ArrayType>(AI->getAllocatedType());
Craig Topperf40110f2014-04-25 05:29:35 +0000164 if (!ATy || AI->isArrayAllocation())
Chris Lattnerd3374e82009-08-27 06:29:33 +0000165 continue;
166
167 // Get the list of all available allocas for this array type.
168 std::vector<AllocaInst*> &AllocasForType = InlinedArrayAllocas[ATy];
169
170 // Loop over the allocas in AllocasForType to see if we can reuse one. Note
171 // that we have to be careful not to reuse the same "available" alloca for
172 // multiple different allocas that we just inlined, we use the 'UsedAllocas'
173 // set to keep track of which "available" allocas are being used by this
174 // function. Also, AllocasForType can be empty of course!
175 bool MergedAwayAlloca = false;
Yaron Keren62064d62015-06-25 19:28:24 +0000176 for (AllocaInst *AvailableAlloca : AllocasForType) {
Hal Finkel9caa8f72013-07-16 17:10:55 +0000177
178 unsigned Align1 = AI->getAlignment(),
179 Align2 = AvailableAlloca->getAlignment();
Chris Lattnerd3374e82009-08-27 06:29:33 +0000180
181 // The available alloca has to be in the right function, not in some other
182 // function in this SCC.
183 if (AvailableAlloca->getParent() != AI->getParent())
184 continue;
185
186 // If the inlined function already uses this alloca then we can't reuse
187 // it.
David Blaikie70573dc2014-11-19 07:49:26 +0000188 if (!UsedAllocas.insert(AvailableAlloca).second)
Chris Lattnerd3374e82009-08-27 06:29:33 +0000189 continue;
190
191 // Otherwise, we *can* reuse it, RAUW AI into AvailableAlloca and declare
192 // success!
Chris Lattner5b6a8652010-12-06 07:38:40 +0000193 DEBUG(dbgs() << " ***MERGED ALLOCA: " << *AI << "\n\t\tINTO: "
194 << *AvailableAlloca << '\n');
Chris Lattnerd3374e82009-08-27 06:29:33 +0000195
Evgeniy Stepanovd8b86f72015-09-29 00:30:19 +0000196 // Move affected dbg.declare calls immediately after the new alloca to
197 // avoid the situation when a dbg.declare preceeds its alloca.
198 if (auto *L = LocalAsMetadata::getIfExists(AI))
199 if (auto *MDV = MetadataAsValue::getIfExists(AI->getContext(), L))
200 for (User *U : MDV->users())
201 if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(U))
202 DDI->moveBefore(AvailableAlloca->getNextNode());
203
Chris Lattnerd3374e82009-08-27 06:29:33 +0000204 AI->replaceAllUsesWith(AvailableAlloca);
Hal Finkel9caa8f72013-07-16 17:10:55 +0000205
Hal Finkelec7cd262013-07-17 14:32:41 +0000206 if (Align1 != Align2) {
207 if (!Align1 || !Align2) {
Mehdi Amini46a43552015-03-04 18:43:29 +0000208 const DataLayout &DL = Caller->getParent()->getDataLayout();
209 unsigned TypeAlign = DL.getABITypeAlignment(AI->getAllocatedType());
Hal Finkelec7cd262013-07-17 14:32:41 +0000210
211 Align1 = Align1 ? Align1 : TypeAlign;
212 Align2 = Align2 ? Align2 : TypeAlign;
213 }
214
215 if (Align1 > Align2)
216 AvailableAlloca->setAlignment(AI->getAlignment());
217 }
Hal Finkel9caa8f72013-07-16 17:10:55 +0000218
Chris Lattnerd3374e82009-08-27 06:29:33 +0000219 AI->eraseFromParent();
220 MergedAwayAlloca = true;
221 ++NumMergedAllocas;
Craig Topperf40110f2014-04-25 05:29:35 +0000222 IFI.StaticAllocas[AllocaNo] = nullptr;
Chris Lattnerd3374e82009-08-27 06:29:33 +0000223 break;
224 }
Misha Brukmanb1c93172005-04-21 23:48:37 +0000225
Chris Lattnerd3374e82009-08-27 06:29:33 +0000226 // If we already nuked the alloca, we're done with it.
227 if (MergedAwayAlloca)
228 continue;
Chris Lattner5b6a8652010-12-06 07:38:40 +0000229
Chris Lattnerd3374e82009-08-27 06:29:33 +0000230 // If we were unable to merge away the alloca either because there are no
231 // allocas of the right type available or because we reused them all
232 // already, remember that this alloca came from an inlined function and mark
233 // it used so we don't reuse it for other allocas from this inline
234 // operation.
235 AllocasForType.push_back(AI);
236 UsedAllocas.insert(AI);
Chris Lattner6754b822004-05-23 21:22:17 +0000237 }
Chris Lattnerd3374e82009-08-27 06:29:33 +0000238
Chris Lattner6754b822004-05-23 21:22:17 +0000239 return true;
Chris Lattnerd075cc22003-08-31 19:10:30 +0000240}
Jakob Stoklund Olesen8a19d3c2010-01-20 17:51:28 +0000241
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000242static void emitAnalysis(CallSite CS, const Twine &Msg) {
243 Function *Caller = CS.getCaller();
244 LLVMContext &Ctx = Caller->getContext();
245 DebugLoc DLoc = CS.getInstruction()->getDebugLoc();
246 emitOptimizationRemarkAnalysis(Ctx, DEBUG_TYPE, *Caller, DLoc, Msg);
247}
248
Sean Silvaab6a6832016-07-23 04:22:50 +0000249/// Return true if inlining of CS can block the caller from being
250/// inlined which is proved to be more beneficial. \p IC is the
251/// estimated inline cost associated with callsite \p CS.
252/// \p TotalAltCost will be set to the estimated cost of inlining the caller
253/// if \p CS is suppressed for inlining.
254static bool
255shouldBeDeferred(Function *Caller, CallSite CS, InlineCost IC,
256 int &TotalSecondaryCost,
257 std::function<InlineCost(CallSite CS)> &GetInlineCost) {
Xinliang David Li4b2fdcc2016-04-29 22:59:36 +0000258
259 // For now we only handle local or inline functions.
260 if (!Caller->hasLocalLinkage() && !Caller->hasLinkOnceODRLinkage())
261 return false;
262 // Try to detect the case where the current inlining candidate caller (call
263 // it B) is a static or linkonce-ODR function and is an inlining candidate
264 // elsewhere, and the current candidate callee (call it C) is large enough
265 // that inlining it into B would make B too big to inline later. In these
266 // circumstances it may be best not to inline C into B, but to inline B into
267 // its callers.
268 //
269 // This only applies to static and linkonce-ODR functions because those are
270 // expected to be available for inlining in the translation units where they
271 // are used. Thus we will always have the opportunity to make local inlining
272 // decisions. Importantly the linkonce-ODR linkage covers inline functions
273 // and templates in C++.
274 //
275 // FIXME: All of this logic should be sunk into getInlineCost. It relies on
276 // the internal implementation of the inline cost metrics rather than
277 // treating them as truly abstract units etc.
278 TotalSecondaryCost = 0;
279 // The candidate cost to be imposed upon the current function.
280 int CandidateCost = IC.getCost() - (InlineConstants::CallPenalty + 1);
281 // This bool tracks what happens if we do NOT inline C into B.
282 bool callerWillBeRemoved = Caller->hasLocalLinkage();
283 // This bool tracks what happens if we DO inline C into B.
284 bool inliningPreventsSomeOuterInline = false;
285 for (User *U : Caller->users()) {
286 CallSite CS2(U);
287
288 // If this isn't a call to Caller (it could be some other sort
289 // of reference) skip it. Such references will prevent the caller
290 // from being removed.
291 if (!CS2 || CS2.getCalledFunction() != Caller) {
292 callerWillBeRemoved = false;
293 continue;
294 }
295
Sean Silvaab6a6832016-07-23 04:22:50 +0000296 InlineCost IC2 = GetInlineCost(CS2);
Xinliang David Li4b2fdcc2016-04-29 22:59:36 +0000297 ++NumCallerCallersAnalyzed;
298 if (!IC2) {
299 callerWillBeRemoved = false;
300 continue;
301 }
302 if (IC2.isAlways())
303 continue;
304
305 // See if inlining or original callsite would erase the cost delta of
306 // this callsite. We subtract off the penalty for the call instruction,
307 // which we would be deleting.
308 if (IC2.getCostDelta() <= CandidateCost) {
309 inliningPreventsSomeOuterInline = true;
310 TotalSecondaryCost += IC2.getCost();
311 }
312 }
313 // If all outer calls to Caller would get inlined, the cost for the last
314 // one is set very low by getInlineCost, in anticipation that Caller will
315 // be removed entirely. We did not account for this above unless there
316 // is only one caller of Caller.
317 if (callerWillBeRemoved && !Caller->use_empty())
318 TotalSecondaryCost += InlineConstants::LastCallToStaticBonus;
319
320 if (inliningPreventsSomeOuterInline && TotalSecondaryCost < IC.getCost())
321 return true;
322
323 return false;
324}
325
Sanjay Patelf1b0db12015-03-10 16:42:24 +0000326/// Return true if the inliner should attempt to inline at the given CallSite.
Sean Silvaab6a6832016-07-23 04:22:50 +0000327static bool shouldInline(CallSite CS,
328 std::function<InlineCost(CallSite CS)> GetInlineCost) {
329 InlineCost IC = GetInlineCost(CS);
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000330
Daniel Dunbar3933e662008-10-30 19:26:59 +0000331 if (IC.isAlways()) {
David Greene0122fc42010-01-05 01:27:51 +0000332 DEBUG(dbgs() << " Inlining: cost=always"
Bill Wendling2602bb42009-07-31 19:52:24 +0000333 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000334 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName()) +
335 " should always be inlined (cost=always)");
Daniel Dunbar3933e662008-10-30 19:26:59 +0000336 return true;
337 }
338
339 if (IC.isNever()) {
David Greene0122fc42010-01-05 01:27:51 +0000340 DEBUG(dbgs() << " NOT Inlining: cost=never"
Bill Wendling2602bb42009-07-31 19:52:24 +0000341 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000342 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName() +
343 " should never be inlined (cost=never)"));
Daniel Dunbar3933e662008-10-30 19:26:59 +0000344 return false;
345 }
346
Dale Johannesen30599242009-10-09 00:11:32 +0000347 Function *Caller = CS.getCaller();
Chandler Carruth0539c072012-03-31 12:42:41 +0000348 if (!IC) {
349 DEBUG(dbgs() << " NOT Inlining: cost=" << IC.getCost()
350 << ", thres=" << (IC.getCostDelta() + IC.getCost())
Bill Wendling2602bb42009-07-31 19:52:24 +0000351 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000352 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName() +
353 " too costly to inline (cost=") +
354 Twine(IC.getCost()) + ", threshold=" +
355 Twine(IC.getCostDelta() + IC.getCost()) + ")");
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000356 return false;
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000357 }
Dale Johannesen30599242009-10-09 00:11:32 +0000358
Xinliang David Li4b2fdcc2016-04-29 22:59:36 +0000359 int TotalSecondaryCost = 0;
Sean Silvaab6a6832016-07-23 04:22:50 +0000360 if (shouldBeDeferred(Caller, CS, IC, TotalSecondaryCost, GetInlineCost)) {
Xinliang David Li4b2fdcc2016-04-29 22:59:36 +0000361 DEBUG(dbgs() << " NOT Inlining: " << *CS.getInstruction()
362 << " Cost = " << IC.getCost()
363 << ", outer Cost = " << TotalSecondaryCost << '\n');
364 emitAnalysis(CS, Twine("Not inlining. Cost of inlining " +
365 CS.getCalledFunction()->getName() +
366 " increases the cost of inlining " +
367 CS.getCaller()->getName() + " in other contexts"));
368 return false;
Dale Johannesen30599242009-10-09 00:11:32 +0000369 }
370
Chandler Carruth0539c072012-03-31 12:42:41 +0000371 DEBUG(dbgs() << " Inlining: cost=" << IC.getCost()
372 << ", thres=" << (IC.getCostDelta() + IC.getCost())
Dale Johannesen30599242009-10-09 00:11:32 +0000373 << ", Call: " << *CS.getInstruction() << '\n');
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000374 emitAnalysis(
375 CS, CS.getCalledFunction()->getName() + Twine(" can be inlined into ") +
376 CS.getCaller()->getName() + " with cost=" + Twine(IC.getCost()) +
377 " (threshold=" + Twine(IC.getCostDelta() + IC.getCost()) + ")");
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000378 return true;
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000379}
Chris Lattnerd075cc22003-08-31 19:10:30 +0000380
Sanjay Patelf1b0db12015-03-10 16:42:24 +0000381/// Return true if the specified inline history ID
Chris Lattnere8262672010-05-01 01:05:10 +0000382/// indicates an inline history that includes the specified function.
383static bool InlineHistoryIncludes(Function *F, int InlineHistoryID,
384 const SmallVectorImpl<std::pair<Function*, int> > &InlineHistory) {
385 while (InlineHistoryID != -1) {
386 assert(unsigned(InlineHistoryID) < InlineHistory.size() &&
387 "Invalid inline history ID");
388 if (InlineHistory[InlineHistoryID].first == F)
389 return true;
390 InlineHistoryID = InlineHistory[InlineHistoryID].second;
391 }
392 return false;
393}
394
Piotr Padlewski84abc742016-07-29 00:27:16 +0000395bool Inliner::doInitialization(CallGraph &CG) {
396 if (InlinerFunctionImportStats != InlinerFunctionImportStatsOpts::No)
397 ImportedFunctionsStats.setModuleInfo(CG.getModule());
398 return false; // No changes to CallGraph.
399}
400
Chris Lattner4422d312010-04-16 22:42:17 +0000401bool Inliner::runOnSCC(CallGraphSCC &SCC) {
Andrew Kayloraa641a52016-04-22 22:06:11 +0000402 if (skipSCC(SCC))
403 return false;
Andrew Kaylor9c81d0f2016-05-23 21:57:54 +0000404 return inlineCalls(SCC);
405}
Andrew Kayloraa641a52016-04-22 22:06:11 +0000406
Sean Silvaab6a6832016-07-23 04:22:50 +0000407static bool
408inlineCallsImpl(CallGraphSCC &SCC, CallGraph &CG,
409 std::function<AssumptionCache &(Function &)> GetAssumptionCache,
410 ProfileSummaryInfo *PSI, TargetLibraryInfo &TLI,
411 bool InsertLifetime,
412 std::function<InlineCost(CallSite CS)> GetInlineCost,
Piotr Padlewski84abc742016-07-29 00:27:16 +0000413 std::function<AAResults &(Function &)> AARGetter,
414 ImportedFunctionsInliningStatistics &ImportedFunctionsStats) {
Dale Johannesen32dfb352009-03-23 23:39:20 +0000415 SmallPtrSet<Function*, 8> SCCFunctions;
David Greene0122fc42010-01-05 01:27:51 +0000416 DEBUG(dbgs() << "Inliner visiting SCC:");
Yaron Keren4c548f22015-06-20 07:12:33 +0000417 for (CallGraphNode *Node : SCC) {
418 Function *F = Node->getFunction();
Chris Lattner6754b822004-05-23 21:22:17 +0000419 if (F) SCCFunctions.insert(F);
David Greene0122fc42010-01-05 01:27:51 +0000420 DEBUG(dbgs() << " " << (F ? F->getName() : "INDIRECTNODE"));
Chris Lattnerd075cc22003-08-31 19:10:30 +0000421 }
Chris Lattnerd075cc22003-08-31 19:10:30 +0000422
Chris Lattner6754b822004-05-23 21:22:17 +0000423 // Scan through and identify all call sites ahead of time so that we only
424 // inline call sites in the original functions, not call sites that result
425 // from inlining other functions.
Chris Lattnere8262672010-05-01 01:05:10 +0000426 SmallVector<std::pair<CallSite, int>, 16> CallSites;
427
428 // When inlining a callee produces new call sites, we want to keep track of
429 // the fact that they were inlined from the callee. This allows us to avoid
430 // infinite inlining in some obscure cases. To represent this, we use an
431 // index into the InlineHistory vector.
432 SmallVector<std::pair<Function*, int>, 8> InlineHistory;
Chris Lattner6754b822004-05-23 21:22:17 +0000433
Yaron Keren4c548f22015-06-20 07:12:33 +0000434 for (CallGraphNode *Node : SCC) {
435 Function *F = Node->getFunction();
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000436 if (!F) continue;
437
Yaron Keren4c548f22015-06-20 07:12:33 +0000438 for (BasicBlock &BB : *F)
439 for (Instruction &I : BB) {
440 CallSite CS(cast<Value>(&I));
Dale Johannesen30599242009-10-09 00:11:32 +0000441 // If this isn't a call, or it is a call to an intrinsic, it can
Chris Lattner9e507472009-08-31 05:34:32 +0000442 // never be inlined.
Gabor Greif62f0aac2010-07-28 22:50:26 +0000443 if (!CS || isa<IntrinsicInst>(I))
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000444 continue;
445
Chris Lattner9e507472009-08-31 05:34:32 +0000446 // If this is a direct call to an external function, we can never inline
447 // it. If it is an indirect call, inlining may resolve it to be a
448 // direct call, so we keep it.
Yaron Kerenc66c06b2015-07-19 15:48:07 +0000449 if (Function *Callee = CS.getCalledFunction())
450 if (Callee->isDeclaration())
451 continue;
Chris Lattner9e507472009-08-31 05:34:32 +0000452
Chris Lattnere8262672010-05-01 01:05:10 +0000453 CallSites.push_back(std::make_pair(CS, -1));
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000454 }
455 }
Chris Lattnerd075cc22003-08-31 19:10:30 +0000456
David Greene0122fc42010-01-05 01:27:51 +0000457 DEBUG(dbgs() << ": " << CallSites.size() << " call sites.\n");
Misha Brukmanb1c93172005-04-21 23:48:37 +0000458
Chris Lattnera5cdd5e2010-04-20 00:47:08 +0000459 // If there are no calls in this function, exit early.
460 if (CallSites.empty())
461 return false;
Yaron Keren6967cbb2015-07-02 14:25:09 +0000462
Chris Lattner6754b822004-05-23 21:22:17 +0000463 // Now that we have all of the call sites, move the ones to functions in the
464 // current SCC to the end of the list.
465 unsigned FirstCallInSCC = CallSites.size();
466 for (unsigned i = 0; i < FirstCallInSCC; ++i)
Chris Lattnere8262672010-05-01 01:05:10 +0000467 if (Function *F = CallSites[i].first.getCalledFunction())
Chris Lattner6754b822004-05-23 21:22:17 +0000468 if (SCCFunctions.count(F))
469 std::swap(CallSites[i--], CallSites[--FirstCallInSCC]);
Misha Brukmanb1c93172005-04-21 23:48:37 +0000470
Chris Lattnerd3374e82009-08-27 06:29:33 +0000471
472 InlinedArrayAllocasTy InlinedArrayAllocas;
Sean Silvaab6a6832016-07-23 04:22:50 +0000473 InlineFunctionInfo InlineInfo(&CG, &GetAssumptionCache);
Chandler Carruth66b31302015-01-04 12:03:27 +0000474
Chris Lattner6754b822004-05-23 21:22:17 +0000475 // Now that we have all of the call sites, loop over them and inline them if
476 // it looks profitable to do so.
477 bool Changed = false;
478 bool LocalChange;
479 do {
480 LocalChange = false;
481 // Iterate over the outer loop because inlining functions can cause indirect
482 // calls to become direct calls.
Yaron Keren4c548f22015-06-20 07:12:33 +0000483 // CallSites may be modified inside so ranged for loop can not be used.
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000484 for (unsigned CSi = 0; CSi != CallSites.size(); ++CSi) {
Chris Lattnere8262672010-05-01 01:05:10 +0000485 CallSite CS = CallSites[CSi].first;
Chris Lattnerd3374e82009-08-27 06:29:33 +0000486
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000487 Function *Caller = CS.getCaller();
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000488 Function *Callee = CS.getCalledFunction();
489
490 // If this call site is dead and it is to a readonly function, we should
491 // just delete the call instead of trying to inline it, regardless of
492 // size. This happens because IPSCCP propagates the result out of the
493 // call and then we're left with the dead call.
Chandler Carruth7b560d42015-09-09 17:55:00 +0000494 if (isInstructionTriviallyDead(CS.getInstruction(), &TLI)) {
David Greene0122fc42010-01-05 01:27:51 +0000495 DEBUG(dbgs() << " -> Deleting dead call: "
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000496 << *CS.getInstruction() << "\n");
497 // Update the call graph by deleting the edge from Callee to Caller.
498 CG[Caller]->removeCallEdgeFor(CS);
499 CS.getInstruction()->eraseFromParent();
500 ++NumCallsDeleted;
501 } else {
502 // We can only inline direct calls to non-declarations.
Craig Topperf40110f2014-04-25 05:29:35 +0000503 if (!Callee || Callee->isDeclaration()) continue;
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000504
Eric Christopherea2820342010-07-13 18:27:13 +0000505 // If this call site was obtained by inlining another function, verify
Chris Lattnere8262672010-05-01 01:05:10 +0000506 // that the include path for the function did not include the callee
Chris Lattner5b6a8652010-12-06 07:38:40 +0000507 // itself. If so, we'd be recursively inlining the same function,
Chris Lattnere8262672010-05-01 01:05:10 +0000508 // which would provide the same callsites, which would cause us to
509 // infinitely inline.
510 int InlineHistoryID = CallSites[CSi].second;
511 if (InlineHistoryID != -1 &&
512 InlineHistoryIncludes(Callee, InlineHistoryID, InlineHistory))
513 continue;
514
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000515 LLVMContext &CallerCtx = Caller->getContext();
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000516
NAKAMURA Takumicd1fc4b2014-04-17 12:22:14 +0000517 // Get DebugLoc to report. CS will be invalid after Inliner.
518 DebugLoc DLoc = CS.getInstruction()->getDebugLoc();
519
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000520 // If the policy determines that we should inline this function,
521 // try to do so.
Sean Silvaab6a6832016-07-23 04:22:50 +0000522 if (!shouldInline(CS, GetInlineCost)) {
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000523 emitOptimizationRemarkMissed(CallerCtx, DEBUG_TYPE, *Caller, DLoc,
524 Twine(Callee->getName() +
525 " will not be inlined into " +
526 Caller->getName()));
527 continue;
528 }
529
Chris Lattner2eee5d32010-04-22 23:37:35 +0000530 // Attempt to inline the function.
Sean Silvaab6a6832016-07-23 04:22:50 +0000531 if (!InlineCallIfPossible(CS, InlineInfo, InlinedArrayAllocas,
Piotr Padlewski84abc742016-07-29 00:27:16 +0000532 InlineHistoryID, InsertLifetime, AARGetter,
533 ImportedFunctionsStats)) {
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000534 emitOptimizationRemarkMissed(CallerCtx, DEBUG_TYPE, *Caller, DLoc,
535 Twine(Callee->getName() +
536 " will not be inlined into " +
537 Caller->getName()));
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000538 continue;
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000539 }
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000540 ++NumInlined;
Diego Novilloa9298b22014-04-08 16:42:34 +0000541
542 // Report the inline decision.
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000543 emitOptimizationRemark(
544 CallerCtx, DEBUG_TYPE, *Caller, DLoc,
Diego Novilloa9298b22014-04-08 16:42:34 +0000545 Twine(Callee->getName() + " inlined into " + Caller->getName()));
546
Chris Lattnerc2432b92010-05-01 01:26:13 +0000547 // If inlining this function gave us any new call sites, throw them
Chris Lattner2eee5d32010-04-22 23:37:35 +0000548 // onto our worklist to process. They are useful inline candidates.
Chris Lattnerc2432b92010-05-01 01:26:13 +0000549 if (!InlineInfo.InlinedCalls.empty()) {
Chris Lattnere8262672010-05-01 01:05:10 +0000550 // Create a new inline history entry for this, so that we remember
551 // that these new callsites came about due to inlining Callee.
552 int NewHistoryID = InlineHistory.size();
553 InlineHistory.push_back(std::make_pair(Callee, InlineHistoryID));
554
Yaron Keren4c548f22015-06-20 07:12:33 +0000555 for (Value *Ptr : InlineInfo.InlinedCalls)
Chandler Carruth21211992012-03-25 04:03:40 +0000556 CallSites.push_back(std::make_pair(CallSite(Ptr), NewHistoryID));
Chris Lattnerc691de32010-04-23 18:37:01 +0000557 }
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000558 }
559
560 // If we inlined or deleted the last possible call site to the function,
561 // delete the function body now.
562 if (Callee && Callee->use_empty() && Callee->hasLocalLinkage() &&
Chris Lattner9e507472009-08-31 05:34:32 +0000563 // TODO: Can remove if in SCC now.
Chris Lattner081375b2009-08-31 03:15:49 +0000564 !SCCFunctions.count(Callee) &&
Chris Lattner9e507472009-08-31 05:34:32 +0000565
Chris Lattner081375b2009-08-31 03:15:49 +0000566 // The function may be apparently dead, but if there are indirect
567 // callgraph references to the node, we cannot delete it yet, this
568 // could invalidate the CGSCC iterator.
569 CG[Callee]->getNumReferences() == 0) {
David Greene0122fc42010-01-05 01:27:51 +0000570 DEBUG(dbgs() << " -> Deleting dead function: "
Chris Lattnerd3374e82009-08-27 06:29:33 +0000571 << Callee->getName() << "\n");
572 CallGraphNode *CalleeNode = CG[Callee];
Yaron Keren6967cbb2015-07-02 14:25:09 +0000573
Chris Lattnerd3374e82009-08-27 06:29:33 +0000574 // Remove any call graph edges from the callee to its callees.
575 CalleeNode->removeAllCalledFunctions();
576
Chris Lattnerd3374e82009-08-27 06:29:33 +0000577 // Removing the node for callee from the call graph and delete it.
Easwaran Ramanb1bd3982016-03-08 00:36:35 +0000578 delete CG.removeFunctionFromModule(CalleeNode);
Chris Lattnerd3374e82009-08-27 06:29:33 +0000579 ++NumDeleted;
580 }
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000581
582 // Remove this call site from the list. If possible, use
583 // swap/pop_back for efficiency, but do not use it if doing so would
584 // move a call site to a function in this SCC before the
585 // 'FirstCallInSCC' barrier.
Chris Lattner4422d312010-04-16 22:42:17 +0000586 if (SCC.isSingular()) {
Benjamin Kramer5ac57e32010-05-31 12:50:41 +0000587 CallSites[CSi] = CallSites.back();
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000588 CallSites.pop_back();
589 } else {
590 CallSites.erase(CallSites.begin()+CSi);
591 }
592 --CSi;
593
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000594 Changed = true;
595 LocalChange = true;
596 }
Chris Lattner6754b822004-05-23 21:22:17 +0000597 } while (LocalChange);
Chris Lattner4d25c862004-04-08 06:34:31 +0000598
Chris Lattnerd075cc22003-08-31 19:10:30 +0000599 return Changed;
600}
601
Sean Silvaab6a6832016-07-23 04:22:50 +0000602bool Inliner::inlineCalls(CallGraphSCC &SCC) {
603 CallGraph &CG = getAnalysis<CallGraphWrapperPass>().getCallGraph();
604 ACT = &getAnalysis<AssumptionCacheTracker>();
605 PSI = getAnalysis<ProfileSummaryInfoWrapperPass>().getPSI(CG.getModule());
606 auto &TLI = getAnalysis<TargetLibraryInfoWrapperPass>().getTLI();
607 // We compute dedicated AA results for each function in the SCC as needed. We
608 // use a lambda referencing external objects so that they live long enough to
609 // be queried, but we re-use them each time.
610 Optional<BasicAAResult> BAR;
611 Optional<AAResults> AAR;
612 auto AARGetter = [&](Function &F) -> AAResults & {
613 BAR.emplace(createLegacyPMBasicAAResult(*this, F));
614 AAR.emplace(createLegacyPMAAResults(*this, F, *BAR));
615 return *AAR;
616 };
617 auto GetAssumptionCache = [&](Function &F) -> AssumptionCache & {
618 return ACT->getAssumptionCache(F);
619 };
620 return inlineCallsImpl(SCC, CG, GetAssumptionCache, PSI, TLI, InsertLifetime,
621 [this](CallSite CS) { return getInlineCost(CS); },
Piotr Padlewski84abc742016-07-29 00:27:16 +0000622 AARGetter, ImportedFunctionsStats);
Sean Silvaab6a6832016-07-23 04:22:50 +0000623}
624
Sanjay Patelf1b0db12015-03-10 16:42:24 +0000625/// Remove now-dead linkonce functions at the end of
626/// processing to avoid breaking the SCC traversal.
Chris Lattnerc87784f2004-04-20 22:06:53 +0000627bool Inliner::doFinalization(CallGraph &CG) {
Piotr Padlewski84abc742016-07-29 00:27:16 +0000628 if (InlinerFunctionImportStats != InlinerFunctionImportStatsOpts::No)
629 ImportedFunctionsStats.dump(InlinerFunctionImportStats ==
630 InlinerFunctionImportStatsOpts::Verbose);
Devang Patelf0ef3572008-11-05 01:39:16 +0000631 return removeDeadFunctions(CG);
632}
633
Sanjay Patelf1b0db12015-03-10 16:42:24 +0000634/// Remove dead functions that are not included in DNR (Do Not Remove) list.
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000635bool Inliner::removeDeadFunctions(CallGraph &CG, bool AlwaysInlineOnly) {
636 SmallVector<CallGraphNode*, 16> FunctionsToRemove;
David Majnemerac256cf2015-05-05 20:14:22 +0000637 SmallVector<CallGraphNode *, 16> DeadFunctionsInComdats;
638 SmallDenseMap<const Comdat *, int, 16> ComdatEntriesAlive;
639
640 auto RemoveCGN = [&](CallGraphNode *CGN) {
641 // Remove any call graph edges from the function to its callees.
642 CGN->removeAllCalledFunctions();
643
644 // Remove any edges from the external node to the function's call graph
645 // node. These edges might have been made irrelegant due to
646 // optimization of the program.
647 CG.getExternalCallingNode()->removeAnyCallEdgeTo(CGN);
648
649 // Removing the node for callee from the call graph and delete it.
650 FunctionsToRemove.push_back(CGN);
651 };
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000652
653 // Scan for all of the functions, looking for ones that should now be removed
654 // from the program. Insert the dead ones in the FunctionsToRemove set.
David Blaikiea5d7de92015-08-05 20:55:50 +0000655 for (const auto &I : CG) {
656 CallGraphNode *CGN = I.second.get();
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000657 Function *F = CGN->getFunction();
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000658 if (!F || F->isDeclaration())
659 continue;
660
661 // Handle the case when this function is called and we only want to care
662 // about always-inline functions. This is a bit of a hack to share code
663 // between here and the InlineAlways pass.
Duncan P. N. Exon Smith2c79ad92015-02-14 01:11:29 +0000664 if (AlwaysInlineOnly && !F->hasFnAttribute(Attribute::AlwaysInline))
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000665 continue;
666
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000667 // If the only remaining users of the function are dead constants, remove
668 // them.
669 F->removeDeadConstantUsers();
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000670
Eli Friedman1923a332011-10-20 05:23:42 +0000671 if (!F->isDefTriviallyDead())
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000672 continue;
David Majnemerac077032014-10-08 19:32:32 +0000673
674 // It is unsafe to drop a function with discardable linkage from a COMDAT
675 // without also dropping the other members of the COMDAT.
676 // The inliner doesn't visit non-function entities which are in COMDAT
677 // groups so it is unsafe to do so *unless* the linkage is local.
David Majnemerac256cf2015-05-05 20:14:22 +0000678 if (!F->hasLocalLinkage()) {
679 if (const Comdat *C = F->getComdat()) {
680 --ComdatEntriesAlive[C];
681 DeadFunctionsInComdats.push_back(CGN);
682 continue;
683 }
684 }
Devang Patelf0ef3572008-11-05 01:39:16 +0000685
David Majnemerac256cf2015-05-05 20:14:22 +0000686 RemoveCGN(CGN);
Chris Lattnerc87784f2004-04-20 22:06:53 +0000687 }
David Majnemerac256cf2015-05-05 20:14:22 +0000688 if (!DeadFunctionsInComdats.empty()) {
689 // Count up all the entities in COMDAT groups
690 auto ComdatGroupReferenced = [&](const Comdat *C) {
691 auto I = ComdatEntriesAlive.find(C);
692 if (I != ComdatEntriesAlive.end())
693 ++(I->getSecond());
694 };
695 for (const Function &F : CG.getModule())
696 if (const Comdat *C = F.getComdat())
697 ComdatGroupReferenced(C);
698 for (const GlobalVariable &GV : CG.getModule().globals())
699 if (const Comdat *C = GV.getComdat())
700 ComdatGroupReferenced(C);
701 for (const GlobalAlias &GA : CG.getModule().aliases())
702 if (const Comdat *C = GA.getComdat())
703 ComdatGroupReferenced(C);
704 for (CallGraphNode *CGN : DeadFunctionsInComdats) {
705 Function *F = CGN->getFunction();
706 const Comdat *C = F->getComdat();
707 int NumAlive = ComdatEntriesAlive[C];
708 // We can remove functions in a COMDAT group if the entire group is dead.
709 assert(NumAlive >= 0);
710 if (NumAlive > 0)
711 continue;
712
713 RemoveCGN(CGN);
714 }
715 }
716
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000717 if (FunctionsToRemove.empty())
718 return false;
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000719
720 // Now that we know which functions to delete, do so. We didn't want to do
721 // this inline, because that would invalidate our CallGraph::iterator
722 // objects. :(
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000723 //
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000724 // Note that it doesn't matter that we are iterating over a non-stable order
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000725 // here to do this, it doesn't matter which order the functions are deleted
726 // in.
Chandler Carruth45ae88f2012-04-01 10:41:24 +0000727 array_pod_sort(FunctionsToRemove.begin(), FunctionsToRemove.end());
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000728 FunctionsToRemove.erase(std::unique(FunctionsToRemove.begin(),
729 FunctionsToRemove.end()),
730 FunctionsToRemove.end());
Yaron Keren62064d62015-06-25 19:28:24 +0000731 for (CallGraphNode *CGN : FunctionsToRemove) {
Easwaran Ramanb1bd3982016-03-08 00:36:35 +0000732 delete CG.removeFunctionFromModule(CGN);
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000733 ++NumDeleted;
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000734 }
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000735 return true;
Chris Lattnerc87784f2004-04-20 22:06:53 +0000736}