[PATCH] D66123: [scudo][standalone] Clean-up of Android specific compilation.

Christopher Ferris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 17:17:33 PDT 2019


cferris created this revision.
cferris added reviewers: cryptoad, eugenis, vitalybuka, hctim, morehouse.
cferris added projects: LLVM, Sanitizers.
Herald added a subscriber: srhines.

It is possible to build with ANDROID defined, but without
bionic. However, all of the code under the SCUDO_ANDROID
assumes that it is being compiled for android with bionic.
Make this explicit by changing SCUDO_ANDROID to SCUDO_ANDROID_BIONIC
and set it only if it is really building for bionic.

In addition, use bionic headers for some values rather than redefining
them.

Verified that this builds properly on aosp master.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D66123

Files:
  compiler-rt/lib/scudo/standalone/allocator_config.h
  compiler-rt/lib/scudo/standalone/common.h
  compiler-rt/lib/scudo/standalone/linux.cpp
  compiler-rt/lib/scudo/standalone/linux.h
  compiler-rt/lib/scudo/standalone/platform.h
  compiler-rt/lib/scudo/standalone/tsd_shared.h
  compiler-rt/lib/scudo/standalone/wrappers_c.cpp
  compiler-rt/lib/scudo/standalone/wrappers_c.h
  compiler-rt/lib/scudo/standalone/wrappers_c.inc
  compiler-rt/lib/scudo/standalone/wrappers_c_bionic.cpp
  compiler-rt/lib/scudo/standalone/wrappers_cpp.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66123.214713.patch
Type: text/x-patch
Size: 7907 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190813/e72322b3/attachment.bin>


More information about the llvm-commits mailing list