]> git.ipfire.org Git - thirdparty/glibc.git/commit - math/Makefile
Move -U__LIBC_INTERNAL_MATH_INLINES to test-math-inline.h.
authorJoseph Myers <joseph@codesourcery.com>
Thu, 9 Feb 2017 17:32:48 +0000 (17:32 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Thu, 9 Feb 2017 17:32:48 +0000 (17:32 +0000)
commit9b40563c8b43c122ecbc428349e85cffc9dde94b
tree6a86d481d6b7168ffb76bcb970c9402075125d38
parent3b2f60328eb4a9f9d776ee63931a3538eafe596a
Move -U__LIBC_INTERNAL_MATH_INLINES to test-math-inline.h.

The libm tests of inline functions undefine __NO_MATH_INLINES (from
math-CPPFLAGS) in test-math-inline.h, but __LIBC_INTERNAL_MATH_INLINES
via -U options in the makefile.

This is an odd inconsistency between the handling of the two macros.
It also depends on the ordering (in compilation commands) of the
various variables providing compiler options (which I think is why
it's using CPPFLAGS-*.c instead of CFLAGS-*.c).

This patch moves the undefine to test-math-inline.h, so improving
consistency and eliminating the dependency on the order in which
variables go in the compilation commands.  The CPPFLAGS-*.c variables
are renamed to CFLAGS-*.c, matching those for the non-inline libm
function tests.

By moving the -U option to the header file, this should ensure that
ordering issues don't arise with a subsequent move of the options to
these tests to target-specific CFLAGS += ... variable settings (for
when tests for each function are build separately and so compilation
options need setting for many more makefile targets, for which
target-specific variable settings seem to be the most convenient form
to generate with iterators).

Tested for x86_64.

* math/test-math-inline.h (__LIBC_INTERNAL_MATH_INLINES): Undefine
macro.
* math/Makefile (CPPFLAGS-test-ifloat.c): Rename to ...
(CFLAGS-test-ifloat.c): ... this.  Remove
-U__LIBC_INTERNAL_MATH_INLINES.
* math/Makefile (CPPFLAGS-test-idouble.c): Rename to ...
(CFLAGS-test-idouble.c): ... this.  Remove
-U__LIBC_INTERNAL_MATH_INLINES.
* math/Makefile (CPPFLAGS-test-ildouble.c): Rename to ...
(CFLAGS-test-ildouble.c): ... this.  Remove
-U__LIBC_INTERNAL_MATH_INLINES.
ChangeLog
math/Makefile
math/test-math-inline.h