<div dir="ltr">Looks good. Is it easy to reproduce this with a simple test case and inspect with FileCheck?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 January 2013 15:15, Tim Northover <span dir="ltr"><<a href="mailto:Tim.Northover@arm.com" target="_blank">Tim.Northover@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
Currently, when generating code to print InstAlias<"nop", (HINTi 0)> the<br>
TableGen backend calls getImm() to check whether the provided immediate is<br>
correct.<br>
<br>
However, in some circumstances, the operand may not actually *be* an<br>
immediate, it could be an MCExpr for example (contrived here, but realistic<br>
for more complex instructions). I think TableGen should emit code that checks<br>
whether it *is* an immediate before calling getImm(). This patch implements<br>
that change.<br>
<br>
It's part of the change to generic LLVM code that we had to implement while<br>
developing the AArch64 backend. These are the changes that no other target<br>
exercises, so sending them to the list before would have been a little<br>
motive-less, and often untestable.<br>
<br>
This one is reasonably simple, OK to commit?<br>
<br>
Cheers.<br>
<br>
Tim.<br>
<br>
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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></div><br></div>