> Daniel, I assume we want reviews.llvm.org/D16408 too when that lands,
> right?
Yes, the tests in D16408 aren't blocking any particular build that I know of 
but we should still merge it.

> -----Original Message-----
> From: hwennb...@google.com [mailto:hwennb...@google.com] On Behalf
> Of Hans Wennborg
> Sent: 21 January 2016 17:06
> To: Daniel Sanders; Marshall Clow
> Cc: cfe-commits
> Subject: Re: [libcxx] r258403 - [libcxx] Add appropriate 'REQUIRE' directives 
> to
> tests that require en_US.UTF-8.
> 
> Hi Marshall,
> 
> Is this OK for 3.8?
> 
> Daniel, I assume we want reviews.llvm.org/D16408 too when that lands,
> right?
> 
> Thanks,
> Hans
> 
> On Thu, Jan 21, 2016 at 7:35 AM, Daniel Sanders via cfe-commits
> <cfe-commits@lists.llvm.org> wrote:
> > Author: dsanders
> > Date: Thu Jan 21 09:35:15 2016
> > New Revision: 258403
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=258403&view=rev
> > Log:
> > [libcxx] Add appropriate 'REQUIRE' directives to tests that require
> en_US.UTF-8.
> >
> > Reviewers: mclow.lists, hans
> >
> > Subscribers: bcraig, cfe-commits
> >
> > Differential Revision: http://reviews.llvm.org/D16406
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to