[PATCH] D72882: [scudo][standalone] WIP: make release interval configurable

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 09:55:44 PST 2020


cryptoad marked an inline comment as done.
cryptoad added inline comments.


================
Comment at: compiler-rt/lib/scudo/standalone/wrappers_c.inc:162
+        case 0:
+          value = 500;
+          break;
----------------
eugenis wrote:
> This value, and the one below, do not seem to match the description in https://android.googlesource.com/platform/bionic/+/master/docs/native_allocator.md
> 
> A comment explaining the choice of the values would be great.
This CL is just a WIP that I put here to share with cferris at .
It will look different once we settle on what we need to do, you can ignore it :)
But thanks for the comments!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72882





More information about the llvm-commits mailing list