Chris Lattner | 52f7e90 | 2001-10-13 07:03:50 +0000 | [diff] [blame] | 1 | //===- Linker.cpp - Module Linker Implementation --------------------------===// |
| 2 | // |
| 3 | // This file implements the LLVM module linker. |
| 4 | // |
| 5 | // Specifically, this: |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 6 | // * Merges global variables between the two modules |
| 7 | // * Uninit + Uninit = Init, Init + Uninit = Init, Init + Init = Error if != |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 8 | // * Merges functions between two modules |
Chris Lattner | 52f7e90 | 2001-10-13 07:03:50 +0000 | [diff] [blame] | 9 | // |
| 10 | //===----------------------------------------------------------------------===// |
| 11 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 12 | #include "llvm/Transforms/Utils/Linker.h" |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 13 | #include "llvm/Module.h" |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 14 | #include "llvm/SymbolTable.h" |
| 15 | #include "llvm/DerivedTypes.h" |
| 16 | #include "llvm/iOther.h" |
Chris Lattner | 31bcdb8 | 2002-04-28 19:55:58 +0000 | [diff] [blame] | 17 | #include "llvm/Constants.h" |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 18 | |
| 19 | // Error - Simple wrapper function to conditionally assign to E and return true. |
| 20 | // This just makes error return conditions a little bit simpler... |
| 21 | // |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 22 | static inline bool Error(std::string *E, std::string Message) { |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 23 | if (E) *E = Message; |
| 24 | return true; |
| 25 | } |
| 26 | |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 27 | // LinkTypes - Go through the symbol table of the Src module and see if any |
| 28 | // types are named in the src module that are not named in the Dst module. |
| 29 | // Make sure there are no type name conflicts. |
| 30 | // |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 31 | static bool LinkTypes(Module *Dest, const Module *Src, std::string *Err) { |
Chris Lattner | 6e6026b | 2002-11-20 18:36:02 +0000 | [diff] [blame] | 32 | SymbolTable *DestST = &Dest->getSymbolTable(); |
| 33 | const SymbolTable *SrcST = &Src->getSymbolTable(); |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 34 | |
| 35 | // Look for a type plane for Type's... |
| 36 | SymbolTable::const_iterator PI = SrcST->find(Type::TypeTy); |
| 37 | if (PI == SrcST->end()) return false; // No named types, do nothing. |
| 38 | |
| 39 | const SymbolTable::VarMap &VM = PI->second; |
| 40 | for (SymbolTable::type_const_iterator I = VM.begin(), E = VM.end(); |
| 41 | I != E; ++I) { |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 42 | const std::string &Name = I->first; |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 43 | const Type *RHS = cast<Type>(I->second); |
| 44 | |
| 45 | // Check to see if this type name is already in the dest module... |
| 46 | const Type *Entry = cast_or_null<Type>(DestST->lookup(Type::TypeTy, Name)); |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 47 | if (Entry && !isa<OpaqueType>(Entry)) { // Yup, the value already exists... |
Chris Lattner | 2f6bb2b | 2003-01-30 20:53:43 +0000 | [diff] [blame] | 48 | if (Entry != RHS) { |
| 49 | if (OpaqueType *OT = dyn_cast<OpaqueType>(const_cast<Type*>(RHS))) { |
| 50 | OT->refineAbstractTypeTo(Entry); |
| 51 | } else { |
| 52 | // If it's the same, noop. Otherwise, error. |
| 53 | return Error(Err, "Type named '" + Name + |
| 54 | "' of different shape in modules.\n Src='" + |
| 55 | Entry->getDescription() + "'.\n Dst='" + |
| 56 | RHS->getDescription() + "'"); |
| 57 | } |
| 58 | } |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 59 | } else { // Type not in dest module. Add it now. |
Chris Lattner | 2f6bb2b | 2003-01-30 20:53:43 +0000 | [diff] [blame] | 60 | if (Entry) { |
| 61 | OpaqueType *OT = cast<OpaqueType>(const_cast<Type*>(Entry)); |
| 62 | OT->refineAbstractTypeTo(RHS); |
| 63 | } |
| 64 | |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 65 | // TODO: FIXME WHEN TYPES AREN'T CONST |
| 66 | DestST->insert(Name, const_cast<Type*>(RHS)); |
| 67 | } |
| 68 | } |
| 69 | return false; |
| 70 | } |
| 71 | |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 72 | static void PrintMap(const std::map<const Value*, Value*> &M) { |
| 73 | for (std::map<const Value*, Value*>::const_iterator I = M.begin(), E =M.end(); |
Chris Lattner | 2d3e8bb | 2001-11-03 03:27:29 +0000 | [diff] [blame] | 74 | I != E; ++I) { |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 75 | std::cerr << " Fr: " << (void*)I->first << " "; |
Chris Lattner | 87182ae | 2002-04-07 22:31:23 +0000 | [diff] [blame] | 76 | I->first->dump(); |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 77 | std::cerr << " To: " << (void*)I->second << " "; |
Chris Lattner | 87182ae | 2002-04-07 22:31:23 +0000 | [diff] [blame] | 78 | I->second->dump(); |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 79 | std::cerr << "\n"; |
Chris Lattner | 2d3e8bb | 2001-11-03 03:27:29 +0000 | [diff] [blame] | 80 | } |
| 81 | } |
| 82 | |
| 83 | |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 84 | // RemapOperand - Use LocalMap and GlobalMap to convert references from one |
| 85 | // module to another. This is somewhat sophisticated in that it can |
| 86 | // automatically handle constant references correctly as well... |
| 87 | // |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 88 | static Value *RemapOperand(const Value *In, |
| 89 | std::map<const Value*, Value*> &LocalMap, |
| 90 | std::map<const Value*, Value*> *GlobalMap) { |
| 91 | std::map<const Value*,Value*>::const_iterator I = LocalMap.find(In); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 92 | if (I != LocalMap.end()) return I->second; |
| 93 | |
| 94 | if (GlobalMap) { |
| 95 | I = GlobalMap->find(In); |
| 96 | if (I != GlobalMap->end()) return I->second; |
| 97 | } |
| 98 | |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 99 | // Check to see if it's a constant that we are interesting in transforming... |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 100 | if (const Constant *CPV = dyn_cast<Constant>(In)) { |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 101 | if (!isa<DerivedType>(CPV->getType()) && !isa<ConstantExpr>(CPV)) |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 102 | return const_cast<Constant*>(CPV); // Simple constants stay identical... |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 103 | |
Chris Lattner | e9bb2df | 2001-12-03 22:26:30 +0000 | [diff] [blame] | 104 | Constant *Result = 0; |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 105 | |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 106 | if (const ConstantArray *CPA = dyn_cast<ConstantArray>(CPV)) { |
Chris Lattner | 697954c | 2002-01-20 22:54:45 +0000 | [diff] [blame] | 107 | const std::vector<Use> &Ops = CPA->getValues(); |
| 108 | std::vector<Constant*> Operands(Ops.size()); |
Chris Lattner | e306d94 | 2002-07-18 02:31:03 +0000 | [diff] [blame] | 109 | for (unsigned i = 0, e = Ops.size(); i != e; ++i) |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 110 | Operands[i] = |
Chris Lattner | e9bb2df | 2001-12-03 22:26:30 +0000 | [diff] [blame] | 111 | cast<Constant>(RemapOperand(Ops[i], LocalMap, GlobalMap)); |
| 112 | Result = ConstantArray::get(cast<ArrayType>(CPA->getType()), Operands); |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 113 | } else if (const ConstantStruct *CPS = dyn_cast<ConstantStruct>(CPV)) { |
Chris Lattner | 697954c | 2002-01-20 22:54:45 +0000 | [diff] [blame] | 114 | const std::vector<Use> &Ops = CPS->getValues(); |
| 115 | std::vector<Constant*> Operands(Ops.size()); |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 116 | for (unsigned i = 0; i < Ops.size(); ++i) |
| 117 | Operands[i] = |
Chris Lattner | e9bb2df | 2001-12-03 22:26:30 +0000 | [diff] [blame] | 118 | cast<Constant>(RemapOperand(Ops[i], LocalMap, GlobalMap)); |
| 119 | Result = ConstantStruct::get(cast<StructType>(CPS->getType()), Operands); |
| 120 | } else if (isa<ConstantPointerNull>(CPV)) { |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 121 | Result = const_cast<Constant*>(CPV); |
| 122 | } else if (const ConstantPointerRef *CPR = |
| 123 | dyn_cast<ConstantPointerRef>(CPV)) { |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 124 | Value *V = RemapOperand(CPR->getValue(), LocalMap, GlobalMap); |
Chris Lattner | e9bb2df | 2001-12-03 22:26:30 +0000 | [diff] [blame] | 125 | Result = ConstantPointerRef::get(cast<GlobalValue>(V)); |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 126 | } else if (const ConstantExpr *CE = dyn_cast<ConstantExpr>(CPV)) { |
Chris Lattner | b319faf | 2002-08-20 19:35:11 +0000 | [diff] [blame] | 127 | if (CE->getOpcode() == Instruction::GetElementPtr) { |
| 128 | Value *Ptr = RemapOperand(CE->getOperand(0), LocalMap, GlobalMap); |
| 129 | std::vector<Constant*> Indices; |
| 130 | Indices.reserve(CE->getNumOperands()-1); |
| 131 | for (unsigned i = 1, e = CE->getNumOperands(); i != e; ++i) |
| 132 | Indices.push_back(cast<Constant>(RemapOperand(CE->getOperand(i), |
| 133 | LocalMap, GlobalMap))); |
| 134 | |
| 135 | Result = ConstantExpr::getGetElementPtr(cast<Constant>(Ptr), Indices); |
| 136 | } else if (CE->getNumOperands() == 1) { |
Chris Lattner | ad33348 | 2002-08-14 18:24:09 +0000 | [diff] [blame] | 137 | // Cast instruction |
| 138 | assert(CE->getOpcode() == Instruction::Cast); |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 139 | Value *V = RemapOperand(CE->getOperand(0), LocalMap, GlobalMap); |
Chris Lattner | ad33348 | 2002-08-14 18:24:09 +0000 | [diff] [blame] | 140 | Result = ConstantExpr::getCast(cast<Constant>(V), CE->getType()); |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 141 | } else if (CE->getNumOperands() == 2) { |
| 142 | // Binary operator... |
| 143 | Value *V1 = RemapOperand(CE->getOperand(0), LocalMap, GlobalMap); |
| 144 | Value *V2 = RemapOperand(CE->getOperand(1), LocalMap, GlobalMap); |
| 145 | |
| 146 | Result = ConstantExpr::get(CE->getOpcode(), cast<Constant>(V1), |
Chris Lattner | e8e4605 | 2002-07-30 18:54:22 +0000 | [diff] [blame] | 147 | cast<Constant>(V2)); |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 148 | } else { |
Chris Lattner | b319faf | 2002-08-20 19:35:11 +0000 | [diff] [blame] | 149 | assert(0 && "Unknown constant expr type!"); |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 150 | } |
| 151 | |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 152 | } else { |
| 153 | assert(0 && "Unknown type of derived type constant value!"); |
| 154 | } |
| 155 | |
| 156 | // Cache the mapping in our local map structure... |
Chris Lattner | d149c05 | 2002-09-23 18:14:15 +0000 | [diff] [blame] | 157 | if (GlobalMap) |
| 158 | GlobalMap->insert(std::make_pair(In, Result)); |
| 159 | else |
| 160 | LocalMap.insert(std::make_pair(In, Result)); |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 161 | return Result; |
| 162 | } |
Chris Lattner | 2d3e8bb | 2001-11-03 03:27:29 +0000 | [diff] [blame] | 163 | |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 164 | std::cerr << "XXX LocalMap: \n"; |
Chris Lattner | 2d3e8bb | 2001-11-03 03:27:29 +0000 | [diff] [blame] | 165 | PrintMap(LocalMap); |
| 166 | |
| 167 | if (GlobalMap) { |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 168 | std::cerr << "XXX GlobalMap: \n"; |
Chris Lattner | 2d3e8bb | 2001-11-03 03:27:29 +0000 | [diff] [blame] | 169 | PrintMap(*GlobalMap); |
| 170 | } |
| 171 | |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 172 | std::cerr << "Couldn't remap value: " << (void*)In << " " << *In << "\n"; |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 173 | assert(0 && "Couldn't remap value!"); |
| 174 | return 0; |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 175 | } |
| 176 | |
| 177 | |
| 178 | // LinkGlobals - Loop through the global variables in the src module and merge |
| 179 | // them into the dest module... |
| 180 | // |
| 181 | static bool LinkGlobals(Module *Dest, const Module *Src, |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 182 | std::map<const Value*, Value*> &ValueMap, |
| 183 | std::string *Err) { |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 184 | // We will need a module level symbol table if the src module has a module |
| 185 | // level symbol table... |
Chris Lattner | b91b657 | 2002-12-03 18:32:30 +0000 | [diff] [blame] | 186 | SymbolTable *ST = (SymbolTable*)&Dest->getSymbolTable(); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 187 | |
| 188 | // Loop over all of the globals in the src module, mapping them over as we go |
| 189 | // |
| 190 | for (Module::const_giterator I = Src->gbegin(), E = Src->gend(); I != E; ++I){ |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 191 | const GlobalVariable *SGV = I; |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 192 | GlobalVariable *DGV = 0; |
| 193 | if (SGV->hasName()) { |
| 194 | // A same named thing is a global variable, because the only two things |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 195 | // that may be in a module level symbol table are Global Vars and |
| 196 | // Functions, and they both have distinct, nonoverlapping, possible types. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 197 | // |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 198 | DGV = cast_or_null<GlobalVariable>(ST->lookup(SGV->getType(), |
| 199 | SGV->getName())); |
| 200 | } |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 201 | |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 202 | assert(SGV->hasInitializer() || SGV->hasExternalLinkage() && |
| 203 | "Global must either be external or have an initializer!"); |
| 204 | |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 205 | bool SGExtern = SGV->isExternal(); |
| 206 | bool DGExtern = DGV ? DGV->isExternal() : false; |
| 207 | |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 208 | if (!DGV || DGV->hasInternalLinkage() || SGV->hasInternalLinkage()) { |
| 209 | // No linking to be performed, simply create an identical version of the |
| 210 | // symbol over in the dest module... the initializer will be filled in |
| 211 | // later by LinkGlobalInits... |
| 212 | // |
Chris Lattner | 2719bac | 2003-04-21 21:15:04 +0000 | [diff] [blame] | 213 | GlobalVariable *NewDGV = |
| 214 | new GlobalVariable(SGV->getType()->getElementType(), |
| 215 | SGV->isConstant(), SGV->getLinkage(), /*init*/0, |
| 216 | SGV->getName(), Dest); |
| 217 | |
| 218 | // If the LLVM runtime renamed the global, but it is an externally visible |
| 219 | // symbol, DGV must be an existing global with internal linkage. Rename |
| 220 | // it. |
| 221 | if (NewDGV->getName() != SGV->getName() && !NewDGV->hasInternalLinkage()){ |
| 222 | assert(DGV && DGV->getName() == SGV->getName() && |
| 223 | DGV->hasInternalLinkage()); |
| 224 | DGV->setName(""); |
| 225 | NewDGV->setName(SGV->getName()); // Force the name back |
| 226 | DGV->setName(SGV->getName()); // This will cause a renaming |
| 227 | assert(NewDGV->getName() == SGV->getName() && |
| 228 | DGV->getName() != SGV->getName()); |
| 229 | } |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 230 | |
| 231 | // Make sure to remember this mapping... |
Chris Lattner | 2719bac | 2003-04-21 21:15:04 +0000 | [diff] [blame] | 232 | ValueMap.insert(std::make_pair(SGV, NewDGV)); |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 233 | } else if (!SGExtern && !DGExtern && SGV->getLinkage() !=DGV->getLinkage()){ |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 234 | return Error(Err, "Global variables named '" + SGV->getName() + |
| 235 | "' have different linkage specifiers!"); |
| 236 | } else if (SGV->hasExternalLinkage() || SGV->hasLinkOnceLinkage() || |
| 237 | SGV->hasAppendingLinkage()) { |
| 238 | // If the global variable has a name, and that name is already in use in |
| 239 | // the Dest module, make sure that the name is a compatible global |
| 240 | // variable... |
| 241 | // |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 242 | // Check to see if the two GV's have the same Const'ness... |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 243 | if (SGV->isConstant() != DGV->isConstant()) |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 244 | return Error(Err, "Global Variable Collision on '" + |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 245 | SGV->getType()->getDescription() + "':%" + SGV->getName() + |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 246 | " - Global variables differ in const'ness"); |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 247 | if (DGExtern) |
| 248 | DGV->setLinkage(SGV->getLinkage()); |
| 249 | |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 250 | // Okay, everything is cool, remember the mapping... |
Chris Lattner | 697954c | 2002-01-20 22:54:45 +0000 | [diff] [blame] | 251 | ValueMap.insert(std::make_pair(SGV, DGV)); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 252 | } else { |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 253 | assert(0 && "Unknown linkage!"); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 254 | } |
| 255 | } |
| 256 | return false; |
| 257 | } |
| 258 | |
| 259 | |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 260 | // LinkGlobalInits - Update the initializers in the Dest module now that all |
| 261 | // globals that may be referenced are in Dest. |
| 262 | // |
| 263 | static bool LinkGlobalInits(Module *Dest, const Module *Src, |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 264 | std::map<const Value*, Value*> &ValueMap, |
| 265 | std::string *Err) { |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 266 | |
| 267 | // Loop over all of the globals in the src module, mapping them over as we go |
| 268 | // |
| 269 | for (Module::const_giterator I = Src->gbegin(), E = Src->gend(); I != E; ++I){ |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 270 | const GlobalVariable *SGV = I; |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 271 | |
| 272 | if (SGV->hasInitializer()) { // Only process initialized GV's |
| 273 | // Figure out what the initializer looks like in the dest module... |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 274 | Constant *SInit = |
Chris Lattner | 2f6bb2b | 2003-01-30 20:53:43 +0000 | [diff] [blame] | 275 | cast<Constant>(RemapOperand(SGV->getInitializer(), ValueMap, 0)); |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 276 | |
| 277 | GlobalVariable *DGV = cast<GlobalVariable>(ValueMap[SGV]); |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 278 | if (DGV->hasInitializer()) { |
| 279 | assert(SGV->getLinkage() == DGV->getLinkage()); |
| 280 | if (SGV->hasExternalLinkage()) { |
| 281 | if (DGV->getInitializer() != SInit) |
| 282 | return Error(Err, "Global Variable Collision on '" + |
| 283 | SGV->getType()->getDescription() +"':%"+SGV->getName()+ |
| 284 | " - Global variables have different initializers"); |
| 285 | } else if (DGV->hasLinkOnceLinkage()) { |
| 286 | // Nothing is required, mapped values will take the new global |
| 287 | // automatically. |
| 288 | } else if (DGV->hasAppendingLinkage()) { |
| 289 | assert(0 && "Appending linkage unimplemented!"); |
| 290 | } else { |
| 291 | assert(0 && "Unknown linkage!"); |
| 292 | } |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 293 | } else { |
| 294 | // Copy the initializer over now... |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 295 | DGV->setInitializer(SInit); |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 296 | } |
| 297 | } |
| 298 | } |
| 299 | return false; |
| 300 | } |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 301 | |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 302 | // LinkFunctionProtos - Link the functions together between the two modules, |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 303 | // without doing function bodies... this just adds external function prototypes |
| 304 | // to the Dest function... |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 305 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 306 | static bool LinkFunctionProtos(Module *Dest, const Module *Src, |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 307 | std::map<const Value*, Value*> &ValueMap, |
| 308 | std::string *Err) { |
Chris Lattner | b91b657 | 2002-12-03 18:32:30 +0000 | [diff] [blame] | 309 | SymbolTable *ST = (SymbolTable*)&Dest->getSymbolTable(); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 310 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 311 | // Loop over all of the functions in the src module, mapping them over as we |
| 312 | // go |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 313 | // |
| 314 | for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 315 | const Function *SF = I; // SrcFunction |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 316 | Function *DF = 0; |
| 317 | if (SF->hasName()) |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 318 | // The same named thing is a Function, because the only two things |
| 319 | // that may be in a module level symbol table are Global Vars and |
| 320 | // Functions, and they both have distinct, nonoverlapping, possible types. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 321 | // |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 322 | DF = cast_or_null<Function>(ST->lookup(SF->getType(), SF->getName())); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 323 | |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 324 | bool SFExtern = SF->isExternal(); |
| 325 | bool DFExtern = DF ? DF->isExternal() : false; |
| 326 | |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 327 | if (!DF || SF->hasInternalLinkage() || DF->hasInternalLinkage()) { |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 328 | // Function does not already exist, simply insert an function signature |
| 329 | // identical to SF into the dest module... |
Chris Lattner | 2719bac | 2003-04-21 21:15:04 +0000 | [diff] [blame] | 330 | Function *NewDF = new Function(SF->getFunctionType(), SF->getLinkage(), |
| 331 | SF->getName(), Dest); |
| 332 | |
| 333 | // If the LLVM runtime renamed the function, but it is an externally |
| 334 | // visible symbol, DF must be an existing function with internal linkage. |
| 335 | // Rename it. |
| 336 | if (NewDF->getName() != SF->getName() && !NewDF->hasInternalLinkage()) { |
| 337 | assert(DF && DF->getName() == SF->getName() &&DF->hasInternalLinkage()); |
| 338 | DF->setName(""); |
| 339 | NewDF->setName(SF->getName()); // Force the name back |
| 340 | DF->setName(SF->getName()); // This will cause a renaming |
| 341 | assert(NewDF->getName() == SF->getName() && |
| 342 | DF->getName() != SF->getName()); |
| 343 | } |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 344 | |
| 345 | // ... and remember this mapping... |
Chris Lattner | 2719bac | 2003-04-21 21:15:04 +0000 | [diff] [blame] | 346 | ValueMap.insert(std::make_pair(SF, NewDF)); |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 347 | } else if (SF->getLinkage() == GlobalValue::AppendingLinkage) { |
| 348 | return Error(Err, "Functions named '" + SF->getName() + |
| 349 | "' have appending linkage!"); |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 350 | } else if (!SFExtern && !DFExtern && SF->getLinkage() != DF->getLinkage()) { |
| 351 | return Error(Err, "Functions named '" + SF->getName() + |
| 352 | "' have different linkage specifiers!"); |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 353 | } else if (SF->getLinkage() == GlobalValue::ExternalLinkage) { |
| 354 | // If the function has a name, and that name is already in use in the Dest |
| 355 | // module, make sure that the name is a compatible function... |
| 356 | // |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 357 | // Check to make sure the function is not defined in both modules... |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 358 | if (!SF->isExternal() && !DF->isExternal()) |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 359 | return Error(Err, "Function '" + |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 360 | SF->getFunctionType()->getDescription() + "':\"" + |
| 361 | SF->getName() + "\" - Function is already defined!"); |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 362 | |
Chris Lattner | 0fec08e | 2003-04-21 21:07:05 +0000 | [diff] [blame] | 363 | if (DFExtern) |
| 364 | DF->setLinkage(SF->getLinkage()); |
| 365 | |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 366 | // Otherwise, just remember this mapping... |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 367 | ValueMap.insert(std::make_pair(SF, DF)); |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 368 | } else if (SF->getLinkage() == GlobalValue::LinkOnceLinkage) { |
| 369 | // Completely ignore the source function. |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 370 | ValueMap.insert(std::make_pair(SF, DF)); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 371 | } |
| 372 | } |
| 373 | return false; |
| 374 | } |
| 375 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 376 | // LinkFunctionBody - Copy the source function over into the dest function and |
| 377 | // fix up references to values. At this point we know that Dest is an external |
| 378 | // function, and that Src is not. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 379 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 380 | static bool LinkFunctionBody(Function *Dest, const Function *Src, |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 381 | std::map<const Value*, Value*> &GlobalMap, |
| 382 | std::string *Err) { |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 383 | assert(Src && Dest && Dest->isExternal() && !Src->isExternal()); |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 384 | std::map<const Value*, Value*> LocalMap; // Map for function local values |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 385 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 386 | // Go through and convert function arguments over... |
Chris Lattner | 69da5cf | 2002-10-13 20:57:00 +0000 | [diff] [blame] | 387 | Function::aiterator DI = Dest->abegin(); |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 388 | for (Function::const_aiterator I = Src->abegin(), E = Src->aend(); |
Chris Lattner | 69da5cf | 2002-10-13 20:57:00 +0000 | [diff] [blame] | 389 | I != E; ++I, ++DI) { |
| 390 | DI->setName(I->getName()); // Copy the name information over... |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 391 | |
| 392 | // Add a mapping to our local map |
Chris Lattner | 69da5cf | 2002-10-13 20:57:00 +0000 | [diff] [blame] | 393 | LocalMap.insert(std::make_pair(I, DI)); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 394 | } |
| 395 | |
| 396 | // Loop over all of the basic blocks, copying the instructions over... |
| 397 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 398 | for (Function::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 399 | // Create new basic block and add to mapping and the Dest function... |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 400 | BasicBlock *DBB = new BasicBlock(I->getName(), Dest); |
| 401 | LocalMap.insert(std::make_pair(I, DBB)); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 402 | |
| 403 | // Loop over all of the instructions in the src basic block, copying them |
| 404 | // over. Note that this is broken in a strict sense because the cloned |
| 405 | // instructions will still be referencing values in the Src module, not |
| 406 | // the remapped values. In our case, however, we will not get caught and |
| 407 | // so we can delay patching the values up until later... |
| 408 | // |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 409 | for (BasicBlock::const_iterator II = I->begin(), IE = I->end(); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 410 | II != IE; ++II) { |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 411 | Instruction *DI = II->clone(); |
| 412 | DI->setName(II->getName()); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 413 | DBB->getInstList().push_back(DI); |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 414 | LocalMap.insert(std::make_pair(II, DI)); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 415 | } |
| 416 | } |
| 417 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 418 | // At this point, all of the instructions and values of the function are now |
| 419 | // copied over. The only problem is that they are still referencing values in |
| 420 | // the Source function as operands. Loop through all of the operands of the |
| 421 | // functions and patch them up to point to the local versions... |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 422 | // |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 423 | for (Function::iterator BB = Dest->begin(), BE = Dest->end(); BB != BE; ++BB) |
| 424 | for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I) |
| 425 | for (Instruction::op_iterator OI = I->op_begin(), OE = I->op_end(); |
Chris Lattner | 221d688 | 2002-02-12 21:07:25 +0000 | [diff] [blame] | 426 | OI != OE; ++OI) |
| 427 | *OI = RemapOperand(*OI, LocalMap, &GlobalMap); |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 428 | |
| 429 | return false; |
| 430 | } |
| 431 | |
| 432 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 433 | // LinkFunctionBodies - Link in the function bodies that are defined in the |
| 434 | // source module into the DestModule. This consists basically of copying the |
| 435 | // function over and fixing up references to values. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 436 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 437 | static bool LinkFunctionBodies(Module *Dest, const Module *Src, |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 438 | std::map<const Value*, Value*> &ValueMap, |
| 439 | std::string *Err) { |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 440 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 441 | // Loop over all of the functions in the src module, mapping them over as we |
| 442 | // go |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 443 | // |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 444 | for (Module::const_iterator SF = Src->begin(), E = Src->end(); SF != E; ++SF){ |
| 445 | if (!SF->isExternal()) { // No body if function is external |
| 446 | Function *DF = cast<Function>(ValueMap[SF]); // Destination function |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 447 | |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 448 | // DF not external SF external? |
| 449 | if (!DF->isExternal()) { |
Chris Lattner | 4ad02e7 | 2003-04-16 20:28:45 +0000 | [diff] [blame] | 450 | if (DF->hasLinkOnceLinkage()) continue; // No relinkage for link-once! |
Chris Lattner | c2d774b | 2001-10-23 20:43:42 +0000 | [diff] [blame] | 451 | if (Err) |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 452 | *Err = "Function '" + (SF->hasName() ? SF->getName() :std::string("")) |
| 453 | + "' body multiply defined!"; |
Chris Lattner | c2d774b | 2001-10-23 20:43:42 +0000 | [diff] [blame] | 454 | return true; |
| 455 | } |
| 456 | |
Chris Lattner | 1896150 | 2002-06-25 16:12:52 +0000 | [diff] [blame] | 457 | if (LinkFunctionBody(DF, SF, ValueMap, Err)) return true; |
Chris Lattner | c2d774b | 2001-10-23 20:43:42 +0000 | [diff] [blame] | 458 | } |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 459 | } |
| 460 | return false; |
| 461 | } |
| 462 | |
Chris Lattner | 52f7e90 | 2001-10-13 07:03:50 +0000 | [diff] [blame] | 463 | |
| 464 | |
| 465 | // LinkModules - This function links two modules together, with the resulting |
| 466 | // left module modified to be the composite of the two input modules. If an |
| 467 | // error occurs, true is returned and ErrorMsg (if not null) is set to indicate |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 468 | // the problem. Upon failure, the Dest module could be in a modified state, and |
| 469 | // shouldn't be relied on to be consistent. |
Chris Lattner | 52f7e90 | 2001-10-13 07:03:50 +0000 | [diff] [blame] | 470 | // |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 471 | bool LinkModules(Module *Dest, const Module *Src, std::string *ErrorMsg) { |
Chris Lattner | 43a9994 | 2003-04-22 19:13:20 +0000 | [diff] [blame^] | 472 | if (Dest->getEndianness() != Src->getEndianness()) |
| 473 | std::cerr << "WARNING: Linking two modules of different endianness!\n"; |
| 474 | if (Dest->getPointerSize() != Src->getPointerSize()) |
| 475 | std::cerr << "WARNING: Linking two modules of different pointer size!\n"; |
Chris Lattner | 2c236f3 | 2001-11-03 05:18:24 +0000 | [diff] [blame] | 476 | |
| 477 | // LinkTypes - Go through the symbol table of the Src module and see if any |
| 478 | // types are named in the src module that are not named in the Dst module. |
| 479 | // Make sure there are no type name conflicts. |
| 480 | // |
| 481 | if (LinkTypes(Dest, Src, ErrorMsg)) return true; |
| 482 | |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 483 | // ValueMap - Mapping of values from what they used to be in Src, to what they |
| 484 | // are now in Dest. |
| 485 | // |
Chris Lattner | 5c2d335 | 2003-01-30 19:53:34 +0000 | [diff] [blame] | 486 | std::map<const Value*, Value*> ValueMap; |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 487 | |
Chris Lattner | 8d2de8a | 2001-10-15 03:12:52 +0000 | [diff] [blame] | 488 | // Insert all of the globals in src into the Dest module... without |
| 489 | // initializers |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 490 | if (LinkGlobals(Dest, Src, ValueMap, ErrorMsg)) return true; |
| 491 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 492 | // Link the functions together between the two modules, without doing function |
| 493 | // bodies... this just adds external function prototypes to the Dest |
| 494 | // function... We do this so that when we begin processing function bodies, |
| 495 | // all of the global values that may be referenced are available in our |
| 496 | // ValueMap. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 497 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 498 | if (LinkFunctionProtos(Dest, Src, ValueMap, ErrorMsg)) return true; |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 499 | |
Chris Lattner | 6cdf197 | 2002-07-18 00:13:08 +0000 | [diff] [blame] | 500 | // Update the initializers in the Dest module now that all globals that may |
| 501 | // be referenced are in Dest. |
| 502 | // |
| 503 | if (LinkGlobalInits(Dest, Src, ValueMap, ErrorMsg)) return true; |
| 504 | |
Chris Lattner | c8cc4cb | 2002-05-07 18:36:35 +0000 | [diff] [blame] | 505 | // Link in the function bodies that are defined in the source module into the |
| 506 | // DestModule. This consists basically of copying the function over and |
| 507 | // fixing up references to values. |
Chris Lattner | 5c377c5 | 2001-10-14 23:29:15 +0000 | [diff] [blame] | 508 | // |
Chris Lattner | 79df7c0 | 2002-03-26 18:01:55 +0000 | [diff] [blame] | 509 | if (LinkFunctionBodies(Dest, Src, ValueMap, ErrorMsg)) return true; |
Chris Lattner | 52f7e90 | 2001-10-13 07:03:50 +0000 | [diff] [blame] | 510 | |
| 511 | return false; |
| 512 | } |
Vikram S. Adve | 9466f51 | 2001-10-28 21:38:02 +0000 | [diff] [blame] | 513 | |