<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">A question, can you explain why when SOExpr->EvaluateAsAbsolute(Value) returns false, it's safe to return <span style="font-family:arial,sans-serif;font-size:14.399999618530273px">Match_Success? </span></div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<span style="font-family:arial,sans-serif;font-size:14.399999618530273px">How do we guarantee </span>EvaluateAsAbsolute can always solve the expression and return an immediate value in encoding stage via MCStreamer? Looking at your patch, it seems there might be some assumptions here, or you want to simply use "assertion" in encoding stage to detect any invalid expression in future? If it is yes, it doesn't sound a good solution to me from design level of point.<br>
</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
-Jiangning</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-03-13 14:23 GMT+08:00 Saleem Abdulrasool <span dir="ltr"><<a href="mailto:compnerd@compnerd.org" target="_blank">compnerd@compnerd.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Wed, Mar 12, 2014 at 11:12 PM, Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>></span> wrote:<br>

<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"><div dir="ltr"><div style="font-family:arial,helvetica,sans-serif;font-size:small">

Hi,</div><div style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div style="font-family:arial,helvetica,sans-serif;font-size:small">
Attached patch is to raise an error message when finding an expression at compile time can't really be solved as a constant for immediate operand. Without this patch, llvm compiler would crash.</div></div></blockquote>

<div><br></div></div><div>The attached patch seems wrong.  With the attached change, validateTargetOperandClass always returns Match_InvalidOperand for any immediate in this addressing mode.  If the value is a constant value, then it should return Match_Success.</div>
<span class="HOEnZb"><font color="#888888">
<div> </div><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"><div dir="ltr"><span><font color="#888888">-- <br>


<div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning<div style="font-family:arial,helvetica,sans-serif;font-size:small;display:inline">
</div></font></div></div>
</font></span></div>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></font></span></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</font></span></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>