[PATCH] D27873: Reduce the size of quarantine cache in ASAN_LOW_MEMORY configuration.

Evgeniy Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 20 14:34:44 PST 2016


eugenis accepted this revision.
eugenis added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/asan/asan_allocator.cc:235
+      // quarantine is set to 64Kb.
+      (ASAN_LOW_MEMORY) ? 64 << 10 : FIRST_32_SECOND_64(1 << 18, 1 << 20);
 
----------------
1 << 16


================
Comment at: lib/asan/asan_internal.h:39
 #ifndef ASAN_LOW_MEMORY
-# if SANITIZER_IOS || (SANITIZER_WORDSIZE == 32)
+# if SANITIZER_IOS || SANITIZER_ANDROID || (SANITIZER_WORDSIZE == 32)
 #  define ASAN_LOW_MEMORY 1
----------------
Please remove a line in lib/asan/CMakeLists.txt that sets ASAN_LOW_MEMORY for Android.


https://reviews.llvm.org/D27873





More information about the llvm-commits mailing list