]> git.nega.tv - josh/narcissus/commit
Fix reduction using the wrong rounding mode (#1)
authorJosh Simmons <josh@nega.tv>
Fri, 4 Nov 2022 20:59:59 +0000 (21:59 +0100)
committerGitHub <noreply@github.com>
Fri, 4 Nov 2022 20:59:59 +0000 (21:59 +0100)
commit1b87d0565a0b16fc5ab03dd7ef267933dfe0a8fa
tree46e0439a3577c0f52505750d41afe02c7d702b37
parent41afd7a7b0303449aeda09386edceeb416a46d0d
Fix reduction using the wrong rounding mode (#1)

Previously we were using `f32::round` in the reduction step, however
that isn't correct. We really need IEEE-754 tiesToEven semantics for the
rounding step, so manually implement that.
narcissus-maths/src/lib.rs
narcissus-maths/src/sin_cos_pi.rs
narcissus-maths/src/tan_pi.rs
narcissus-maths/tests/exhaustive_f32.rs