<br><br><div class="gmail_quote">On Mon, Dec 26, 2011 at 5:17 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Sat, Dec 24, 2011 at 9:06 PM, Craig Topper <<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>> wrote:<br>
> Modified: cfe/trunk/lib/Headers/immintrin.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Headers/immintrin.h?rev=147262&r1=147261&r2=147262&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Headers/immintrin.h?rev=147262&r1=147261&r2=147262&view=diff</a><br>

> ==============================================================================<br>
> --- cfe/trunk/lib/Headers/immintrin.h (original)<br>
> +++ cfe/trunk/lib/Headers/immintrin.h Sat Dec 24 23:06:45 2011<br>
> @@ -48,7 +48,7 @@<br>
>  #include <smmintrin.h><br>
>  #endif<br>
><br>
> -#if defined (__AES__) || defined (__PCLMUL__)<br>
> +#if defined (__AES__)<br>
>  #include <wmmintrin.h><br>
>  #endif<br>
<br>
</div>This change doesn't appear to be related to the rest of the patch; why<br>
are you changing this?<br></blockquote><div><br>Clang doesn't define PCLMUL and the header file itself doesn't have any PCLMUL intrinsics in it and the header file explicitly checks that AES must be defined. So if clang did define PCLMUL and could be made to do so with defining AES, the header would break. I'll try to work on adding the intrinsics soon.<br>
<br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Eli<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>