diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2011-10-20 05:23:42 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2011-10-20 05:23:42 +0000 |
| commit | 1923a330e66073944560fe7d751b399798656d89 (patch) | |
| tree | c5cf6c5182ab557feeb6a77d11a513ccc82d639f /llvm/lib/Transforms/IPO/GlobalOpt.cpp | |
| parent | 337c5b880c5f5328cf557ab4177726bbf0a93100 (diff) | |
| download | bcm5719-llvm-1923a330e66073944560fe7d751b399798656d89.tar.gz bcm5719-llvm-1923a330e66073944560fe7d751b399798656d89.zip | |
Refactor code from inlining and globalopt that checks whether a function definition is unused, and enhance it so it can tell that functions which are only used by a blockaddress are in fact dead. This probably doesn't happen much on most code, but the Linux kernel's _THIS_IP_ can trigger this issue with blockaddress. (GlobalDCE can also handle the given tescase, but we only run that at -O3.) Found while looking at PR11180.
llvm-svn: 142572
Diffstat (limited to 'llvm/lib/Transforms/IPO/GlobalOpt.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/GlobalOpt.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/IPO/GlobalOpt.cpp b/llvm/lib/Transforms/IPO/GlobalOpt.cpp index 3552d03919b..c57e9fc0e8d 100644 --- a/llvm/lib/Transforms/IPO/GlobalOpt.cpp +++ b/llvm/lib/Transforms/IPO/GlobalOpt.cpp @@ -1890,7 +1890,7 @@ bool GlobalOpt::OptimizeFunctions(Module &M) { if (!F->hasName() && !F->isDeclaration()) F->setLinkage(GlobalValue::InternalLinkage); F->removeDeadConstantUsers(); - if (F->use_empty() && (F->hasLocalLinkage() || F->hasLinkOnceLinkage())) { + if (F->isDefTriviallyDead()) { F->eraseFromParent(); Changed = true; ++NumFnDeleted; |
