[PATCH] D22774: [MSVC] Add ARM support to intrin.h for MSVC compatibility
Martin Storsjö via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 1 23:19:22 PDT 2016
mstorsjo added inline comments.
Comment at: lib/Headers/armintr.h:26
@@ +25,3 @@
> Hmm, why not do __has_header instead? armv7-windows-itanium won't define `_MSC_VER` but you may have the header from the SDK.
Hmm, so you mean like this?
// Normal file content
typedef enum ...
That would cause use in armv7-windows-msvc mode to use the SDK's armintr.h instead of ours - which as far as I know also is fine. (The issue I'm mainly fixing is that including `<intrin.h>` is expected to bring in this, but currently it doesn't.)
I'm not sure I follow your argument completely though - armv7-windows-itanium doesn't define `_MSC_VER`, and thus would already try to include the next armintr.h, which would be picked up from the SDK if it exists there.
More information about the cfe-commits