[PATCH] D61206: [sanitizer] Calculate SizeClassAllocator32::ByteMap type from Params::kSpaceSize and Params::kRegionSizeLog

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 26 11:28:15 PDT 2019


eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_allocator_primary32.h:62
+  using ByteMap = typename conditional<
+      (TwoLevelByteMapSize1 > 128),
+      TwoLevelByteMap<TwoLevelByteMapSize1, 1 << 12, AddressSpaceView>,
----------------
make 128 a named constant


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61206/new/

https://reviews.llvm.org/D61206





More information about the llvm-commits mailing list