diff options
author | Philip Reames <listmail@philipreames.com> | 2022-01-10 09:50:01 -0800 |
---|---|---|
committer | Philip Reames <listmail@philipreames.com> | 2022-01-10 15:43:39 -0800 |
commit | 5265ac72c6686d73edcf5de37b019b64c814d00c (patch) | |
tree | 44b8376e7aad9a6250d9a6c3184d2830330b69b9 /llvm/lib/Transforms/InstCombine/InstructionCombining.cpp | |
parent | 38b30eb2b218ad481855d5c691ef4501711ca3a6 (diff) | |
download | llvm-5265ac72c6686d73edcf5de37b019b64c814d00c.zip llvm-5265ac72c6686d73edcf5de37b019b64c814d00c.tar.gz llvm-5265ac72c6686d73edcf5de37b019b64c814d00c.tar.bz2 |
[MemoryBuiltin] Add an API for checking if an unused allocation can be removed [NFC]
Not all allocation functions are removable if unused. An example of a non-removable allocation would be a direct call to the replaceable global allocation function in C++. An example of a removable one - at least according to historical practice - would be malloc.
Diffstat (limited to 'llvm/lib/Transforms/InstCombine/InstructionCombining.cpp')
-rw-r--r-- | llvm/lib/Transforms/InstCombine/InstructionCombining.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp index 31329ae..6d5548d 100644 --- a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp +++ b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp @@ -2696,6 +2696,8 @@ static bool isAllocSiteRemovable(Instruction *AI, } Instruction *InstCombinerImpl::visitAllocSite(Instruction &MI) { + assert(isa<AllocaInst>(MI) || isAllocRemovable(&cast<CallBase>(MI), &TLI)); + // If we have a malloc call which is only used in any amount of comparisons to // null and free calls, delete the calls and replace the comparisons with true // or false as appropriate. |