[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