[PATCH] D61745: [scudo][standalone] Introduce the Primary(s) and LocalCache
Kostya Kortchinsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 17 10:34:42 PDT 2019
cryptoad marked an inline comment as done.
cryptoad added inline comments.
================
Comment at: lib/scudo/standalone/local_cache.h:149
+
+ NOINLINE bool refill(PerClass *C, uptr ClassId) {
+ initCacheMaybe(C);
----------------
morehouse wrote:
> We always return true, so why not void?
With B==nullptr this will return false. Updating.
Repository:
rCRT Compiler Runtime
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61745/new/
https://reviews.llvm.org/D61745
More information about the llvm-commits
mailing list