> 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