On Thu, May 16, 2013 at 4:45 AM, Enea Zaffanella <span dir="ltr"><<a href="mailto:zaffanella@cs.unipr.it" target="_blank">zaffanella@cs.unipr.it</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 05/15/2013 10:25 PM, Richard Smith wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please move the tests into test/CodeGenCXX, merge into existing test<br>
files, and ensure that -emit-llvm is on the RUN: line. Maybe<br>
cxx11-thread-local.cpp for the first test and anonymous-namespaces.cpp<br>
for the second one?<br>
<br>
Otherwise, LGTM, thanks!<br>
</blockquote>
<br></div>
Added the patch and one of the tests in r181998.<br>
<br>
As for the other test, adding it at the end of cxx11-thread-local.cpp results in a CHECK line failing to find its target. Since I know very little about llvm bytecode, I left this addition to someone more knowledgeable ... if you prefer, I can add those lines in their own test file.</blockquote>
<div><br></div><div>Added in r182043.</div></div>