export additional header modules from xmmintrin

[jtsoftware]

Summary

If someone only includes xmmintrin.h, but references any definitions from stdlib.h (included via mm_malloc.h), if module are enabled it will fail, but if modules are not enable, it will not fail to compile. This will fix this (if STDC_HOSTED is defined).

[rsmith]

lib/Headers/module.modulemap

48

Did you intentionally remove this comment?

56–58

You shouldn't need to add anything here. sse2 exports sse, which in turn exports mm_malloc and mmx, so users of sse2 get mm_malloc and mmx whether or not sse2 lists them. It's better for it to not do so, as they aren't direct dependencies.

[jtsoftware]

Right. Also undoing moving stuff around. How about this? Thanks.

[rsmith]

LGTM

[Eugene.Zelenko]

Committed in r264092.

lib/Headers/module.modulemap

explicit module mmx {
       header "mmintrin.h"
     }
 
     explicit module f16c {
       header "f16cintrin.h"
     }
 
     explicit module sse {
+      export mm_malloc
       export mmx
       export sse2 // note: for hackish <emmintrin.h> dependency
       header "xmmintrin.h"
     }
 
     explicit module sse2 {
       export sse
       header "emmintrin.h"
     }
 
     explicit module sse3 {
       export sse2
       header "pmmintrin.h"
     }
 
     explicit module ssse3 {
       export sse3
       header "tmmintrin.h"
     }

test/Headers/xmmintrin.c

 }
 
 // Make sure that including <xmmintrin.h> also makes <emmintrin.h>'s content available.
 // This is an ugly hack for GCC compatibility.
 __m128 test_xmmintrin_provides_emmintrin(__m128d __a, __m128d __b) {
   return _mm_add_sd(__a, __b);
 }
 
+#if __STDC_HOSTED__
+// Make sure stdlib.h symbols are accessible.
+void *p = NULL;
+#endif