<div class="gmail_quote">On Mon, Sep 12, 2011 at 5:58 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="HOEnZb"><div></div><div class="h5">On Mon, Sep 12, 2011 at 5:52 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> On Mon, Sep 12, 2011 at 5:27 PM, Eli Friedman <<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Author: efriedma<br>
>> Date: Mon Sep 12 19:27:04 2011<br>
>> New Revision: 139563<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=139563&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=139563&view=rev</a><br>
>> Log:<br>
>> Fix the assembler strings for a couple of atomic instructions.  Doesn't<br>
>> really matter much in practice, but it's a bit cleaner.<br>
><br>
> *cough* test case? ;] I know it doesn't matter much, but these look trivial<br>
> to test anyways...<br>
<br>
</div></div>We don't really have any tests for the Intel-style asm-printer;<br>
outside of that, this is only cosmetic.</blockquote><div><br></div><div>Ahh, didn't realize it was on the intel side. Yea, w/o a test framework not worth it.</div></div>