[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