[PATCH] D90700: [scudo][standalone] Simplify populateFreelist
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 5 15:02:42 PST 2020
pcc added a comment.
LGTM
With this change we will end up with blocks distributed randomly among all of the transfer batches that we create at one time instead of having all of a transfer batch's blocks be consecutive. So we improve randomness at the cost of some locality which could impact performance. I'd be fine with letting this land though and we can see if it significantly impacts performance in practice once it's picked up downstream.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90700/new/
https://reviews.llvm.org/D90700
More information about the llvm-commits
mailing list