[PATCH] D21959: [X86] Add xgetbv xsetbv intrinsics

Craig Topper via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 00:17:24 PDT 2016


craig.topper added a comment.

This change seems consistent with similar being done in r250158 when the other xsave intrinsics were added. intrin.h includes x86intrin.h which in turn includes xsaveintrin.h. So this just makes xgetbv/xsetbv available on non-microsoft platforms.


================
Comment at: lib/Headers/intrin.h:289
@@ -288,3 +288,2 @@
 static __inline__
 #define _XCR_XFEATURE_ENABLED_MASK 0
-unsigned __int64 __cdecl _xgetbv(unsigned int);
----------------
Should _XCR_XFEATURE_ENABLE_MASK be moved?

Also what is that "static __inline__" bound to? It looks like when _XCR_XFEATURE_ENABLE_MASK was added, the "static __inline__" was separated from the declaration of _xgetbv.


https://reviews.llvm.org/D21959





More information about the cfe-commits mailing list