[patch] [libcxx] msvcrt revised support.h

G M gmisocpp at gmail.com
Wed Oct 9 17:45:32 PDT 2013


HI Marshall

I think you must have missed this epic email:
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20130930/089995.html
The test case is attached to that. The original patch is correct and the
test case hasn't changed.

The revised patch makes a stylist change as preferred by Arthur and removes
some unneeded functions.
One can apply both, just the earlier one, or just the later one.

On Thu, Oct 10, 2013 at 8:08 AM, Marshall Clow <mclow.lists at gmail.com>wrote:

> On Oct 9, 2013, at 11:00 AM, G M <gmisocpp at gmail.com> wrote:
>
> > Hi Everyone
> >
> > Attached is a revised patch to support.h posted a week ago that fixes
> some bugs.
> >
> > It has some revisions to match Athurs suggestions and I've removed some
> functions that are now provided by the base platform.
>
> I don't see any tests here.
> Do these fix failures in the current test suite?
> If so, which ones?
>
> Thanks,
> -- Marshall
>
> Marshall Clow     Idio Software   <mailto:mclow.lists at gmail.com>
>
> A.D. 1517: Martin Luther nails his 95 Theses to the church door and is
> promptly moderated down to (-1, Flamebait).
>         -- Yu Suzuki
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131010/4f62cae3/attachment.html>


More information about the cfe-commits mailing list