[PATCH] D46408: [sanitizer] s/TestOnlyInit/Init for the allocator ByteMap (NFC)

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 7 11:33:59 PDT 2018


cryptoad added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_allocator_bytemap.h:21
  public:
-  void TestOnlyInit() {
+  void Init() {
     internal_memset(map_, 0, sizeof(map_));
----------------
vitalybuka wrote:
> I'd call it Reset().
I feel like Reset could be confusing as it could lead someone to think it can be called for more purposes than initialization.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D46408





More information about the llvm-commits mailing list