[PATCH] D13015: [X86] Make f16c intrinsics accessible through emmintrin.h, per Intel docs

[Paul Robinson via cfe-commits]

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