Hi Chris,<br><br>Thanks for working on this! For avoidance of doubt, in A64 'mul a, b' is just an alias for 'mla a, b, xzr'. So Tim is suggesting to update the mla patterns too, which makes sense to me. <br><br>James <br><div class="gmail_quote"><div dir="ltr">On Thu, 14 Jan 2016 at 17:26, Tim Northover via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">t.p.northover added a comment.<br>
<br>
It's probably worth adding patterns for the non-XZR case too while we're at it (the basic ones are in AArch64InstrFormats.td around WideMulAccum, but since that's only a class we can't add them there).<br>
<br>
<br>
<a href="http://reviews.llvm.org/D16195" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16195</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>