std::set<std::string> ExternalNames;
bool DontInternalize;
public:
- InternalizePass(bool InternalizeEverything = true);
- InternalizePass(const std::vector <const char *>& exportList);
+ static char ID; // Pass identification, replacement for typeid
+ explicit InternalizePass(bool InternalizeEverything = true);
+ explicit InternalizePass(const std::vector <const char *>& exportList);
void LoadFile(const char *Filename);
virtual bool runOnModule(Module &M);
};
+ char InternalizePass::ID = 0;
RegisterPass<InternalizePass> X("internalize", "Internalize Global Symbols");
} // end anonymous namespace
InternalizePass::InternalizePass(bool InternalizeEverything)
- : DontInternalize(false){
+ : ModulePass((intptr_t)&ID), DontInternalize(false){
if (!APIFile.empty()) // If a filename is specified, use it
LoadFile(APIFile.c_str());
else if (!APIList.empty()) // Else, if a list is specified, use it.
}
InternalizePass::InternalizePass(const std::vector<const char *>&exportList)
- : DontInternalize(false){
+ : ModulePass((intptr_t)&ID), DontInternalize(false){
for(std::vector<const char *>::const_iterator itr = exportList.begin();
itr != exportList.end(); itr++) {
ExternalNames.insert(*itr);
ExternalNames.insert("llvm.dbg.compile_units");
ExternalNames.insert("llvm.dbg.global_variables");
ExternalNames.insert("llvm.dbg.subprograms");
+ ExternalNames.insert("llvm.global_ctors");
+ ExternalNames.insert("llvm.global_dtors");
// Mark all global variables with initializers as internal as well.
for (Module::global_iterator I = M.global_begin(), E = M.global_end();
I != E; ++I)
if (!I->isDeclaration() && !I->hasInternalLinkage() &&
!ExternalNames.count(I->getName())) {
- // Special case handling of the global ctor and dtor list. When we
- // internalize it, we mark it constant, which allows elimination of
- // the list if it's empty.
- //
- if (I->hasAppendingLinkage() && (I->getName() == "llvm.global_ctors" ||
- I->getName() == "llvm.global_dtors")) {
- // If the global ctors/dtors list has no uses, do not internalize it, as
- // there is no __main in this program, so the asmprinter should handle
- // it.
- if (I->use_empty()) continue;
-
- // Otherwise, also mark the list constant, as we know that it will not
- // be mutated any longer, and the makes simple IPO xforms automatically
- // better.
- I->setConstant(true);
- }
-
I->setLinkage(GlobalValue::InternalLinkage);
Changed = true;
++NumGlobals;