From 28c110dd4fd8c0cbd78e9d591f4890d0461560e4 Mon Sep 17 00:00:00 2001
From: halibobo1205 <halibobo1205@gmail.com>
Date: Wed, 12 Jun 2024 17:51:41 +0800
Subject: [PATCH] log(math): add log for x87 ret.

---
 .../main/java/org/tron/common/math/MathWrapper.java |  8 +++++++-
 .../java/org/tron/common/math/PowDataForJdk8.java   | 13 +++++++++++--
 .../test/java/org/tron/common/math/MathTest.java    |  3 +--
 3 files changed, 19 insertions(+), 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
index 4a1acf6c0f3..f643dea05b0 100644
--- 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
index a1dcc0f71d0..022a268c708 100644
--- 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
index 0ca55a07c3c..ba49d8024bd 100644
--- 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));
   }
 }