Adding some testcases that use the load patterns would probably be good too. I've been fixing a lot of these sort of patterns lately because they didn't understand the integer promotion.<br><br>~Craig<br><br><div class="gmail_quote">
On Tue, Jan 3, 2012 at 10:14 AM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
All integer vector loads in patterns need to be memopv2i64(!28-bit) or memopv4i64(256-bit) and bitcasted to the correct type because all integer vector loads are promoted to v2i64 or v4i64. So for instance<br><br>+def : Pat<(int_x86_xop_vphsubwd (alignedloadv8i16 addr:$src1)),<br>

+          (VPHSUBWDrm addr:$src1)>;<br><br>Needs to be <br><br>+def : Pat<(int_x86_xop_vphsubwd (bc_v8i16 (memopv2i64 addr:$src1))),<br>+          (VPHSUBWDrm addr:$src1)>;<br><br>~Craig<br><br><div class="gmail_quote">
<div class="im">
On Tue, Jan 3, 2012 at 8:12 AM, Jan Sjodin <span dir="ltr"><<a href="mailto:jan_sjodin@yahoo.com" target="_blank">jan_sjodin@yahoo.com</a>></span> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
<div><div style="color:#000;background-color:#fff;font-family:arial,helvetica,sans-serif;font-size:10pt"><div>This patch adds XOP intrinsics (including tests). Ok to commit?<span><font color="#888888"><br></font></span></div>

<span><font color="#888888"><div><br></div><div>- Jan<br></div></font></span></div></div><br></div><div class="im">_______________________________________________<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></div></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br>~Craig<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>