rust-analyzer icon indicating copy to clipboard operation
rust-analyzer copied to clipboard

Add `f16` and `f128` support (add to `FloatTy`)

Open tgross35 opened this issue 1 year ago • 3 comments

FloatTy in rustc has f16 and f128 now. It would probably be pretty easy to update RA's FloatTy to match and propagate that out.

tgross35 avatar Jun 19 '24 03:06 tgross35

Probably pretty easy

@rustbot label +E-easy +A-hir

tgross35 avatar Jun 19 '24 03:06 tgross35

Error: Label E-easy can only be set by Rust team members

Please file an issue on GitHub at triagebot if there's a problem with this bot, or reach out on #t-infra on Zulip.

rustbot avatar Jun 19 '24 03:06 rustbot

That will require changes to chalk

Veykril avatar Jun 19 '24 07:06 Veykril

Now that #17572 has been merged, all that's left for this issue is to:

  • Add support for f16/f128 intrinsics.
  • Replace rustc_apfloat with the types themselves once they've been stabilised.

Most of the former will probably require the later since rustc_apfloat only supports basic arithmetic operations.

beetrees avatar Jul 10 '24 10:07 beetrees