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 | c66330504c3f433430a28cd7f7f981e555c51bce (patch) | |
tree | 9d881c567e25847a240ddc05ec82c8abe5290d95 /lib/Transforms/IPO/Inliner.cpp | |
parent | d7e5264b462738084649310055ba8baea33fe9c8 (diff) | |
download | external_llvm-c66330504c3f433430a28cd7f7f981e555c51bce.zip external_llvm-c66330504c3f433430a28cd7f7f981e555c51bce.tar.gz external_llvm-c66330504c3f433430a28cd7f7f981e555c51bce.tar.bz2 |
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.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@142572 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms/IPO/Inliner.cpp')
-rw-r--r-- | lib/Transforms/IPO/Inliner.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/lib/Transforms/IPO/Inliner.cpp b/lib/Transforms/IPO/Inliner.cpp index f00935b..bdc9fe4 100644 --- a/lib/Transforms/IPO/Inliner.cpp +++ b/lib/Transforms/IPO/Inliner.cpp @@ -533,10 +533,7 @@ bool Inliner::removeDeadFunctions(CallGraph &CG, if (DNR && DNR->count(F)) continue; - if (!F->hasLinkOnceLinkage() && !F->hasLocalLinkage() && - !F->hasAvailableExternallyLinkage()) - continue; - if (!F->use_empty()) + if (!F->isDefTriviallyDead()) continue; // Remove any call graph edges from the function to its callees. |