<div dir="ltr"><div>Commited @ r212722.</div><div><br></div><div>FYI, I have another alias related patch brewing --- for add & sub.</div><div><br></div>Thanks Tim !<div><br></div><div>Cheers,</div><div>--</div><div>Arnaud A. de Grandmaison</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 5:03 PM, Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Arnaud,<br>
<div class=""><br>
> I had to move it from ouside of the multiclass, to where it is used inside<br>
> the multiclass, because InstAlias does not have those. As you point, the<br>
> alias have no codegen pattern.<br>
<br>
</div>Ah, sorry. I missed that.<br>
<div class=""><br>
> I have not changed the functionality there, so no additionnal .ll test is<br>
> necessary -- I think.<br>
<br>
</div>Agreed. You were quite right the first time.<br>
<div class=""><br>
> By the way, this could applies in many places around. If you believe it is<br>
> worth, than I can fix those other places (and refactor a bit) in a separate<br>
> patch. This would make the code more readable.<br>
<br>
</div>That'd be great if you're willing! No need to ask for review for<br>
something that simple (depending on quite what you mean by "refactor"<br>
of course).<br>
<div class=""><br>
> I attached the updated patch. I left the fixme note in<br>
> basic-a64-diagnostics; I will remove it before committing if you think we<br>
> just don't care. I also changed pseudo to alias in the commit message or<br>
> comments.<br>
<br>
</div>I think this patch looks good (either way with the FIXME). Thanks for<br>
updating it.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br></div>