[PATCH] D49505: [scudo] Simplify internal names (NFC)

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 18 15:26:53 PDT 2018


alekseyshl added a comment.

Well, now it is not consistent. Why some types end with T and some not (AllocatorCacheT vs PrimaryAllocator)? Why combined allocator is called Backend and struct Allocator is not called Frontend?

To be honest, I don't have better names in mind at the moment. I kind of get used to the current naming, so I might be biased, but it is more or less a matter of taste. If you keep it consistent, I can live with your original PrimaryT and SecondaryT. Sorry for the churn.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49505





More information about the llvm-commits mailing list