Am 09.08.2015 um 12:57 schrieb Jose Fonseca: > Neither MSVC nor MinGW defines LONG_BIT. For MSVC this was not a problem as > it doesn't define __x86_64__ macro (it's GCC specific.) > > However on Windows long type is guaranteed to be 32bits. > > Also add an #error, as GCC will just warn, not throw any error, when no > value is returned. > > Trivial. > --- > src/util/rounding.h | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/src/util/rounding.h b/src/util/rounding.h > index b0c9918..ec31b47 100644 > --- a/src/util/rounding.h > +++ b/src/util/rounding.h > @@ -96,8 +96,10 @@ _mesa_lroundevenf(float x) > #ifdef __x86_64__ > #if LONG_BIT == 64 > return _mm_cvtss_si64(_mm_load_ss(&x)); > -#elif LONG_BIT == 32 > +#elif LONG_BIT == 32 || defined(_WIN32) > return _mm_cvtss_si32(_mm_load_ss(&x)); > +#else > +#error "Unsupported or undefined LONG_BIT" > #endif > #else > return lrintf(x); > @@ -114,8 +116,10 @@ _mesa_lroundeven(double x) > #ifdef __x86_64__ > #if LONG_BIT == 64 > return _mm_cvtsd_si64(_mm_load_sd(&x)); > -#elif LONG_BIT == 32 > +#elif LONG_BIT == 32 || defined(_WIN32) > return _mm_cvtsd_si32(_mm_load_sd(&x)); > +#else > +#error "Unsupported or undefined LONG_BIT" > #endif > #else > return lrint(x); >
For the series: Reviewed-by: Roland Scheidegger <srol...@vmware.co> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev