[PATCH] D47160: [asan] Use dynamic allocator space address on Android/AArch64.

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis created this revision.
eugenis added reviewers: kcc, alekseyshl.
Herald added subscribers: kristof.beyls, kubamracek, srhines, rengolin.
Herald added a reviewer: javed.absar.

We need one library to support all of 39, 42 and 48 bit VMAs, and
there is no common address that works for all of them.


https://reviews.llvm.org/D47160

Files:
  compiler-rt/lib/asan/asan_allocator.h


Index: compiler-rt/lib/asan/asan_allocator.h
===================================================================
--- compiler-rt/lib/asan/asan_allocator.h
+++ compiler-rt/lib/asan/asan_allocator.h
@@ -129,7 +129,8 @@
 const uptr kAllocatorSize  =  0x20000000000ULL;  // 2T.
 typedef DefaultSizeClassMap SizeClassMap;
 # elif defined(__aarch64__) && SANITIZER_ANDROID
-const uptr kAllocatorSpace =  0x3000000000ULL;
+// Android needs to support 39, 42 and 48 bit VMA.
+const uptr kAllocatorSpace =  ~(uptr)0;
 const uptr kAllocatorSize  =  0x2000000000ULL;  // 128G.
 typedef VeryCompactSizeClassMap SizeClassMap;
 # elif defined(__aarch64__)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47160.147864.patch
Type: text/x-patch
Size: 642 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180521/8ee78115/attachment.bin>