aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/Include/pymath.h
diff options
context:
space:
mode:
authorSebastian Berg <sebastianb@nvidia.com>2023-05-10 18:44:52 +0200
committerGitHub <noreply@github.com>2023-05-10 17:44:52 +0100
commit7a3b03509e5e3e72d8c47137579cccb52548a318 (patch)
treecbb1adddb46665552cb1e93ccae135aac0507e80 /Include/pymath.h
parenta7a2dbbf72aceef61bfb50901bfa39bfb8d6d229 (diff)
downloadcpython-7a3b03509e5e3e72d8c47137579cccb52548a318.tar.gz
cpython-7a3b03509e5e3e72d8c47137579cccb52548a318.zip
gh-104263: Rely on Py_NAN and introduce Py_INFINITY (GH-104202)
This PR removes `_Py_dg_stdnan` and `_Py_dg_infinity` in favour of using the standard `NAN` and `INFINITY` macros provided by C99. This change has the side-effect of fixing a bug on MIPS where the hard-coded value used by `_Py_dg_stdnan` gave a signalling NaN rather than a quiet NaN. --------- Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
Diffstat (limited to 'Include/pymath.h')
-rw-r--r--Include/pymath.h25
1 files changed, 11 insertions, 14 deletions
diff --git a/Include/pymath.h b/Include/pymath.h
index 772b67e4977..4c1e3d99848 100644
--- a/Include/pymath.h
+++ b/Include/pymath.h
@@ -39,27 +39,24 @@
// Return 1 if float or double arg is neither infinite nor NAN, else 0.
#define Py_IS_FINITE(X) isfinite(X)
-/* HUGE_VAL is supposed to expand to a positive double infinity. Python
- * uses Py_HUGE_VAL instead because some platforms are broken in this
- * respect. We used to embed code in pyport.h to try to worm around that,
- * but different platforms are broken in conflicting ways. If you're on
- * a platform where HUGE_VAL is defined incorrectly, fiddle your Python
- * config to #define Py_HUGE_VAL to something that works on your platform.
+// Py_INFINITY: Value that evaluates to a positive double infinity.
+#ifndef Py_INFINITY
+# define Py_INFINITY ((double)INFINITY)
+#endif
+
+/* Py_HUGE_VAL should always be the same as Py_INFINITY. But historically
+ * this was not reliable and Python did not require IEEE floats and C99
+ * conformity. Prefer Py_INFINITY for new code.
*/
#ifndef Py_HUGE_VAL
# define Py_HUGE_VAL HUGE_VAL
#endif
-// Py_NAN: Value that evaluates to a quiet Not-a-Number (NaN).
+/* Py_NAN: Value that evaluates to a quiet Not-a-Number (NaN). The sign is
+ * undefined and normally not relevant, but e.g. fixed for float("nan").
+ */
#if !defined(Py_NAN)
-# if _Py__has_builtin(__builtin_nan)
- // Built-in implementation of the ISO C99 function nan(): quiet NaN.
-# define Py_NAN (__builtin_nan(""))
-#else
- // Use C99 NAN constant: quiet Not-A-Number.
- // NAN is a float, Py_NAN is a double: cast to double.
# define Py_NAN ((double)NAN)
-# endif
#endif
#endif /* Py_PYMATH_H */