[PATCH] Change internal allocator in sanitizer tools
    Alexey Samsonov 
    samsonov at google.com
       
    Tue Apr 16 05:11:29 PDT 2013
    
    
  
================
Comment at: lib/sanitizer_common/sanitizer_allocator.cc:20
@@ +19,3 @@
+#if SANITIZER_WORDSIZE == 64
+static const uptr kInternalAllocatorSpace = 0x690000000000ULL;
+static const uptr kInternalAllocatorSize =   0x10000000000ULL;  // 1T
----------------
Dmitry Vyukov wrote:
> THis won't work on windows (for gotsan). windows address space is 43-bit.
ok... I glanced at tsan_platform.h for Go-Windows and couldn't find a place for yet another mapping...
================
Comment at: lib/sanitizer_common/tests/sanitizer_allocator_test.cc:585
@@ +584,3 @@
+  u64 end = space + size;
+  CHECK_LE(end, (SANITIZER_WORDSIZE == 64) ? 0x800000000000ULL
+                                           : 0x100000000ULL);
----------------
Evgeniy Stepanov wrote:
> I'd prefer if (SANITIZER_WORDSIZE == 64) {
> all-checks-for-64-bits
> } else {
> all-checks-for-32-bits
> }
Done
http://llvm-reviews.chandlerc.com/D671
    
    
More information about the llvm-commits
mailing list