[PATCH] D88457: [scudo][standalone] Fix some primary tests
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 29 11:44:06 PDT 2020
pcc added a comment.
I'm not really a fan of long lists of template arguments. So I would favor something like
3. Have the SizeClassAllocator classes take something like a Config type (see allocator_config.h) as a template argument and have it read the configuration out of that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88457/new/
https://reviews.llvm.org/D88457
More information about the llvm-commits
mailing list