[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