[CELT-dev] WIN64 issue

[Jean-Marc Valin]

Thanks for the patch. It's in the git repo now.

	Jean-Marc

On 10-10-16 12:17 PM, John Ridges wrote:
>   Hi Jean-Marc,
> 
> I've been compiling CELT for 64-bit Windows and it all seems to work 
> fine (with the occasional benign warning message) except for one place, 
> and that is in "float_cast.h" where there is some inline assembly 
> language that gets complied for Windows. Since the Microsoft 64-bit 
> compiler won't allow inline assembly language (and *still* doesn't have 
> "lrintf"), I suggest the following change to use intrinsics for the 
> 64-bit compiler:
> 
>   --- float_cast.h    Sat Oct 17 19:51:32 2009
> +++ new\float_cast.h    Fri Oct 15 12:09:38 2010
> @@ -72,8 +72,14 @@
>   #include <math.h>
>   #define float2int(x) lrint(x)
> 
> -#elif (defined (WIN32) || defined (_WIN32))
> +#elif (defined (WIN64) || defined (_WIN64))
> +    #include <xmmintrin.h>
> 
> +    __inline long int float2int(float value)
> +    {
> +        return _mm_cvtss_si32(_mm_load_ss(&value));
> +    }
> +#elif (defined (WIN32) || defined (_WIN32))
>       #include <math.h>
> 
>       /*    Win32 doesn't seem to have these functions.
> 
> 
> BTW, I have also compiled CELT for 64-bit Mac OS X with no problems.
> 
> Cheers,
> 
> John Ridges
> 
> 
> 
> 
> _______________________________________________
> celt-dev mailing list
> celt-dev at xiph.org
> http://lists.xiph.org/mailman/listinfo/celt-dev
> 
>