Nothing Special   »   [go: up one dir, main page]

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

Add f16 and f128 support #17572

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Add f16 and f128 support #17572

merged 1 commit into from
Jul 10, 2024

Conversation

beetrees
Copy link
Contributor
@beetrees beetrees commented Jul 10, 2024

Adds f16 and f128 support, using the rustc_apfloat library (also used by rustc) for parsing/arithmetic/displaying since the types aren't stable yet so can't be used by rust-analyzer itself.

Issue: #17451

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 10, 2024
@beetrees
Copy link
Contributor Author

CI should pass now (fixed the Clippy warning).

@Veykril
Copy link
Member
Veykril commented Jul 10, 2024

Thanks!
@bors r+

@bors
Copy link
Collaborator
bors commented Jul 10, 2024

📌 Commit b58e52b has been approved by Veykril

It is now in the queue for this repository.

bors added a commit that referenced this pull request Jul 10, 2024
Add `f16` and `f128` support

Adds `f16` and `f128` support, using the `rustc_apfloat` library (also used by `rustc`) for parsing/arithmetic/displaying since the types aren't stable yet so can't be used by rust-analyzer itself.
@bors
Copy link
Collaborator
bors commented Jul 10, 2024

⌛ Testing commit b58e52b with merge f8349fb...

@beetrees
Copy link
Contributor Author

Clippy warning fixed. I've checked locally with cargo clippy --all-targets, so CI should pass now.

@Veykril
Copy link
Member
Veykril commented Jul 10, 2024

@bors r+

@bors
Copy link
Collaborator
bors commented Jul 10, 2024

📌 Commit d5db933 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Collaborator
bors commented Jul 10, 2024

⌛ Testing commit d5db933 with merge 2bfab90...

@bors
Copy link
Collaborator
bors commented Jul 10, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 2bfab90 to master...

@bors bors merged commit 2bfab90 into rust-lang:master Jul 10, 2024
11 checks passed
@beetrees beetrees deleted the f16-f128 branch July 10, 2024 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants