blob: 3cb6479a10c767d82bf9639a7c02cff557d68fc8 [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/Transforms/IPO/InlinerPass.h"
17#include "llvm/ADT/SmallPtrSet.h"
18#include "llvm/ADT/Statistic.h"
Hal Finkel0c083022014-09-01 09:01:39 +000019#include "llvm/Analysis/AliasAnalysis.h"
Chris Lattnerd075cc22003-08-31 19:10:30 +000020#include "llvm/Analysis/CallGraph.h"
Dan Gohman4552e3c2009-10-13 18:30:07 +000021#include "llvm/Analysis/InlineCost.h"
Chandler Carruth219b89b2014-03-04 11:01:28 +000022#include "llvm/IR/CallSite.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000023#include "llvm/IR/DataLayout.h"
Diego Novilloa9298b22014-04-08 16:42:34 +000024#include "llvm/IR/DiagnosticInfo.h"
Chandler Carruth9fb823b2013-01-02 11:36:10 +000025#include "llvm/IR/Instructions.h"
26#include "llvm/IR/IntrinsicInst.h"
27#include "llvm/IR/Module.h"
Reid Spencer7c16caa2004-09-01 22:55:40 +000028#include "llvm/Support/CommandLine.h"
29#include "llvm/Support/Debug.h"
Daniel Dunbar0dd5e1e2009-07-25 00:23:56 +000030#include "llvm/Support/raw_ostream.h"
Chandler Carruthed0881b2012-12-03 16:50:05 +000031#include "llvm/Target/TargetLibraryInfo.h"
32#include "llvm/Transforms/Utils/Cloning.h"
33#include "llvm/Transforms/Utils/Local.h"
Chris Lattnera82f1312003-11-21 21:45:31 +000034using namespace llvm;
Brian Gaeke960707c2003-11-11 22:41:34 +000035
Chandler Carruth964daaa2014-04-22 02:55:47 +000036#define DEBUG_TYPE "inline"
37
Chris Lattner1631bcb2006-12-19 22:09:18 +000038STATISTIC(NumInlined, "Number of functions inlined");
Chris Lattnerb49a6222010-05-01 17:19:38 +000039STATISTIC(NumCallsDeleted, "Number of call sites deleted, not inlined");
Chris Lattner1631bcb2006-12-19 22:09:18 +000040STATISTIC(NumDeleted, "Number of functions deleted because all callers found");
Chris Lattnerd3374e82009-08-27 06:29:33 +000041STATISTIC(NumMergedAllocas, "Number of allocas merged together");
Chris Lattner1631bcb2006-12-19 22:09:18 +000042
Benjamin Kramerbde91762012-06-02 10:20:22 +000043// This weirdly named statistic tracks the number of times that, when attempting
Chandler Carruth7ae90d42012-04-11 10:15:10 +000044// to inline a function A into B, we analyze the callers of B in order to see
45// if those would be more profitable and blocked inline steps.
46STATISTIC(NumCallerCallersAnalyzed, "Number of caller-callers analyzed");
47
Dan Gohmand78c4002008-05-13 00:00:25 +000048static cl::opt<int>
Jakob Stoklund Olesen113fb542010-02-04 18:48:20 +000049InlineLimit("inline-threshold", cl::Hidden, cl::init(225), cl::ZeroOrMore,
50 cl::desc("Control the amount of inlining to perform (default = 225)"));
Chris Lattnerd075cc22003-08-31 19:10:30 +000051
Jakob Stoklund Olesen492b8b42010-02-13 01:51:53 +000052static cl::opt<int>
53HintThreshold("inlinehint-threshold", cl::Hidden, cl::init(325),
54 cl::desc("Threshold for inlining functions with inline hint"));
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +000055
Manman Rend4612442014-02-06 01:59:22 +000056// We instroduce this threshold to help performance of instrumentation based
57// PGO before we actually hook up inliner with analysis passes such as BPI and
58// BFI.
Manman Rene8781b12014-02-05 22:53:44 +000059static cl::opt<int>
Manman Rend4612442014-02-06 01:59:22 +000060ColdThreshold("inlinecold-threshold", cl::Hidden, cl::init(225),
Manman Rene8781b12014-02-05 22:53:44 +000061 cl::desc("Threshold for inlining functions with cold attribute"));
62
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +000063// Threshold to use when optsize is specified (and there is no -inline-limit).
64const int OptSizeThreshold = 75;
65
Owen Andersona7aed182010-08-06 18:33:48 +000066Inliner::Inliner(char &ID)
Chad Rosier07d37bc2012-02-25 02:56:01 +000067 : CallGraphSCCPass(ID), InlineThreshold(InlineLimit), InsertLifetime(true) {}
Chris Lattnerd075cc22003-08-31 19:10:30 +000068
Chad Rosier07d37bc2012-02-25 02:56:01 +000069Inliner::Inliner(char &ID, int Threshold, bool InsertLifetime)
Jakob Stoklund Olesen31a7eb42010-11-02 23:40:26 +000070 : CallGraphSCCPass(ID), InlineThreshold(InlineLimit.getNumOccurrences() > 0 ?
Chad Rosier07d37bc2012-02-25 02:56:01 +000071 InlineLimit : Threshold),
72 InsertLifetime(InsertLifetime) {}
Chris Lattner22ad7ab2008-01-12 06:49:13 +000073
Chris Lattnerf94bed32007-01-30 23:28:39 +000074/// getAnalysisUsage - For this class, we declare that we require and preserve
75/// the call graph. If the derived class implements this method, it should
76/// always explicitly call the implementation here.
Chandler Carruthe40e60e2012-12-27 11:17:15 +000077void Inliner::getAnalysisUsage(AnalysisUsage &AU) const {
Hal Finkel0c083022014-09-01 09:01:39 +000078 AU.addRequired<AliasAnalysis>();
Chandler Carruthe40e60e2012-12-27 11:17:15 +000079 CallGraphSCCPass::getAnalysisUsage(AU);
Chris Lattnerf94bed32007-01-30 23:28:39 +000080}
81
Chris Lattnerd3374e82009-08-27 06:29:33 +000082
Chris Lattner229907c2011-07-18 04:54:35 +000083typedef DenseMap<ArrayType*, std::vector<AllocaInst*> >
Chris Lattnerd3374e82009-08-27 06:29:33 +000084InlinedArrayAllocasTy;
85
Bill Wendlingd154e2832013-01-23 06:41:41 +000086/// \brief If the inlined function had a higher stack protection level than the
87/// calling function, then bump up the caller's stack protection level.
88static void AdjustCallerSSPLevel(Function *Caller, Function *Callee) {
89 // If upgrading the SSP attribute, clear out the old SSP Attributes first.
90 // Having multiple SSP attributes doesn't actually hurt, but it adds useless
91 // clutter to the IR.
92 AttrBuilder B;
93 B.addAttribute(Attribute::StackProtect)
94 .addAttribute(Attribute::StackProtectStrong);
95 AttributeSet OldSSPAttr = AttributeSet::get(Caller->getContext(),
96 AttributeSet::FunctionIndex,
97 B);
98 AttributeSet CallerAttr = Caller->getAttributes(),
99 CalleeAttr = Callee->getAttributes();
100
101 if (CalleeAttr.hasAttribute(AttributeSet::FunctionIndex,
102 Attribute::StackProtectReq)) {
103 Caller->removeAttributes(AttributeSet::FunctionIndex, OldSSPAttr);
104 Caller->addFnAttr(Attribute::StackProtectReq);
105 } else if (CalleeAttr.hasAttribute(AttributeSet::FunctionIndex,
106 Attribute::StackProtectStrong) &&
107 !CallerAttr.hasAttribute(AttributeSet::FunctionIndex,
108 Attribute::StackProtectReq)) {
109 Caller->removeAttributes(AttributeSet::FunctionIndex, OldSSPAttr);
110 Caller->addFnAttr(Attribute::StackProtectStrong);
111 } else if (CalleeAttr.hasAttribute(AttributeSet::FunctionIndex,
112 Attribute::StackProtect) &&
113 !CallerAttr.hasAttribute(AttributeSet::FunctionIndex,
114 Attribute::StackProtectReq) &&
115 !CallerAttr.hasAttribute(AttributeSet::FunctionIndex,
116 Attribute::StackProtectStrong))
117 Caller->addFnAttr(Attribute::StackProtect);
118}
119
Chris Lattnerd3374e82009-08-27 06:29:33 +0000120/// InlineCallIfPossible - If it is possible to inline the specified call site,
121/// do so and update the CallGraph for this operation.
122///
123/// This function also does some basic book-keeping to update the IR. The
Chris Lattner0e890182009-08-28 04:48:54 +0000124/// InlinedArrayAllocas map keeps track of any allocas that are already
125/// available from other functions inlined into the caller. If we are able to
126/// inline this call site we attempt to reuse already available allocas or add
127/// any new allocas to the set if not possible.
Chris Lattner4ba01ec2010-04-22 23:07:58 +0000128static bool InlineCallIfPossible(CallSite CS, InlineFunctionInfo &IFI,
Chris Lattnerfb212de2010-12-06 07:52:42 +0000129 InlinedArrayAllocasTy &InlinedArrayAllocas,
Hal Finkel9caa8f72013-07-16 17:10:55 +0000130 int InlineHistory, bool InsertLifetime,
Rafael Espindola612886f2014-02-21 01:53:35 +0000131 const DataLayout *DL) {
Chris Lattner6754b822004-05-23 21:22:17 +0000132 Function *Callee = CS.getCalledFunction();
Bill Wendlingf5260d22008-11-21 00:09:21 +0000133 Function *Caller = CS.getCaller();
134
Chris Lattnerd3374e82009-08-27 06:29:33 +0000135 // Try to inline the function. Get the list of static allocas that were
136 // inlined.
Chad Rosier07d37bc2012-02-25 02:56:01 +0000137 if (!InlineFunction(CS, IFI, InsertLifetime))
Chris Lattnerb9d0a962009-08-27 04:32:07 +0000138 return false;
Misha Brukmanb1c93172005-04-21 23:48:37 +0000139
Bill Wendlingd154e2832013-01-23 06:41:41 +0000140 AdjustCallerSSPLevel(Caller, Callee);
Bill Wendling26c6a3e2008-11-21 00:06:32 +0000141
Chris Lattnerd3374e82009-08-27 06:29:33 +0000142 // Look at all of the allocas that we inlined through this call site. If we
143 // have already inlined other allocas through other calls into this function,
144 // then we know that they have disjoint lifetimes and that we can merge them.
145 //
146 // There are many heuristics possible for merging these allocas, and the
147 // different options have different tradeoffs. One thing that we *really*
148 // don't want to hurt is SRoA: once inlining happens, often allocas are no
149 // longer address taken and so they can be promoted.
150 //
151 // Our "solution" for that is to only merge allocas whose outermost type is an
152 // array type. These are usually not promoted because someone is using a
153 // variable index into them. These are also often the most important ones to
154 // merge.
155 //
156 // A better solution would be to have real memory lifetime markers in the IR
157 // and not have the inliner do any merging of allocas at all. This would
158 // allow the backend to do proper stack slot coloring of all allocas that
159 // *actually make it to the backend*, which is really what we want.
160 //
161 // Because we don't have this information, we do this simple and useful hack.
162 //
163 SmallPtrSet<AllocaInst*, 16> UsedAllocas;
164
Chris Lattnerfb212de2010-12-06 07:52:42 +0000165 // When processing our SCC, check to see if CS was inlined from some other
166 // call site. For example, if we're processing "A" in this code:
167 // A() { B() }
168 // B() { x = alloca ... C() }
169 // C() { y = alloca ... }
170 // Assume that C was not inlined into B initially, and so we're processing A
171 // and decide to inline B into A. Doing this makes an alloca available for
172 // reuse and makes a callsite (C) available for inlining. When we process
173 // the C call site we don't want to do any alloca merging between X and Y
174 // because their scopes are not disjoint. We could make this smarter by
175 // keeping track of the inline history for each alloca in the
176 // InlinedArrayAllocas but this isn't likely to be a significant win.
177 if (InlineHistory != -1) // Only do merging for top-level call sites in SCC.
178 return true;
179
Chris Lattnerd3374e82009-08-27 06:29:33 +0000180 // Loop over all the allocas we have so far and see if they can be merged with
181 // a previously inlined alloca. If not, remember that we had it.
Chris Lattner4ba01ec2010-04-22 23:07:58 +0000182 for (unsigned AllocaNo = 0, e = IFI.StaticAllocas.size();
Chris Lattnerd3374e82009-08-27 06:29:33 +0000183 AllocaNo != e; ++AllocaNo) {
Chris Lattner4ba01ec2010-04-22 23:07:58 +0000184 AllocaInst *AI = IFI.StaticAllocas[AllocaNo];
Chris Lattnerd3374e82009-08-27 06:29:33 +0000185
186 // Don't bother trying to merge array allocations (they will usually be
187 // canonicalized to be an allocation *of* an array), or allocations whose
188 // type is not itself an array (because we're afraid of pessimizing SRoA).
Chris Lattner229907c2011-07-18 04:54:35 +0000189 ArrayType *ATy = dyn_cast<ArrayType>(AI->getAllocatedType());
Craig Topperf40110f2014-04-25 05:29:35 +0000190 if (!ATy || AI->isArrayAllocation())
Chris Lattnerd3374e82009-08-27 06:29:33 +0000191 continue;
192
193 // Get the list of all available allocas for this array type.
194 std::vector<AllocaInst*> &AllocasForType = InlinedArrayAllocas[ATy];
195
196 // Loop over the allocas in AllocasForType to see if we can reuse one. Note
197 // that we have to be careful not to reuse the same "available" alloca for
198 // multiple different allocas that we just inlined, we use the 'UsedAllocas'
199 // set to keep track of which "available" allocas are being used by this
200 // function. Also, AllocasForType can be empty of course!
201 bool MergedAwayAlloca = false;
202 for (unsigned i = 0, e = AllocasForType.size(); i != e; ++i) {
203 AllocaInst *AvailableAlloca = AllocasForType[i];
Hal Finkel9caa8f72013-07-16 17:10:55 +0000204
205 unsigned Align1 = AI->getAlignment(),
206 Align2 = AvailableAlloca->getAlignment();
207 // If we don't have data layout information, and only one alloca is using
208 // the target default, then we can't safely merge them because we can't
209 // pick the greater alignment.
Rafael Espindola612886f2014-02-21 01:53:35 +0000210 if (!DL && (!Align1 || !Align2) && Align1 != Align2)
Hal Finkel9caa8f72013-07-16 17:10:55 +0000211 continue;
Chris Lattnerd3374e82009-08-27 06:29:33 +0000212
213 // The available alloca has to be in the right function, not in some other
214 // function in this SCC.
215 if (AvailableAlloca->getParent() != AI->getParent())
216 continue;
217
218 // If the inlined function already uses this alloca then we can't reuse
219 // it.
220 if (!UsedAllocas.insert(AvailableAlloca))
221 continue;
222
223 // Otherwise, we *can* reuse it, RAUW AI into AvailableAlloca and declare
224 // success!
Chris Lattner5b6a8652010-12-06 07:38:40 +0000225 DEBUG(dbgs() << " ***MERGED ALLOCA: " << *AI << "\n\t\tINTO: "
226 << *AvailableAlloca << '\n');
Chris Lattnerd3374e82009-08-27 06:29:33 +0000227
228 AI->replaceAllUsesWith(AvailableAlloca);
Hal Finkel9caa8f72013-07-16 17:10:55 +0000229
Hal Finkelec7cd262013-07-17 14:32:41 +0000230 if (Align1 != Align2) {
231 if (!Align1 || !Align2) {
Rafael Espindola612886f2014-02-21 01:53:35 +0000232 assert(DL && "DataLayout required to compare default alignments");
233 unsigned TypeAlign = DL->getABITypeAlignment(AI->getAllocatedType());
Hal Finkelec7cd262013-07-17 14:32:41 +0000234
235 Align1 = Align1 ? Align1 : TypeAlign;
236 Align2 = Align2 ? Align2 : TypeAlign;
237 }
238
239 if (Align1 > Align2)
240 AvailableAlloca->setAlignment(AI->getAlignment());
241 }
Hal Finkel9caa8f72013-07-16 17:10:55 +0000242
Chris Lattnerd3374e82009-08-27 06:29:33 +0000243 AI->eraseFromParent();
244 MergedAwayAlloca = true;
245 ++NumMergedAllocas;
Craig Topperf40110f2014-04-25 05:29:35 +0000246 IFI.StaticAllocas[AllocaNo] = nullptr;
Chris Lattnerd3374e82009-08-27 06:29:33 +0000247 break;
248 }
Misha Brukmanb1c93172005-04-21 23:48:37 +0000249
Chris Lattnerd3374e82009-08-27 06:29:33 +0000250 // If we already nuked the alloca, we're done with it.
251 if (MergedAwayAlloca)
252 continue;
Chris Lattner5b6a8652010-12-06 07:38:40 +0000253
Chris Lattnerd3374e82009-08-27 06:29:33 +0000254 // If we were unable to merge away the alloca either because there are no
255 // allocas of the right type available or because we reused them all
256 // already, remember that this alloca came from an inlined function and mark
257 // it used so we don't reuse it for other allocas from this inline
258 // operation.
259 AllocasForType.push_back(AI);
260 UsedAllocas.insert(AI);
Chris Lattner6754b822004-05-23 21:22:17 +0000261 }
Chris Lattnerd3374e82009-08-27 06:29:33 +0000262
Chris Lattner6754b822004-05-23 21:22:17 +0000263 return true;
Chris Lattnerd075cc22003-08-31 19:10:30 +0000264}
Jakob Stoklund Olesen8a19d3c2010-01-20 17:51:28 +0000265
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +0000266unsigned Inliner::getInlineThreshold(CallSite CS) const {
Patrik Hägglund8a1e3162012-05-23 13:42:57 +0000267 int thres = InlineThreshold; // -inline-threshold or else selected by
268 // overall opt level
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +0000269
Patrik Hägglund8a1e3162012-05-23 13:42:57 +0000270 // If -inline-threshold is not given, listen to the optsize attribute when it
271 // would decrease the threshold.
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +0000272 Function *Caller = CS.getCaller();
Patrik Hägglund8a1e3162012-05-23 13:42:57 +0000273 bool OptSize = Caller && !Caller->isDeclaration() &&
Bill Wendling698e84f2012-12-30 10:32:01 +0000274 Caller->getAttributes().hasAttribute(AttributeSet::FunctionIndex,
275 Attribute::OptimizeForSize);
Nadav Rotem97d44342012-09-13 16:27:32 +0000276 if (!(InlineLimit.getNumOccurrences() > 0) && OptSize &&
277 OptSizeThreshold < thres)
Jakob Stoklund Olesen492b8b42010-02-13 01:51:53 +0000278 thres = OptSizeThreshold;
Jakob Stoklund Olesen74bb06c2010-02-06 01:16:28 +0000279
Quentin Colombetc0dba202012-12-13 01:05:25 +0000280 // Listen to the inlinehint attribute when it would increase the threshold
281 // and the caller does not need to minimize its size.
Jakob Stoklund Olesen492b8b42010-02-13 01:51:53 +0000282 Function *Callee = CS.getCalledFunction();
Patrik Hägglund8a1e3162012-05-23 13:42:57 +0000283 bool InlineHint = Callee && !Callee->isDeclaration() &&
Bill Wendling698e84f2012-12-30 10:32:01 +0000284 Callee->getAttributes().hasAttribute(AttributeSet::FunctionIndex,
285 Attribute::InlineHint);
Quentin Colombetc0dba202012-12-13 01:05:25 +0000286 if (InlineHint && HintThreshold > thres
Bill Wendling698e84f2012-12-30 10:32:01 +0000287 && !Caller->getAttributes().hasAttribute(AttributeSet::FunctionIndex,
288 Attribute::MinSize))
Jakob Stoklund Olesen492b8b42010-02-13 01:51:53 +0000289 thres = HintThreshold;
290
Manman Rene8781b12014-02-05 22:53:44 +0000291 // Listen to the cold attribute when it would decrease the threshold.
292 bool ColdCallee = Callee && !Callee->isDeclaration() &&
293 Callee->getAttributes().hasAttribute(AttributeSet::FunctionIndex,
294 Attribute::Cold);
Manman Ren3c440672014-04-25 17:34:55 +0000295 // Command line argument for InlineLimit will override the default
296 // ColdThreshold. If we have -inline-threshold but no -inlinecold-threshold,
297 // do not use the default cold threshold even if it is smaller.
298 if ((InlineLimit.getNumOccurrences() == 0 ||
299 ColdThreshold.getNumOccurrences() > 0) && ColdCallee &&
300 ColdThreshold < thres)
Manman Rene8781b12014-02-05 22:53:44 +0000301 thres = ColdThreshold;
302
Jakob Stoklund Olesen492b8b42010-02-13 01:51:53 +0000303 return thres;
Jakob Stoklund Olesen8a19d3c2010-01-20 17:51:28 +0000304}
305
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000306static void emitAnalysis(CallSite CS, const Twine &Msg) {
307 Function *Caller = CS.getCaller();
308 LLVMContext &Ctx = Caller->getContext();
309 DebugLoc DLoc = CS.getInstruction()->getDebugLoc();
310 emitOptimizationRemarkAnalysis(Ctx, DEBUG_TYPE, *Caller, DLoc, Msg);
311}
312
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000313/// shouldInline - Return true if the inliner should attempt to inline
314/// at the given CallSite.
315bool Inliner::shouldInline(CallSite CS) {
Daniel Dunbar3933e662008-10-30 19:26:59 +0000316 InlineCost IC = getInlineCost(CS);
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000317
Daniel Dunbar3933e662008-10-30 19:26:59 +0000318 if (IC.isAlways()) {
David Greene0122fc42010-01-05 01:27:51 +0000319 DEBUG(dbgs() << " Inlining: cost=always"
Bill Wendling2602bb42009-07-31 19:52:24 +0000320 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000321 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName()) +
322 " should always be inlined (cost=always)");
Daniel Dunbar3933e662008-10-30 19:26:59 +0000323 return true;
324 }
325
326 if (IC.isNever()) {
David Greene0122fc42010-01-05 01:27:51 +0000327 DEBUG(dbgs() << " NOT Inlining: cost=never"
Bill Wendling2602bb42009-07-31 19:52:24 +0000328 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000329 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName() +
330 " should never be inlined (cost=never)"));
Daniel Dunbar3933e662008-10-30 19:26:59 +0000331 return false;
332 }
333
Dale Johannesen30599242009-10-09 00:11:32 +0000334 Function *Caller = CS.getCaller();
Chandler Carruth0539c072012-03-31 12:42:41 +0000335 if (!IC) {
336 DEBUG(dbgs() << " NOT Inlining: cost=" << IC.getCost()
337 << ", thres=" << (IC.getCostDelta() + IC.getCost())
Bill Wendling2602bb42009-07-31 19:52:24 +0000338 << ", Call: " << *CS.getInstruction() << "\n");
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000339 emitAnalysis(CS, Twine(CS.getCalledFunction()->getName() +
340 " too costly to inline (cost=") +
341 Twine(IC.getCost()) + ", threshold=" +
342 Twine(IC.getCostDelta() + IC.getCost()) + ")");
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000343 return false;
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000344 }
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000345
Chandler Carruth30b84162012-03-14 20:16:41 +0000346 // Try to detect the case where the current inlining candidate caller (call
347 // it B) is a static or linkonce-ODR function and is an inlining candidate
348 // elsewhere, and the current candidate callee (call it C) is large enough
349 // that inlining it into B would make B too big to inline later. In these
350 // circumstances it may be best not to inline C into B, but to inline B into
351 // its callers.
352 //
353 // This only applies to static and linkonce-ODR functions because those are
354 // expected to be available for inlining in the translation units where they
355 // are used. Thus we will always have the opportunity to make local inlining
356 // decisions. Importantly the linkonce-ODR linkage covers inline functions
357 // and templates in C++.
Chandler Carruth0539c072012-03-31 12:42:41 +0000358 //
359 // FIXME: All of this logic should be sunk into getInlineCost. It relies on
360 // the internal implementation of the inline cost metrics rather than
361 // treating them as truly abstract units etc.
Rafael Espindola3cf4af12014-07-30 15:57:51 +0000362 if (Caller->hasLocalLinkage() || Caller->hasLinkOnceODRLinkage()) {
Dale Johannesen30599242009-10-09 00:11:32 +0000363 int TotalSecondaryCost = 0;
Chandler Carruth0539c072012-03-31 12:42:41 +0000364 // The candidate cost to be imposed upon the current function.
365 int CandidateCost = IC.getCost() - (InlineConstants::CallPenalty + 1);
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000366 // This bool tracks what happens if we do NOT inline C into B.
Chandler Carruthb9e35fb2012-03-27 10:48:28 +0000367 bool callerWillBeRemoved = Caller->hasLocalLinkage();
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000368 // This bool tracks what happens if we DO inline C into B.
369 bool inliningPreventsSomeOuterInline = false;
Chandler Carruthcdf47882014-03-09 03:16:01 +0000370 for (User *U : Caller->users()) {
371 CallSite CS2(U);
Dale Johannesen30599242009-10-09 00:11:32 +0000372
373 // If this isn't a call to Caller (it could be some other sort
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000374 // of reference) skip it. Such references will prevent the caller
375 // from being removed.
376 if (!CS2 || CS2.getCalledFunction() != Caller) {
377 callerWillBeRemoved = false;
Dale Johannesen30599242009-10-09 00:11:32 +0000378 continue;
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000379 }
Dale Johannesen30599242009-10-09 00:11:32 +0000380
381 InlineCost IC2 = getInlineCost(CS2);
Chandler Carruth7ae90d42012-04-11 10:15:10 +0000382 ++NumCallerCallersAnalyzed;
Chandler Carruth0539c072012-03-31 12:42:41 +0000383 if (!IC2) {
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000384 callerWillBeRemoved = false;
Chandler Carruth0539c072012-03-31 12:42:41 +0000385 continue;
386 }
387 if (IC2.isAlways())
Dale Johannesen30599242009-10-09 00:11:32 +0000388 continue;
389
Chandler Carruth0539c072012-03-31 12:42:41 +0000390 // See if inlining or original callsite would erase the cost delta of
391 // this callsite. We subtract off the penalty for the call instruction,
392 // which we would be deleting.
393 if (IC2.getCostDelta() <= CandidateCost) {
Dale Johannesena71d2cc2011-01-04 19:01:54 +0000394 inliningPreventsSomeOuterInline = true;
Chandler Carruth0539c072012-03-31 12:42:41 +0000395 TotalSecondaryCost += IC2.getCost();
Dale Johannesen30599242009-10-09 00:11:32 +0000396 }
397 }
398 // If all outer calls to Caller would get inlined, the cost for the last
399 // one is set very low by getInlineCost, in anticipation that Caller will
400 // be removed entirely. We did not account for this above unless there
401 // is only one caller of Caller.
Chandler Carruthcdf47882014-03-09 03:16:01 +0000402 if (callerWillBeRemoved && !Caller->use_empty())
Dale Johannesen96a5b872009-10-09 21:42:02 +0000403 TotalSecondaryCost += InlineConstants::LastCallToStaticBonus;
Dale Johannesen30599242009-10-09 00:11:32 +0000404
Chandler Carruth0539c072012-03-31 12:42:41 +0000405 if (inliningPreventsSomeOuterInline && TotalSecondaryCost < IC.getCost()) {
406 DEBUG(dbgs() << " NOT Inlining: " << *CS.getInstruction() <<
407 " Cost = " << IC.getCost() <<
Dale Johannesen30599242009-10-09 00:11:32 +0000408 ", outer Cost = " << TotalSecondaryCost << '\n');
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000409 emitAnalysis(
410 CS, Twine("Not inlining. Cost of inlining " +
411 CS.getCalledFunction()->getName() +
412 " increases the cost of inlining " +
413 CS.getCaller()->getName() + " in other contexts"));
Dale Johannesen30599242009-10-09 00:11:32 +0000414 return false;
415 }
416 }
417
Chandler Carruth0539c072012-03-31 12:42:41 +0000418 DEBUG(dbgs() << " Inlining: cost=" << IC.getCost()
419 << ", thres=" << (IC.getCostDelta() + IC.getCost())
Dale Johannesen30599242009-10-09 00:11:32 +0000420 << ", Call: " << *CS.getInstruction() << '\n');
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000421 emitAnalysis(
422 CS, CS.getCalledFunction()->getName() + Twine(" can be inlined into ") +
423 CS.getCaller()->getName() + " with cost=" + Twine(IC.getCost()) +
424 " (threshold=" + Twine(IC.getCostDelta() + IC.getCost()) + ")");
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000425 return true;
Daniel Dunbare7fbf9f42008-10-29 01:02:02 +0000426}
Chris Lattnerd075cc22003-08-31 19:10:30 +0000427
Chris Lattnere8262672010-05-01 01:05:10 +0000428/// InlineHistoryIncludes - Return true if the specified inline history ID
429/// indicates an inline history that includes the specified function.
430static bool InlineHistoryIncludes(Function *F, int InlineHistoryID,
431 const SmallVectorImpl<std::pair<Function*, int> > &InlineHistory) {
432 while (InlineHistoryID != -1) {
433 assert(unsigned(InlineHistoryID) < InlineHistory.size() &&
434 "Invalid inline history ID");
435 if (InlineHistory[InlineHistoryID].first == F)
436 return true;
437 InlineHistoryID = InlineHistory[InlineHistoryID].second;
438 }
439 return false;
440}
441
Chris Lattner4422d312010-04-16 22:42:17 +0000442bool Inliner::runOnSCC(CallGraphSCC &SCC) {
Chandler Carruth6378cf52013-11-26 04:19:30 +0000443 CallGraph &CG = getAnalysis<CallGraphWrapperPass>().getCallGraph();
Rafael Espindola93512512014-02-25 17:30:31 +0000444 DataLayoutPass *DLP = getAnalysisIfAvailable<DataLayoutPass>();
Craig Topperf40110f2014-04-25 05:29:35 +0000445 const DataLayout *DL = DLP ? &DLP->getDataLayout() : nullptr;
Benjamin Kramer8bcc9712012-08-29 15:32:21 +0000446 const TargetLibraryInfo *TLI = getAnalysisIfAvailable<TargetLibraryInfo>();
Hal Finkel0c083022014-09-01 09:01:39 +0000447 AliasAnalysis *AA = &getAnalysis<AliasAnalysis>();
Chris Lattnerd075cc22003-08-31 19:10:30 +0000448
Dale Johannesen32dfb352009-03-23 23:39:20 +0000449 SmallPtrSet<Function*, 8> SCCFunctions;
David Greene0122fc42010-01-05 01:27:51 +0000450 DEBUG(dbgs() << "Inliner visiting SCC:");
Chris Lattner4422d312010-04-16 22:42:17 +0000451 for (CallGraphSCC::iterator I = SCC.begin(), E = SCC.end(); I != E; ++I) {
452 Function *F = (*I)->getFunction();
Chris Lattner6754b822004-05-23 21:22:17 +0000453 if (F) SCCFunctions.insert(F);
David Greene0122fc42010-01-05 01:27:51 +0000454 DEBUG(dbgs() << " " << (F ? F->getName() : "INDIRECTNODE"));
Chris Lattnerd075cc22003-08-31 19:10:30 +0000455 }
Chris Lattnerd075cc22003-08-31 19:10:30 +0000456
Chris Lattner6754b822004-05-23 21:22:17 +0000457 // Scan through and identify all call sites ahead of time so that we only
458 // inline call sites in the original functions, not call sites that result
459 // from inlining other functions.
Chris Lattnere8262672010-05-01 01:05:10 +0000460 SmallVector<std::pair<CallSite, int>, 16> CallSites;
461
462 // When inlining a callee produces new call sites, we want to keep track of
463 // the fact that they were inlined from the callee. This allows us to avoid
464 // infinite inlining in some obscure cases. To represent this, we use an
465 // index into the InlineHistory vector.
466 SmallVector<std::pair<Function*, int>, 8> InlineHistory;
Chris Lattner6754b822004-05-23 21:22:17 +0000467
Chris Lattner4422d312010-04-16 22:42:17 +0000468 for (CallGraphSCC::iterator I = SCC.begin(), E = SCC.end(); I != E; ++I) {
469 Function *F = (*I)->getFunction();
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000470 if (!F) continue;
471
472 for (Function::iterator BB = F->begin(), E = F->end(); BB != E; ++BB)
473 for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I) {
Gabor Greif62f0aac2010-07-28 22:50:26 +0000474 CallSite CS(cast<Value>(I));
Dale Johannesen30599242009-10-09 00:11:32 +0000475 // If this isn't a call, or it is a call to an intrinsic, it can
Chris Lattner9e507472009-08-31 05:34:32 +0000476 // never be inlined.
Gabor Greif62f0aac2010-07-28 22:50:26 +0000477 if (!CS || isa<IntrinsicInst>(I))
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000478 continue;
479
Chris Lattner9e507472009-08-31 05:34:32 +0000480 // If this is a direct call to an external function, we can never inline
481 // it. If it is an indirect call, inlining may resolve it to be a
482 // direct call, so we keep it.
483 if (CS.getCalledFunction() && CS.getCalledFunction()->isDeclaration())
484 continue;
485
Chris Lattnere8262672010-05-01 01:05:10 +0000486 CallSites.push_back(std::make_pair(CS, -1));
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000487 }
488 }
Chris Lattnerd075cc22003-08-31 19:10:30 +0000489
David Greene0122fc42010-01-05 01:27:51 +0000490 DEBUG(dbgs() << ": " << CallSites.size() << " call sites.\n");
Misha Brukmanb1c93172005-04-21 23:48:37 +0000491
Chris Lattnera5cdd5e2010-04-20 00:47:08 +0000492 // If there are no calls in this function, exit early.
493 if (CallSites.empty())
494 return false;
495
Chris Lattner6754b822004-05-23 21:22:17 +0000496 // Now that we have all of the call sites, move the ones to functions in the
497 // current SCC to the end of the list.
498 unsigned FirstCallInSCC = CallSites.size();
499 for (unsigned i = 0; i < FirstCallInSCC; ++i)
Chris Lattnere8262672010-05-01 01:05:10 +0000500 if (Function *F = CallSites[i].first.getCalledFunction())
Chris Lattner6754b822004-05-23 21:22:17 +0000501 if (SCCFunctions.count(F))
502 std::swap(CallSites[i--], CallSites[--FirstCallInSCC]);
Misha Brukmanb1c93172005-04-21 23:48:37 +0000503
Chris Lattnerd3374e82009-08-27 06:29:33 +0000504
505 InlinedArrayAllocasTy InlinedArrayAllocas;
Hal Finkel0c083022014-09-01 09:01:39 +0000506 InlineFunctionInfo InlineInfo(&CG, DL, AA);
Chris Lattnerd3374e82009-08-27 06:29:33 +0000507
Chris Lattner6754b822004-05-23 21:22:17 +0000508 // Now that we have all of the call sites, loop over them and inline them if
509 // it looks profitable to do so.
510 bool Changed = false;
511 bool LocalChange;
512 do {
513 LocalChange = false;
514 // Iterate over the outer loop because inlining functions can cause indirect
515 // calls to become direct calls.
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000516 for (unsigned CSi = 0; CSi != CallSites.size(); ++CSi) {
Chris Lattnere8262672010-05-01 01:05:10 +0000517 CallSite CS = CallSites[CSi].first;
Chris Lattnerd3374e82009-08-27 06:29:33 +0000518
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000519 Function *Caller = CS.getCaller();
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000520 Function *Callee = CS.getCalledFunction();
521
522 // If this call site is dead and it is to a readonly function, we should
523 // just delete the call instead of trying to inline it, regardless of
524 // size. This happens because IPSCCP propagates the result out of the
525 // call and then we're left with the dead call.
Benjamin Kramer8bcc9712012-08-29 15:32:21 +0000526 if (isInstructionTriviallyDead(CS.getInstruction(), TLI)) {
David Greene0122fc42010-01-05 01:27:51 +0000527 DEBUG(dbgs() << " -> Deleting dead call: "
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000528 << *CS.getInstruction() << "\n");
529 // Update the call graph by deleting the edge from Callee to Caller.
530 CG[Caller]->removeCallEdgeFor(CS);
531 CS.getInstruction()->eraseFromParent();
532 ++NumCallsDeleted;
533 } else {
534 // We can only inline direct calls to non-declarations.
Craig Topperf40110f2014-04-25 05:29:35 +0000535 if (!Callee || Callee->isDeclaration()) continue;
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000536
Eric Christopherea2820342010-07-13 18:27:13 +0000537 // If this call site was obtained by inlining another function, verify
Chris Lattnere8262672010-05-01 01:05:10 +0000538 // that the include path for the function did not include the callee
Chris Lattner5b6a8652010-12-06 07:38:40 +0000539 // itself. If so, we'd be recursively inlining the same function,
Chris Lattnere8262672010-05-01 01:05:10 +0000540 // which would provide the same callsites, which would cause us to
541 // infinitely inline.
542 int InlineHistoryID = CallSites[CSi].second;
543 if (InlineHistoryID != -1 &&
544 InlineHistoryIncludes(Callee, InlineHistoryID, InlineHistory))
545 continue;
546
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000547 LLVMContext &CallerCtx = Caller->getContext();
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000548
NAKAMURA Takumicd1fc4b2014-04-17 12:22:14 +0000549 // Get DebugLoc to report. CS will be invalid after Inliner.
550 DebugLoc DLoc = CS.getInstruction()->getDebugLoc();
551
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000552 // If the policy determines that we should inline this function,
553 // try to do so.
554 if (!shouldInline(CS)) {
555 emitOptimizationRemarkMissed(CallerCtx, DEBUG_TYPE, *Caller, DLoc,
556 Twine(Callee->getName() +
557 " will not be inlined into " +
558 Caller->getName()));
559 continue;
560 }
561
Chris Lattner2eee5d32010-04-22 23:37:35 +0000562 // Attempt to inline the function.
Chris Lattnerfb212de2010-12-06 07:52:42 +0000563 if (!InlineCallIfPossible(CS, InlineInfo, InlinedArrayAllocas,
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000564 InlineHistoryID, InsertLifetime, DL)) {
565 emitOptimizationRemarkMissed(CallerCtx, DEBUG_TYPE, *Caller, DLoc,
566 Twine(Callee->getName() +
567 " will not be inlined into " +
568 Caller->getName()));
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000569 continue;
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000570 }
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000571 ++NumInlined;
Diego Novilloa9298b22014-04-08 16:42:34 +0000572
573 // Report the inline decision.
Diego Novillo7f8af8b2014-05-22 14:19:46 +0000574 emitOptimizationRemark(
575 CallerCtx, DEBUG_TYPE, *Caller, DLoc,
Diego Novilloa9298b22014-04-08 16:42:34 +0000576 Twine(Callee->getName() + " inlined into " + Caller->getName()));
577
Chris Lattnerc2432b92010-05-01 01:26:13 +0000578 // If inlining this function gave us any new call sites, throw them
Chris Lattner2eee5d32010-04-22 23:37:35 +0000579 // onto our worklist to process. They are useful inline candidates.
Chris Lattnerc2432b92010-05-01 01:26:13 +0000580 if (!InlineInfo.InlinedCalls.empty()) {
Chris Lattnere8262672010-05-01 01:05:10 +0000581 // Create a new inline history entry for this, so that we remember
582 // that these new callsites came about due to inlining Callee.
583 int NewHistoryID = InlineHistory.size();
584 InlineHistory.push_back(std::make_pair(Callee, InlineHistoryID));
585
Chris Lattnerc2432b92010-05-01 01:26:13 +0000586 for (unsigned i = 0, e = InlineInfo.InlinedCalls.size();
Chris Lattnere8262672010-05-01 01:05:10 +0000587 i != e; ++i) {
Chris Lattnerc2432b92010-05-01 01:26:13 +0000588 Value *Ptr = InlineInfo.InlinedCalls[i];
Chandler Carruth21211992012-03-25 04:03:40 +0000589 CallSites.push_back(std::make_pair(CallSite(Ptr), NewHistoryID));
Chris Lattnere8262672010-05-01 01:05:10 +0000590 }
Chris Lattnerc691de32010-04-23 18:37:01 +0000591 }
Chris Lattnereb9acbf2009-11-12 07:56:08 +0000592 }
593
594 // If we inlined or deleted the last possible call site to the function,
595 // delete the function body now.
596 if (Callee && Callee->use_empty() && Callee->hasLocalLinkage() &&
Chris Lattner9e507472009-08-31 05:34:32 +0000597 // TODO: Can remove if in SCC now.
Chris Lattner081375b2009-08-31 03:15:49 +0000598 !SCCFunctions.count(Callee) &&
Chris Lattner9e507472009-08-31 05:34:32 +0000599
Chris Lattner081375b2009-08-31 03:15:49 +0000600 // The function may be apparently dead, but if there are indirect
601 // callgraph references to the node, we cannot delete it yet, this
602 // could invalidate the CGSCC iterator.
603 CG[Callee]->getNumReferences() == 0) {
David Greene0122fc42010-01-05 01:27:51 +0000604 DEBUG(dbgs() << " -> Deleting dead function: "
Chris Lattnerd3374e82009-08-27 06:29:33 +0000605 << Callee->getName() << "\n");
606 CallGraphNode *CalleeNode = CG[Callee];
607
608 // Remove any call graph edges from the callee to its callees.
609 CalleeNode->removeAllCalledFunctions();
610
Chris Lattnerd3374e82009-08-27 06:29:33 +0000611 // Removing the node for callee from the call graph and delete it.
612 delete CG.removeFunctionFromModule(CalleeNode);
613 ++NumDeleted;
614 }
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000615
616 // Remove this call site from the list. If possible, use
617 // swap/pop_back for efficiency, but do not use it if doing so would
618 // move a call site to a function in this SCC before the
619 // 'FirstCallInSCC' barrier.
Chris Lattner4422d312010-04-16 22:42:17 +0000620 if (SCC.isSingular()) {
Benjamin Kramer5ac57e32010-05-31 12:50:41 +0000621 CallSites[CSi] = CallSites.back();
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000622 CallSites.pop_back();
623 } else {
624 CallSites.erase(CallSites.begin()+CSi);
625 }
626 --CSi;
627
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000628 Changed = true;
629 LocalChange = true;
630 }
Chris Lattner6754b822004-05-23 21:22:17 +0000631 } while (LocalChange);
Chris Lattner4d25c862004-04-08 06:34:31 +0000632
Chris Lattnerd075cc22003-08-31 19:10:30 +0000633 return Changed;
634}
635
Chris Lattnerc87784f2004-04-20 22:06:53 +0000636// doFinalization - Remove now-dead linkonce functions at the end of
637// processing to avoid breaking the SCC traversal.
638bool Inliner::doFinalization(CallGraph &CG) {
Devang Patelf0ef3572008-11-05 01:39:16 +0000639 return removeDeadFunctions(CG);
640}
641
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000642/// removeDeadFunctions - Remove dead functions that are not included in
643/// DNR (Do Not Remove) list.
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000644bool Inliner::removeDeadFunctions(CallGraph &CG, bool AlwaysInlineOnly) {
645 SmallVector<CallGraphNode*, 16> FunctionsToRemove;
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000646
647 // Scan for all of the functions, looking for ones that should now be removed
648 // from the program. Insert the dead ones in the FunctionsToRemove set.
649 for (CallGraph::iterator I = CG.begin(), E = CG.end(); I != E; ++I) {
650 CallGraphNode *CGN = I->second;
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000651 Function *F = CGN->getFunction();
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000652 if (!F || F->isDeclaration())
653 continue;
654
655 // Handle the case when this function is called and we only want to care
656 // about always-inline functions. This is a bit of a hack to share code
657 // between here and the InlineAlways pass.
Bill Wendlingc9b22d72012-10-09 07:45:08 +0000658 if (AlwaysInlineOnly &&
Bill Wendling698e84f2012-12-30 10:32:01 +0000659 !F->getAttributes().hasAttribute(AttributeSet::FunctionIndex,
660 Attribute::AlwaysInline))
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000661 continue;
662
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000663 // If the only remaining users of the function are dead constants, remove
664 // them.
665 F->removeDeadConstantUsers();
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000666
Eli Friedman1923a332011-10-20 05:23:42 +0000667 if (!F->isDefTriviallyDead())
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000668 continue;
669
670 // Remove any call graph edges from the function to its callees.
671 CGN->removeAllCalledFunctions();
Devang Patelf0ef3572008-11-05 01:39:16 +0000672
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000673 // Remove any edges from the external node to the function's call graph
674 // node. These edges might have been made irrelegant due to
675 // optimization of the program.
676 CG.getExternalCallingNode()->removeAnyCallEdgeTo(CGN);
Chris Lattner6455c512004-09-18 21:37:03 +0000677
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000678 // Removing the node for callee from the call graph and delete it.
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000679 FunctionsToRemove.push_back(CGN);
Chris Lattnerc87784f2004-04-20 22:06:53 +0000680 }
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000681 if (FunctionsToRemove.empty())
682 return false;
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000683
684 // Now that we know which functions to delete, do so. We didn't want to do
685 // this inline, because that would invalidate our CallGraph::iterator
686 // objects. :(
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000687 //
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000688 // Note that it doesn't matter that we are iterating over a non-stable order
Chris Lattner5eef6ad2009-08-27 03:51:50 +0000689 // here to do this, it doesn't matter which order the functions are deleted
690 // in.
Chandler Carruth45ae88f2012-04-01 10:41:24 +0000691 array_pod_sort(FunctionsToRemove.begin(), FunctionsToRemove.end());
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000692 FunctionsToRemove.erase(std::unique(FunctionsToRemove.begin(),
693 FunctionsToRemove.end()),
694 FunctionsToRemove.end());
695 for (SmallVectorImpl<CallGraphNode *>::iterator I = FunctionsToRemove.begin(),
696 E = FunctionsToRemove.end();
697 I != E; ++I) {
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000698 delete CG.removeFunctionFromModule(*I);
699 ++NumDeleted;
Chris Lattnerbe8bb802004-04-21 20:44:33 +0000700 }
Chandler Carruthd7a5f2a2012-03-16 06:10:13 +0000701 return true;
Chris Lattnerc87784f2004-04-20 22:06:53 +0000702}