Hi Nicolas,<br>
<br>
On Fri, Jan 4, 2019 at 7:39 AM Nicolas Cellier <notifications@github.com><br>
wrote:<br>
<br>
> Now that I'm fairly confident that my patches are OK, I suggest this:<br>
><br>
>    - we keep fdlibm for backward compatibility;<br>
>    - we maintain our own patched version;<br>
>    - we don't add any new feature nor try to patch anything but correct<br>
>    compilation of existing source; this should generate very low level of<br>
>    activity (only if C standard change and break another thing);<br>
>    - for this we clone https://github.com/nicolas-cellier-aka-nice/fdlibm/<br>
>    on opensmalltalk so as to not depend on an exotic repository, and replace<br>
>    the fdlibm files by using a git submodule;<br>
>    - we start an alternate FloatMathPlugin implementation based on<br>
>    another library (I suggest CRLibm but other suggestions are welcome).<br>
><br>
> Does it sound OK?<br>
><br>
<br>
Sounds perfect to me.  I would add that once we have an alternate<br>
FloatMathPlugin implementation based on another library we can do the<br>
performance analysis and if it is fast enough we should implement normal<br>
floating point arithmetic with the same support and get rid of<br>
FloatMathPlugin altogether, having bit-identical floating point provided by<br>
the normal primitives.<br>
<br>
_,,,^..^,,,_<br>
best, Eliot<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/315#issuecomment-451485149">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyWxR-LhThGDIw5BHhmm3oBNzCyPKeks5u_3qYgaJpZM4Y_jtg">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW4JvXxgPN3w4TqWjzgB4XrZQJXBHks5u_3qYgaJpZM4Y_jtg.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eliotmiranda in #315: Hi Nicolas,\n\nOn Fri, Jan 4, 2019 at 7:39 AM Nicolas Cellier \u003cnotifications@github.com\u003e\nwrote:\n\n\u003e Now that I'm fairly confident that my patches are OK, I suggest this:\n\u003e\n\u003e    - we keep fdlibm for backward compatibility;\n\u003e    - we maintain our own patched version;\n\u003e    - we don't add any new feature nor try to patch anything but correct\n\u003e    compilation of existing source; this should generate very low level of\n\u003e    activity (only if C standard change and break another thing);\n\u003e    - for this we clone https://github.com/nicolas-cellier-aka-nice/fdlibm/\n\u003e    on opensmalltalk so as to not depend on an exotic repository, and replace\n\u003e    the fdlibm files by using a git submodule;\n\u003e    - we start an alternate FloatMathPlugin implementation based on\n\u003e    another library (I suggest CRLibm but other suggestions are welcome).\n\u003e\n\u003e Does it sound OK?\n\u003e\n\nSounds perfect to me.  I would add that once we have an alternate\nFloatMathPlugin implementation based on another library we can do the\nperformance analysis and if it is fast enough we should implement normal\nfloating point arithmetic with the same support and get rid of\nFloatMathPlugin altogether, having bit-identical floating point provided by\nthe normal primitives.\n\n_,,,^..^,,,_\nbest, Eliot\n"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/315#issuecomment-451485149"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/315#issuecomment-451485149",
"url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/315#issuecomment-451485149",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>