-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Importer support for float80 #14971
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
Importer support for float80 #14971
Conversation
@swift-ci Please test. |
@moiseev The stdlib changes here are pretty minimal, but can you take a quick look? |
Importer side looks good, and none of the rest looks like anything alarming. |
Got build failures on Linux on Z (s390x) related to this commit:
(and others) Any ideas on how to fix this? Thanks |
Looks like Steve forgot one of the |
ah OK. I have a patch ready. Will create a PR. Thanks @jrose-apple! |
Great, thanks @vivkong! I'm happy to review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Teach the importer to map between C's
long double
type andFloat80
on non-Windows x86_64 and i386 platforms, and take advantage of this mapping to clean up a few details in the stdlib.This also makes available the full set of platform math functions for
Float80
.Resolves SR-2046.