bruno added inline comments.
> eladcohen wrote in module.modulemap:133
> emmintrin.h is already included explicitly in wmmintrin.h & __wmmintrin_aes.h.
>
> If a user includes / there is no problem, since the
> intel submodule has an 'export *' directive and both aes & sse2 will be
> imported.
eladcohen added inline comments.
Comment at: lib/Headers/module.modulemap:133
@@ -131,2 +132,3 @@
explicit module aes {
+ export sse2
header "__wmmintrin_aes.h"
bruno wrote:
> The mmx case above makes sense to me, but I find conceptually odd that
bruno added a subscriber: bruno.
Comment at: lib/Headers/module.modulemap:133
@@ -131,2 +132,3 @@
explicit module aes {
+ export sse2
header "__wmmintrin_aes.h"
The mmx case above makes sense to me, but I find conceptually odd that we need
to re-
eladcohen added a comment.
ping
https://reviews.llvm.org/D24752
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
eladcohen created this revision.
eladcohen added a reviewer: rsmith.
eladcohen added a subscriber: cfe-commits.
X86 intrinsic header files mm3dnow.h and wmmintrin.h include and depend on
mmintrin.h and emmintrin.h respectively.
This patch adds these missing dependencies to the corresponding submo
