aboutsummaryrefslogtreecommitdiff
path: root/lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h
diff options
context:
space:
mode:
authorAugie Fackler <augie@google.com>2022-01-14 14:38:44 -0500
committerAugie Fackler <augie@google.com>2022-03-04 10:41:10 -0500
commit5e4c75db3b242fc2411e50d9016c63ebd69e8a97 (patch)
tree6902d9ab4d9ab8de14ece7fcee25e13f207b2275 /lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h
parent43a0016f3dcff9d3932fb8d97e99460333ba7107 (diff)
downloadllvm-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