[PATCH] D54904: Introduce `AddressSpaceView` template parameter to `SizeClassAllocator32`, `FlatByteMap`, and `TwoLevelByteMap`.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 27 08:58:25 PST 2018


delcypher added a comment.

In D54904#1309831 <https://reviews.llvm.org/D54904#1309831>, @kubamracek wrote:

> Please make this patch independent on the other (type_traits).


Is this really necessary? Without the static_assert it really isn't obvious why `ByteMap::AddressSpaceView` exists.


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D54904





More information about the llvm-commits mailing list