summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnton Korobeynikov <asl@math.spbu.ru>2008-03-11 00:24:53 +0000
committerAnton Korobeynikov <asl@math.spbu.ru>2008-03-11 00:24:53 +0000
commit2591afca0a600f2df902865cb1ab4ffeeb380f89 (patch)
treef18afc337cabd8cf6a9fd0a9f2f00a91ab2a90c6
parentc5b9cc70cede944833d6c5fe3a47cce825353171 (diff)
downloadbcm5719-llvm-2591afca0a600f2df902865cb1ab4ffeeb380f89.tar.gz
bcm5719-llvm-2591afca0a600f2df902865cb1ab4ffeeb380f89.zip
Fix thinko: alias always defines new symbol. Even is aliasee itself is undefined.
llvm-svn: 48203
-rw-r--r--llvm/lib/Archive/Archive.cpp6
-rw-r--r--llvm/lib/Linker/LinkArchives.cpp9
2 files changed, 4 insertions, 11 deletions
diff --git a/llvm/lib/Archive/Archive.cpp b/llvm/lib/Archive/Archive.cpp
index 7df5e48f9fa..6c95dbe372e 100644
--- a/llvm/lib/Archive/Archive.cpp
+++ b/llvm/lib/Archive/Archive.cpp
@@ -207,10 +207,8 @@ static void getSymbols(Module*M, std::vector<std::string>& symbols) {
// Loop over aliases
for (Module::alias_iterator AI = M->alias_begin(), AE = M->alias_end();
AI != AE; ++AI) {
- const GlobalValue *Aliased = AI->getAliasedGlobal();
- if (!Aliased->isDeclaration())
- if (AI->hasName())
- symbols.push_back(AI->getName());
+ if (AI->hasName())
+ symbols.push_back(AI->getName());
}
}
diff --git a/llvm/lib/Linker/LinkArchives.cpp b/llvm/lib/Linker/LinkArchives.cpp
index 308a775acd2..6cab77c6010 100644
--- a/llvm/lib/Linker/LinkArchives.cpp
+++ b/llvm/lib/Linker/LinkArchives.cpp
@@ -71,13 +71,8 @@ GetAllUndefinedSymbols(Module *M, std::set<std::string> &UndefinedSymbols) {
for (Module::alias_iterator I = M->alias_begin(), E = M->alias_end();
I != E; ++I)
- if (I->hasName()) {
- const GlobalValue *Aliased = I->getAliasedGlobal();
- if (Aliased->isDeclaration())
- UndefinedSymbols.insert(I->getName());
- else
- DefinedSymbols.insert(I->getName());
- }
+ if (I->hasName())
+ DefinedSymbols.insert(I->getName());
// Prune out any defined symbols from the undefined symbols set...
for (std::set<std::string>::iterator I = UndefinedSymbols.begin();
OpenPOWER on IntegriCloud