[PATCH] D53912: [Headers] [MS] Add intrin0.h
Azharuddin Mohammed via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 1 16:31:01 PDT 2018
azharudd added a comment.
In https://reviews.llvm.org/D53912#1281584, @rnk wrote:
> This sounds like it would defeat what I'm assuming is the intended purpose of intrin0.h, which is to reduce compile time. intrin.h is kind of enormous, and the compile time problems are well-documented. We should investigate what's up with intrin0.h and implement as many builtins as we need to support it.
I agree. This currently resolves issues with building for ARM64 target using Visual Studio 2017. The missing intrinsics it complains about are already present in intrin.h. We could add those to intrin0.h too but that would duplicate. Please let me know what you think.
Repository:
rC Clang
https://reviews.llvm.org/D53912
More information about the cfe-commits
mailing list