Used copysign to avoid unnecessary branches.

Centril · Aug 20, 2019 · 535efa4 · 535efa4
1 parent 64e3a10
commit 535efa4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 18 deletions.
diff --git a/src/libstd/f32.rs b/src/libstd/f32.rs
@@ -910,15 +910,7 @@ impl f32 {
     pub fn asinh(self) -> f32 {
         match self {
             x if x == NEG_INFINITY => NEG_INFINITY,
-            x if x.is_sign_negative() => {
-                let v = (x + ((x * x) + 1.0).sqrt()).ln();
-                if v.is_sign_negative() {
-                    v
-                } else {
-                    -v
-                }
-            }
-            x => (x + ((x * x) + 1.0).sqrt()).ln()
+            x => (x + ((x * x) + 1.0).sqrt()).ln().copysign(self)
         }
     }
 

diff --git a/src/libstd/f64.rs b/src/libstd/f64.rs
@@ -833,15 +833,7 @@ impl f64 {
     pub fn asinh(self) -> f64 {
         match self {
             x if x == NEG_INFINITY => NEG_INFINITY,
-            x if x.is_sign_negative() => {
-                let v = (x + ((x * x) + 1.0).sqrt()).ln();
-                if v.is_sign_negative() {
-                    v
-                } else {
-                    -v
-                }
-            }
-            x => (x + ((x * x) + 1.0).sqrt()).ln()
+            x => (x + ((x * x) + 1.0).sqrt()).ln().copysign(self)
         }
     }