This is an archive of the discontinued LLVM Phabricator instance.

libc++: support newlib's ctype
ClosedPublic

Authored by jfb on Feb 25 2015, 1:31 PM.

Details

Summary

Newlib supports ctype differently from other platforms, this patch teaches libc++ about yet another platform that does ctype differently.

Diff Detail

Repository
rL LLVM

Event Timeline

jfb updated this revision to Diff 20698.Feb 25 2015, 1:31 PM
jfb retitled this revision from to libc++: support newlib's ctype.
jfb updated this object.
jfb added a reviewer: jroelofs.
jfb added subscribers: mclow.lists, jvoung, EricWF and 2 others.
jfb added a comment.Feb 25 2015, 1:44 PM

@jroelofs points out this bug that @danalbert filed:
http://llvm.org/bugs/show_bug.cgi?id=21506

Should I wait for this mythical classic_table, or go ahead with more ctype stew?

jroelofs accepted this revision.Feb 25 2015, 1:57 PM
jroelofs edited edge metadata.

LGTM for now.

The "right" fix is something like this: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20141208/119787.html

This revision is now accepted and ready to land.Feb 25 2015, 1:57 PM
This revision was automatically updated to reflect the committed changes.