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

Evgeniy Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 09:28:51 PST 2016


eugenis added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_allocator_primary64.h:466
+    const uptr page_size = GetPageSizeCached();
+    CHECK_GE(end_ptr - beg_ptr, page_size);
+    beg_ptr = RoundUpTo(beg_ptr, page_size);
----------------
I don't think this can be a CHECK any longer. The caller does not guarantee page size granularity.


https://reviews.llvm.org/D27003





More information about the llvm-commits mailing list