[PATCH] D75551: [scudo][standalone] Do not fill 32b regions at once

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 10:39:20 PST 2020


cryptoad updated this revision to Diff 248235.
cryptoad added a comment.

Commenting `populateFreeList`, with some additional white-line
separation for code blocks.

Additionally I removed the update to `LastReleaseAtNs` in said function,
we check for enough `PushedBlocks` in `releaseToOS` anyway, which is
not affected by `populateFreeList`, so updating the timer didn't add
anything.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75551/new/

https://reviews.llvm.org/D75551

Files:
  compiler-rt/lib/scudo/standalone/primary32.h
  compiler-rt/lib/scudo/standalone/primary64.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75551.248235.patch
Type: text/x-patch
Size: 8536 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200304/340367f9/attachment.bin>


More information about the llvm-commits mailing list