Page MenuHomeFreeBSD

riscv: remove more riscv64sf support
ClosedPublic

Authored by brooks on Mar 12 2024, 11:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 19, 5:57 AM
Unknown Object (File)
Thu, May 16, 10:03 AM
Unknown Object (File)
Mon, May 6, 4:10 AM
Unknown Object (File)
Sat, May 4, 1:22 PM
Unknown Object (File)
Thu, May 2, 8:06 PM
Unknown Object (File)
Sun, Apr 28, 12:42 AM
Unknown Object (File)
Sat, Apr 27, 12:25 PM
Unknown Object (File)
Sat, Apr 27, 11:10 AM
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 Not Applicable
Unit
Tests Not Applicable

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
73–76

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
73–76

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

lib/msun/riscv/fenv.h
73–76

So delete the whole block?

lib/msun/riscv/fenv.h
73–76

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.