Missed the CRC32 instructions. They have now been fixed in r147411.<br><br><div class="gmail_quote">On Sun, Jan 1, 2012 at 1:44 PM, 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">Fixed sfence, mfence, lfence, clflush, monitor, and mwait in r147409.<div class="HOEnZb"><div class="h5"><br><br><div class="gmail_quote">
On Sun, Jan 1, 2012 at 1:06 PM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com" target="_blank">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">This is similar to the fix for the prefetch instruction in r146163. I think the fence instructions and clflush are similarly broken. I'll see if I can find any others and I'll commit a fix.<br>

<br><div class="gmail_quote"><div><div>
On Sun, Jan 1, 2012 at 10:18 AM, Umansky, Victor <span dir="ltr"><<a href="mailto:victor.umansky@intel.com" target="_blank">victor.umansky@intel.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div>







<div>
<font face="Calibri"><span style="font-size:11pt">
<div>Hi Evan,</div>
<div> </div>
<div>I noticed that in X86Subtarget constructor you set ‘<font face="Courier New"><span style="font-size:10pt">X86SSELevel’</span></font><font face="Courier New"><span style="font-size:10pt"> </span></font>member of the class as  ‘<font face="Courier New"><span style="font-size:10pt">NoMMXSSE</span></font>’
in the case when  HasAVX member is set to ‘true’.</div>
<div>Effectively that invalidates SSE features for AVX architecture - because hasSSEn() accessors return ‘false’ when HasAVXn() is ‘true’.</div>
<div>I wonder whether this is the behavior which you’d like to enforce – as conceptually AVX architecture complements SSE rather than replaces it completely.</div>
<div> </div>
<div>I noticed this problem after discovering that LLVM fails to lower “sse2.fence” intrinsic when generating a code for AVX architecture – because this intrinsic is conditioned on hasSSE2() being ‘true’.</div>
<div>Is that case was somehow missed from regression testing, or there is another way to lower that intrinsic?</div>
<div> </div>
<div>I’d appreciate your clarifications.</div>
<div> </div>
<div>Best Regards,</div>
<div>     Victor Umansky</div>
<div> </div>
</span></font>
<font face="monospace">---------------------------------------------------------------------<br>
Intel Israel (74) Limited<br>
<br>
This e-mail and any attachments may contain confidential material for<br>
the sole use of the intended recipient(s). Any review or distribution<br>
by others is strictly prohibited. If you are not the intended<br>
recipient, please contact the sender and delete all copies.</font></div>

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