[PATCH] D13782: [sanitizer] [asan] Use same shadow offset for aarch64

[Adhemerval Zanella via llvm-commits]

zatrazz added inline comments.

================
Comment at: lib/sanitizer_common/sanitizer_platform.h:111
@@ -111,7 +110,3 @@
 #if defined(__aarch64__)
-# if SANITIZER_AARCH64_VMA == 39
-#  define SANITIZER_MMAP_RANGE_SIZE FIRST_32_SECOND_64(1ULL << 32, 1ULL << 39)
-# elif SANITIZER_AARCH64_VMA == 42
-#  define SANITIZER_MMAP_RANGE_SIZE FIRST_32_SECOND_64(1ULL << 32, 1ULL << 42)
-# endif
+# define SANITIZER_MMAP_RANGE_SIZE FIRST_32_SECOND_64(1ULL << 32, 1ULL << 42)
 #elif defined(__mips__)
----------------
eugenis wrote:
> I'm pretty sure this won't work on 48 bit VMA.
> Could you evaluate the cost of lifting the upper limit to 48? Last time I looked, this should only affect .bss size (so, no real RAM cost), and not really that much.
I won't, but I am also not sure if kAArch64_ShadowOffset64 = 1ULL << 36 will work correctly on 48-bit kernels. But it will check this out.


http://reviews.llvm.org/D13782