Skip to content

Commit

Permalink
Revert "Build glibc with -frounding-math"
Browse files Browse the repository at this point in the history
This reverts commit bd7b0e5.

Conflicts:

	ChangeLog
  • Loading branch information
ajaeger committed May 7, 2012
1 parent 495fd99 commit 20f244a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
8 changes: 0 additions & 8 deletions ChangeLog
Original file line number Diff line number Diff line change
Expand Up @@ -64,14 +64,6 @@
* math/libm-test.inc (check_float_internal): Correct ulp
calculation for subnormal expected results.

2012-05-06 Andreas Jaeger <[email protected]>

* Makeconfig (+math-flags): New, set to -frounding-math.
(+cflags): Add +math-flags so that all of glibc gets compiled with
it.

* sysdeps/x86_64/fpu/libm-test-ulps: Updated.

2012-05-05 Joseph Myers <[email protected]>

* math/libm-test.inc (hypot_test) [TEST_DOUBLE && TEST_INLINE]:
Expand Down
7 changes: 1 addition & 6 deletions Makeconfig
Original file line number Diff line number Diff line change
Expand Up @@ -574,11 +574,6 @@ endif
# actually different, so allow the compiler to merge them all.
+merge-constants = -fmerge-all-constants

# We have to assume that glibc functions are called in any rounding
# mode and also change the rounding mode in a few functions. So,
# disable any optimization that assume default rounding mode.
+math-flags = -frounding-math

# This is the program that generates makefile dependencies from C source files.
# The -MP flag tells GCC >= 3.2 (which we now require) to produce dummy
# targets for headers so that removed headers don't break the build.
Expand Down Expand Up @@ -638,7 +633,7 @@ ifeq "$(strip $(+cflags))" ""
+cflags := $(default_cflags)
endif # $(+cflags) == ""

+cflags += $(cflags-cpu) $(+gccwarn) $(+merge-constants) $(+math-flags)
+cflags += $(cflags-cpu) $(+gccwarn) $(+merge-constants)
+gcc-nowarn := -w

# Don't duplicate options if we inherited variables from the parent.
Expand Down

0 comments on commit 20f244a

Please sign in to comment.