From: Chris Lattner Date: Tue, 7 May 2002 18:36:35 +0000 (+0000) Subject: Updates to move some header files out of include/llvm/Transforms into X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=c8cc4cb03bd90f89be7fe1649542a2d5ae689632;p=oota-llvm.git Updates to move some header files out of include/llvm/Transforms into the Scalar and Utils subdirectories git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@2523 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Analysis/PostDominators.cpp b/lib/Analysis/PostDominators.cpp index 9697f1f99a2..59a61cbdd27 100644 --- a/lib/Analysis/PostDominators.cpp +++ b/lib/Analysis/PostDominators.cpp @@ -6,7 +6,7 @@ //===----------------------------------------------------------------------===// #include "llvm/Analysis/Dominators.h" -#include "llvm/Transforms/UnifyFunctionExitNodes.h" +#include "llvm/Transforms/Scalar/UnifyFunctionExitNodes.h" #include "llvm/Support/CFG.h" #include "Support/DepthFirstIterator.h" #include "Support/STLExtras.h" diff --git a/lib/ExecutionEngine/Interpreter/UserInput.cpp b/lib/ExecutionEngine/Interpreter/UserInput.cpp index 018e6f7c0f4..91addb6dd68 100644 --- a/lib/ExecutionEngine/Interpreter/UserInput.cpp +++ b/lib/ExecutionEngine/Interpreter/UserInput.cpp @@ -7,7 +7,7 @@ #include "Interpreter.h" #include "llvm/Bytecode/Reader.h" #include "llvm/DerivedTypes.h" -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include using std::string; using std::cout; diff --git a/lib/Linker/LinkModules.cpp b/lib/Linker/LinkModules.cpp index 2de6f81fdf9..7d43047c0af 100644 --- a/lib/Linker/LinkModules.cpp +++ b/lib/Linker/LinkModules.cpp @@ -5,11 +5,11 @@ // Specifically, this: // * Merges global variables between the two modules // * Uninit + Uninit = Init, Init + Uninit = Init, Init + Init = Error if != -// * Merges methods between two modules +// * Merges functions between two modules // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Module.h" #include "llvm/Function.h" #include "llvm/BasicBlock.h" @@ -235,8 +235,8 @@ static bool LinkGlobalInits(Module *Dest, const Module *Src, } // LinkFunctionProtos - Link the functions together between the two modules, -// without doing method bodies... this just adds external method prototypes to -// the Dest function... +// without doing function bodies... this just adds external function prototypes +// to the Dest function... // static bool LinkFunctionProtos(Module *Dest, const Module *Src, map &ValueMap, @@ -245,14 +245,15 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // level symbol table... SymbolTable *ST = Src->getSymbolTable() ? Dest->getSymbolTableSure() : 0; - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const Function *SM = *I; // SrcFunction Value *V; - // If the method has a name, and that name is already in use in the - // Dest module, make sure that the name is a compatible method... + // If the function has a name, and that name is already in use in the Dest + // module, make sure that the name is a compatible function... // if (SM->hasExternalLinkage() && SM->hasName() && (V = ST->lookup(SM->getType(), SM->getName())) && @@ -263,7 +264,7 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Function *DM = cast(V); // DestFunction - // Check to make sure the method is not defined in both modules... + // Check to make sure the function is not defined in both modules... if (!SM->isExternal() && !DM->isExternal()) return Error(Err, "Function '" + SM->getFunctionType()->getDescription() + "':\"" + @@ -272,13 +273,13 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Otherwise, just remember this mapping... ValueMap.insert(std::make_pair(SM, DM)); } else { - // Function does not already exist, simply insert an external method + // Function does not already exist, simply insert an external function // signature identical to SM into the dest module... Function *DM = new Function(SM->getFunctionType(), SM->hasInternalLinkage(), SM->getName()); - // Add the method signature to the dest module... + // Add the function signature to the dest module... Dest->getFunctionList().push_back(DM); // ... and remember this mapping... @@ -288,23 +289,23 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, return false; } -// LinkFunctionBody - Copy the source method over into the dest method -// and fix up references to values. At this point we know that Dest -// is an external method, and that Src is not. +// LinkFunctionBody - Copy the source function over into the dest function and +// fix up references to values. At this point we know that Dest is an external +// function, and that Src is not. // static bool LinkFunctionBody(Function *Dest, const Function *Src, const map &GlobalMap, string *Err = 0) { assert(Src && Dest && Dest->isExternal() && !Src->isExternal()); - map LocalMap; // Map for method local values + map LocalMap; // Map for function local values - // Go through and convert method arguments over... + // Go through and convert function arguments over... for (Function::ArgumentListType::const_iterator I = Src->getArgumentList().begin(), E = Src->getArgumentList().end(); I != E; ++I) { const Argument *SMA = *I; - // Create the new method argument and add to the dest method... + // Create the new function argument and add to the dest function... Argument *DMA = new Argument(SMA->getType(), SMA->getName()); Dest->getArgumentList().push_back(DMA); @@ -317,7 +318,7 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, for (Function::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const BasicBlock *SBB = *I; - // Create new basic block and add to mapping and the Dest method... + // Create new basic block and add to mapping and the Dest function... BasicBlock *DBB = new BasicBlock(SBB->getName(), Dest); LocalMap.insert(std::make_pair(SBB, DBB)); @@ -337,10 +338,10 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } } - // At this point, all of the instructions and values of the method are now - // copied over. The only problem is that they are still referencing values - // in the Source method as operands. Loop through all of the operands of the - // methods and patch them up to point to the local versions... + // At this point, all of the instructions and values of the function are now + // copied over. The only problem is that they are still referencing values in + // the Source function as operands. Loop through all of the operands of the + // functions and patch them up to point to the local versions... // for (Function::iterator BI = Dest->begin(), BE = Dest->end(); BI != BE; ++BI) { @@ -358,20 +359,21 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } -// LinkFunctionBodies - Link in the method bodies that are defined in the source -// module into the DestModule. This consists basically of copying the method -// over and fixing up references to values. +// LinkFunctionBodies - Link in the function bodies that are defined in the +// source module into the DestModule. This consists basically of copying the +// function over and fixing up references to values. // static bool LinkFunctionBodies(Module *Dest, const Module *Src, map &ValueMap, string *Err = 0) { - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { - const Function *SM = *I; // Source Function - if (!SM->isExternal()) { // No body if method is external - Function *DM = cast(ValueMap[SM]); // Destination method + const Function *SM = *I; // Source Function + if (!SM->isExternal()) { // No body if function is external + Function *DM = cast(ValueMap[SM]); // Destination function // DM not external SM external? if (!DM->isExternal()) { @@ -417,16 +419,17 @@ bool LinkModules(Module *Dest, const Module *Src, string *ErrorMsg = 0) { // if (LinkGlobalInits(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link the methods together between the two modules, without doing method - // bodies... this just adds external method prototypes to the Dest method... - // We do this so that when we begin processing method bodies, all of the - // global values that may be referenced are available in our ValueMap. + // Link the functions together between the two modules, without doing function + // bodies... this just adds external function prototypes to the Dest + // function... We do this so that when we begin processing function bodies, + // all of the global values that may be referenced are available in our + // ValueMap. // if (LinkFunctionProtos(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link in the method bodies that are defined in the source module into the - // DestModule. This consists basically of copying the method over and fixing - // up references to values. + // Link in the function bodies that are defined in the source module into the + // DestModule. This consists basically of copying the function over and + // fixing up references to values. // if (LinkFunctionBodies(Dest, Src, ValueMap, ErrorMsg)) return true; diff --git a/lib/Transforms/IPO/OldPoolAllocate.cpp b/lib/Transforms/IPO/OldPoolAllocate.cpp index fe30daadd15..5190dd2fd8f 100644 --- a/lib/Transforms/IPO/OldPoolAllocate.cpp +++ b/lib/Transforms/IPO/OldPoolAllocate.cpp @@ -10,7 +10,7 @@ //===----------------------------------------------------------------------===// #include "llvm/Transforms/IPO/PoolAllocate.h" -#include "llvm/Transforms/CloneFunction.h" +#include "llvm/Transforms/Utils/CloneFunction.h" #include "llvm/Analysis/DataStructureGraph.h" #include "llvm/Module.h" #include "llvm/Function.h" diff --git a/lib/Transforms/Instrumentation/ProfilePaths/ProfilePaths.cpp b/lib/Transforms/Instrumentation/ProfilePaths/ProfilePaths.cpp index ebd290cd915..afc53053d08 100644 --- a/lib/Transforms/Instrumentation/ProfilePaths/ProfilePaths.cpp +++ b/lib/Transforms/Instrumentation/ProfilePaths/ProfilePaths.cpp @@ -25,7 +25,7 @@ //===----------------------------------------------------------------------===// #include "llvm/Transforms/Instrumentation/ProfilePaths.h" -#include "llvm/Transforms/UnifyFunctionExitNodes.h" +#include "llvm/Transforms/Scalar/UnifyFunctionExitNodes.h" #include "llvm/Support/CFG.h" #include "llvm/Constants.h" #include "llvm/DerivedTypes.h" diff --git a/lib/Transforms/Scalar/SymbolStripping.cpp b/lib/Transforms/Scalar/SymbolStripping.cpp index 8320716d54b..84531c77e4d 100644 --- a/lib/Transforms/Scalar/SymbolStripping.cpp +++ b/lib/Transforms/Scalar/SymbolStripping.cpp @@ -14,7 +14,7 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/SymbolStripping.h" +#include "llvm/Transforms/Scalar/SymbolStripping.h" #include "llvm/Module.h" #include "llvm/Function.h" #include "llvm/SymbolTable.h" diff --git a/lib/Transforms/Utils/CloneFunction.cpp b/lib/Transforms/Utils/CloneFunction.cpp index 808ea48e673..6b1b24ac576 100644 --- a/lib/Transforms/Utils/CloneFunction.cpp +++ b/lib/Transforms/Utils/CloneFunction.cpp @@ -1,7 +1,7 @@ // FIXME: document -#include "llvm/Transforms/CloneFunction.h" +#include "llvm/Transforms/Utils/CloneFunction.h" #include "llvm/Function.h" #include "llvm/BasicBlock.h" #include "llvm/Instruction.h" diff --git a/lib/Transforms/Utils/Linker.cpp b/lib/Transforms/Utils/Linker.cpp index 2de6f81fdf9..7d43047c0af 100644 --- a/lib/Transforms/Utils/Linker.cpp +++ b/lib/Transforms/Utils/Linker.cpp @@ -5,11 +5,11 @@ // Specifically, this: // * Merges global variables between the two modules // * Uninit + Uninit = Init, Init + Uninit = Init, Init + Init = Error if != -// * Merges methods between two modules +// * Merges functions between two modules // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Module.h" #include "llvm/Function.h" #include "llvm/BasicBlock.h" @@ -235,8 +235,8 @@ static bool LinkGlobalInits(Module *Dest, const Module *Src, } // LinkFunctionProtos - Link the functions together between the two modules, -// without doing method bodies... this just adds external method prototypes to -// the Dest function... +// without doing function bodies... this just adds external function prototypes +// to the Dest function... // static bool LinkFunctionProtos(Module *Dest, const Module *Src, map &ValueMap, @@ -245,14 +245,15 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // level symbol table... SymbolTable *ST = Src->getSymbolTable() ? Dest->getSymbolTableSure() : 0; - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const Function *SM = *I; // SrcFunction Value *V; - // If the method has a name, and that name is already in use in the - // Dest module, make sure that the name is a compatible method... + // If the function has a name, and that name is already in use in the Dest + // module, make sure that the name is a compatible function... // if (SM->hasExternalLinkage() && SM->hasName() && (V = ST->lookup(SM->getType(), SM->getName())) && @@ -263,7 +264,7 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Function *DM = cast(V); // DestFunction - // Check to make sure the method is not defined in both modules... + // Check to make sure the function is not defined in both modules... if (!SM->isExternal() && !DM->isExternal()) return Error(Err, "Function '" + SM->getFunctionType()->getDescription() + "':\"" + @@ -272,13 +273,13 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Otherwise, just remember this mapping... ValueMap.insert(std::make_pair(SM, DM)); } else { - // Function does not already exist, simply insert an external method + // Function does not already exist, simply insert an external function // signature identical to SM into the dest module... Function *DM = new Function(SM->getFunctionType(), SM->hasInternalLinkage(), SM->getName()); - // Add the method signature to the dest module... + // Add the function signature to the dest module... Dest->getFunctionList().push_back(DM); // ... and remember this mapping... @@ -288,23 +289,23 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, return false; } -// LinkFunctionBody - Copy the source method over into the dest method -// and fix up references to values. At this point we know that Dest -// is an external method, and that Src is not. +// LinkFunctionBody - Copy the source function over into the dest function and +// fix up references to values. At this point we know that Dest is an external +// function, and that Src is not. // static bool LinkFunctionBody(Function *Dest, const Function *Src, const map &GlobalMap, string *Err = 0) { assert(Src && Dest && Dest->isExternal() && !Src->isExternal()); - map LocalMap; // Map for method local values + map LocalMap; // Map for function local values - // Go through and convert method arguments over... + // Go through and convert function arguments over... for (Function::ArgumentListType::const_iterator I = Src->getArgumentList().begin(), E = Src->getArgumentList().end(); I != E; ++I) { const Argument *SMA = *I; - // Create the new method argument and add to the dest method... + // Create the new function argument and add to the dest function... Argument *DMA = new Argument(SMA->getType(), SMA->getName()); Dest->getArgumentList().push_back(DMA); @@ -317,7 +318,7 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, for (Function::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const BasicBlock *SBB = *I; - // Create new basic block and add to mapping and the Dest method... + // Create new basic block and add to mapping and the Dest function... BasicBlock *DBB = new BasicBlock(SBB->getName(), Dest); LocalMap.insert(std::make_pair(SBB, DBB)); @@ -337,10 +338,10 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } } - // At this point, all of the instructions and values of the method are now - // copied over. The only problem is that they are still referencing values - // in the Source method as operands. Loop through all of the operands of the - // methods and patch them up to point to the local versions... + // At this point, all of the instructions and values of the function are now + // copied over. The only problem is that they are still referencing values in + // the Source function as operands. Loop through all of the operands of the + // functions and patch them up to point to the local versions... // for (Function::iterator BI = Dest->begin(), BE = Dest->end(); BI != BE; ++BI) { @@ -358,20 +359,21 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } -// LinkFunctionBodies - Link in the method bodies that are defined in the source -// module into the DestModule. This consists basically of copying the method -// over and fixing up references to values. +// LinkFunctionBodies - Link in the function bodies that are defined in the +// source module into the DestModule. This consists basically of copying the +// function over and fixing up references to values. // static bool LinkFunctionBodies(Module *Dest, const Module *Src, map &ValueMap, string *Err = 0) { - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { - const Function *SM = *I; // Source Function - if (!SM->isExternal()) { // No body if method is external - Function *DM = cast(ValueMap[SM]); // Destination method + const Function *SM = *I; // Source Function + if (!SM->isExternal()) { // No body if function is external + Function *DM = cast(ValueMap[SM]); // Destination function // DM not external SM external? if (!DM->isExternal()) { @@ -417,16 +419,17 @@ bool LinkModules(Module *Dest, const Module *Src, string *ErrorMsg = 0) { // if (LinkGlobalInits(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link the methods together between the two modules, without doing method - // bodies... this just adds external method prototypes to the Dest method... - // We do this so that when we begin processing method bodies, all of the - // global values that may be referenced are available in our ValueMap. + // Link the functions together between the two modules, without doing function + // bodies... this just adds external function prototypes to the Dest + // function... We do this so that when we begin processing function bodies, + // all of the global values that may be referenced are available in our + // ValueMap. // if (LinkFunctionProtos(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link in the method bodies that are defined in the source module into the - // DestModule. This consists basically of copying the method over and fixing - // up references to values. + // Link in the function bodies that are defined in the source module into the + // DestModule. This consists basically of copying the function over and + // fixing up references to values. // if (LinkFunctionBodies(Dest, Src, ValueMap, ErrorMsg)) return true; diff --git a/lib/Transforms/Utils/UnifyFunctionExitNodes.cpp b/lib/Transforms/Utils/UnifyFunctionExitNodes.cpp index 0fa87f8e8d2..37dc4ab9544 100644 --- a/lib/Transforms/Utils/UnifyFunctionExitNodes.cpp +++ b/lib/Transforms/Utils/UnifyFunctionExitNodes.cpp @@ -5,7 +5,7 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/UnifyFunctionExitNodes.h" +#include "llvm/Transforms/Scalar/UnifyFunctionExitNodes.h" #include "llvm/BasicBlock.h" #include "llvm/Function.h" #include "llvm/iTerminators.h" diff --git a/lib/VMCore/Dominators.cpp b/lib/VMCore/Dominators.cpp index 9697f1f99a2..59a61cbdd27 100644 --- a/lib/VMCore/Dominators.cpp +++ b/lib/VMCore/Dominators.cpp @@ -6,7 +6,7 @@ //===----------------------------------------------------------------------===// #include "llvm/Analysis/Dominators.h" -#include "llvm/Transforms/UnifyFunctionExitNodes.h" +#include "llvm/Transforms/Scalar/UnifyFunctionExitNodes.h" #include "llvm/Support/CFG.h" #include "Support/DepthFirstIterator.h" #include "Support/STLExtras.h" diff --git a/lib/VMCore/Linker.cpp b/lib/VMCore/Linker.cpp index 2de6f81fdf9..7d43047c0af 100644 --- a/lib/VMCore/Linker.cpp +++ b/lib/VMCore/Linker.cpp @@ -5,11 +5,11 @@ // Specifically, this: // * Merges global variables between the two modules // * Uninit + Uninit = Init, Init + Uninit = Init, Init + Init = Error if != -// * Merges methods between two modules +// * Merges functions between two modules // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Module.h" #include "llvm/Function.h" #include "llvm/BasicBlock.h" @@ -235,8 +235,8 @@ static bool LinkGlobalInits(Module *Dest, const Module *Src, } // LinkFunctionProtos - Link the functions together between the two modules, -// without doing method bodies... this just adds external method prototypes to -// the Dest function... +// without doing function bodies... this just adds external function prototypes +// to the Dest function... // static bool LinkFunctionProtos(Module *Dest, const Module *Src, map &ValueMap, @@ -245,14 +245,15 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // level symbol table... SymbolTable *ST = Src->getSymbolTable() ? Dest->getSymbolTableSure() : 0; - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const Function *SM = *I; // SrcFunction Value *V; - // If the method has a name, and that name is already in use in the - // Dest module, make sure that the name is a compatible method... + // If the function has a name, and that name is already in use in the Dest + // module, make sure that the name is a compatible function... // if (SM->hasExternalLinkage() && SM->hasName() && (V = ST->lookup(SM->getType(), SM->getName())) && @@ -263,7 +264,7 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Function *DM = cast(V); // DestFunction - // Check to make sure the method is not defined in both modules... + // Check to make sure the function is not defined in both modules... if (!SM->isExternal() && !DM->isExternal()) return Error(Err, "Function '" + SM->getFunctionType()->getDescription() + "':\"" + @@ -272,13 +273,13 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, // Otherwise, just remember this mapping... ValueMap.insert(std::make_pair(SM, DM)); } else { - // Function does not already exist, simply insert an external method + // Function does not already exist, simply insert an external function // signature identical to SM into the dest module... Function *DM = new Function(SM->getFunctionType(), SM->hasInternalLinkage(), SM->getName()); - // Add the method signature to the dest module... + // Add the function signature to the dest module... Dest->getFunctionList().push_back(DM); // ... and remember this mapping... @@ -288,23 +289,23 @@ static bool LinkFunctionProtos(Module *Dest, const Module *Src, return false; } -// LinkFunctionBody - Copy the source method over into the dest method -// and fix up references to values. At this point we know that Dest -// is an external method, and that Src is not. +// LinkFunctionBody - Copy the source function over into the dest function and +// fix up references to values. At this point we know that Dest is an external +// function, and that Src is not. // static bool LinkFunctionBody(Function *Dest, const Function *Src, const map &GlobalMap, string *Err = 0) { assert(Src && Dest && Dest->isExternal() && !Src->isExternal()); - map LocalMap; // Map for method local values + map LocalMap; // Map for function local values - // Go through and convert method arguments over... + // Go through and convert function arguments over... for (Function::ArgumentListType::const_iterator I = Src->getArgumentList().begin(), E = Src->getArgumentList().end(); I != E; ++I) { const Argument *SMA = *I; - // Create the new method argument and add to the dest method... + // Create the new function argument and add to the dest function... Argument *DMA = new Argument(SMA->getType(), SMA->getName()); Dest->getArgumentList().push_back(DMA); @@ -317,7 +318,7 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, for (Function::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { const BasicBlock *SBB = *I; - // Create new basic block and add to mapping and the Dest method... + // Create new basic block and add to mapping and the Dest function... BasicBlock *DBB = new BasicBlock(SBB->getName(), Dest); LocalMap.insert(std::make_pair(SBB, DBB)); @@ -337,10 +338,10 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } } - // At this point, all of the instructions and values of the method are now - // copied over. The only problem is that they are still referencing values - // in the Source method as operands. Loop through all of the operands of the - // methods and patch them up to point to the local versions... + // At this point, all of the instructions and values of the function are now + // copied over. The only problem is that they are still referencing values in + // the Source function as operands. Loop through all of the operands of the + // functions and patch them up to point to the local versions... // for (Function::iterator BI = Dest->begin(), BE = Dest->end(); BI != BE; ++BI) { @@ -358,20 +359,21 @@ static bool LinkFunctionBody(Function *Dest, const Function *Src, } -// LinkFunctionBodies - Link in the method bodies that are defined in the source -// module into the DestModule. This consists basically of copying the method -// over and fixing up references to values. +// LinkFunctionBodies - Link in the function bodies that are defined in the +// source module into the DestModule. This consists basically of copying the +// function over and fixing up references to values. // static bool LinkFunctionBodies(Module *Dest, const Module *Src, map &ValueMap, string *Err = 0) { - // Loop over all of the methods in the src module, mapping them over as we go + // Loop over all of the functions in the src module, mapping them over as we + // go // for (Module::const_iterator I = Src->begin(), E = Src->end(); I != E; ++I) { - const Function *SM = *I; // Source Function - if (!SM->isExternal()) { // No body if method is external - Function *DM = cast(ValueMap[SM]); // Destination method + const Function *SM = *I; // Source Function + if (!SM->isExternal()) { // No body if function is external + Function *DM = cast(ValueMap[SM]); // Destination function // DM not external SM external? if (!DM->isExternal()) { @@ -417,16 +419,17 @@ bool LinkModules(Module *Dest, const Module *Src, string *ErrorMsg = 0) { // if (LinkGlobalInits(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link the methods together between the two modules, without doing method - // bodies... this just adds external method prototypes to the Dest method... - // We do this so that when we begin processing method bodies, all of the - // global values that may be referenced are available in our ValueMap. + // Link the functions together between the two modules, without doing function + // bodies... this just adds external function prototypes to the Dest + // function... We do this so that when we begin processing function bodies, + // all of the global values that may be referenced are available in our + // ValueMap. // if (LinkFunctionProtos(Dest, Src, ValueMap, ErrorMsg)) return true; - // Link in the method bodies that are defined in the source module into the - // DestModule. This consists basically of copying the method over and fixing - // up references to values. + // Link in the function bodies that are defined in the source module into the + // DestModule. This consists basically of copying the function over and + // fixing up references to values. // if (LinkFunctionBodies(Dest, Src, ValueMap, ErrorMsg)) return true; diff --git a/tools/gccld/gccld.cpp b/tools/gccld/gccld.cpp index 8d783050af2..9d81198ab74 100644 --- a/tools/gccld/gccld.cpp +++ b/tools/gccld/gccld.cpp @@ -14,14 +14,14 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Module.h" #include "llvm/PassManager.h" #include "llvm/Bytecode/Reader.h" #include "llvm/Bytecode/WriteBytecodePass.h" -#include "llvm/Transforms/SymbolStripping.h" #include "llvm/Transforms/CleanupGCCOutput.h" #include "llvm/Transforms/ConstantMerge.h" +#include "llvm/Transforms/ScalarSymbolStripping.h" #include "llvm/Transforms/IPO/GlobalDCE.h" #include "llvm/Transforms/IPO/Internalize.h" #include "Support/CommandLine.h" diff --git a/tools/link/link.cpp b/tools/link/link.cpp index 2abde5f7cdb..c975abc3f64 100644 --- a/tools/link/link.cpp +++ b/tools/link/link.cpp @@ -9,7 +9,7 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Bytecode/Reader.h" #include "llvm/Bytecode/Writer.h" #include "llvm/Module.h" diff --git a/tools/llvm-link/llvm-link.cpp b/tools/llvm-link/llvm-link.cpp index 2abde5f7cdb..c975abc3f64 100644 --- a/tools/llvm-link/llvm-link.cpp +++ b/tools/llvm-link/llvm-link.cpp @@ -9,7 +9,7 @@ // //===----------------------------------------------------------------------===// -#include "llvm/Transforms/Linker.h" +#include "llvm/Transforms/Utils/Linker.h" #include "llvm/Bytecode/Reader.h" #include "llvm/Bytecode/Writer.h" #include "llvm/Module.h" diff --git a/tools/opt/opt.cpp b/tools/opt/opt.cpp index cc2b0a6ce34..6a983abbc89 100644 --- a/tools/opt/opt.cpp +++ b/tools/opt/opt.cpp @@ -12,24 +12,24 @@ #include "llvm/Bytecode/WriteBytecodePass.h" #include "llvm/Assembly/PrintModulePass.h" #include "llvm/Analysis/Verifier.h" -#include "llvm/Transforms/UnifyFunctionExitNodes.h" #include "llvm/Transforms/ConstantMerge.h" #include "llvm/Transforms/CleanupGCCOutput.h" #include "llvm/Transforms/LevelChange.h" #include "llvm/Transforms/FunctionInlining.h" -#include "llvm/Transforms/SymbolStripping.h" #include "llvm/Transforms/ChangeAllocations.h" #include "llvm/Transforms/IPO/SimpleStructMutation.h" #include "llvm/Transforms/IPO/Internalize.h" #include "llvm/Transforms/IPO/GlobalDCE.h" #include "llvm/Transforms/IPO/PoolAllocate.h" -#include "llvm/Transforms/Scalar/DCE.h" #include "llvm/Transforms/Scalar/ConstantProp.h" +#include "llvm/Transforms/Scalar/DCE.h" +#include "llvm/Transforms/Scalar/DecomposeMultiDimRefs.h" #include "llvm/Transforms/Scalar/GCSE.h" #include "llvm/Transforms/Scalar/IndVarSimplify.h" #include "llvm/Transforms/Scalar/InstructionCombining.h" #include "llvm/Transforms/Scalar/PromoteMemoryToRegister.h" -#include "llvm/Transforms/Scalar/DecomposeMultiDimRefs.h" +#include "llvm/Transforms/Scalar/SymbolStripping.h" +#include "llvm/Transforms/Scalar/UnifyFunctionExitNodes.h" #include "llvm/Transforms/Instrumentation/TraceValues.h" #include "llvm/Transforms/Instrumentation/ProfilePaths.h" #include "Support/CommandLine.h"