Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 1 | //===- FunctionResolution.cpp - Resolve declarations to implementations ---===// |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 2 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 3 | // 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 Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 7 | // |
John Criswell | b576c94 | 2003-10-20 19:43:21 +0000 | [diff] [blame] | 8 | //===----------------------------------------------------------------------===// |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 9 | // |
| 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 Lattner | ff1be26 | 2002-07-23 22:04:02 +0000 | [diff] [blame] | 21 | #include "llvm/Transforms/IPO.h" |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 22 | #include "llvm/Module.h" |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 23 | #include "llvm/DerivedTypes.h" |
| 24 | #include "llvm/Pass.h" |
Misha Brukman | 47b14a4 | 2004-07-29 17:30:56 +0000 | [diff] [blame] | 25 | #include "llvm/Instructions.h" |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 26 | #include "llvm/Constants.h" |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 27 | #include "llvm/Support/CallSite.h" |
Chris Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 28 | #include "llvm/Target/TargetData.h" |
Chris Lattner | 250d91b | 2003-08-13 22:15:04 +0000 | [diff] [blame] | 29 | #include "llvm/Assembly/Writer.h" |
Reid Spencer | 551ccae | 2004-09-01 22:55:40 +0000 | [diff] [blame] | 30 | #include "llvm/ADT/Statistic.h" |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 31 | #include <algorithm> |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 32 | using namespace llvm; |
Brian Gaeke | d0fde30 | 2003-11-11 22:41:34 +0000 | [diff] [blame] | 33 | |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 34 | namespace { |
Chris Lattner | ac0b6ae | 2006-12-06 17:46:33 +0000 | [diff] [blame] | 35 | Statistic NumResolved("funcresolve", "Number of varargs functions resolved"); |
| 36 | Statistic NumGlobals("funcresolve", "Number of global variables resolved"); |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 37 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 38 | struct FunctionResolvingPass : public ModulePass { |
Chris Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 39 | virtual void getAnalysisUsage(AnalysisUsage &AU) const { |
| 40 | AU.addRequired<TargetData>(); |
| 41 | } |
| 42 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 43 | bool runOnModule(Module &M); |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 44 | }; |
Chris Lattner | 7f8897f | 2006-08-27 22:42:52 +0000 | [diff] [blame] | 45 | RegisterPass<FunctionResolvingPass> X("funcresolve", "Resolve Functions"); |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 46 | } |
| 47 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 48 | ModulePass *llvm::createFunctionResolvingPass() { |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 49 | return new FunctionResolvingPass(); |
| 50 | } |
| 51 | |
Chris Lattner | da902ba | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 52 | static bool ResolveFunctions(Module &M, std::vector<GlobalValue*> &Globals, |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 53 | 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 Spencer | 2f18907 | 2005-12-13 19:56:51 +0000 | [diff] [blame] | 58 | const FunctionType *OldFT = Old->getFunctionType(); |
| 59 | const FunctionType *ConcreteFT = Concrete->getFunctionType(); |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 60 | |
Reid Spencer | 2f18907 | 2005-12-13 19:56:51 +0000 | [diff] [blame] | 61 | if (OldFT->getNumParams() > ConcreteFT->getNumParams() && |
| 62 | !ConcreteFT->isVarArg()) |
Chris Lattner | 4e2fd75 | 2003-02-27 20:55:48 +0000 | [diff] [blame] | 63 | if (!Old->use_empty()) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 64 | cerr << "WARNING: Linking function '" << Old->getName() |
| 65 | << "' is causing arguments to be dropped.\n"; |
| 66 | cerr << "WARNING: Prototype: "; |
Bill Wendling | f5da133 | 2006-12-07 22:21:48 +0000 | [diff] [blame] | 67 | WriteAsOperand(*cerr.stream(), Old); |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 68 | cerr << " resolved to "; |
Bill Wendling | f5da133 | 2006-12-07 22:21:48 +0000 | [diff] [blame] | 69 | WriteAsOperand(*cerr.stream(), Concrete); |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 70 | cerr << "\n"; |
Chris Lattner | 4e2fd75 | 2003-02-27 20:55:48 +0000 | [diff] [blame] | 71 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 72 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 73 | // Check to make sure that if there are specified types, that they |
| 74 | // match... |
| 75 | // |
Reid Spencer | 2f18907 | 2005-12-13 19:56:51 +0000 | [diff] [blame] | 76 | unsigned NumArguments = std::min(OldFT->getNumParams(), |
| 77 | ConcreteFT->getNumParams()); |
Chris Lattner | 4e2fd75 | 2003-02-27 20:55:48 +0000 | [diff] [blame] | 78 | |
Chris Lattner | dde601d | 2003-03-03 19:57:46 +0000 | [diff] [blame] | 79 | if (!Old->use_empty() && !Concrete->use_empty()) |
| 80 | for (unsigned i = 0; i < NumArguments; ++i) |
Reid Spencer | 2f18907 | 2005-12-13 19:56:51 +0000 | [diff] [blame] | 81 | if (OldFT->getParamType(i) != ConcreteFT->getParamType(i)) |
| 82 | if (OldFT->getParamType(i)->getTypeID() != |
| 83 | ConcreteFT->getParamType(i)->getTypeID()) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 84 | cerr << "WARNING: Function [" << Old->getName() |
| 85 | << "]: Parameter types conflict for: '"; |
Bill Wendling | f5da133 | 2006-12-07 22:21:48 +0000 | [diff] [blame] | 86 | WriteTypeSymbolic(*cerr.stream(), OldFT, &M); |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 87 | cerr << "' (in " |
| 88 | << Old->getParent()->getModuleIdentifier() << ") and '"; |
Bill Wendling | f5da133 | 2006-12-07 22:21:48 +0000 | [diff] [blame] | 89 | WriteTypeSymbolic(*cerr.stream(), ConcreteFT, &M); |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 90 | cerr << "'(in " |
| 91 | << Concrete->getParent()->getModuleIdentifier() << ")\n"; |
Chris Lattner | b29170f | 2003-08-20 23:50:38 +0000 | [diff] [blame] | 92 | return Changed; |
Chris Lattner | a0f85e5 | 2003-08-23 20:03:05 +0000 | [diff] [blame] | 93 | } |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 94 | |
Chris Lattner | 1fd95af | 2003-04-28 01:23:29 +0000 | [diff] [blame] | 95 | // 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 Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 98 | // |
Chris Lattner | 1fd95af | 2003-04-28 01:23:29 +0000 | [diff] [blame] | 99 | // 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 Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 102 | // |
Chris Lattner | d514d82 | 2005-02-01 01:23:31 +0000 | [diff] [blame] | 103 | if (!Old->use_empty()) { |
Chris Lattner | 1078d11 | 2003-07-23 22:03:18 +0000 | [diff] [blame] | 104 | Value *Replacement = Concrete; |
| 105 | if (Concrete->getType() != Old->getType()) |
Reid Spencer | 4da4912 | 2006-12-12 05:05:00 +0000 | [diff] [blame^] | 106 | Replacement = ConstantExpr::getBitCast(Concrete, Old->getType()); |
Chris Lattner | d514d82 | 2005-02-01 01:23:31 +0000 | [diff] [blame] | 107 | NumResolved += Old->getNumUses(); |
Chris Lattner | 1078d11 | 2003-07-23 22:03:18 +0000 | [diff] [blame] | 108 | Old->replaceAllUsesWith(Replacement); |
| 109 | } |
Chris Lattner | 12ce59d | 2003-05-31 21:08:45 +0000 | [diff] [blame] | 110 | |
| 111 | // Since there are no uses of Old anymore, remove it from the module. |
| 112 | M.getFunctionList().erase(Old); |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 113 | } |
| 114 | return Changed; |
| 115 | } |
| 116 | |
| 117 | |
Chris Lattner | da902ba | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 118 | static bool ResolveGlobalVariables(Module &M, |
| 119 | std::vector<GlobalValue*> &Globals, |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 120 | GlobalVariable *Concrete) { |
| 121 | bool Changed = false; |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 122 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 123 | for (unsigned i = 0; i != Globals.size(); ++i) |
| 124 | if (Globals[i] != Concrete) { |
Reid Spencer | 4da4912 | 2006-12-12 05:05:00 +0000 | [diff] [blame^] | 125 | Constant *Cast = ConstantExpr::getBitCast(Concrete,Globals[i]->getType()); |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 126 | Globals[i]->replaceAllUsesWith(Cast); |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 127 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 128 | // Since there are no uses of Old anymore, remove it from the module. |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 129 | M.getGlobalList().erase(cast<GlobalVariable>(Globals[i])); |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 130 | |
| 131 | ++NumGlobals; |
| 132 | Changed = true; |
| 133 | } |
| 134 | return Changed; |
| 135 | } |
| 136 | |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 137 | // Check to see if all of the callers of F ignore the return value. |
| 138 | static 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 Spencer | 518310c | 2004-07-18 00:44:37 +0000 | [diff] [blame] | 141 | if (GlobalValue *GV = dyn_cast<GlobalValue>(*I)) { |
| 142 | for (Value::use_iterator I = GV->use_begin(), E = GV->use_end(); |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 143 | 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 Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 157 | static bool ProcessGlobalsWithSameName(Module &M, TargetData &TD, |
Chris Lattner | da902ba | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 158 | std::vector<GlobalValue*> &Globals) { |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 159 | assert(!Globals.empty() && "Globals list shouldn't be empty here!"); |
| 160 | |
| 161 | bool isFunction = isa<Function>(Globals[0]); // Is this group all functions? |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 162 | GlobalValue *Concrete = 0; // The most concrete implementation to resolve to |
| 163 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 164 | for (unsigned i = 0; i != Globals.size(); ) { |
| 165 | if (isa<Function>(Globals[i]) != isFunction) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 166 | cerr << "WARNING: Found function and global variable with the " |
| 167 | << "same name: '" << Globals[i]->getName() << "'.\n"; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 168 | return false; // Don't know how to handle this, bail out! |
| 169 | } |
| 170 | |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 171 | if (isFunction) { |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 172 | // 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 Lattner | 6f23963 | 2002-11-08 00:38:20 +0000 | [diff] [blame] | 176 | if (!F->isExternal()) { |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 177 | if (Concrete && !Concrete->isExternal()) |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 178 | return false; // Found two different functions types. Can't choose! |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 179 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 180 | Concrete = Globals[i]; |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 181 | } else if (Concrete) { |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 182 | if (Concrete->isExternal()) // If we have multiple external symbols... |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 183 | if (F->getFunctionType()->getNumParams() > |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 184 | cast<Function>(Concrete)->getFunctionType()->getNumParams()) |
| 185 | Concrete = F; // We are more concrete than "Concrete"! |
| 186 | |
| 187 | } else { |
| 188 | Concrete = F; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 189 | } |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 190 | } else { |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 191 | GlobalVariable *GV = cast<GlobalVariable>(Globals[i]); |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 192 | if (!GV->isExternal()) { |
| 193 | if (Concrete) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 194 | cerr << "WARNING: Two global variables with external linkage" |
| 195 | << " exist with the same name: '" << GV->getName() |
| 196 | << "'!\n"; |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 197 | return false; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 198 | } |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 199 | Concrete = GV; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 200 | } |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 201 | } |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 202 | ++i; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 203 | } |
| 204 | |
| 205 | if (Globals.size() > 1) { // Found a multiply defined global... |
Chris Lattner | c16e631 | 2003-05-31 21:57:06 +0000 | [diff] [blame] | 206 | // 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 Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 218 | |
Chris Lattner | c16e631 | 2003-05-31 21:57:06 +0000 | [diff] [blame] | 219 | if (!HasExternal && NumInstancesWithExternalLinkage <= 1) |
| 220 | return false; // Nothing to do? Must have multiple internal definitions. |
| 221 | |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 222 | // 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 Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 228 | // and the return values match (or was never used), do not warn. |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 229 | if (Function *ConcreteF = dyn_cast<Function>(Concrete)) |
| 230 | if (Function *OtherF = dyn_cast<Function>(Other)) |
Chris Lattner | 03fb8b2 | 2003-11-20 21:21:31 +0000 | [diff] [blame] | 231 | if ((ConcreteF->getReturnType() == OtherF->getReturnType() || |
| 232 | CallersAllIgnoreReturnValue(*OtherF)) && |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 233 | OtherF->getFunctionType()->isVarArg() && |
Chris Lattner | d5d8996 | 2004-02-09 04:14:01 +0000 | [diff] [blame] | 234 | OtherF->getFunctionType()->getNumParams() == 0) |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 235 | DontPrintWarning = true; |
Misha Brukman | fd93908 | 2005-04-21 23:48:37 +0000 | [diff] [blame] | 236 | |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 237 | // 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 Lattner | 4e4c444 | 2004-08-20 00:30:39 +0000 | [diff] [blame] | 241 | 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 Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 250 | } |
Chris Lattner | c16e631 | 2003-05-31 21:57:06 +0000 | [diff] [blame] | 251 | |
Chris Lattner | 23367a7 | 2004-09-30 00:12:29 +0000 | [diff] [blame] | 252 | if (0 && !DontPrintWarning) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 253 | cerr << "WARNING: Found global types that are not compatible:\n"; |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 254 | for (unsigned i = 0; i < Globals.size(); ++i) { |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 255 | cerr << "\t"; |
Bill Wendling | f5da133 | 2006-12-07 22:21:48 +0000 | [diff] [blame] | 256 | WriteTypeSymbolic(*cerr.stream(), Globals[i]->getType(), &M); |
Bill Wendling | 832171c | 2006-12-07 20:04:42 +0000 | [diff] [blame] | 257 | cerr << " %" << Globals[i]->getName() << "\n"; |
Chris Lattner | ce9431955 | 2003-10-22 23:03:38 +0000 | [diff] [blame] | 258 | } |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 259 | } |
| 260 | |
Chris Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 261 | if (!Concrete) |
| 262 | Concrete = Globals[0]; |
Chris Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 263 | 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 Lattner | 5858e1e | 2003-10-21 23:17:56 +0000 | [diff] [blame] | 288 | |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 289 | if (isFunction) |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 290 | return ResolveFunctions(M, Globals, cast<Function>(Concrete)); |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 291 | else |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 292 | return ResolveGlobalVariables(M, Globals, |
| 293 | cast<GlobalVariable>(Concrete)); |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 294 | } |
Chris Lattner | ea2294a | 2003-04-19 00:15:27 +0000 | [diff] [blame] | 295 | return false; |
Chris Lattner | a45ec54 | 2002-10-09 21:10:06 +0000 | [diff] [blame] | 296 | } |
| 297 | |
Chris Lattner | b12914b | 2004-09-20 04:48:05 +0000 | [diff] [blame] | 298 | bool FunctionResolvingPass::runOnModule(Module &M) { |
Chris Lattner | da902ba | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 299 | std::map<std::string, std::vector<GlobalValue*> > Globals; |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 300 | |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 301 | // 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 Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 304 | // |
Chris Lattner | 6be5e56 | 2003-10-22 04:43:18 +0000 | [diff] [blame] | 305 | bool Changed = false; |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 306 | for (Module::iterator I = M.begin(), E = M.end(); I != E; ) { |
| 307 | Function *F = I++; |
Chris Lattner | 6be5e56 | 2003-10-22 04:43:18 +0000 | [diff] [blame] | 308 | if (F->use_empty() && F->isExternal()) { |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 309 | M.getFunctionList().erase(F); |
Chris Lattner | 6be5e56 | 2003-10-22 04:43:18 +0000 | [diff] [blame] | 310 | Changed = true; |
Chris Lattner | e5ad50b | 2004-06-18 05:50:48 +0000 | [diff] [blame] | 311 | } else if (!F->hasInternalLinkage() && !F->getName().empty() && |
| 312 | !F->getIntrinsicID()) |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 313 | Globals[F->getName()].push_back(F); |
| 314 | } |
| 315 | |
Chris Lattner | 7f8897f | 2006-08-27 22:42:52 +0000 | [diff] [blame] | 316 | for (Module::global_iterator I = M.global_begin(), E = M.global_end(); |
| 317 | I != E; ) { |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 318 | GlobalVariable *GV = I++; |
Chris Lattner | 6be5e56 | 2003-10-22 04:43:18 +0000 | [diff] [blame] | 319 | if (GV->use_empty() && GV->isExternal()) { |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 320 | M.getGlobalList().erase(GV); |
Chris Lattner | 6be5e56 | 2003-10-22 04:43:18 +0000 | [diff] [blame] | 321 | Changed = true; |
| 322 | } else if (!GV->hasInternalLinkage() && !GV->getName().empty()) |
Chris Lattner | 4cb766a | 2003-10-22 04:42:20 +0000 | [diff] [blame] | 323 | Globals[GV->getName()].push_back(GV); |
| 324 | } |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 325 | |
Chris Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 326 | TargetData &TD = getAnalysis<TargetData>(); |
| 327 | |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 328 | // 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 Lattner | da902ba | 2003-01-30 18:22:32 +0000 | [diff] [blame] | 331 | for (std::map<std::string, std::vector<GlobalValue*> >::iterator |
| 332 | I = Globals.begin(), E = Globals.end(); I != E; ++I) |
Chris Lattner | efd47ba | 2003-10-22 03:35:34 +0000 | [diff] [blame] | 333 | Changed |= ProcessGlobalsWithSameName(M, TD, I->second); |
Chris Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 334 | |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 335 | // 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 Lattner | 7f8897f | 2006-08-27 22:42:52 +0000 | [diff] [blame] | 349 | for (Module::global_iterator I = M.global_begin(), E = M.global_end(); |
| 350 | I != E; ) |
Chris Lattner | a2b8d7b | 2002-11-10 03:36:55 +0000 | [diff] [blame] | 351 | 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 Lattner | 22ee3eb | 2002-05-24 20:42:13 +0000 | [diff] [blame] | 361 | return Changed; |
| 362 | } |