[PATCH] D71772: [scudo][standalone] Support __BIONIC__
Kostya Kortchinsky via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 20 09:30:01 PST 2019
cryptoad added inline comments.
================
Comment at: compiler-rt/lib/scudo/standalone/platform.h:18
-#if defined(__ANDROID__)
+#if defined(__ANDROID__) || defined(__BIONIC__)
#define SCUDO_ANDROID 1
----------------
pcc wrote:
> Maybe just `#if defined(__BIONIC__)`?
We still want Scudo to work on Android outside of Bionic (eg: .so or just statically linked to a binary) in which case I don't think __BIONIC__ will be defined.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71772/new/
https://reviews.llvm.org/D71772
More information about the llvm-commits
mailing list