Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 1 | //===- FunctionResolution.cpp - Resolve declarations to implementations ---===// |
| 2 | // |
| 3 | // Loop over the functions that are in the module and look for functions that |
| 4 | // have the same name. More often than not, there will be things like: |
| 5 | // |
| 6 | // declare void %foo(...) |
| 7 | // void %foo(int, int) { ... } |
| 8 | // |
| 9 | // because of the way things are declared in C. If this is the case, patch |
| 10 | // things up. |
| 11 | // |
| 12 | //===----------------------------------------------------------------------===// |
| 13 | |
Chris Lattner | 5afe2f2 | 2002-07-23 22:04:02 +0000 | [diff] [blame] | 14 | #include "llvm/Transforms/IPO.h" |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 15 | #include "llvm/Module.h" |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 16 | #include "llvm/SymbolTable.h" |
| 17 | #include "llvm/DerivedTypes.h" |
| 18 | #include "llvm/Pass.h" |
| 19 | #include "llvm/iOther.h" |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 20 | #include "llvm/Constants.h" |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 21 | #include "llvm/Assembly/Writer.h" // FIXME: remove when varargs implemented |
Chris Lattner | bf3a099 | 2002-10-01 22:38:41 +0000 | [diff] [blame] | 22 | #include "Support/Statistic.h" |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 23 | #include <algorithm> |
| 24 | |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 25 | namespace { |
Chris Lattner | bf3a099 | 2002-10-01 22:38:41 +0000 | [diff] [blame] | 26 | Statistic<>NumResolved("funcresolve", "Number of varargs functions resolved"); |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 27 | Statistic<> NumGlobals("funcresolve", "Number of global variables resolved"); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 28 | |
| 29 | struct FunctionResolvingPass : public Pass { |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 30 | bool run(Module &M); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 31 | }; |
Chris Lattner | a2c0985 | 2002-07-26 21:12:44 +0000 | [diff] [blame] | 32 | RegisterOpt<FunctionResolvingPass> X("funcresolve", "Resolve Functions"); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 33 | } |
| 34 | |
| 35 | Pass *createFunctionResolvingPass() { |
| 36 | return new FunctionResolvingPass(); |
| 37 | } |
| 38 | |
| 39 | // ConvertCallTo - Convert a call to a varargs function with no arg types |
| 40 | // specified to a concrete nonvarargs function. |
| 41 | // |
| 42 | static void ConvertCallTo(CallInst *CI, Function *Dest) { |
| 43 | const FunctionType::ParamTypes &ParamTys = |
| 44 | Dest->getFunctionType()->getParamTypes(); |
| 45 | BasicBlock *BB = CI->getParent(); |
| 46 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 47 | // Keep an iterator to where we want to insert cast instructions if the |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 48 | // argument types don't agree. |
| 49 | // |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 50 | BasicBlock::iterator BBI = CI; |
Chris Lattner | 68f63f7 | 2003-01-30 22:38:44 +0000 | [diff] [blame] | 51 | unsigned NumArgsToCopy = CI->getNumOperands()-1; |
Chris Lattner | 04c9274 | 2003-02-14 19:12:29 +0000 | [diff] [blame] | 52 | if (NumArgsToCopy != ParamTys.size() && |
| 53 | !(NumArgsToCopy > ParamTys.size() && |
Chris Lattner | 68f63f7 | 2003-01-30 22:38:44 +0000 | [diff] [blame] | 54 | Dest->getFunctionType()->isVarArg())) { |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 55 | std::cerr << "WARNING: Call arguments do not match expected number of" |
| 56 | << " parameters.\n"; |
| 57 | std::cerr << "WARNING: In function '" |
| 58 | << CI->getParent()->getParent()->getName() << "': call: " << *CI; |
| 59 | std::cerr << "Function resolved to: "; |
| 60 | WriteAsOperand(std::cerr, Dest); |
| 61 | std::cerr << "\n"; |
Chris Lattner | 04c9274 | 2003-02-14 19:12:29 +0000 | [diff] [blame] | 62 | if (NumArgsToCopy > ParamTys.size()) |
| 63 | NumArgsToCopy = ParamTys.size(); |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 64 | } |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 65 | |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 66 | std::vector<Value*> Params; |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 67 | |
| 68 | // Convert all of the call arguments over... inserting cast instructions if |
| 69 | // the types are not compatible. |
Chris Lattner | 68f63f7 | 2003-01-30 22:38:44 +0000 | [diff] [blame] | 70 | for (unsigned i = 1; i <= NumArgsToCopy; ++i) { |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 71 | Value *V = CI->getOperand(i); |
| 72 | |
Chris Lattner | 68f63f7 | 2003-01-30 22:38:44 +0000 | [diff] [blame] | 73 | if (i-1 < ParamTys.size() && V->getType() != ParamTys[i-1]) { |
| 74 | // Must insert a cast... |
Chris Lattner | 5c44786 | 2002-09-10 17:03:06 +0000 | [diff] [blame] | 75 | V = new CastInst(V, ParamTys[i-1], "argcast", BBI); |
Chris Lattner | 68f63f7 | 2003-01-30 22:38:44 +0000 | [diff] [blame] | 76 | } |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 77 | |
| 78 | Params.push_back(V); |
| 79 | } |
| 80 | |
| 81 | // Replace the old call instruction with a new call instruction that calls |
| 82 | // the real function. |
| 83 | // |
Chris Lattner | 5c44786 | 2002-09-10 17:03:06 +0000 | [diff] [blame] | 84 | Instruction *NewCall = new CallInst(Dest, Params, "", BBI); |
Chris Lattner | 1eb9e6c | 2002-05-24 21:33:26 +0000 | [diff] [blame] | 85 | |
| 86 | // Remove the old call instruction from the program... |
| 87 | BB->getInstList().remove(BBI); |
| 88 | |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 89 | // Transfer the name over... |
Chris Lattner | ed37856 | 2002-07-30 02:42:49 +0000 | [diff] [blame] | 90 | if (NewCall->getType() != Type::VoidTy) |
| 91 | NewCall->setName(CI->getName()); |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 92 | |
Chris Lattner | 1eb9e6c | 2002-05-24 21:33:26 +0000 | [diff] [blame] | 93 | // Replace uses of the old instruction with the appropriate values... |
| 94 | // |
| 95 | if (NewCall->getType() == CI->getType()) { |
| 96 | CI->replaceAllUsesWith(NewCall); |
| 97 | NewCall->setName(CI->getName()); |
| 98 | |
| 99 | } else if (NewCall->getType() == Type::VoidTy) { |
| 100 | // Resolved function does not return a value but the prototype does. This |
| 101 | // often occurs because undefined functions default to returning integers. |
| 102 | // Just replace uses of the call (which are broken anyway) with dummy |
| 103 | // values. |
| 104 | CI->replaceAllUsesWith(Constant::getNullValue(CI->getType())); |
| 105 | } else if (CI->getType() == Type::VoidTy) { |
| 106 | // If we are gaining a new return value, we don't have to do anything |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 107 | // special here, because it will automatically be ignored. |
Chris Lattner | 1eb9e6c | 2002-05-24 21:33:26 +0000 | [diff] [blame] | 108 | } else { |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 109 | // Insert a cast instruction to convert the return value of the function |
| 110 | // into it's new type. Of course we only need to do this if the return |
| 111 | // value of the function is actually USED. |
| 112 | // |
| 113 | if (!CI->use_empty()) { |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 114 | // Insert the new cast instruction... |
Chris Lattner | 5c44786 | 2002-09-10 17:03:06 +0000 | [diff] [blame] | 115 | CastInst *NewCast = new CastInst(NewCall, CI->getType(), |
| 116 | NewCall->getName(), BBI); |
| 117 | CI->replaceAllUsesWith(NewCast); |
Chris Lattner | aacadd1 | 2002-07-30 00:50:49 +0000 | [diff] [blame] | 118 | } |
Chris Lattner | 1eb9e6c | 2002-05-24 21:33:26 +0000 | [diff] [blame] | 119 | } |
| 120 | |
| 121 | // The old instruction is no longer needed, destroy it! |
| 122 | delete CI; |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 123 | } |
| 124 | |
| 125 | |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 126 | static bool ResolveFunctions(Module &M, std::vector<GlobalValue*> &Globals, |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 127 | Function *Concrete) { |
| 128 | bool Changed = false; |
| 129 | for (unsigned i = 0; i != Globals.size(); ++i) |
| 130 | if (Globals[i] != Concrete) { |
| 131 | Function *Old = cast<Function>(Globals[i]); |
| 132 | const FunctionType *OldMT = Old->getFunctionType(); |
| 133 | const FunctionType *ConcreteMT = Concrete->getFunctionType(); |
| 134 | |
| 135 | assert(OldMT->getParamTypes().size() <= |
| 136 | ConcreteMT->getParamTypes().size() && |
| 137 | "Concrete type must have more specified parameters!"); |
| 138 | |
| 139 | // Check to make sure that if there are specified types, that they |
| 140 | // match... |
| 141 | // |
| 142 | for (unsigned i = 0; i < OldMT->getParamTypes().size(); ++i) |
| 143 | if (OldMT->getParamTypes()[i] != ConcreteMT->getParamTypes()[i]) { |
Chris Lattner | 4794895 | 2003-01-30 21:33:07 +0000 | [diff] [blame] | 144 | std::cerr << "funcresolve: Function [" << Old->getName() |
| 145 | << "]: Parameter types conflict for: '" << OldMT |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 146 | << "' and '" << ConcreteMT << "'\n"; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 147 | return Changed; |
| 148 | } |
| 149 | |
| 150 | // Attempt to convert all of the uses of the old function to the |
| 151 | // concrete form of the function. If there is a use of the fn that |
| 152 | // we don't understand here we punt to avoid making a bad |
| 153 | // transformation. |
| 154 | // |
| 155 | // At this point, we know that the return values are the same for |
| 156 | // our two functions and that the Old function has no varargs fns |
| 157 | // specified. In otherwords it's just <retty> (...) |
| 158 | // |
| 159 | for (unsigned i = 0; i < Old->use_size(); ) { |
| 160 | User *U = *(Old->use_begin()+i); |
| 161 | if (CastInst *CI = dyn_cast<CastInst>(U)) { |
| 162 | // Convert casts directly |
| 163 | assert(CI->getOperand(0) == Old); |
| 164 | CI->setOperand(0, Concrete); |
| 165 | Changed = true; |
| 166 | ++NumResolved; |
| 167 | } else if (CallInst *CI = dyn_cast<CallInst>(U)) { |
| 168 | // Can only fix up calls TO the argument, not args passed in. |
| 169 | if (CI->getCalledValue() == Old) { |
| 170 | ConvertCallTo(CI, Concrete); |
| 171 | Changed = true; |
| 172 | ++NumResolved; |
| 173 | } else { |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 174 | std::cerr << "Couldn't cleanup this function call, must be an" |
| 175 | << " argument or something!" << CI; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 176 | ++i; |
| 177 | } |
| 178 | } else { |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 179 | std::cerr << "Cannot convert use of function: " << U << "\n"; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 180 | ++i; |
| 181 | } |
| 182 | } |
| 183 | } |
| 184 | return Changed; |
| 185 | } |
| 186 | |
| 187 | |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 188 | static bool ResolveGlobalVariables(Module &M, |
| 189 | std::vector<GlobalValue*> &Globals, |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 190 | GlobalVariable *Concrete) { |
| 191 | bool Changed = false; |
| 192 | assert(isa<ArrayType>(Concrete->getType()->getElementType()) && |
| 193 | "Concrete version should be an array type!"); |
| 194 | |
| 195 | // Get the type of the things that may be resolved to us... |
| 196 | const Type *AETy = |
| 197 | cast<ArrayType>(Concrete->getType()->getElementType())->getElementType(); |
| 198 | |
| 199 | std::vector<Constant*> Args; |
| 200 | Args.push_back(Constant::getNullValue(Type::LongTy)); |
| 201 | Args.push_back(Constant::getNullValue(Type::LongTy)); |
| 202 | ConstantExpr *Replacement = |
| 203 | ConstantExpr::getGetElementPtr(ConstantPointerRef::get(Concrete), Args); |
| 204 | |
| 205 | for (unsigned i = 0; i != Globals.size(); ++i) |
| 206 | if (Globals[i] != Concrete) { |
| 207 | GlobalVariable *Old = cast<GlobalVariable>(Globals[i]); |
| 208 | if (Old->getType()->getElementType() != AETy) { |
| 209 | std::cerr << "WARNING: Two global variables exist with the same name " |
| 210 | << "that cannot be resolved!\n"; |
| 211 | return false; |
| 212 | } |
| 213 | |
| 214 | // In this case, Old is a pointer to T, Concrete is a pointer to array of |
| 215 | // T. Because of this, replace all uses of Old with a constantexpr |
| 216 | // getelementptr that returns the address of the first element of the |
| 217 | // array. |
| 218 | // |
| 219 | Old->replaceAllUsesWith(Replacement); |
| 220 | // Since there are no uses of Old anymore, remove it from the module. |
| 221 | M.getGlobalList().erase(Old); |
| 222 | |
| 223 | ++NumGlobals; |
| 224 | Changed = true; |
| 225 | } |
| 226 | return Changed; |
| 227 | } |
| 228 | |
| 229 | static bool ProcessGlobalsWithSameName(Module &M, |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 230 | std::vector<GlobalValue*> &Globals) { |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 231 | assert(!Globals.empty() && "Globals list shouldn't be empty here!"); |
| 232 | |
| 233 | bool isFunction = isa<Function>(Globals[0]); // Is this group all functions? |
| 234 | bool Changed = false; |
| 235 | GlobalValue *Concrete = 0; // The most concrete implementation to resolve to |
| 236 | |
| 237 | assert((isFunction ^ isa<GlobalVariable>(Globals[0])) && |
| 238 | "Should either be function or gvar!"); |
| 239 | |
| 240 | for (unsigned i = 0; i != Globals.size(); ) { |
| 241 | if (isa<Function>(Globals[i]) != isFunction) { |
| 242 | std::cerr << "WARNING: Found function and global variable with the " |
| 243 | << "same name: '" << Globals[i]->getName() << "'.\n"; |
| 244 | return false; // Don't know how to handle this, bail out! |
| 245 | } |
| 246 | |
Chris Lattner | 52b8fc0 | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 247 | if (isFunction) { |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 248 | // For functions, we look to merge functions definitions of "int (...)" |
| 249 | // to 'int (int)' or 'int ()' or whatever else is not completely generic. |
| 250 | // |
| 251 | Function *F = cast<Function>(Globals[i]); |
Chris Lattner | 5997c3d | 2002-11-08 00:38:20 +0000 | [diff] [blame] | 252 | if (!F->isExternal()) { |
Chris Lattner | 52b8fc0 | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 253 | if (Concrete && !Concrete->isExternal()) |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 254 | return false; // Found two different functions types. Can't choose! |
| 255 | |
| 256 | Concrete = Globals[i]; |
Chris Lattner | 52b8fc0 | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 257 | } else if (Concrete) { |
| 258 | if (Concrete->isExternal()) // If we have multiple external symbols...x |
| 259 | if (F->getFunctionType()->getNumParams() > |
| 260 | cast<Function>(Concrete)->getFunctionType()->getNumParams()) |
| 261 | Concrete = F; // We are more concrete than "Concrete"! |
| 262 | |
| 263 | } else { |
| 264 | Concrete = F; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 265 | } |
| 266 | ++i; |
| 267 | } else { |
| 268 | // For global variables, we have to merge C definitions int A[][4] with |
| 269 | // int[6][4] |
| 270 | GlobalVariable *GV = cast<GlobalVariable>(Globals[i]); |
| 271 | if (Concrete == 0) { |
| 272 | if (isa<ArrayType>(GV->getType()->getElementType())) |
| 273 | Concrete = GV; |
| 274 | } else { // Must have different types... one is an array of the other? |
| 275 | const ArrayType *AT = |
| 276 | dyn_cast<ArrayType>(GV->getType()->getElementType()); |
| 277 | |
| 278 | // If GV is an array of Concrete, then GV is the array. |
| 279 | if (AT && AT->getElementType() == Concrete->getType()->getElementType()) |
| 280 | Concrete = GV; |
| 281 | else { |
| 282 | // Concrete must be an array type, check to see if the element type of |
| 283 | // concrete is already GV. |
| 284 | AT = cast<ArrayType>(Concrete->getType()->getElementType()); |
| 285 | if (AT->getElementType() != GV->getType()->getElementType()) |
| 286 | Concrete = 0; // Don't know how to handle it! |
| 287 | } |
| 288 | } |
| 289 | |
| 290 | ++i; |
| 291 | } |
| 292 | } |
| 293 | |
| 294 | if (Globals.size() > 1) { // Found a multiply defined global... |
| 295 | // We should find exactly one concrete function definition, which is |
| 296 | // probably the implementation. Change all of the function definitions and |
| 297 | // uses to use it instead. |
| 298 | // |
| 299 | if (!Concrete) { |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 300 | std::cerr << "WARNING: Found function types that are not compatible:\n"; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 301 | for (unsigned i = 0; i < Globals.size(); ++i) { |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 302 | std::cerr << "\t" << Globals[i]->getType()->getDescription() << " %" |
| 303 | << Globals[i]->getName() << "\n"; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 304 | } |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 305 | std::cerr << " No linkage of globals named '" << Globals[0]->getName() |
| 306 | << "' performed!\n"; |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 307 | return Changed; |
| 308 | } |
| 309 | |
| 310 | if (isFunction) |
| 311 | return Changed | ResolveFunctions(M, Globals, cast<Function>(Concrete)); |
| 312 | else |
| 313 | return Changed | ResolveGlobalVariables(M, Globals, |
| 314 | cast<GlobalVariable>(Concrete)); |
| 315 | } |
| 316 | return Changed; |
| 317 | } |
| 318 | |
Chris Lattner | 113f4f4 | 2002-06-25 16:13:24 +0000 | [diff] [blame] | 319 | bool FunctionResolvingPass::run(Module &M) { |
Chris Lattner | 98cf1f5 | 2002-11-20 18:36:02 +0000 | [diff] [blame] | 320 | SymbolTable &ST = M.getSymbolTable(); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 321 | |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 322 | std::map<std::string, std::vector<GlobalValue*> > Globals; |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 323 | |
| 324 | // Loop over the entries in the symbol table. If an entry is a func pointer, |
| 325 | // then add it to the Functions map. We do a two pass algorithm here to avoid |
| 326 | // problems with iterators getting invalidated if we did a one pass scheme. |
| 327 | // |
Chris Lattner | 98cf1f5 | 2002-11-20 18:36:02 +0000 | [diff] [blame] | 328 | for (SymbolTable::iterator I = ST.begin(), E = ST.end(); I != E; ++I) |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 329 | if (const PointerType *PT = dyn_cast<PointerType>(I->first)) { |
| 330 | SymbolTable::VarMap &Plane = I->second; |
| 331 | for (SymbolTable::type_iterator PI = Plane.begin(), PE = Plane.end(); |
| 332 | PI != PE; ++PI) { |
| 333 | GlobalValue *GV = cast<GlobalValue>(PI->second); |
| 334 | assert(PI->first == GV->getName() && |
| 335 | "Global name and symbol table do not agree!"); |
| 336 | if (GV->hasExternalLinkage()) // Only resolve decls to external fns |
| 337 | Globals[PI->first].push_back(GV); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 338 | } |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 339 | } |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 340 | |
| 341 | bool Changed = false; |
| 342 | |
| 343 | // Now we have a list of all functions with a particular name. If there is |
| 344 | // more than one entry in a list, merge the functions together. |
| 345 | // |
Chris Lattner | b2d9f7d | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 346 | for (std::map<std::string, std::vector<GlobalValue*> >::iterator |
| 347 | I = Globals.begin(), E = Globals.end(); I != E; ++I) |
Chris Lattner | 013eca00 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 348 | Changed |= ProcessGlobalsWithSameName(M, I->second); |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 349 | |
Chris Lattner | 52b8fc0 | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 350 | // Now loop over all of the globals, checking to see if any are trivially |
| 351 | // dead. If so, remove them now. |
| 352 | |
| 353 | for (Module::iterator I = M.begin(), E = M.end(); I != E; ) |
| 354 | if (I->isExternal() && I->use_empty()) { |
| 355 | Function *F = I; |
| 356 | ++I; |
| 357 | M.getFunctionList().erase(F); |
| 358 | ++NumResolved; |
| 359 | Changed = true; |
| 360 | } else { |
| 361 | ++I; |
| 362 | } |
| 363 | |
| 364 | for (Module::giterator I = M.gbegin(), E = M.gend(); I != E; ) |
| 365 | if (I->isExternal() && I->use_empty()) { |
| 366 | GlobalVariable *GV = I; |
| 367 | ++I; |
| 368 | M.getGlobalList().erase(GV); |
| 369 | ++NumGlobals; |
| 370 | Changed = true; |
| 371 | } else { |
| 372 | ++I; |
| 373 | } |
| 374 | |
Chris Lattner | 5aa9e3e | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 375 | return Changed; |
| 376 | } |