This makes more sense. LGTM.

-- Sean Silva

On Thu, May 21, 2015 at 6:03 PM, John Thompson <
[email protected]> wrote:

> It turns out that these are the exports needed, since the wmmintrin.h file
> includes the aes and pclmul headers.
>
>
> http://reviews.llvm.org/D9842
>
> Files:
>   lib/Headers/module.modulemap
>
> Index: lib/Headers/module.modulemap
> ===================================================================
> --- lib/Headers/module.modulemap
> +++ lib/Headers/module.modulemap
> @@ -157,6 +157,8 @@
>      explicit module aes_pclmul {
>        requires aes, pclmul
>        header "wmmintrin.h"
> +      export aes
> +      export pclmul
>      }
>
>      explicit module aes {
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to