This is the mail archive of the
glibc-cvs@sourceware.org
mailing list for the glibc project.
GNU C Library master sources branch master updated. glibc-2.28.9000-128-g221e4ba
- From: jsm28 at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 12 Sep 2018 20:31:50 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.28.9000-128-g221e4ba
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 "GNU C Library master sources".
The branch, master has been updated
via 221e4babca17b363df2c56e839572e9f7ab7d127 (commit)
from 3e08ff544b86834cd24795de159f16b8c65c7b8f (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 -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=221e4babca17b363df2c56e839572e9f7ab7d127
commit 221e4babca17b363df2c56e839572e9f7ab7d127
Author: Joseph Myers <joseph@codesourcery.com>
Date: Wed Sep 12 20:31:24 2018 +0000
Include most of elf/ modules-names in modules-names-tests.
I'm testing a patch to let the compiler expand calls to floor in libm
as built-in function calls as much as possible, instead of calling
__floor, so that no architecture-specific __floor inlines are needed,
and then to arrange for non-inlined calls to end up calling __floor,
as done with sqrt and __ieee754_sqrt.
This shows up elf/tst-relsort1mod2.c calling floor, which must not be
converted to a call to __floor. Now, while an IS_IN (libm)
conditional could be added to the existing conditionals on such
redirections in include/math.h, the _ISOMAC conditional ought to
suffice (code in other glibc libraries shouldn't be calling floor or
sqrt anyway, as they aren't provided in libc and the other libraries
don't link with libm). But while tests are mostly now built with
_ISOMAC defined, test modules in modules-names aren't unless also
listed in modules-names-tests.
As far as I can see, all the modules in modules-names in elf/ are in
fact parts of tests and so listing them in modules-names-tests is
appropriate, so they get built with something closer to the headers
used for user code, except in a few cases that actually rely on
something from internal headers. This patch duly sets
modules-names-tests there accordingly (filtering out those tests that
fail to build without internal headers).
Tested for x86_64, and with build-many-glibcs.py.
* elf/Makefile (modules-names-tests): New variable.
diff --git a/ChangeLog b/ChangeLog
index 696e456..5bbf568 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2018-09-12 Joseph Myers <joseph@codesourcery.com>
+
+ * elf/Makefile (modules-names-tests): New variable.
+
2018-09-12 Szabolcs Nagy <szabolcs.nagy@arm.com>
* NEWS: Mention log2 improvements.
diff --git a/elf/Makefile b/elf/Makefile
index cd07713..4a4ca84 100644
--- a/elf/Makefile
+++ b/elf/Makefile
@@ -274,6 +274,10 @@ modules-names = testobj1 testobj2 testobj3 testobj4 testobj5 testobj6 \
tst-nodelete-dlclose-dso tst-nodelete-dlclose-plugin \
tst-main1mod tst-libc_dlvsym-dso tst-absolute-sym-lib \
tst-absolute-zero-lib tst-big-note-lib
+# Most modules build with _ISOMAC defined, but those filtered out
+# depend on internal headers.
+modules-names-tests = $(filter-out ifuncmod% tst-libc_dlvsym-dso tst-tlsmod%,\
+ $(modules-names))
ifeq (yes,$(have-mtls-dialect-gnu2))
tests += tst-gnu2-tls1
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 4 ++++
elf/Makefile | 4 ++++
2 files changed, 8 insertions(+), 0 deletions(-)
hooks/post-receive
--
GNU C Library master sources