[PATCH] D43088: [sanitizer] Size class map & local cache improvements

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad added inline comments.


================
Comment at: lib/sanitizer_common/sanitizer_allocator_local_cache.h:101
+      c->max_count = 2 * SizeClassMap::MaxCachedHint(size);
+      c->class_size = size;
     }
----------------
alekseyshl wrote:
> Does
> 
>       c->class_size = Allocator::ClassIdToSize(i);
>       c->max_count = 2 * SizeClassMap::MaxCachedHint(c->class_size);
> 
> lead to less optimal code?
It leads to almost exactly the same asm (at least for x64).
Do you prefer yours?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D43088