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

Evgeniy Stepanov via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 11:09:22 PDT 2015


eugenis 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__)
----------------
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.


http://reviews.llvm.org/D13782





More information about the llvm-commits mailing list