blob: 9b8e0a1fc4d032c83bf7e7ac28ec64564f229a62 [file] [log] [blame]
Chris Lattnercf5933a2004-06-20 04:11:48 +00001//===- Inliner.cpp - Code common to all inliners --------------------------===//
Misha Brukmanfd939082005-04-21 23:48:37 +00002//
John Criswellb576c942003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
Chris Lattner4ee451d2007-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 Brukmanfd939082005-04-21 23:48:37 +00007//
John Criswellb576c942003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattner237ef562003-08-31 19:10:30 +00009//
Chris Lattnerbefa4992004-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 Lattner237ef562003-08-31 19:10:30 +000013//
14//===----------------------------------------------------------------------===//
15
Chris Lattner86453c52006-12-19 22:09:18 +000016#define DEBUG_TYPE "inline"
Chris Lattner237ef562003-08-31 19:10:30 +000017#include "llvm/Module.h"
Misha Brukman47b14a42004-07-29 17:30:56 +000018#include "llvm/Instructions.h"
Dale Johannesen1f67ce42009-03-19 18:03:56 +000019#include "llvm/IntrinsicInst.h"
Chris Lattner237ef562003-08-31 19:10:30 +000020#include "llvm/Analysis/CallGraph.h"
21#include "llvm/Support/CallSite.h"
Chris Lattnerff2dad32007-01-30 23:28:39 +000022#include "llvm/Target/TargetData.h"
Tanya Lattner6f7426e2007-06-19 22:29:50 +000023#include "llvm/Transforms/IPO/InlinerPass.h"
Chris Lattner12f0bab2009-08-27 04:32:07 +000024#include "llvm/Transforms/Utils/InlineCost.h"
Chris Lattner237ef562003-08-31 19:10:30 +000025#include "llvm/Transforms/Utils/Cloning.h"
Reid Spencer551ccae2004-09-01 22:55:40 +000026#include "llvm/Support/CommandLine.h"
27#include "llvm/Support/Debug.h"
Daniel Dunbarce63ffb2009-07-25 00:23:56 +000028#include "llvm/Support/raw_ostream.h"
Chris Lattner12f0bab2009-08-27 04:32:07 +000029#include "llvm/ADT/SmallPtrSet.h"
Reid Spencer551ccae2004-09-01 22:55:40 +000030#include "llvm/ADT/Statistic.h"
Chris Lattnerbefa4992004-05-23 21:22:17 +000031#include <set>
Chris Lattnera51bcb52003-11-21 21:45:31 +000032using namespace llvm;
Brian Gaeked0fde302003-11-11 22:41:34 +000033
Chris Lattner86453c52006-12-19 22:09:18 +000034STATISTIC(NumInlined, "Number of functions inlined");
35STATISTIC(NumDeleted, "Number of functions deleted because all callers found");
Chris Lattner199ba422009-08-27 06:29:33 +000036STATISTIC(NumMergedAllocas, "Number of allocas merged together");
Chris Lattner86453c52006-12-19 22:09:18 +000037
Dan Gohman844731a2008-05-13 00:00:25 +000038static cl::opt<int>
Dale Johannesen5fee49e2009-08-25 01:13:58 +000039InlineLimit("inline-threshold", cl::Hidden, cl::init(200), cl::ZeroOrMore,
Dale Johannesencbfdf962009-01-12 22:11:50 +000040 cl::desc("Control the amount of inlining to perform (default = 200)"));
Chris Lattner237ef562003-08-31 19:10:30 +000041
Dan Gohmanae73dc12008-09-04 17:05:41 +000042Inliner::Inliner(void *ID)
43 : CallGraphSCCPass(ID), InlineThreshold(InlineLimit) {}
Chris Lattner237ef562003-08-31 19:10:30 +000044
Dan Gohmanae73dc12008-09-04 17:05:41 +000045Inliner::Inliner(void *ID, int Threshold)
46 : CallGraphSCCPass(ID), InlineThreshold(Threshold) {}
Chris Lattner120d0532008-01-12 06:49:13 +000047
Chris Lattnerff2dad32007-01-30 23:28:39 +000048/// getAnalysisUsage - For this class, we declare that we require and preserve
49/// the call graph. If the derived class implements this method, it should
50/// always explicitly call the implementation here.
51void Inliner::getAnalysisUsage(AnalysisUsage &Info) const {
Chris Lattnerff2dad32007-01-30 23:28:39 +000052 CallGraphSCCPass::getAnalysisUsage(Info);
53}
54
Chris Lattner199ba422009-08-27 06:29:33 +000055
56typedef DenseMap<const ArrayType*, std::vector<AllocaInst*> >
57InlinedArrayAllocasTy;
58
59/// InlineCallIfPossible - If it is possible to inline the specified call site,
60/// do so and update the CallGraph for this operation.
61///
62/// This function also does some basic book-keeping to update the IR. The
Chris Lattnercc0a0292009-08-28 04:48:54 +000063/// InlinedArrayAllocas map keeps track of any allocas that are already
64/// available from other functions inlined into the caller. If we are able to
65/// inline this call site we attempt to reuse already available allocas or add
66/// any new allocas to the set if not possible.
Chris Lattner199ba422009-08-27 06:29:33 +000067static bool InlineCallIfPossible(CallSite CS, CallGraph &CG,
68 const TargetData *TD,
69 InlinedArrayAllocasTy &InlinedArrayAllocas) {
Chris Lattnerbefa4992004-05-23 21:22:17 +000070 Function *Callee = CS.getCalledFunction();
Bill Wendling66c75aa2008-11-21 00:09:21 +000071 Function *Caller = CS.getCaller();
72
Chris Lattner199ba422009-08-27 06:29:33 +000073 // Try to inline the function. Get the list of static allocas that were
74 // inlined.
75 SmallVector<AllocaInst*, 16> StaticAllocas;
76 if (!InlineFunction(CS, &CG, TD, &StaticAllocas))
Chris Lattner12f0bab2009-08-27 04:32:07 +000077 return false;
Misha Brukmanfd939082005-04-21 23:48:37 +000078
Bill Wendling8c1604e2008-11-21 00:06:32 +000079 // If the inlined function had a higher stack protection level than the
80 // calling function, then bump up the caller's stack protection level.
Bill Wendling8c1604e2008-11-21 00:06:32 +000081 if (Callee->hasFnAttr(Attribute::StackProtectReq))
82 Caller->addFnAttr(Attribute::StackProtectReq);
83 else if (Callee->hasFnAttr(Attribute::StackProtect) &&
84 !Caller->hasFnAttr(Attribute::StackProtectReq))
85 Caller->addFnAttr(Attribute::StackProtect);
86
Chris Lattner199ba422009-08-27 06:29:33 +000087
88 // Look at all of the allocas that we inlined through this call site. If we
89 // have already inlined other allocas through other calls into this function,
90 // then we know that they have disjoint lifetimes and that we can merge them.
91 //
92 // There are many heuristics possible for merging these allocas, and the
93 // different options have different tradeoffs. One thing that we *really*
94 // don't want to hurt is SRoA: once inlining happens, often allocas are no
95 // longer address taken and so they can be promoted.
96 //
97 // Our "solution" for that is to only merge allocas whose outermost type is an
98 // array type. These are usually not promoted because someone is using a
99 // variable index into them. These are also often the most important ones to
100 // merge.
101 //
102 // A better solution would be to have real memory lifetime markers in the IR
103 // and not have the inliner do any merging of allocas at all. This would
104 // allow the backend to do proper stack slot coloring of all allocas that
105 // *actually make it to the backend*, which is really what we want.
106 //
107 // Because we don't have this information, we do this simple and useful hack.
108 //
109 SmallPtrSet<AllocaInst*, 16> UsedAllocas;
110
111 // Loop over all the allocas we have so far and see if they can be merged with
112 // a previously inlined alloca. If not, remember that we had it.
113 for (unsigned AllocaNo = 0, e = StaticAllocas.size();
114 AllocaNo != e; ++AllocaNo) {
115 AllocaInst *AI = StaticAllocas[AllocaNo];
116
117 // Don't bother trying to merge array allocations (they will usually be
118 // canonicalized to be an allocation *of* an array), or allocations whose
119 // type is not itself an array (because we're afraid of pessimizing SRoA).
120 const ArrayType *ATy = dyn_cast<ArrayType>(AI->getAllocatedType());
121 if (ATy == 0 || AI->isArrayAllocation())
122 continue;
123
124 // Get the list of all available allocas for this array type.
125 std::vector<AllocaInst*> &AllocasForType = InlinedArrayAllocas[ATy];
126
127 // Loop over the allocas in AllocasForType to see if we can reuse one. Note
128 // that we have to be careful not to reuse the same "available" alloca for
129 // multiple different allocas that we just inlined, we use the 'UsedAllocas'
130 // set to keep track of which "available" allocas are being used by this
131 // function. Also, AllocasForType can be empty of course!
132 bool MergedAwayAlloca = false;
133 for (unsigned i = 0, e = AllocasForType.size(); i != e; ++i) {
134 AllocaInst *AvailableAlloca = AllocasForType[i];
135
136 // The available alloca has to be in the right function, not in some other
137 // function in this SCC.
138 if (AvailableAlloca->getParent() != AI->getParent())
139 continue;
140
141 // If the inlined function already uses this alloca then we can't reuse
142 // it.
143 if (!UsedAllocas.insert(AvailableAlloca))
144 continue;
145
146 // Otherwise, we *can* reuse it, RAUW AI into AvailableAlloca and declare
147 // success!
148 DEBUG(errs() << " ***MERGED ALLOCA: " << *AI);
149
150 AI->replaceAllUsesWith(AvailableAlloca);
151 AI->eraseFromParent();
152 MergedAwayAlloca = true;
153 ++NumMergedAllocas;
154 break;
155 }
Misha Brukmanfd939082005-04-21 23:48:37 +0000156
Chris Lattner199ba422009-08-27 06:29:33 +0000157 // If we already nuked the alloca, we're done with it.
158 if (MergedAwayAlloca)
159 continue;
Misha Brukmanfd939082005-04-21 23:48:37 +0000160
Chris Lattner199ba422009-08-27 06:29:33 +0000161 // If we were unable to merge away the alloca either because there are no
162 // allocas of the right type available or because we reused them all
163 // already, remember that this alloca came from an inlined function and mark
164 // it used so we don't reuse it for other allocas from this inline
165 // operation.
166 AllocasForType.push_back(AI);
167 UsedAllocas.insert(AI);
Chris Lattnerbefa4992004-05-23 21:22:17 +0000168 }
Chris Lattner199ba422009-08-27 06:29:33 +0000169
Chris Lattnerbefa4992004-05-23 21:22:17 +0000170 return true;
Chris Lattner237ef562003-08-31 19:10:30 +0000171}
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000172
173/// shouldInline - Return true if the inliner should attempt to inline
174/// at the given CallSite.
175bool Inliner::shouldInline(CallSite CS) {
Daniel Dunbarc5e1ec42008-10-30 19:26:59 +0000176 InlineCost IC = getInlineCost(CS);
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000177
Daniel Dunbarc5e1ec42008-10-30 19:26:59 +0000178 if (IC.isAlways()) {
Bill Wendling84a832f2009-07-31 19:52:24 +0000179 DEBUG(errs() << " Inlining: cost=always"
180 << ", Call: " << *CS.getInstruction() << "\n");
Daniel Dunbarc5e1ec42008-10-30 19:26:59 +0000181 return true;
182 }
183
184 if (IC.isNever()) {
Bill Wendling84a832f2009-07-31 19:52:24 +0000185 DEBUG(errs() << " NOT Inlining: cost=never"
186 << ", Call: " << *CS.getInstruction() << "\n");
Daniel Dunbarc5e1ec42008-10-30 19:26:59 +0000187 return false;
188 }
189
190 int Cost = IC.getValue();
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000191 int CurrentThreshold = InlineThreshold;
192 Function *Fn = CS.getCaller();
Bill Wendling84a832f2009-07-31 19:52:24 +0000193 if (Fn && !Fn->isDeclaration() &&
194 Fn->hasFnAttr(Attribute::OptimizeForSize) &&
195 InlineThreshold != 50)
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000196 CurrentThreshold = 50;
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000197
Chris Lattner135755d2009-08-27 03:51:50 +0000198 float FudgeFactor = getInlineFudgeFactor(CS);
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000199 if (Cost >= (int)(CurrentThreshold * FudgeFactor)) {
Bill Wendling84a832f2009-07-31 19:52:24 +0000200 DEBUG(errs() << " NOT Inlining: cost=" << Cost
201 << ", Call: " << *CS.getInstruction() << "\n");
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000202 return false;
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000203 }
Chris Lattner135755d2009-08-27 03:51:50 +0000204
205 DEBUG(errs() << " Inlining: cost=" << Cost
206 << ", Call: " << *CS.getInstruction() << "\n");
207 return true;
Daniel Dunbar1a99dbf2008-10-29 01:02:02 +0000208}
Chris Lattner237ef562003-08-31 19:10:30 +0000209
Chris Lattner5095e3d2009-08-31 00:19:58 +0000210bool Inliner::runOnSCC(std::vector<CallGraphNode*> &SCC) {
Chris Lattner237ef562003-08-31 19:10:30 +0000211 CallGraph &CG = getAnalysis<CallGraph>();
Dan Gohman02a436c2009-07-24 18:13:53 +0000212 const TargetData *TD = getAnalysisIfAvailable<TargetData>();
Chris Lattner237ef562003-08-31 19:10:30 +0000213
Dale Johannesen16581bf2009-03-23 23:39:20 +0000214 SmallPtrSet<Function*, 8> SCCFunctions;
Bill Wendling84a832f2009-07-31 19:52:24 +0000215 DEBUG(errs() << "Inliner visiting SCC:");
Chris Lattner237ef562003-08-31 19:10:30 +0000216 for (unsigned i = 0, e = SCC.size(); i != e; ++i) {
Chris Lattnerbefa4992004-05-23 21:22:17 +0000217 Function *F = SCC[i]->getFunction();
218 if (F) SCCFunctions.insert(F);
Daniel Dunbarce63ffb2009-07-25 00:23:56 +0000219 DEBUG(errs() << " " << (F ? F->getName() : "INDIRECTNODE"));
Chris Lattner237ef562003-08-31 19:10:30 +0000220 }
Chris Lattner237ef562003-08-31 19:10:30 +0000221
Chris Lattnerbefa4992004-05-23 21:22:17 +0000222 // Scan through and identify all call sites ahead of time so that we only
223 // inline call sites in the original functions, not call sites that result
224 // from inlining other functions.
Chris Lattner199ba422009-08-27 06:29:33 +0000225 SmallVector<CallSite, 16> CallSites;
Chris Lattnerbefa4992004-05-23 21:22:17 +0000226
Chris Lattner135755d2009-08-27 03:51:50 +0000227 for (unsigned i = 0, e = SCC.size(); i != e; ++i) {
228 Function *F = SCC[i]->getFunction();
229 if (!F) continue;
230
231 for (Function::iterator BB = F->begin(), E = F->end(); BB != E; ++BB)
232 for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I) {
233 CallSite CS = CallSite::get(I);
234 if (CS.getInstruction() == 0 || isa<DbgInfoIntrinsic>(I))
235 continue;
236
237 if (CS.getCalledFunction() == 0 ||
238 !CS.getCalledFunction()->isDeclaration())
239 CallSites.push_back(CS);
240 }
241 }
Chris Lattner237ef562003-08-31 19:10:30 +0000242
Bill Wendling84a832f2009-07-31 19:52:24 +0000243 DEBUG(errs() << ": " << CallSites.size() << " call sites.\n");
Misha Brukmanfd939082005-04-21 23:48:37 +0000244
Chris Lattnerbefa4992004-05-23 21:22:17 +0000245 // Now that we have all of the call sites, move the ones to functions in the
246 // current SCC to the end of the list.
247 unsigned FirstCallInSCC = CallSites.size();
248 for (unsigned i = 0; i < FirstCallInSCC; ++i)
249 if (Function *F = CallSites[i].getCalledFunction())
250 if (SCCFunctions.count(F))
251 std::swap(CallSites[i--], CallSites[--FirstCallInSCC]);
Misha Brukmanfd939082005-04-21 23:48:37 +0000252
Chris Lattner199ba422009-08-27 06:29:33 +0000253
254 InlinedArrayAllocasTy InlinedArrayAllocas;
255
Chris Lattnerbefa4992004-05-23 21:22:17 +0000256 // Now that we have all of the call sites, loop over them and inline them if
257 // it looks profitable to do so.
258 bool Changed = false;
259 bool LocalChange;
260 do {
261 LocalChange = false;
262 // Iterate over the outer loop because inlining functions can cause indirect
263 // calls to become direct calls.
Chris Lattner135755d2009-08-27 03:51:50 +0000264 for (unsigned CSi = 0; CSi != CallSites.size(); ++CSi) {
265 // We can only inline direct calls.
Chris Lattner199ba422009-08-27 06:29:33 +0000266 CallSite CS = CallSites[CSi];
267
268 Function *Callee = CS.getCalledFunction();
Chris Lattner135755d2009-08-27 03:51:50 +0000269 if (!Callee) continue;
270
271 // Calls to external functions are never inlinable.
272 if (Callee->isDeclaration()) {
273 if (SCC.size() == 1) {
274 std::swap(CallSites[CSi], CallSites.back());
275 CallSites.pop_back();
276 } else {
277 // Keep the 'in SCC / not in SCC' boundary correct.
278 CallSites.erase(CallSites.begin()+CSi);
Chris Lattnerbefa4992004-05-23 21:22:17 +0000279 }
Chris Lattner135755d2009-08-27 03:51:50 +0000280 --CSi;
281 continue;
Chris Lattner775cbdd2004-04-08 06:34:31 +0000282 }
Chris Lattner135755d2009-08-27 03:51:50 +0000283
284 // If the policy determines that we should inline this function,
285 // try to do so.
Chris Lattner135755d2009-08-27 03:51:50 +0000286 if (!shouldInline(CS))
287 continue;
288
289 Function *Caller = CS.getCaller();
290 // Attempt to inline the function...
Chris Lattner199ba422009-08-27 06:29:33 +0000291 if (!InlineCallIfPossible(CS, CG, TD, InlinedArrayAllocas))
Chris Lattner135755d2009-08-27 03:51:50 +0000292 continue;
293
Chris Lattner199ba422009-08-27 06:29:33 +0000294 // If we inlined the last possible call site to the function, delete the
295 // function body now.
Chris Lattnerb374b902009-08-31 03:15:49 +0000296 if (Callee->use_empty() && Callee->hasLocalLinkage() &&
297 !SCCFunctions.count(Callee) &&
298 // The function may be apparently dead, but if there are indirect
299 // callgraph references to the node, we cannot delete it yet, this
300 // could invalidate the CGSCC iterator.
301 CG[Callee]->getNumReferences() == 0) {
Chris Lattner199ba422009-08-27 06:29:33 +0000302 DEBUG(errs() << " -> Deleting dead function: "
303 << Callee->getName() << "\n");
304 CallGraphNode *CalleeNode = CG[Callee];
305
306 // Remove any call graph edges from the callee to its callees.
307 CalleeNode->removeAllCalledFunctions();
308
309 resetCachedCostInfo(Callee);
310
311 // Removing the node for callee from the call graph and delete it.
312 delete CG.removeFunctionFromModule(CalleeNode);
313 ++NumDeleted;
314 }
315
Chris Lattner135755d2009-08-27 03:51:50 +0000316 // Remove any cached cost info for this caller, as inlining the
317 // callee has increased the size of the caller (which may be the
318 // same as the callee).
319 resetCachedCostInfo(Caller);
320
321 // Remove this call site from the list. If possible, use
322 // swap/pop_back for efficiency, but do not use it if doing so would
323 // move a call site to a function in this SCC before the
324 // 'FirstCallInSCC' barrier.
325 if (SCC.size() == 1) {
326 std::swap(CallSites[CSi], CallSites.back());
327 CallSites.pop_back();
328 } else {
329 CallSites.erase(CallSites.begin()+CSi);
330 }
331 --CSi;
332
333 ++NumInlined;
334 Changed = true;
335 LocalChange = true;
336 }
Chris Lattnerbefa4992004-05-23 21:22:17 +0000337 } while (LocalChange);
Chris Lattner775cbdd2004-04-08 06:34:31 +0000338
Chris Lattner237ef562003-08-31 19:10:30 +0000339 return Changed;
340}
341
Chris Lattner68d57e72004-04-20 22:06:53 +0000342// doFinalization - Remove now-dead linkonce functions at the end of
343// processing to avoid breaking the SCC traversal.
344bool Inliner::doFinalization(CallGraph &CG) {
Devang Patelb7c6bf12008-11-05 01:39:16 +0000345 return removeDeadFunctions(CG);
346}
347
Chris Lattner135755d2009-08-27 03:51:50 +0000348/// removeDeadFunctions - Remove dead functions that are not included in
349/// DNR (Do Not Remove) list.
Devang Patelb7c6bf12008-11-05 01:39:16 +0000350bool Inliner::removeDeadFunctions(CallGraph &CG,
Chris Lattner135755d2009-08-27 03:51:50 +0000351 SmallPtrSet<const Function *, 16> *DNR) {
352 SmallPtrSet<CallGraphNode*, 16> FunctionsToRemove;
Chris Lattner3e1358a2004-04-21 20:44:33 +0000353
354 // Scan for all of the functions, looking for ones that should now be removed
355 // from the program. Insert the dead ones in the FunctionsToRemove set.
356 for (CallGraph::iterator I = CG.begin(), E = CG.end(); I != E; ++I) {
357 CallGraphNode *CGN = I->second;
Chris Lattnerb374b902009-08-31 03:15:49 +0000358 if (CGN->getFunction() == 0)
Chris Lattner135755d2009-08-27 03:51:50 +0000359 continue;
360
361 Function *F = CGN->getFunction();
362
363 // If the only remaining users of the function are dead constants, remove
364 // them.
365 F->removeDeadConstantUsers();
Chris Lattner3e1358a2004-04-21 20:44:33 +0000366
Chris Lattner135755d2009-08-27 03:51:50 +0000367 if (DNR && DNR->count(F))
368 continue;
Chris Lattnerb374b902009-08-31 03:15:49 +0000369 if (!F->hasLinkOnceLinkage() && !F->hasLocalLinkage() &&
370 !F->hasAvailableExternallyLinkage())
Chris Lattner135755d2009-08-27 03:51:50 +0000371 continue;
372 if (!F->use_empty())
373 continue;
374
375 // Remove any call graph edges from the function to its callees.
376 CGN->removeAllCalledFunctions();
Devang Patelb7c6bf12008-11-05 01:39:16 +0000377
Chris Lattner135755d2009-08-27 03:51:50 +0000378 // Remove any edges from the external node to the function's call graph
379 // node. These edges might have been made irrelegant due to
380 // optimization of the program.
381 CG.getExternalCallingNode()->removeAnyCallEdgeTo(CGN);
Chris Lattner0c0aa712004-09-18 21:37:03 +0000382
Chris Lattner135755d2009-08-27 03:51:50 +0000383 // Removing the node for callee from the call graph and delete it.
384 FunctionsToRemove.insert(CGN);
Chris Lattner68d57e72004-04-20 22:06:53 +0000385 }
Chris Lattner3e1358a2004-04-21 20:44:33 +0000386
387 // Now that we know which functions to delete, do so. We didn't want to do
388 // this inline, because that would invalidate our CallGraph::iterator
389 // objects. :(
Chris Lattner135755d2009-08-27 03:51:50 +0000390 //
391 // Note that it doesn't matter that we are iterating over a non-stable set
392 // here to do this, it doesn't matter which order the functions are deleted
393 // in.
Chris Lattner3e1358a2004-04-21 20:44:33 +0000394 bool Changed = false;
Chris Lattner135755d2009-08-27 03:51:50 +0000395 for (SmallPtrSet<CallGraphNode*, 16>::iterator I = FunctionsToRemove.begin(),
396 E = FunctionsToRemove.end(); I != E; ++I) {
Dale Johannesen1f67ce42009-03-19 18:03:56 +0000397 resetCachedCostInfo((*I)->getFunction());
Chris Lattner3e1358a2004-04-21 20:44:33 +0000398 delete CG.removeFunctionFromModule(*I);
399 ++NumDeleted;
400 Changed = true;
401 }
402
Chris Lattner68d57e72004-04-20 22:06:53 +0000403 return Changed;
404}