From 41bdcded65eefd1b82a1f18dd49a61473d7072be Mon Sep 17 00:00:00 2001 From: Andrey Turbanov Date: Tue, 8 Aug 2023 11:38:15 +0000 Subject: [PATCH] 8313875: Use literals instead of static fields in java.util.Math: twoToTheDoubleScaleUp, twoToTheDoubleScaleDown Reviewed-by: redestad, darcy, bpb, rgiulietti --- src/java.base/share/classes/java/lang/Math.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/java.base/share/classes/java/lang/Math.java b/src/java.base/share/classes/java/lang/Math.java index fd5c8bb0491..67a0e3bc0cd 100644 --- a/src/java.base/share/classes/java/lang/Math.java +++ b/src/java.base/share/classes/java/lang/Math.java @@ -3346,12 +3346,12 @@ public final class Math { if(scaleFactor < 0) { scaleFactor = Math.max(scaleFactor, -MAX_SCALE); scale_increment = -512; - exp_delta = twoToTheDoubleScaleDown; + exp_delta = 0x1p-512; } else { scaleFactor = Math.min(scaleFactor, MAX_SCALE); scale_increment = 512; - exp_delta = twoToTheDoubleScaleUp; + exp_delta = 0x1p512; } // Calculate (scaleFactor % +/-512), 512 = 2^9, using @@ -3420,10 +3420,6 @@ public final class Math { return (float)((double)f*powerOfTwoD(scaleFactor)); } - // Constants used in scalb - static double twoToTheDoubleScaleUp = powerOfTwoD(512); - static double twoToTheDoubleScaleDown = powerOfTwoD(-512); - /** * Returns a floating-point power of two in the normal range. */