log(math): add log for x87 ret.

halibobo1205 · Jun 12, 2024 · 28c110d · 28c110d
1 parent ce7a0c1
commit 28c110d
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 5 deletions.
diff --git a/common/src/main/java/org/tron/common/math/MathWrapper.java b/common/src/main/java/org/tron/common/math/MathWrapper.java
@@ -140,6 +140,12 @@ public static double pow(double a, double b) {
   public static double powForExchange(double a, double b) {
     Preconditions.checkArgument(b > -1 && b < 1, "b must be -1<b<1");
     Preconditions.checkArgument(a > 1 && a < 2, "a must be 1<a<2");
-    return PowDataForJdk8.getData().getOrDefault(a, StrictMathWrapper.pow(a, b));
+    Double ret = PowDataForJdk8.get(a);
+    if (ret != null) {
+      logger.info("get from x87, a: {}, ret: {}", PowDataForJdk8.doubleToHex(a),
+          PowDataForJdk8.doubleToHex(ret));
+      return ret;
+    }
+    return pow(a, b);
   }
 }
diff --git a/common/src/main/java/org/tron/common/math/PowDataForJdk8.java b/common/src/main/java/org/tron/common/math/PowDataForJdk8.java
@@ -3,7 +3,6 @@
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
-import lombok.Getter;
 
 /**
  * This class is used to provide the same pow data for jdk8.
@@ -12,7 +11,6 @@
  */
 class PowDataForJdk8 {
 
-  @Getter
   private  static final Map<Double, Double> data = Collections.synchronizedMap(new HashMap<>());
 
   /**
@@ -371,6 +369,17 @@ private static void addData(String a, String x87Ret) {
     data.put(hexToDouble(a), hexToDouble(x87Ret));
   }
 
+  public static Double get(double key) {
+    return data.get(key);
+  }
+
+  static String doubleToHex(double input) {
+    // Convert the double to a long
+    long inputAsLong = Double.doubleToLongBits(input);
+    // and then convert the long to a hex string
+    return Long.toHexString(inputAsLong);
+  }
+
   static double hexToDouble(String input) {
     // Convert the hex string to a long
     long hexAsLong = Long.parseLong(input, 16);

diff --git a/common/src/test/java/org/tron/common/math/MathTest.java b/common/src/test/java/org/tron/common/math/MathTest.java
@@ -15,8 +15,7 @@ public void testPow() {
     double ret3 = StrictMathWrapper.pow(a, 0.0005);
     Assert.assertNotEquals(0, Double.compare(ret, ret2));
     Assert.assertNotEquals(0, Double.compare(ret2, ret3));
-    Assert.assertEquals(0, Double.compare(
-        PowDataForJdk8.getData().get(a), ret2));
+    Assert.assertEquals(0, Double.compare(PowDataForJdk8.get(a), ret2));
     Assert.assertEquals(0, Double.compare(ret, ret3));
   }
 }