<div dir="ltr">Agree with Ahmed. Also all patches are supposed to go to llvm-commits not llvm-dev.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 24, 2015 at 9:56 AM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank">ahmed.bougacha@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Mar 24, 2015 at 7:54 AM, Jun Koi <<a href="mailto:junkoi2004@gmail.com">junkoi2004@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> This patch fixes outs/ins of MOV16mr instruction of X86.<br>
><br>
> Thanks.<br>
><br>
><br>
> diff --git a/lib/Target/X86/X86InstrInfo.td b/lib/Target/X86/X86InstrInfo.td<br>
> index e9a0431..f5b2064 100644<br>
> --- a/lib/Target/X86/X86InstrInfo.td<br>
> +++ b/lib/Target/X86/X86InstrInfo.td<br>
> @@ -1412,7 +1412,7 @@ let SchedRW = [WriteStore] in {<br>
>  def MOV8mr  : I<0x88, MRMDestMem, (outs), (ins i8mem :$dst, GR8 :$src),<br>
>                  "mov{b}\t{$src, $dst|$dst, $src}",<br>
>                  [(store GR8:$src, addr:$dst)], IIC_MOV_MEM>;<br>
> -def MOV16mr : I<0x89, MRMDestMem, (outs), (ins i16mem:$dst, GR16:$src),<br>
> +def MOV16mr : I<0x89, MRMDestMem, (outs i16mem:$dst), (ins GR16:$src),<br>
>                  "mov{w}\t{$src, $dst|$dst, $src}",<br>
>                  [(store GR16:$src, addr:$dst)], IIC_MOV_MEM>, OpSize16;<br>
<br>
Why?  i16mem here stands for the pointer, not the actual memory.  A<br>
store doesn't define a pointer, so why would it be in "outs"?<br>
Also, what's special about i16? You'd need to change the various other<br>
*mr instructions, for instance the MOV8mr right above.<br>
<br>
-Ahmed<br>
<br>
>  def MOV32mr : I<0x89, MRMDestMem, (outs), (ins i32mem:$dst, GR32:$src),<br>
><br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">~Craig</div>
</div>