<div dir="ltr"><div style>Homer,</div><div style><br></div><div style>One quick comment for this patch, please see the following change:</div><br><div class="gmail_extra">@@ -2169,6 +2247,8 @@ namespace gbe<br>       } else if (dst.isdf()) {<br>
         ir::Register r = sel.reg(ir::RegisterFamily::FAMILY_QWORD);<br>         sel.MOV_DF(dst, src, sel.selReg(r));<br>+      } else if (dst.isint64()) {<br>+        sel.MOV_INT64(dst, src);<br>       } else<br>         sel.MOV(dst, src);<br>
       return true;<br><br>Here, what we want to do is to convert a different data type to int64 type.</div><div class="gmail_extra">But according to your previous patch, MOV_INT64 is used to move two</div><div class="gmail_extra">
int64 type registers, right? So I think it may not be correct, and you may</div><div class="gmail_extra">need to add a new instruction to handle similar CVT instruction as MOV_DF</div><div class="gmail_extra">or you can extent MOV_DF to support int64 too. Any idea?<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><font color="#888888"><br>

</font></span></blockquote></div><br></div></div>