From 1b9f9c5f67027011e8c8f376871197401bd40c84 Mon Sep 17 00:00:00 2001 From: Chris Lattner Date: Sun, 13 Feb 2005 17:42:11 +0000 Subject: [PATCH] Do not put internal symbols into the symbol table. This shrinks the symbol table for archives in common cases, and prevents trying to resolve a external reference with an internal reference. This shrinks the libpython.a symbol table from 126302 to 19770 bytes. llvm-svn: 20151 --- llvm/lib/Bytecode/Reader/ReaderWrappers.cpp | 30 +++++++-------------- 1 file changed, 10 insertions(+), 20 deletions(-) diff --git a/llvm/lib/Bytecode/Reader/ReaderWrappers.cpp b/llvm/lib/Bytecode/Reader/ReaderWrappers.cpp index 9b8491327fa4..ce54b7a4e256 100644 --- a/llvm/lib/Bytecode/Reader/ReaderWrappers.cpp +++ b/llvm/lib/Bytecode/Reader/ReaderWrappers.cpp @@ -328,28 +328,18 @@ bool llvm::GetBytecodeDependentLibraries(const std::string &fname, } } -namespace { -void getSymbols(Module*M, std::vector& symbols) { +static void getSymbols(Module*M, std::vector& symbols) { // Loop over global variables - for (Module::giterator GI = M->gbegin(), GE=M->gend(); GI != GE; ++GI) { - if (GI->hasInitializer()) { - std::string name ( GI->getName() ); - if (!name.empty()) { - symbols.push_back(name); - } - } - } + for (Module::giterator GI = M->gbegin(), GE=M->gend(); GI != GE; ++GI) + if (GI->hasInitializer() && !GI->hasInternalLinkage()) + if (!GI->getName().empty()) + symbols.push_back(GI->getName()); - //Loop over functions - for (Module::iterator FI = M->begin(), FE=M->end(); FI != FE; ++FI) { - if (!FI->isExternal()) { - std::string name ( FI->getName() ); - if (!name.empty()) { - symbols.push_back(name); - } - } - } -} + // Loop over functions. + for (Module::iterator FI = M->begin(), FE = M->end(); FI != FE; ++FI) + if (!FI->isExternal() && !FI->hasInternalLinkage()) + if (!FI->getName().empty()) + symbols.push_back(FI->getName()); } // Get just the externally visible defined symbols from the bytecode