re PR target/19116 (-funsafe-math-optimizations make nan compares equal to one…

re PR target/19116 (-funsafe-math-optimizations make nan compares equal to one another (-finite-math-only should be doing that))

2006-10-21  Richard Guenther  <rguenther@suse.de>

	PR target/19116
	* config/i386/i386.c (override_options): Do not set MASK_IEEE_FP
	if flag_unsafe_math_optimizations is specified.  We have
	flag_finite_math_only for that.
	* config/i386/i386.md (sqrtxf2): Do not require TARGET_IEEE_FP
	or flag_unsafe_math_optimizations.

From-SVN: r117930

gcc/ChangeLog

 2006-10-21  Richard Guenther  <rguenther@suse.de>
 
+	PR target/19116
+	* config/i386/i386.c (override_options): Do not set MASK_IEEE_FP
+	if flag_unsafe_math_optimizations is specified.  We have
+	flag_finite_math_only for that.
+	* config/i386/i386.md (sqrtxf2): Do not require TARGET_IEEE_FP
+	or flag_unsafe_math_optimizations.
+
+2006-10-21  Richard Guenther  <rguenther@suse.de>
+
 	* builtins.c (fold_builtin_classify): Fix typo.
 
 2006-10-21  Richard Guenther  <rguenther@suse.de>

gcc/config/i386/i386.c

@@ -1847,7 +1847,7 @@ override_options (void)
 
   /* If we're doing fast math, we don't care about comparison order
      wrt NaNs.  This lets us use a shorter comparison sequence.  */
-  if (flag_unsafe_math_optimizations)
+  if (flag_finite_math_only)
     target_flags &= ~MASK_IEEE_FP;
 
   /* If the architecture always has an FPU, turn off NO_FANCY_MATH_387,

gcc/config/i386/i386.md

@@ -15560,8 +15560,7 @@
 (define_insn "sqrtxf2"
   [(set (match_operand:XF 0 "register_operand" "=f")
 	(sqrt:XF (match_operand:XF 1 "register_operand" "0")))]
-  "TARGET_USE_FANCY_MATH_387 
-   && (TARGET_IEEE_FP || flag_unsafe_math_optimizations) "
+  "TARGET_USE_FANCY_MATH_387"
   "fsqrt"
   [(set_attr "type" "fpspc")
    (set_attr "mode" "XF")