[ODE] building 64bit ODE?

[Krystian Ligenza]

Tanguy Fautre pisze:
> Tanguy Fautre wrote:
>> Krystian Ligenza wrote:
>>> Tanguy Fautre pisze:
>>>> Krystian Ligenza wrote:
>>>>> I have tried to build a 64bit version of ODE, but get lots of 
>>>>> errors from opcode:
>>>>> error C4235: nonstandard extension used : '__asm' keyword not 
>>>>> supported on this architecture    \opcode\Ice\IceFPU.h    47
>>
>> Ok, I've looked at the IceFPU.h file. Here is the faulty part:
>>
>> [...]
>
> I was a bit too quick to think it was the only inline assembly in this 
> file... There are tons of it. :-(
>
>
> I guess that an easy (and probably good) solution would be to remove 
> the Visual C++ versions (i.e. ifdef _MSC_VER) of all these functions. 
> Unless someone can come up with a good reason why we should keep them 
> ? Cause to me it looks like old and now-worthless optimizations (e.g. 
> most x64 compilers use SSE to handle floating points instead of the 
> FPU), and this is probably not worth the troubles anymore.
>
>
> Otherwise, another solution (but that I don't agree with) could be to 
> replace in this file all the:
>
> #ifdef _MSC_VER
>
> with
>
> #if defined _MSC_VER && ! defined WIN64

thank you for investigating this. Don't know how big optimization it 
does for 32bits, so I'm waiting for others opinion.

Cheers,
Krystian