msun/riscv: limit exposure of fegetexcept

[brooks]

Summary

fegetexcept is a static inline function in the default hardfloat
ABI so only expose it the softfloat case.

Given we've removed riscv64sf from the tree we might want to GC this
entierly at some point.

[jrtc27]

Oh I had it as an in-progress patch to make these *not* inline-only. I don't think it's a bug that they're only inline, but every other architecture exposes them as actual symbols too and GCC at least relies on that. See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272759 (comment 5 in particular) and https://cgit.freebsd.org/src/commit/?id=448c505c33cc334193590f3844406d6a74f26e2a.

[brooks]

In D44331#1011041, @jrtc27 wrote:

Oh I had it as an in-progress patch to make these *not* inline-only. I don't think it's a bug that they're only inline, but every other architecture exposes them as actual symbols too and GCC at least relies on that. See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272759 (comment 5 in particular) and https://cgit.freebsd.org/src/commit/?id=448c505c33cc334193590f3844406d6a74f26e2a.

Ah, I'll grab that and apply it.

[jrtc27]

In D44331#1011093, @brooks wrote:

In D44331#1011041, @jrtc27 wrote:

Oh I had it as an in-progress patch to make these *not* inline-only. I don't think it's a bug that they're only inline, but every other architecture exposes them as actual symbols too and GCC at least relies on that. See https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=272759 (comment 5 in particular) and https://cgit.freebsd.org/src/commit/?id=448c505c33cc334193590f3844406d6a74f26e2a.

Ah, I'll grab that and apply it.

Apparently, and not surprisingly, my completely-untested patch wasn't quite right, but whatever I missed is probably straightforward to fix.

[brooks]

Reduce scope to fegetexcept

[jhb]

I'd just skip to removing outright I think.

[emaste]

OK but I agree with jhb that skipping ahead to removal is fine

[brooks]

Will squash with D44334