diff options
| author | Augie Fackler <augie@google.com> | 2022-01-14 14:38:44 -0500 |
|---|---|---|
| committer | Augie Fackler <augie@google.com> | 2022-03-04 10:41:10 -0500 |
| commit | 5e4c75db3b242fc2411e50d9016c63ebd69e8a97 (patch) | |
| tree | 6902d9ab4d9ab8de14ece7fcee25e13f207b2275 /lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h | |
| parent | 43a0016f3dcff9d3932fb8d97e99460333ba7107 (diff) | |
| download | llvm-5e4c75db3b242fc2411e50d9016c63ebd69e8a97.zip llvm-5e4c75db3b242fc2411e50d9016c63ebd69e8a97.tar.gz llvm-5e4c75db3b242fc2411e50d9016c63ebd69e8a97.tar.bz2 | |
InstructionCombining: avoid eliding mismatched alloc/free pairs
Prior to this change LLVM would happily elide a call to any allocation
function and a call to any free function operating on the same unused
pointer. This can cause problems in some obscure cases, for example if
the body of operator::new can be inlined but the body of
operator::delete can't, as in this example from jyknight:
#include <stdlib.h>
#include <stdio.h>
int allocs = 0;
void *operator new(size_t n) {
allocs++;
void *mem = malloc(n);
if (!mem) abort();
return mem;
}
__attribute__((noinline)) void operator delete(void *mem) noexcept {
allocs--;
free(mem);
}
void deleteit(int*i) { delete i; }
int main() {
int*i = new int;
deleteit(i);
if (allocs != 0)
printf("MEMORY LEAK! allocs: %d\n", allocs);
}
This patch addresses the issue by introducing the concept of an
allocator function family and uses it to make sure that alloc/free
function pairs are only removed if they're in the same family.
Differential Revision: https://reviews.llvm.org/D117356
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h')
0 files changed, 0 insertions, 0 deletions
