<div dir="ltr">Here's a version with a comment and the fixed CHECK statement.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 12, 2013 at 1:53 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 09/12/2013 10:50 PM, Tom Roeder wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Yes, that triggers it. Here's an updated patch that uses the test you<br>
suggest and uses FileCheck to make sure that the right IR is generated from<br>
llvm-dis. I'm new to the LLVM testing infrastructure, so please tell me if<br>
there's a better way to do this check.<br>
</blockquote>
<br></div>
It is probably enough to CHECK: just for the line that contains the select.<br>
<br>
Also, please add a short comment to the test case (or call the function accordingly) such that it is clear we test the CS select and not any select.<br>
<br>
With those changes applied, this patch seems good to go.<br>
<br>
Thank you,<br>
Tobias<br>
<br>
<br>
</blockquote></div><br></div>