diff options
| author | Chris Lattner <sabre@nondot.org> | 2004-10-14 19:53:50 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2004-10-14 19:53:50 +0000 |
| commit | 5d33e8e73ae2a7a3b6c120bf512005be0ffa3cea (patch) | |
| tree | 6f0decbb03999479ab36fb0948379c6734df9393 /llvm/lib/Transforms | |
| parent | 1f864b5583784dde778b68d3b2ec1d28c6a90e78 (diff) | |
| download | bcm5719-llvm-5d33e8e73ae2a7a3b6c120bf512005be0ffa3cea.tar.gz bcm5719-llvm-5d33e8e73ae2a7a3b6c120bf512005be0ffa3cea.zip | |
Fix a bug John tracked down in libstdc++ where we were incorrectly deleting
weak functions. Thanks for finding this John!
llvm-svn: 16997
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/IPO/GlobalOpt.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/IPO/GlobalOpt.cpp b/llvm/lib/Transforms/IPO/GlobalOpt.cpp index dc8832c4f36..e800e0c6001 100644 --- a/llvm/lib/Transforms/IPO/GlobalOpt.cpp +++ b/llvm/lib/Transforms/IPO/GlobalOpt.cpp @@ -808,7 +808,8 @@ bool GlobalOpt::runOnModule(Module &M) { for (Module::iterator FI = M.begin(), E = M.end(); FI != E; ) { Function *F = FI++; F->removeDeadConstantUsers(); - if (F->use_empty() && (F->hasInternalLinkage() || F->hasWeakLinkage())) { + if (F->use_empty() && (F->hasInternalLinkage() || + F->hasLinkOnceLinkage())) { M.getFunctionList().erase(F); LocalChange = true; ++NumFnDeleted; |
