Move __cpuid() definition for gcc in types.h

This will allow to use the function also for other
purposes then detecting POPCNT.

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>

diff --git a/src/bitcount.h b/src/bitcount.h
index 7a05970d2abcb45a96333a34a070504941ac3b20..43895180c1e9ae7566e16e3f35f29b93e00e18eb 100644 (file)
--- a/src/bitcount.h
+++ b/src/bitcount.h
@@ -29,8 +29,6 @@
 
 #if defined(__INTEL_COMPILER) && defined(USE_POPCNT) // Intel compiler
 
-#include <nmmintrin.h>
-
 inline bool cpu_has_popcnt() {
 
   int CPUInfo[4] = {-1};
@@ -42,8 +40,6 @@ inline bool cpu_has_popcnt() {
 
 #elif defined(_MSC_VER) && defined(USE_POPCNT) // Microsoft compiler
 
-#include <intrin.h>
-
 inline bool cpu_has_popcnt() {
 
   int CPUInfo[4] = {-1};
@@ -55,16 +51,6 @@ inline bool cpu_has_popcnt() {
 
 #elif defined(__GNUC__) && defined(USE_POPCNT) // Gcc compiler
 
-inline void __cpuid(unsigned int op,
-                    unsigned int *eax, unsigned int *ebx,
-                    unsigned int *ecx, unsigned int *edx)
-{
-  *eax = op;
-  *ecx = 0;
-  __asm__("cpuid" : "=a" (*eax), "=b" (*ebx), "=c" (*ecx), "=d" (*edx)
-                  : "0" (*eax), "2" (*ecx));
-}
-
 inline bool cpu_has_popcnt() {
 
   unsigned int eax, ebx, ecx, edx;

diff --git a/src/types.h b/src/types.h
index 0d0d8485ecdbec7f4f248a828e4a27fa9a08bee2..d91009df453748d6701ee607b177921368a7a60e 100644 (file)
--- a/src/types.h
+++ b/src/types.h
@@ -73,4 +73,22 @@ typedef uint64_t Bitboard;
 #define CACHE_LINE_ALIGNMENT  __attribute__ ((aligned(64)))
 #endif
 
+// Define a __cpuid() function for gcc compilers, for Intel and MSVC
+// is already available as an intrinsic.
+#if defined(__INTEL_COMPILER)
+#include <nmmintrin.h>
+#elif defined(_MSC_VER)
+#include <intrin.h>
+#elif defined(__GNUC__)
+inline void __cpuid(unsigned int op,
+                    unsigned int *eax, unsigned int *ebx,
+                    unsigned int *ecx, unsigned int *edx)
+{
+  *eax = op;
+  *ecx = 0;
+  __asm__("cpuid" : "=a" (*eax), "=b" (*ebx), "=c" (*ecx), "=d" (*edx)
+                  : "0" (*eax), "2" (*ecx));
+}
+#endif
+
 #endif // !defined(TYPES_H_INCLUDED)