[PATCH] D27228: Release memory to OS only when the requested range covers the entire page

Evgeniy Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 14:51:15 PST 2016


eugenis added a comment.

Nice.
I understand this is never a problem in practice, because the kernel would never release a region of size (uptr)(-4096).



================
Comment at: lib/sanitizer_common/sanitizer_posix_libcdep.cc:59
 
 void ReleaseMemoryToOS(uptr addr, uptr size) {
   madvise((void*)addr, size, MADV_DONTNEED);
----------------
Is this function still used?


https://reviews.llvm.org/D27228





More information about the llvm-commits mailing list