Page MenuHomeFreeBSD

riscv: remove more riscv64sf support
ClosedPublic

Authored by brooks on Mar 12 2024, 11:20 PM.
Tags
None
Referenced Files
F106201212: D44334.id135682.diff
Fri, Dec 27, 1:52 AM
F106167983: D44334.diff
Thu, Dec 26, 12:24 PM
Unknown Object (File)
Nov 14 2024, 2:24 PM
Unknown Object (File)
Nov 14 2024, 2:11 PM
Unknown Object (File)
Nov 13 2024, 11:11 PM
Unknown Object (File)
Nov 9 2024, 6:36 PM
Unknown Object (File)
Nov 9 2024, 6:35 PM
Unknown Object (File)
Nov 9 2024, 6:35 PM
Subscribers

Details

Summary

Remove a few more bits of riscv64sf support in libc and libm.

Fixes: 1ca12bd927d7 Remove the riscv64sf architecture.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 56570
Build 53458: arc lint + arc unit

Event Timeline

brooks retitled this revision from riscv: remove a more riscv64sf support to riscv: remove more riscv64sf support.Mar 12 2024, 11:21 PM
brooks added a reviewer: jhb.
lib/msun/riscv/fenv.h
74–83

Don't think we need to spend 10 lines of code to make the errors marginally more friendly. Arguably even this is unnecessary; you'll get an error from the linker if you mix different float ABIs, and crt* are hard float.

Drop __riscv_float_abi_soft in fenv.h

I think tis is good... not sure about jrtc's comments

This revision is now accepted and ready to land.Mar 13 2024, 1:57 AM
lib/msun/riscv/fenv.h
74–83

Yeah, I would just go with Jess' suggestion here.

lib/msun/riscv/fenv.h
74–83

So delete the whole block?

lib/msun/riscv/fenv.h
74–83

Either that or the suggested code block in Jess's comment that has a single #ifndef around an #error. I don't have a strong preference.

  • Squash with D44331
  • Reduce ABI checks as suggested
This revision now requires review to proceed.Mar 19 2024, 9:13 PM
This revision is now accepted and ready to land.Mar 21 2024, 6:51 PM
This revision was automatically updated to reflect the committed changes.