[PATCH] D12160: [asan] Enable ASan for AArch64/42-bit VMA

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 19 10:27:54 PDT 2015


rengolin added inline comments.

================
Comment at: lib/sanitizer_common/sanitizer_platform.h:85
@@ +84,3 @@
+// VMA size definition for architecture that support multiple sizes.
+// AArch64 has 3 VMA sizes: 39, 42 and 48.
+#if SANITIZER_LINUX && defined(__aarch64__)
----------------
You haven't added any logic for 48 bits. Would it be simple to do it?

================
Comment at: lib/sanitizer_common/sanitizer_platform.h:89
@@ +88,3 @@
+#  define SANITIZER_AARCH64_VMA 39
+# endif
+#endif
----------------
Would also be good to have some kind of failsafe (#error) for when users choose a VMA that is not supported.


http://reviews.llvm.org/D12160





More information about the llvm-commits mailing list