[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 15:34:18 PST 2016


eugenis added a comment.

I just don't like that now we have two functions doing the same thing, one takes (start, end) pointers, the other - (start, size), and the difference in function names (the word "Pages") has nothing to do with the difference in their behavior.

How about we add the rounding logic to ReleaseMemoryToOS() and remove it from the call sites?


https://reviews.llvm.org/D27228





More information about the llvm-commits mailing list