[cfe-commits] r73021 - /cfe/trunk/lib/Headers/xmmintrin.h

[Chris Lattner]

On Jun 7, 2009, at 1:48 AM, Eli Friedman wrote:

> On Sun, Jun 7, 2009 at 1:33 AM, Daniel Dunbar<daniel at zuster.org>  
> wrote:
>> Author: ddunbar
>> Date: Sun Jun  7 03:33:23 2009
>> New Revision: 73021
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=73021&view=rev
>> Log:
>> xmmintrin needs to include emmintrin, Darwin system headers seem to  
>> depend on it
>> defining m128[id], at least.
>
> There's an issue with this approach: if xmmintrin.h includes
> emmintrin.h, it's impossible to use xmmintrin.h without SSE2.

I don't follow.  emmintrin is SS1 and xmmintrin is SSE2.

-Chris


>  I'm not
> sure if that's actually a signficant issue, considering any remotely
> recent computer has SSE2 support, though; should we care?  We could
> always put it in an ifdef __APPLE__, I suppose...
>
> -Eli
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits