[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:44:35 PST 2016


eugenis added a comment.

Btw ASAN_LOW_MEMORY was set for all 32-bit targets in r171052 by kcc.
I don't understand why.
IMHO, it should be a property of environment (i.e. OS), not architecture. If necessary, we can always test for 32-bit platform directly. And we do, see all the FIRST_32_SECOND_64 macros!

I'd support removing "|| (SANITIZER_WORDSIZE == 32)" from this definition.


https://reviews.llvm.org/D27873





More information about the llvm-commits mailing list