This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Fix checks for Android.
ClosedPublic

Authored by danalbert on Oct 18 2014, 5:31 PM.

Details

Reviewers
eugenis
srhines
Summary

ANDROID is a define that comes from the toolchain when building
for Android targets. ANDROID has a different meaning. ANDROID is
defined for _every_ Android build, including those done for host
modules. For host modules, we want to build the regular Linux
sanitizers and builtins, not the one for Android devices. This hasn't
been a problem until now because we only just started building the
sanitizers for the host.

Diff Detail

Event Timeline

danalbert updated this revision to Diff 15125.Oct 18 2014, 5:31 PM
danalbert retitled this revision from to [compiler-rt] Fix checks for Android..
danalbert updated this object.
danalbert edited the test plan for this revision. (Show Details)
danalbert added reviewers: eugenis, srhines.
danalbert added a subscriber: Unknown Object (MLST).
eugenis accepted this revision.Oct 18 2014, 9:26 PM
eugenis edited edge metadata.

LGTM, thanks!

This revision is now accepted and ready to land.Oct 18 2014, 9:26 PM
danalbert closed this revision.Oct 20 2014, 8:45 AM

Submitted as r220203.