[PATCH] D21900: [compiler-rt] Fix sanitizer memory allocator on win64.

[Etienne Bergeron via llvm-commits]

etienneb added a comment.

I'll move forward with adding a new flavor of mmap.
I can't tell how hard that will be. :)


================
Comment at: lib/sanitizer_common/tests/sanitizer_allocator_test.cc:32-34
@@ -31,5 +31,5 @@
 #if SANITIZER_CAN_USE_ALLOCATOR64
-static const uptr kAllocatorSpace = 0x700000000000ULL;
-static const uptr kAllocatorSize  = 0x010000000000ULL;  // 1T.
-static const u64 kAddressSpaceSize = 1ULL << 47;
+static const uptr kAllocatorSpace = 0x10000000000ULL;
+static const uptr kAllocatorSize  =  0x10000000000ULL;  // 1T.
+static const u64 kAddressSpaceSize = 1ULL << 40;
 
----------------
rnk wrote:
> Should these changes be under #if SANITIZER_WINDOWS so that we test the full address space range on Linux?
I should take a deeper look to the range that Windows can accept.


http://reviews.llvm.org/D21900