Fixed math.fabs

otethal · otethal · commit d8ff4e5ec646 · 2020-07-13T18:39:05.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/MathModuleBuiltins.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/MathModuleBuiltins.java
@@ -1944,9 +1944,8 @@ public double fabs(long value) {
         }
 
         @Specialization
-        public PInt fabs(PInt value) {
-            BigInteger xabs = value.abs();
-            return factory().createInt(xabs);
+        public double fabs(PInt value) {
+            return Math.abs(value.doubleValueWithOverflow(getRaiseNode()));
         }
 
         @Specialization

Original file line number	Diff line number	Diff line change
`@@ -1944,9 +1944,8 @@ public double fabs(long value) {`
`1944`	`1944`	`}`
`1945`	`1945`
`1946`	`1946`	`@Specialization`
`1947`		`- public PInt fabs(PInt value) {`
`1948`		`- BigInteger xabs = value.abs();`
`1949`		`- return factory().createInt(xabs);`
	`1947`	`+ public double fabs(PInt value) {`
	`1948`	`+ return Math.abs(value.doubleValueWithOverflow(getRaiseNode()));`
`1950`	`1949`	`}`
`1951`	`1950`
`1952`	`1951`	`@Specialization`