[PATCH] D49198: [scudo] Use -Wl,-z,global on Android

[Kostya Kortchinsky via Phabricator via llvm-commits]

cryptoad added inline comments.


================
Comment at: lib/scudo/CMakeLists.txt:24
+  if (COMPILER_RT_HAS_Z_GLOBAL)
+    list(APPEND ASAN_DYNAMIC_LINK_FLAGS -Wl,-z,global)
+  endif()
----------------
eugenis wrote:
> ASAN_DYNAMIC_LINK_FLAGS <- is this a typo?
> 
> Can this be done in a common code for all sanitizers that intercept malloc?
> 
> Also, I don't think github is the canonical source for AOSP, could you update the link in the patch description to https://android.googlesource.com/platform/bionic/+/master/android-changes-for-ndk-developers.md ?
Doh! Thanks for catching the error.
Name updated, description updated.
Regarding the last point, you'd want an update to `SANITIZER_COMMON_LINK_FLAGS` based on interception? (not sure if it's possible, just trying to clarify).


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D49198