This is an archive of the discontinued LLVM Phabricator instance.

[libcxxabi] Improve logging of aborts for Android.
ClosedPublic

Authored by danalbert on Aug 18 2014, 5:43 PM.

Details

Reviewers
jroelofs
Summary

In Android, stderr only goes to the console, and as such will only ever be seen by adb shell users. Since very few developers will ever actually see that, also send the abort message to logcat and the tombstone.

Diff Detail

Event Timeline

danalbert updated this revision to Diff 12635.Aug 18 2014, 5:43 PM
danalbert retitled this revision from to [libcxxabi] Improve logging of aborts for Android..
danalbert updated this object.
danalbert edited the test plan for this revision. (Show Details)
danalbert added a reviewer: jroelofs.
danalbert added a subscriber: Unknown Object (MLST).
jroelofs accepted this revision.Aug 18 2014, 6:35 PM
jroelofs edited edge metadata.

LGTM

src/abort_message.cpp
15

Out of curiosity, does the android compiler define this, or does it come from a particular header?

This revision is now accepted and ready to land.Aug 18 2014, 6:35 PM
danalbert closed this revision.Aug 19 2014, 9:39 AM

Submitted as r215983.