-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENH: Use Ryu #20
Comments
Pardon if this is a silly question, but I'm not up to date on how R extensions are built. How much control do you have over what compiler is used? I ask because all major compilers' latest versions basically bundle Ryu as part of the standard library. The reason FMM vendors Ryu is to support older compilers in wide use that do not. Basically if you can use fast_float for the read side might have more value. It's also getting bundled with compilers, but not all yet. MSVC is still in PR I believe, and LLVM only uses it for the C methods and not for |
That's a great question. For packages which are meant for CRAN (the Comprehensive R Archive Network, like PyPi for R), it is generally safe to assume GCC 10 (as part of the Rtools42 compiler toolchain), though up-to-date / current versions should have GCC 12 from Rtools43. I think GCC 10 only had integer support for
Makes sense, will do, thank you :) |
Yeah that's old enough to need Ryu/Dragonbox/fast_float. The upside is that you'll see some very large speedup, especially on 4+ cores. Ryu isn't too hard to add either. Just a few .c files to add to your target, or build and link a small library. |
I checked again, and GCC 12 is the first one with the fast methods bundled. So if you're able to use GCC 12 then there's nothing more to do :) |
Or another floating point library. As is done upstream.
The text was updated successfully, but these errors were encountered: