<div dir="auto">FloatMathPlugin is based on Sun microsystems fdlibm. This library does not compile correctly on modern C compiler because it uses pointer aliasing to access bits of a double as two int32. Nowadays, pointer aliasing is undefined behavior. The same library was used in java and compiled with special flags, but this is fragile. It is easy to replace this UB, but that mean forking this now unmaintained code, unless we find another already existing fork. No time to search now.</div><br><div class="gmail_quote"><div dir="ltr">Le lun. 3 déc. 2018 à 22:31, Eliot Miranda <<a href="mailto:notifications@github.com">notifications@github.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <p>Hi Ben,</p>
<pre><code>ideally we'd have bit-identical FP on all platforms.  Currently this is only achievable using Andreas Raab's FloatMath plugin, used by Croquet and Terf, and not yet tested in 64-bits.  What that has to say about whether we support the default or 53-bit on x64 I don't know.  i guess we should for now simply support the default m ode.  In any case it's unlikely to make much difference because the computational model yields results being truncated to 53 bits after each operation (i.e. other than in the graphics subsystem, floating point operations are performed on double-precision floats so there's no scope for extended calculations to use 64-bit precision internally.  So feel free to include the above as commentary but I think we want to apply your pull request asap and so we should leave the precision argument for a later day when we have more data (such as a 64-bit Terf to test).
</code></pre>

<p style="font-size:small;color:#666">—<br>You are receiving this because you commented.<br>Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/313#issuecomment-443877985" target="_blank" rel="noreferrer">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyW19NRT5NIHT-ZwfbaMs_Mn160JzRks5u1ZhKgaJpZM4Y84NR" target="_blank" rel="noreferrer">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW6Mt1cpxGA4uTNwiFXVgtD80wDqGks5u1ZhKgaJpZM4Y84NR.gif" height="1" width="1" alt=""></p>

</blockquote></div>