Use smaller IPIO2 table for 16 and 32 bit architectures

rust-lang / libm

A port of MUSL's libm to Rust.

Other

540 stars 98 forks source link

Use smaller IPIO2 table for 16 and 32 bit architectures #266

Closed Outurnate closed 2 years ago

Outurnate commented 2 years ago

Fixes #250

All tests pass after change. The longer version of the table is only needed for 128 bit floats, which this lib doesn't support. The short table could be used on all architectures, honestly

Amanieu commented 2 years ago

Yes, we should probably use the short table everywhere. However we may end up adding f128 in the future, so it's good to still have the code in there.