<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jan 21, 2016 at 9:27 AM, Daniel Sanders <span dir="ltr"><<a href="mailto:Daniel.Sanders@imgtec.com" target="_blank">Daniel.Sanders@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">





<div lang="EN-GB" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif">Hi,<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif">It was LGTM'd by bcraig.</span></p></div></div></blockquote><div><br></div><div>Specifically, he said "LGTM, but that doesn't mean much."</div><div><br></div><div>-- Marshall</div><div> </div>







<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div lang="EN-GB" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif"><u></u> <u></u></span></p>
<div style="border-style:none none none solid;border-left-color:blue;border-left-width:1.5pt;padding:0cm 0cm 0cm 4pt">
<div>
<div style="border-style:solid none none;border-top-color:rgb(181,196,223);border-top-width:1pt;padding:3pt 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10pt;font-family:Tahoma,sans-serif">From:</span></b><span lang="EN-US" style="font-size:10pt;font-family:Tahoma,sans-serif"> Marshall Clow [mailto:<a href="mailto:mclow.lists@gmail.com" target="_blank">mclow.lists@gmail.com</a>]
<br>
<b>Sent:</b> 21 January 2016 17:25<br>
<b>To:</b> Hans Wennborg<br>
<b>Cc:</b> Daniel Sanders; cfe-commits<br>
<b>Subject:</b> Re: [libcxx] r258403 - [libcxx] Add appropriate 'REQUIRE' directives to tests that require en_US.UTF-8.<u></u><u></u></span></p>
</div>
</div><div><div class="h5">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<div>
<p class="MsoNormal">On Thu, Jan 21, 2016 at 9:05 AM, Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>> wrote:<u></u><u></u></p>
<p class="MsoNormal">Hi Marshall,<br>
<br>
Is this OK for 3.8?<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I have no idea; this is the first I've seen of it.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal">Apparently it was posted on phab, no one approved it, and it was committed.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">In an hour.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">-- Marshall<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal"> <u></u><u></u></p>
</div>
<blockquote style="border-style:none none none solid;border-left-color:rgb(204,204,204);border-left-width:1pt;padding:0cm 0cm 0cm 6pt;margin-left:4.8pt;margin-right:0cm">
<p class="MsoNormal"><br>
Daniel, I assume we want <a href="http://reviews.llvm.org/D16408" target="_blank">
reviews.llvm.org/D16408</a> too when that lands, right?<br>
<br>
Thanks,<br>
Hans<u></u><u></u></p>
<div>
<div>
<p class="MsoNormal"><br>
On Thu, Jan 21, 2016 at 7:35 AM, Daniel Sanders via cfe-commits<br>
<<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>> wrote:<br>
> Author: dsanders<br>
> Date: Thu Jan 21 09:35:15 2016<br>
> New Revision: 258403<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=258403&view=rev" target="_blank">
http://llvm.org/viewvc/llvm-project?rev=258403&view=rev</a><br>
> Log:<br>
> [libcxx] Add appropriate 'REQUIRE' directives to tests that require en_US.UTF-8.<br>
><br>
> Reviewers: mclow.lists, hans<br>
><br>
> Subscribers: bcraig, cfe-commits<br>
><br>
> Differential Revision: <a href="http://reviews.llvm.org/D16406" target="_blank">
http://reviews.llvm.org/D16406</a><u></u><u></u></p>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>
</div></div></div>
</div>
</div>

</blockquote></div><br></div></div>