blob: 6f1eea0276730e0685b7446d92a16552abc30232 [file] [log] [blame]
Chris Lattner22ee3eb2002-05-24 20:42:13 +00001//===- FunctionResolution.cpp - Resolve declarations to implementations ---===//
Misha Brukmanfd939082005-04-21 23:48:37 +00002//
John Criswellb576c942003-10-20 19:43:21 +00003// The LLVM Compiler Infrastructure
4//
5// This file was developed by the LLVM research group and is distributed under
6// the University of Illinois Open Source License. See LICENSE.TXT for details.
Misha Brukmanfd939082005-04-21 23:48:37 +00007//
John Criswellb576c942003-10-20 19:43:21 +00008//===----------------------------------------------------------------------===//
Chris Lattner22ee3eb2002-05-24 20:42:13 +00009//
10// Loop over the functions that are in the module and look for functions that
11// have the same name. More often than not, there will be things like:
12//
13// declare void %foo(...)
14// void %foo(int, int) { ... }
15//
16// because of the way things are declared in C. If this is the case, patch
17// things up.
18//
19//===----------------------------------------------------------------------===//
20
Chris Lattnerff1be262002-07-23 22:04:02 +000021#include "llvm/Transforms/IPO.h"
Chris Lattner22ee3eb2002-05-24 20:42:13 +000022#include "llvm/Module.h"
Chris Lattner22ee3eb2002-05-24 20:42:13 +000023#include "llvm/DerivedTypes.h"
24#include "llvm/Pass.h"
Misha Brukman47b14a42004-07-29 17:30:56 +000025#include "llvm/Instructions.h"
Chris Lattnera45ec542002-10-09 21:10:06 +000026#include "llvm/Constants.h"
Chris Lattner03fb8b22003-11-20 21:21:31 +000027#include "llvm/Support/CallSite.h"
Chris Lattnerefd47ba2003-10-22 03:35:34 +000028#include "llvm/Target/TargetData.h"
Chris Lattner250d91b2003-08-13 22:15:04 +000029#include "llvm/Assembly/Writer.h"
Reid Spencer551ccae2004-09-01 22:55:40 +000030#include "llvm/ADT/Statistic.h"
Chris Lattner22ee3eb2002-05-24 20:42:13 +000031#include <algorithm>
Chris Lattner03fb8b22003-11-20 21:21:31 +000032using namespace llvm;
Brian Gaeked0fde302003-11-11 22:41:34 +000033
Chris Lattner22ee3eb2002-05-24 20:42:13 +000034namespace {
Chris Lattnerac0b6ae2006-12-06 17:46:33 +000035 Statistic NumResolved("funcresolve", "Number of varargs functions resolved");
36 Statistic NumGlobals("funcresolve", "Number of global variables resolved");
Chris Lattner22ee3eb2002-05-24 20:42:13 +000037
Chris Lattnerb12914b2004-09-20 04:48:05 +000038 struct FunctionResolvingPass : public ModulePass {
Chris Lattnerefd47ba2003-10-22 03:35:34 +000039 virtual void getAnalysisUsage(AnalysisUsage &AU) const {
40 AU.addRequired<TargetData>();
41 }
42
Chris Lattnerb12914b2004-09-20 04:48:05 +000043 bool runOnModule(Module &M);
Chris Lattner22ee3eb2002-05-24 20:42:13 +000044 };
Chris Lattner7f8897f2006-08-27 22:42:52 +000045 RegisterPass<FunctionResolvingPass> X("funcresolve", "Resolve Functions");
Chris Lattner22ee3eb2002-05-24 20:42:13 +000046}
47
Chris Lattnerb12914b2004-09-20 04:48:05 +000048ModulePass *llvm::createFunctionResolvingPass() {
Chris Lattner22ee3eb2002-05-24 20:42:13 +000049 return new FunctionResolvingPass();
50}
51
Chris Lattnerda902ba2003-01-30 18:22:32 +000052static bool ResolveFunctions(Module &M, std::vector<GlobalValue*> &Globals,
Chris Lattnera45ec542002-10-09 21:10:06 +000053 Function *Concrete) {
54 bool Changed = false;
55 for (unsigned i = 0; i != Globals.size(); ++i)
56 if (Globals[i] != Concrete) {
57 Function *Old = cast<Function>(Globals[i]);
Reid Spencer2f189072005-12-13 19:56:51 +000058 const FunctionType *OldFT = Old->getFunctionType();
59 const FunctionType *ConcreteFT = Concrete->getFunctionType();
Misha Brukmanfd939082005-04-21 23:48:37 +000060
Reid Spencer2f189072005-12-13 19:56:51 +000061 if (OldFT->getNumParams() > ConcreteFT->getNumParams() &&
62 !ConcreteFT->isVarArg())
Chris Lattner4e2fd752003-02-27 20:55:48 +000063 if (!Old->use_empty()) {
Bill Wendling832171c2006-12-07 20:04:42 +000064 cerr << "WARNING: Linking function '" << Old->getName()
65 << "' is causing arguments to be dropped.\n";
66 cerr << "WARNING: Prototype: ";
Bill Wendlingf5da1332006-12-07 22:21:48 +000067 WriteAsOperand(*cerr.stream(), Old);
Bill Wendling832171c2006-12-07 20:04:42 +000068 cerr << " resolved to ";
Bill Wendlingf5da1332006-12-07 22:21:48 +000069 WriteAsOperand(*cerr.stream(), Concrete);
Bill Wendling832171c2006-12-07 20:04:42 +000070 cerr << "\n";
Chris Lattner4e2fd752003-02-27 20:55:48 +000071 }
Misha Brukmanfd939082005-04-21 23:48:37 +000072
Chris Lattnera45ec542002-10-09 21:10:06 +000073 // Check to make sure that if there are specified types, that they
74 // match...
75 //
Reid Spencer2f189072005-12-13 19:56:51 +000076 unsigned NumArguments = std::min(OldFT->getNumParams(),
77 ConcreteFT->getNumParams());
Chris Lattner4e2fd752003-02-27 20:55:48 +000078
Chris Lattnerdde601d2003-03-03 19:57:46 +000079 if (!Old->use_empty() && !Concrete->use_empty())
80 for (unsigned i = 0; i < NumArguments; ++i)
Reid Spencer2f189072005-12-13 19:56:51 +000081 if (OldFT->getParamType(i) != ConcreteFT->getParamType(i))
82 if (OldFT->getParamType(i)->getTypeID() !=
83 ConcreteFT->getParamType(i)->getTypeID()) {
Bill Wendling832171c2006-12-07 20:04:42 +000084 cerr << "WARNING: Function [" << Old->getName()
85 << "]: Parameter types conflict for: '";
Bill Wendlingf5da1332006-12-07 22:21:48 +000086 WriteTypeSymbolic(*cerr.stream(), OldFT, &M);
Bill Wendling832171c2006-12-07 20:04:42 +000087 cerr << "' (in "
88 << Old->getParent()->getModuleIdentifier() << ") and '";
Bill Wendlingf5da1332006-12-07 22:21:48 +000089 WriteTypeSymbolic(*cerr.stream(), ConcreteFT, &M);
Bill Wendling832171c2006-12-07 20:04:42 +000090 cerr << "'(in "
91 << Concrete->getParent()->getModuleIdentifier() << ")\n";
Chris Lattnerb29170f2003-08-20 23:50:38 +000092 return Changed;
Chris Lattnera0f85e52003-08-23 20:03:05 +000093 }
Misha Brukmanfd939082005-04-21 23:48:37 +000094
Chris Lattner1fd95af2003-04-28 01:23:29 +000095 // Attempt to convert all of the uses of the old function to the concrete
96 // form of the function. If there is a use of the fn that we don't
97 // understand here we punt to avoid making a bad transformation.
Chris Lattnera45ec542002-10-09 21:10:06 +000098 //
Chris Lattner1fd95af2003-04-28 01:23:29 +000099 // At this point, we know that the return values are the same for our two
100 // functions and that the Old function has no varargs fns specified. In
101 // otherwords it's just <retty> (...)
Chris Lattnera45ec542002-10-09 21:10:06 +0000102 //
Chris Lattnerd514d822005-02-01 01:23:31 +0000103 if (!Old->use_empty()) {
Chris Lattner1078d112003-07-23 22:03:18 +0000104 Value *Replacement = Concrete;
105 if (Concrete->getType() != Old->getType())
Reid Spencer4da49122006-12-12 05:05:00 +0000106 Replacement = ConstantExpr::getBitCast(Concrete, Old->getType());
Chris Lattnerd514d822005-02-01 01:23:31 +0000107 NumResolved += Old->getNumUses();
Chris Lattner1078d112003-07-23 22:03:18 +0000108 Old->replaceAllUsesWith(Replacement);
109 }
Chris Lattner12ce59d2003-05-31 21:08:45 +0000110
111 // Since there are no uses of Old anymore, remove it from the module.
112 M.getFunctionList().erase(Old);
Chris Lattnera45ec542002-10-09 21:10:06 +0000113 }
114 return Changed;
115}
116
117
Chris Lattnerda902ba2003-01-30 18:22:32 +0000118static bool ResolveGlobalVariables(Module &M,
119 std::vector<GlobalValue*> &Globals,
Chris Lattnera45ec542002-10-09 21:10:06 +0000120 GlobalVariable *Concrete) {
121 bool Changed = false;
Chris Lattnerea2294a2003-04-19 00:15:27 +0000122
Chris Lattnera45ec542002-10-09 21:10:06 +0000123 for (unsigned i = 0; i != Globals.size(); ++i)
124 if (Globals[i] != Concrete) {
Reid Spencer4da49122006-12-12 05:05:00 +0000125 Constant *Cast = ConstantExpr::getBitCast(Concrete,Globals[i]->getType());
Chris Lattner5858e1e2003-10-21 23:17:56 +0000126 Globals[i]->replaceAllUsesWith(Cast);
Chris Lattnerea2294a2003-04-19 00:15:27 +0000127
Chris Lattnera45ec542002-10-09 21:10:06 +0000128 // Since there are no uses of Old anymore, remove it from the module.
Chris Lattner5858e1e2003-10-21 23:17:56 +0000129 M.getGlobalList().erase(cast<GlobalVariable>(Globals[i]));
Chris Lattnera45ec542002-10-09 21:10:06 +0000130
131 ++NumGlobals;
132 Changed = true;
133 }
134 return Changed;
135}
136
Chris Lattner03fb8b22003-11-20 21:21:31 +0000137// Check to see if all of the callers of F ignore the return value.
138static bool CallersAllIgnoreReturnValue(Function &F) {
139 if (F.getReturnType() == Type::VoidTy) return true;
140 for (Value::use_iterator I = F.use_begin(), E = F.use_end(); I != E; ++I) {
Reid Spencer518310c2004-07-18 00:44:37 +0000141 if (GlobalValue *GV = dyn_cast<GlobalValue>(*I)) {
142 for (Value::use_iterator I = GV->use_begin(), E = GV->use_end();
Chris Lattner03fb8b22003-11-20 21:21:31 +0000143 I != E; ++I) {
144 CallSite CS = CallSite::get(*I);
145 if (!CS.getInstruction() || !CS.getInstruction()->use_empty())
146 return false;
147 }
148 } else {
149 CallSite CS = CallSite::get(*I);
150 if (!CS.getInstruction() || !CS.getInstruction()->use_empty())
151 return false;
152 }
153 }
154 return true;
155}
156
Chris Lattnerefd47ba2003-10-22 03:35:34 +0000157static bool ProcessGlobalsWithSameName(Module &M, TargetData &TD,
Chris Lattnerda902ba2003-01-30 18:22:32 +0000158 std::vector<GlobalValue*> &Globals) {
Chris Lattnera45ec542002-10-09 21:10:06 +0000159 assert(!Globals.empty() && "Globals list shouldn't be empty here!");
160
161 bool isFunction = isa<Function>(Globals[0]); // Is this group all functions?
Chris Lattnera45ec542002-10-09 21:10:06 +0000162 GlobalValue *Concrete = 0; // The most concrete implementation to resolve to
163
Chris Lattnera45ec542002-10-09 21:10:06 +0000164 for (unsigned i = 0; i != Globals.size(); ) {
165 if (isa<Function>(Globals[i]) != isFunction) {
Bill Wendling832171c2006-12-07 20:04:42 +0000166 cerr << "WARNING: Found function and global variable with the "
167 << "same name: '" << Globals[i]->getName() << "'.\n";
Chris Lattnera45ec542002-10-09 21:10:06 +0000168 return false; // Don't know how to handle this, bail out!
169 }
170
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000171 if (isFunction) {
Chris Lattnera45ec542002-10-09 21:10:06 +0000172 // For functions, we look to merge functions definitions of "int (...)"
173 // to 'int (int)' or 'int ()' or whatever else is not completely generic.
174 //
175 Function *F = cast<Function>(Globals[i]);
Chris Lattner6f239632002-11-08 00:38:20 +0000176 if (!F->isExternal()) {
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000177 if (Concrete && !Concrete->isExternal())
Chris Lattnera45ec542002-10-09 21:10:06 +0000178 return false; // Found two different functions types. Can't choose!
Misha Brukmanfd939082005-04-21 23:48:37 +0000179
Chris Lattnera45ec542002-10-09 21:10:06 +0000180 Concrete = Globals[i];
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000181 } else if (Concrete) {
Chris Lattner03fb8b22003-11-20 21:21:31 +0000182 if (Concrete->isExternal()) // If we have multiple external symbols...
Misha Brukmanfd939082005-04-21 23:48:37 +0000183 if (F->getFunctionType()->getNumParams() >
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000184 cast<Function>(Concrete)->getFunctionType()->getNumParams())
185 Concrete = F; // We are more concrete than "Concrete"!
186
187 } else {
188 Concrete = F;
Chris Lattnera45ec542002-10-09 21:10:06 +0000189 }
Chris Lattnera45ec542002-10-09 21:10:06 +0000190 } else {
Chris Lattnera45ec542002-10-09 21:10:06 +0000191 GlobalVariable *GV = cast<GlobalVariable>(Globals[i]);
Chris Lattner5858e1e2003-10-21 23:17:56 +0000192 if (!GV->isExternal()) {
193 if (Concrete) {
Bill Wendling832171c2006-12-07 20:04:42 +0000194 cerr << "WARNING: Two global variables with external linkage"
195 << " exist with the same name: '" << GV->getName()
196 << "'!\n";
Chris Lattner5858e1e2003-10-21 23:17:56 +0000197 return false;
Chris Lattnera45ec542002-10-09 21:10:06 +0000198 }
Chris Lattner5858e1e2003-10-21 23:17:56 +0000199 Concrete = GV;
Chris Lattnera45ec542002-10-09 21:10:06 +0000200 }
Chris Lattnera45ec542002-10-09 21:10:06 +0000201 }
Chris Lattnerea2294a2003-04-19 00:15:27 +0000202 ++i;
Chris Lattnera45ec542002-10-09 21:10:06 +0000203 }
204
205 if (Globals.size() > 1) { // Found a multiply defined global...
Chris Lattnerc16e6312003-05-31 21:57:06 +0000206 // If there are no external declarations, and there is at most one
207 // externally visible instance of the global, then there is nothing to do.
208 //
209 bool HasExternal = false;
210 unsigned NumInstancesWithExternalLinkage = 0;
211
212 for (unsigned i = 0, e = Globals.size(); i != e; ++i) {
213 if (Globals[i]->isExternal())
214 HasExternal = true;
215 else if (!Globals[i]->hasInternalLinkage())
216 NumInstancesWithExternalLinkage++;
217 }
Misha Brukmanfd939082005-04-21 23:48:37 +0000218
Chris Lattnerc16e6312003-05-31 21:57:06 +0000219 if (!HasExternal && NumInstancesWithExternalLinkage <= 1)
220 return false; // Nothing to do? Must have multiple internal definitions.
221
Chris Lattnerce94319552003-10-22 23:03:38 +0000222 // There are a couple of special cases we don't want to print the warning
223 // for, check them now.
224 bool DontPrintWarning = false;
225 if (Concrete && Globals.size() == 2) {
226 GlobalValue *Other = Globals[Globals[0] == Concrete];
227 // If the non-concrete global is a function which takes (...) arguments,
Chris Lattner03fb8b22003-11-20 21:21:31 +0000228 // and the return values match (or was never used), do not warn.
Chris Lattnerce94319552003-10-22 23:03:38 +0000229 if (Function *ConcreteF = dyn_cast<Function>(Concrete))
230 if (Function *OtherF = dyn_cast<Function>(Other))
Chris Lattner03fb8b22003-11-20 21:21:31 +0000231 if ((ConcreteF->getReturnType() == OtherF->getReturnType() ||
232 CallersAllIgnoreReturnValue(*OtherF)) &&
Chris Lattnerce94319552003-10-22 23:03:38 +0000233 OtherF->getFunctionType()->isVarArg() &&
Chris Lattnerd5d89962004-02-09 04:14:01 +0000234 OtherF->getFunctionType()->getNumParams() == 0)
Chris Lattnerce94319552003-10-22 23:03:38 +0000235 DontPrintWarning = true;
Misha Brukmanfd939082005-04-21 23:48:37 +0000236
Chris Lattnerce94319552003-10-22 23:03:38 +0000237 // Otherwise, if the non-concrete global is a global array variable with a
238 // size of 0, and the concrete global is an array with a real size, don't
239 // warn. This occurs due to declaring 'extern int A[];'.
240 if (GlobalVariable *ConcreteGV = dyn_cast<GlobalVariable>(Concrete))
Chris Lattner4e4c4442004-08-20 00:30:39 +0000241 if (GlobalVariable *OtherGV = dyn_cast<GlobalVariable>(Other)) {
242 const Type *CTy = ConcreteGV->getType();
243 const Type *OTy = OtherGV->getType();
244
245 if (CTy->isSized())
246 if (!OTy->isSized() || !TD.getTypeSize(OTy) ||
247 TD.getTypeSize(OTy) == TD.getTypeSize(CTy))
248 DontPrintWarning = true;
249 }
Chris Lattnerce94319552003-10-22 23:03:38 +0000250 }
Chris Lattnerc16e6312003-05-31 21:57:06 +0000251
Chris Lattner23367a72004-09-30 00:12:29 +0000252 if (0 && !DontPrintWarning) {
Bill Wendling832171c2006-12-07 20:04:42 +0000253 cerr << "WARNING: Found global types that are not compatible:\n";
Chris Lattnerce94319552003-10-22 23:03:38 +0000254 for (unsigned i = 0; i < Globals.size(); ++i) {
Bill Wendling832171c2006-12-07 20:04:42 +0000255 cerr << "\t";
Bill Wendlingf5da1332006-12-07 22:21:48 +0000256 WriteTypeSymbolic(*cerr.stream(), Globals[i]->getType(), &M);
Bill Wendling832171c2006-12-07 20:04:42 +0000257 cerr << " %" << Globals[i]->getName() << "\n";
Chris Lattnerce94319552003-10-22 23:03:38 +0000258 }
Chris Lattnera45ec542002-10-09 21:10:06 +0000259 }
260
Chris Lattner5858e1e2003-10-21 23:17:56 +0000261 if (!Concrete)
262 Concrete = Globals[0];
Chris Lattnerefd47ba2003-10-22 03:35:34 +0000263 else if (GlobalVariable *GV = dyn_cast<GlobalVariable>(Concrete)) {
264 // Handle special case hack to change globals if it will make their types
265 // happier in the long run. The situation we do this is intentionally
266 // extremely limited.
267 if (GV->use_empty() && GV->hasInitializer() &&
268 GV->getInitializer()->isNullValue()) {
269 // Check to see if there is another (external) global with the same size
270 // and a non-empty use-list. If so, we will make IT be the real
271 // implementation.
272 unsigned TS = TD.getTypeSize(Concrete->getType()->getElementType());
273 for (unsigned i = 0, e = Globals.size(); i != e; ++i)
274 if (Globals[i] != Concrete && !Globals[i]->use_empty() &&
275 isa<GlobalVariable>(Globals[i]) &&
276 TD.getTypeSize(Globals[i]->getType()->getElementType()) == TS) {
277 // At this point we want to replace Concrete with Globals[i]. Make
278 // concrete external, and Globals[i] have an initializer.
279 GlobalVariable *NGV = cast<GlobalVariable>(Globals[i]);
280 const Type *ElTy = NGV->getType()->getElementType();
281 NGV->setInitializer(Constant::getNullValue(ElTy));
282 cast<GlobalVariable>(Concrete)->setInitializer(0);
283 Concrete = NGV;
284 break;
285 }
286 }
287 }
Chris Lattner5858e1e2003-10-21 23:17:56 +0000288
Chris Lattnera45ec542002-10-09 21:10:06 +0000289 if (isFunction)
Chris Lattnerea2294a2003-04-19 00:15:27 +0000290 return ResolveFunctions(M, Globals, cast<Function>(Concrete));
Chris Lattnera45ec542002-10-09 21:10:06 +0000291 else
Chris Lattnerea2294a2003-04-19 00:15:27 +0000292 return ResolveGlobalVariables(M, Globals,
293 cast<GlobalVariable>(Concrete));
Chris Lattnera45ec542002-10-09 21:10:06 +0000294 }
Chris Lattnerea2294a2003-04-19 00:15:27 +0000295 return false;
Chris Lattnera45ec542002-10-09 21:10:06 +0000296}
297
Chris Lattnerb12914b2004-09-20 04:48:05 +0000298bool FunctionResolvingPass::runOnModule(Module &M) {
Chris Lattnerda902ba2003-01-30 18:22:32 +0000299 std::map<std::string, std::vector<GlobalValue*> > Globals;
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000300
Chris Lattner4cb766a2003-10-22 04:42:20 +0000301 // Loop over the globals, adding them to the Globals map. We use a two pass
302 // algorithm here to avoid problems with iterators getting invalidated if we
303 // did a one pass scheme.
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000304 //
Chris Lattner6be5e562003-10-22 04:43:18 +0000305 bool Changed = false;
Chris Lattner4cb766a2003-10-22 04:42:20 +0000306 for (Module::iterator I = M.begin(), E = M.end(); I != E; ) {
307 Function *F = I++;
Chris Lattner6be5e562003-10-22 04:43:18 +0000308 if (F->use_empty() && F->isExternal()) {
Chris Lattner4cb766a2003-10-22 04:42:20 +0000309 M.getFunctionList().erase(F);
Chris Lattner6be5e562003-10-22 04:43:18 +0000310 Changed = true;
Chris Lattnere5ad50b2004-06-18 05:50:48 +0000311 } else if (!F->hasInternalLinkage() && !F->getName().empty() &&
312 !F->getIntrinsicID())
Chris Lattner4cb766a2003-10-22 04:42:20 +0000313 Globals[F->getName()].push_back(F);
314 }
315
Chris Lattner7f8897f2006-08-27 22:42:52 +0000316 for (Module::global_iterator I = M.global_begin(), E = M.global_end();
317 I != E; ) {
Chris Lattner4cb766a2003-10-22 04:42:20 +0000318 GlobalVariable *GV = I++;
Chris Lattner6be5e562003-10-22 04:43:18 +0000319 if (GV->use_empty() && GV->isExternal()) {
Chris Lattner4cb766a2003-10-22 04:42:20 +0000320 M.getGlobalList().erase(GV);
Chris Lattner6be5e562003-10-22 04:43:18 +0000321 Changed = true;
322 } else if (!GV->hasInternalLinkage() && !GV->getName().empty())
Chris Lattner4cb766a2003-10-22 04:42:20 +0000323 Globals[GV->getName()].push_back(GV);
324 }
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000325
Chris Lattnerefd47ba2003-10-22 03:35:34 +0000326 TargetData &TD = getAnalysis<TargetData>();
327
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000328 // Now we have a list of all functions with a particular name. If there is
329 // more than one entry in a list, merge the functions together.
330 //
Chris Lattnerda902ba2003-01-30 18:22:32 +0000331 for (std::map<std::string, std::vector<GlobalValue*> >::iterator
332 I = Globals.begin(), E = Globals.end(); I != E; ++I)
Chris Lattnerefd47ba2003-10-22 03:35:34 +0000333 Changed |= ProcessGlobalsWithSameName(M, TD, I->second);
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000334
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000335 // Now loop over all of the globals, checking to see if any are trivially
336 // dead. If so, remove them now.
337
338 for (Module::iterator I = M.begin(), E = M.end(); I != E; )
339 if (I->isExternal() && I->use_empty()) {
340 Function *F = I;
341 ++I;
342 M.getFunctionList().erase(F);
343 ++NumResolved;
344 Changed = true;
345 } else {
346 ++I;
347 }
348
Chris Lattner7f8897f2006-08-27 22:42:52 +0000349 for (Module::global_iterator I = M.global_begin(), E = M.global_end();
350 I != E; )
Chris Lattnera2b8d7b2002-11-10 03:36:55 +0000351 if (I->isExternal() && I->use_empty()) {
352 GlobalVariable *GV = I;
353 ++I;
354 M.getGlobalList().erase(GV);
355 ++NumGlobals;
356 Changed = true;
357 } else {
358 ++I;
359 }
360
Chris Lattner22ee3eb2002-05-24 20:42:13 +0000361 return Changed;
362}