This is an archive of the discontinued LLVM Phabricator instance.

[PR32479] Avoid newlib vasprintf, since it requires gnu extensions
AbandonedPublic

Authored by bcraig on Apr 17 2017, 6:25 PM.

Details

Summary

Newlib 2.5 doesn't define vasprintf unless gnu extensions are turned on. So now, we provide our own implementation backed by vsnprintf.

Diff Detail

Event Timeline

bcraig created this revision.Apr 17 2017, 6:25 PM
bcraig updated this revision to Diff 95520.Apr 17 2017, 6:28 PM
bcraig added reviewers: jyknight, mclow.lists, EricWF.
bcraig added a subscriber: cfe-commits.
EricWF edited edge metadata.Apr 17 2017, 7:02 PM

Why are GNU extensions not turned on? AFAIK Libc++ doesn't compile against any C library without GNU extensions enabled, including glibc and apple-libc.

EricWF resigned from this revision.Apr 20 2017, 8:52 PM

The bug relating to this patch was closed as WONTFIX. This review should be closed.

bcraig abandoned this revision.Apr 21 2017, 6:42 AM