[PATCH] Fixed incorrect disassembly for MOV16o16a when using Intel syntax.
Craig Topper
craig.topper at gmail.com
Tue Jul 30 17:48:53 PDT 2013
Why did the indentation on those lines change? Otherwise LGTM.
On Tue, Jul 30, 2013 at 5:10 PM, Richard Mitton <richard at codersnotes.com>wrote:
> http://llvm-reviews.chandlerc.com/D1241
>
> Files:
> lib/Target/X86/X86InstrInfo.td
>
> Index: lib/Target/X86/X86InstrInfo.td
> ===================================================================
> --- lib/Target/X86/X86InstrInfo.td
> +++ lib/Target/X86/X86InstrInfo.td
> @@ -1079,16 +1079,16 @@
> "mov{b}\t{$src, %al|AL, $src}", [], IIC_MOV_MEM>,
> Requires<[In32BitMode]>;
> def MOV16o16a : Ii32 <0xA1, RawFrm, (outs), (ins offset16:$src),
> - "mov{w}\t{$src, %ax|AL, $src}", [], IIC_MOV_MEM>,
> OpSize,
> + "mov{w}\t{$src, %ax|AX, $src}",
> [], IIC_MOV_MEM>, OpSize,
> Requires<[In32BitMode]>;
> def MOV32o32a : Ii32 <0xA1, RawFrm, (outs), (ins offset32:$src),
> "mov{l}\t{$src, %eax|EAX, $src}", [], IIC_MOV_MEM>,
> Requires<[In32BitMode]>;
> def MOV8ao8 : Ii32 <0xA2, RawFrm, (outs offset8:$dst), (ins),
> "mov{b}\t{%al, $dst|$dst, AL}", [], IIC_MOV_MEM>,
> Requires<[In32BitMode]>;
> def MOV16ao16 : Ii32 <0xA3, RawFrm, (outs offset16:$dst), (ins),
> - "mov{w}\t{%ax, $dst|$dst, AL}", [], IIC_MOV_MEM>,
> OpSize,
> + "mov{w}\t{%ax, $dst|$dst, AX}",
> [], IIC_MOV_MEM>, OpSize,
> Requires<[In32BitMode]>;
> def MOV32ao32 : Ii32 <0xA3, RawFrm, (outs offset32:$dst), (ins),
> "mov{l}\t{%eax, $dst|$dst, EAX}", [], IIC_MOV_MEM>,
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
--
~Craig
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130730/c0868e5c/attachment.html>
More information about the llvm-commits
mailing list