Skip to content
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

Lagoon jets for IEEE 754 floats. #638

Merged
merged 43 commits into from
Sep 11, 2024
Merged

Lagoon jets for IEEE 754 floats. #638

merged 43 commits into from
Sep 11, 2024

Conversation

sigilante
Copy link
Contributor

Pursuant to draft UIP at https://github.com/urbit/UIPs/blob/sigilante/lagoon-reals/UIPS.

Should be released alongside corresponding Hoon code.

@sigilante sigilante requested a review from a team as a code owner April 18, 2024 19:35
@sigilante
Copy link
Contributor Author

The current root problem is that aarch wants a ton of SoftFloat functions that aren't linked yet.

https://github.com/urbit/vere/actions/runs/8819670490/job/24211443275?pr=638

Adding them piecemeal seems to be a fool's errand: any way to just build the whole thing at one go for each? I.e. include all files rather than manually listing the subset of interest?

@sigilante
Copy link
Contributor Author

There seem to be some platform-specific optimized versions linked in as well, so this may need to stay in the same vein to be effectively fast.

@sigilante
Copy link
Contributor Author

@urbit/kernel this is building now thanks to @pkova's sleuthing.

@sigilante
Copy link
Contributor Author

This is ready for review.

@sigilante
Copy link
Contributor Author

When the tests pass here on that last commit, then this is ready to merge. @joemfb @pkova @belisarius222

@joemfb
Copy link
Member

joemfb commented Sep 6, 2024

urbit/urbit#6971

@pkova pkova merged commit 5f47598 into next/kelvin/410 Sep 11, 2024
5 checks passed
@pkova pkova deleted the lagoon-jets branch September 11, 2024 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants