[llvm-commits] [llvm] r139563 - /llvm/trunk/lib/Target/X86/X86InstrCompiler.td

Chandler Carruth chandlerc at google.com
Mon Sep 12 18:02:15 PDT 2011


On Mon, Sep 12, 2011 at 5:58 PM, Eli Friedman <eli.friedman at gmail.com>wrote:

> On Mon, Sep 12, 2011 at 5:52 PM, Chandler Carruth <chandlerc at google.com>
> wrote:
> > On Mon, Sep 12, 2011 at 5:27 PM, Eli Friedman <eli.friedman at gmail.com>
> > wrote:
> >>
> >> Author: efriedma
> >> Date: Mon Sep 12 19:27:04 2011
> >> New Revision: 139563
> >>
> >> URL: http://llvm.org/viewvc/llvm-project?rev=139563&view=rev
> >> Log:
> >> Fix the assembler strings for a couple of atomic instructions.  Doesn't
> >> really matter much in practice, but it's a bit cleaner.
> >
> > *cough* test case? ;] I know it doesn't matter much, but these look
> trivial
> > to test anyways...
>
> We don't really have any tests for the Intel-style asm-printer;
> outside of that, this is only cosmetic.


Ahh, didn't realize it was on the intel side. Yea, w/o a test framework not
worth it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110912/72f76d32/attachment.html>


More information about the llvm-commits mailing list