[PATCH] D47182: [X86] Move all Intel defined intrinsic includes into immintrin.h

[David Kreitzer via Phabricator via cfe-commits]

DavidKreitzer added a comment.

Hi Craig, just one comment on the details. Everything else looks good.



================
Comment at: lib/Headers/x86intrin.h:47
-
-#if !defined(_MSC_VER) || __has_feature(modules) || defined(__POPCNT__)
-#include <popcntintrin.h>
----------------
I see that you are removing this popcntintrin.h inclusion without adding it to immintrin.h. Is that because you are relying on the inclusion from smmintrin.h?

If so, won't that cause a problem on Windows with -mpopcnt for targets that don't include smmintrin.h?


https://reviews.llvm.org/D47182