diff options
| author | Marc Tiehuis <marctiehuis@gmail.com> | 2017-06-20 23:01:04 +1200 |
|---|---|---|
| committer | Marc Tiehuis <marctiehuis@gmail.com> | 2017-06-20 23:10:22 +1200 |
| commit | 5bbec42a4edb3f6758a9d67fcf763c660ac3f204 (patch) | |
| tree | 3dc324d083f142935df53c5900edfc04c0f956f8 /std/math/atanh.zig | |
| parent | c9fc8bd802f5ed52c4cc78b93f18fc5dc9b6bb7f (diff) | |
| download | zig-5bbec42a4edb3f6758a9d67fcf763c660ac3f204.tar.gz zig-5bbec42a4edb3f6758a9d67fcf763c660ac3f204.zip | |
Add math special case tests and general fixes
- Should cover special case inputs for most functions
- Fixed a number of runtime panicking behaviour reliant on shift
overflow/division by zero etc.
Diffstat (limited to 'std/math/atanh.zig')
| -rw-r--r-- | std/math/atanh.zig | 32 |
1 files changed, 30 insertions, 2 deletions
diff --git a/std/math/atanh.zig b/std/math/atanh.zig index 25540d2996..7b1f3d0989 100644 --- a/std/math/atanh.zig +++ b/std/math/atanh.zig @@ -1,3 +1,9 @@ +// Special Cases: +// +// - atanh(+-1) = +-inf with signal +// - atanh(x) = nan if |x| > 1 with signal +// - atanh(nan) = nan + const math = @import("index.zig"); const assert = @import("../debug.zig").assert; @@ -21,6 +27,10 @@ fn atanh_32(x: f32) -> f32 { var y = @bitCast(f32, i); // |x| + if (y == 1.0) { + return math.copysign(f32, math.inf(f32), x); + } + if (u < 0x3F800000 - (1 << 23)) { if (u < 0x3F800000 - (32 << 23)) { // underflow @@ -33,7 +43,6 @@ fn atanh_32(x: f32) -> f32 { y = 0.5 * math.log1p(2 * y + 2 * y * y / (1 - y)); } } else { - // avoid overflow y = 0.5 * math.log1p(2 * (y / (1 - y))); } @@ -47,6 +56,10 @@ fn atanh_64(x: f64) -> f64 { var y = @bitCast(f64, u & (@maxValue(u64) >> 1)); // |x| + if (y == 1.0) { + return math.copysign(f64, math.inf(f64), x); + } + if (e < 0x3FF - 1) { if (e < 0x3FF - 32) { // underflow @@ -59,7 +72,6 @@ fn atanh_64(x: f64) -> f64 { y = 0.5 * math.log1p(2 * y + 2 * y * y / (1 - y)); } } else { - // avoid overflow y = 0.5 * math.log1p(2 * (y / (1 - y))); } @@ -86,3 +98,19 @@ test "math.atanh_64" { assert(math.approxEq(f64, atanh_64(0.2), 0.202733, epsilon)); assert(math.approxEq(f64, atanh_64(0.8923), 1.433099, epsilon)); } + +test "math.atanh32.special" { + assert(math.isPositiveInf(atanh_32(1))); + assert(math.isNegativeInf(atanh_32(-1))); + assert(math.isSignalNan(atanh_32(1.5))); + assert(math.isSignalNan(atanh_32(-1.5))); + assert(math.isNan(atanh_32(math.nan(f32)))); +} + +test "math.atanh64.special" { + assert(math.isPositiveInf(atanh_64(1))); + assert(math.isNegativeInf(atanh_64(-1))); + assert(math.isSignalNan(atanh_64(1.5))); + assert(math.isSignalNan(atanh_64(-1.5))); + assert(math.isNan(atanh_64(math.nan(f64)))); +} |
