ART: Make Touch's stack array smaller under ASAN am: 854b3503ab am: d81d7a9b53 am: 098a630a58

am: 7345cba901

Change-Id: I8aa194e184a167fc742dd9ba46767630613fa6ff

1 files changed, 8 insertions, 1 deletions

diff --git a/runtime/thread.cc b/runtime/thread.cc
index f6b89bc532..d99452fcd1 100644
--- a/runtime/thread.cc
+++ b/runtime/thread.cc
@@ -570,7 +570,14 @@ void Thread::InstallImplicitProtection() {
       // Use a large local volatile array to ensure a large frame size. Do not use anything close
       // to a full page for ASAN. It would be nice to ensure the frame size is at most a page, but
       // there is no pragma support for this.
-      volatile char space[kPageSize - 256];
+      // Note: for ASAN we need to shrink the array a bit, as there's other overhead.
+      constexpr size_t kAsanMultiplier =
+#ifdef ADDRESS_SANITIZER
+          2u;
+#else
+          1u;
+#endif
+      volatile char space[kPageSize - (kAsanMultiplier * 256)];
       char sink ATTRIBUTE_UNUSED = space[zero];
       if (reinterpret_cast<uintptr_t>(space) >= target + kPageSize) {
         Touch(target);