[MinGW-cvs] [git push mingw-org-wsl]Repository: mingw-org-wsl branch 5.0-active updated. 7cd51d6548

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Repository: mingw-org-wsl".

The branch, 5.0-active has been updated
       via  7cd51d65489ca65c0e35725ecc349a51ba0cc95a (commit)
      from  d4d38f3736b22bb3b91b9f9010093a877a0dd0bd (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sf.net/p/mingw/mingw-org-wsl/ci/7cd51d65489ca65c0e35725ecc349a51ba0cc95a/

commit 7cd51d65489ca65c0e35725ecc349a51ba0cc95a
Author: Keith Marshall <ke...@us...>
Date:   Tue Nov 28 17:33:30 2017 +0000

    Handle indeterminate FLT_EVAL_METHOD; fix issue [#2352].

diff --git a/mingwrt/ChangeLog b/mingwrt/ChangeLog
index 1fed35f..9fb4f26 100644
--- a/mingwrt/ChangeLog
+++ b/mingwrt/ChangeLog
@@ -1,3 +1,14 @@
+2017-11-28  Keith Marshall  <ke...@us...>
+
+	Handle indeterminate FLT_EVAL_METHOD; fix issue [#2352].
+
+	* include/math.h [FLT_EVAL_METHOD == 0 || FLT_EVAL_METHOD == 1]
+	(float_t, double_t): Define them explicitly, as appropriate for each
+	of these two specific cases only; otherwise, define them as if for...
+	[FLT_EVAL_METHOD == 2]: ...this explicit case, but also apply it as a
+	suitable default compromise for the indeterminate case, in which...
+	[FLT_EVAL_METHOD != 2]: ...emit indeterminacy warning.
+
 2017-10-25  Keith Marshall  <kei...@us...>
 
 	Include, and make <sys/bsdtypes.h> test suite safe.
diff --git a/mingwrt/include/math.h b/mingwrt/include/math.h
index 2ac331c..ac30253 100644
--- a/mingwrt/include/math.h
+++ b/mingwrt/include/math.h
@@ -326,28 +326,48 @@ _CRTIMP int __cdecl _set_SSE2_enable (int);
 
 # endif /* __MINGW_GNUC_PREREQ(3, 3) */
 
-#ifdef __FLT_EVAL_METHOD__
-/* Use the compiler's builtin definition for FLT_EVAL_METHOD
- * to establish appropriate float_t and double_t typedefs.
+/* Use the compiler's internal definition for FLT_EVAL_METHOD, if one
+ * is available, to establish appropriate float_t and double_t typedefs;
+ * in the case of GCC, this is specified as __FLT_EVAL_METHOD__, which
+ * is expected to be assigned standardized values of 0, 1, or 2, (or
+ * exceptionally, a value of -1, representing indeterminacy).
  */
+#if ! defined __FLT_EVAL_METHOD__ || __valueless(__FLT_EVAL_METHOD__) \
+ || (__FLT_EVAL_METHOD__ - 0) < 0 || (__FLT_EVAL_METHOD__ - 0) > 1
+ /* __FLT_EVAL_METHOD__ has not been defined, or it is defined with no
+  * value, or with a value of -1 (or less), or a value of 2 or more; in
+  * the specific case of a value of 2, this represents an explicit choice
+  * of the IX387 FPU configuration, while in each of the other cases, we
+  * implicitly fall back to this same default configuration.
+  *
+  * NOTE: this configuration is correct for X87 FPU computations, (for
+  * which __FLT_EVAL_METHOD__ is correctly specified as 2); however...
+  */
+# if defined __FLT_EVAL_METHOD__ && (__FLT_EVAL_METHOD__ - 0) != 2
+  /* ...due to a GCC bug, introduced in GCC-6 and persisting into later
+   * versions, it may be selected via __FLT_EVAL_METHOD__ == -1, for the
+   * case of the "-msse -mfpmath=sse" option combination.  In this case,
+   * it is (at best) an unsatisfactory compromise; to avoid it, you may
+   * prefer to adopt "-mfpmath=387", or "-msse2 -mfpmath=sse" instead.
+   */
+#  warning "Default FLT_EVAL_METHOD is inderminate; assuming X87 semantics."
+# endif
+  typedef long double float_t;
+  typedef long double double_t;
+
+#else
+ /* __FLT_EVAL_METHOD__ must have been defined with an explicit value
+  * of either 0 or 1; select the corresponding SSE configuration which
+  * is applicable in each case.
+  */
 # if __FLT_EVAL_METHOD__ == 0
    typedef float float_t;
    typedef double double_t;
 
-# elif __FLT_EVAL_METHOD__ == 1
+# else /*  __FLT_EVAL_METHOD__ == 1 */
    typedef double float_t;
    typedef double double_t;
-
-# elif __FLT_EVAL_METHOD__ == 2
-   typedef long double float_t;
-   typedef long double double_t;
-
 # endif
-#else
- /* ix87 FPU default
-  */
-  typedef long double float_t;
-  typedef long double double_t;
 #endif
 
 /* 7.12.3.1

-----------------------------------------------------------------------

Summary of changes:
 mingwrt/ChangeLog      | 11 +++++++++++
 mingwrt/include/math.h | 48 ++++++++++++++++++++++++++++++++++--------------
 2 files changed, 45 insertions(+), 14 deletions(-)


hooks/post-receive
-- 
Repository: mingw-org-wsl