[PATCH] D27003: Return memory to OS right after free (not in the async thread).

Aleksey Shlyapnikov via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 14:10:56 PST 2016


alekseyshl added inline comments.


================
Comment at: lib/asan/asan_allocator.h:40
   u8 alloc_dealloc_mismatch;
+  s32 release_to_os_interval_ms;
 
----------------
filcab wrote:
> (Unimportant) Nit: Why have a signed number?
It seems that using negative values to represent "do not release memory" state looks a bit saner and safer than a magic value, hence signed.


https://reviews.llvm.org/D27003





More information about the llvm-commits mailing list