diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2014-09-09 14:27:09 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2014-09-09 14:27:09 +0000 |
| commit | 7fc29546f93e1c4ca579be76c69245e7521cce4b (patch) | |
| tree | 26e4174c5b4363ceaa7bdd3b0f2f5b53bdab6c91 /llvm/lib/Linker/LinkModules.cpp | |
| parent | 91ea478d7c69853866125e8a01add7f44922c147 (diff) | |
| download | bcm5719-llvm-7fc29546f93e1c4ca579be76c69245e7521cce4b.tar.gz bcm5719-llvm-7fc29546f93e1c4ca579be76c69245e7521cce4b.zip | |
Prefer common over weak linkage when linking.
This matches the behavior of ELF linkers.
llvm-svn: 217443
Diffstat (limited to 'llvm/lib/Linker/LinkModules.cpp')
| -rw-r--r-- | llvm/lib/Linker/LinkModules.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Linker/LinkModules.cpp b/llvm/lib/Linker/LinkModules.cpp index d02d590ca10..14c6a10c339 100644 --- a/llvm/lib/Linker/LinkModules.cpp +++ b/llvm/lib/Linker/LinkModules.cpp @@ -721,8 +721,9 @@ bool ModuleLinker::getLinkageResult(GlobalValue *Dest, const GlobalValue *Src, } else if (Src->isWeakForLinker()) { assert(!Dest->hasExternalWeakLinkage()); assert(!Dest->hasAvailableExternallyLinkage()); - if (Dest->hasLinkOnceLinkage() && - (Src->hasWeakLinkage() || Src->hasCommonLinkage())) { + if ((Dest->hasLinkOnceLinkage() && Src->hasWeakLinkage()) || + ((Dest->hasLinkOnceLinkage() || Dest->hasWeakLinkage()) && + Src->hasCommonLinkage())) { LinkFromSrc = true; LT = Src->getLinkage(); } else { |
