probinson added a subscriber: probinson.
probinson added a comment.

Regarding _mm[256]_cvtps_ph and ...cvtph_ps, I can find an Intel doc that say 
the _mm_* functions are in emmintrin.h and the _mm256_* are in immintrin.h, so 
putting them all with emmintrin.h is not consistent with what Intel says.  I 
can find a Microsoft doc that say these are all available with immintrin.h (not 
emmintrin.h).  It looks like gcc puts them with immintrin.h also.

Can we move the #include of f16cintrin.h from emmintrin.h to immintrin.h?  It 
would be more compatible with everybody else, and equally inconsistent with 
Intel's doc AFAICT.


Repository:
  rL LLVM

http://reviews.llvm.org/D13015



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to