[PATCH] D39393: [asan] Use dynamic shadow on 32-bit Android.

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_linux.cc:970
 
-uptr GetMaxVirtualAddress() {
+uptr GetMaxVirtualAddress(bool ifunc_safe) {
 #if SANITIZER_NETBSD && defined(__x86_64__)
----------------
Is the `ifunc_safe` change necessary? If `common_flags()` is inlined, I would expect the access to go through `common_flags_dont_use`, which you have marked as hidden. You might want to make `common_flags()` hidden as well, in case it doesn't get inlined.


https://reviews.llvm.org/D39393