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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to