<html><body><div style="color:#000; background-color:#fff; font-family:arial, helvetica, sans-serif;font-size:10pt"><div><span>No functional changes with this patch, the existing tests cover it. The coming XOP patches exercise the same code as well with some twists (5 op instructions).</span></div><div><br><span></span></div><div><span>- Jan<br></span></div><div><br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; margin-top: 5px; padding-left: 5px;"> <div style="font-family: arial, helvetica, sans-serif; font-size: 10pt;"> <div style="font-family: times new roman, new york, times, serif; font-size: 12pt;"> <font face="Arial" size="2"> <hr size="1"> <b><span style="font-weight:bold;">From:</span></b> Bruno Cardoso Lopes <bruno.cardoso@gmail.com><br> <b><span style="font-weight: bold;">To:</span></b> Jan Sjodin <jan_sjodin@yahoo.com> <br><b><span style="font-weight: bold;">Cc:</span></b>
"llvm-commits@cs.uiuc.edu" <llvm-commits@cs.uiuc.edu> <br> <b><span style="font-weight: bold;">Sent:</span></b> Wednesday, December 7, 2011 5:57 PM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: FMA4 cleanup patch<br> </font> <br>
LGTM! Can you add testcases?<br><br>On Wed, Dec 7, 2011 at 1:50 PM, Jan Sjodin <<a ymailto="mailto:jan_sjodin@yahoo.com" href="mailto:jan_sjodin@yahoo.com">jan_sjodin@yahoo.com</a>> wrote:<br>> Patch to clean up the FMA4 encoding. I had accidentally swapped the operands<br>> in the asm strings for the FMA4 rr patterns, which caused some confusion.<br>> Ok to commit?<br>><br>> - Jan<br><br><br><br>-- <br>Bruno Cardoso Lopes<br>http://www.brunocardoso.cc<br><br><br> </div> </div> </blockquote></div> </div></body></html>