X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fbitcount.h;h=aa27c049415bcc3ce98750d48a9194152f8d8486;hp=a2e6c5be30eb5d80fc074b90bcbde4d5de35c124;hb=9e6d38d224e35e509521d9dced4a46cde021fd83;hpb=92b625d04f0d240a80defbfb2c95178a835efc6e diff --git a/src/bitcount.h b/src/bitcount.h index a2e6c5be..aa27c049 100644 --- a/src/bitcount.h +++ b/src/bitcount.h @@ -22,19 +22,12 @@ #if !defined(BITCOUNT_H_INCLUDED) #define BITCOUNT_H_INCLUDED -// To enable POPCNT support uncomment USE_POPCNT define. For PGO compile on a Core i7 -// you may want to collect profile data first with USE_POPCNT disabled and then, in a -// second profiling session, with USE_POPCNT enabled so to exercise both paths. Don't -// forget to leave USE_POPCNT enabled for the final optimized compile though ;-) - -//#define USE_POPCNT - - #include "types.h" -// Select type of intrinsic bit count instruction to use +// Select type of intrinsic bit count instruction to use, see +// README.txt on how to pgo compile with POPCNT support. -#if defined(__INTEL_COMPILER) && defined(IS_64BIT) && defined(USE_POPCNT) // Intel compiler +#if defined(__INTEL_COMPILER) && defined(USE_POPCNT) // Intel compiler #include @@ -45,17 +38,9 @@ inline bool cpu_has_popcnt() { return (CPUInfo[2] >> 23) & 1; } -// Define a dummy template to workaround a compile error if _mm_popcnt_u64() is not defined. -// -// If _mm_popcnt_u64() is defined in it will be choosen first due to -// C++ overload rules that always prefer a function to a template with the same name. -// If not, we avoid a compile error and because cpu_has_popcnt() should return false, -// our templetized _mm_popcnt_u64() is never called anyway. -template unsigned _mm_popcnt_u64(T) { return 0; } // Is never called - #define POPCNT_INTRINSIC(x) _mm_popcnt_u64(x) -#elif defined(_MSC_VER) && defined(IS_64BIT) && defined(USE_POPCNT) // Microsoft compiler +#elif defined(_MSC_VER) && defined(USE_POPCNT) // Microsoft compiler #include @@ -66,9 +51,6 @@ inline bool cpu_has_popcnt() { return (CPUInfo[2] >> 23) & 1; } -// See comment of _mm_popcnt_u64<>() few lines above for an explanation. -template unsigned __popcnt64(T) { return 0; } // Is never called - #define POPCNT_INTRINSIC(x) __popcnt64(x) #else // Safe fallback for unsupported compilers or when USE_POPCNT is disabled