diff options
| author | Craig Topper <craig.topper@gmail.com> | 2017-04-18 21:43:46 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@gmail.com> | 2017-04-18 21:43:46 +0000 |
| commit | 09bb760baa542aadb0712a34f5bf79c34677845b (patch) | |
| tree | d7fd2112c6f10e8135dcfd6261c6880443bc759b /llvm/lib/Analysis/CFLGraph.h | |
| parent | 80fe987b422b2afe59960c020a8c70115aeec140 (diff) | |
| download | bcm5719-llvm-09bb760baa542aadb0712a34f5bf79c34677845b.tar.gz bcm5719-llvm-09bb760baa542aadb0712a34f5bf79c34677845b.zip | |
[MemoryBuiltins] Add isMallocOrCallocLikeFn so BasicAA can check for both at the same time
BasicAA wants to know if a function is either a malloc or calloc like function. Currently we have to check both separately. This means both calls check if its an intrinsic, query TLI, check the nobuiltin attribute, scan the AllocationFnData, etc.
This patch adds a isMallocOrCallocLikeFn so we can go through all of the checks once per call.
This also changes the one other location I saw that called both together.
Differential Revision: https://reviews.llvm.org/D32188
llvm-svn: 300608
Diffstat (limited to 'llvm/lib/Analysis/CFLGraph.h')
| -rw-r--r-- | llvm/lib/Analysis/CFLGraph.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/llvm/lib/Analysis/CFLGraph.h b/llvm/lib/Analysis/CFLGraph.h index e526e0e16aa..75726e84569 100644 --- a/llvm/lib/Analysis/CFLGraph.h +++ b/llvm/lib/Analysis/CFLGraph.h @@ -400,8 +400,7 @@ template <typename CFLAA> class CFLGraphBuilder { // TODO: address other common library functions such as realloc(), // strdup(), // etc. - if (isMallocLikeFn(Inst, &TLI) || isCallocLikeFn(Inst, &TLI) || - isFreeCall(Inst, &TLI)) + if (isMallocOrCallocLikeFn(Inst, &TLI) || isFreeCall(Inst, &TLI)) return; // TODO: Add support for noalias args/all the other fun function |
