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-304-g9a62a93
- From: jsm28 at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 20 Nov 2018 03:55:18 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.28.9000-304-g9a62a93
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 9a62a9397d0a25643922d8d053f04ee895100d9a (commit)
from b15b1a959040fe8cdfbc51092a7b72032f6c9b86 (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=9a62a9397d0a25643922d8d053f04ee895100d9a
commit 9a62a9397d0a25643922d8d053f04ee895100d9a
Author: Joseph Myers <joseph@codesourcery.com>
Date: Tue Nov 20 03:54:53 2018 +0000
Use unique identifiers in conformtest.
This patch makes tests in conformtest use unique identifiers, in
preparation for trying to cover more tests in a single compilation to
speed up these tests as suggested in
<https://sourceware.org/ml/libc-alpha/2018-11/msg00229.html>.
Tests are assigned a number, used in generating identifiers; where a
single call to a run method does multiple compilations (sharing that
number), identifiers are changed as needed to avoid duplication
between those compilations, so they can be combined in future.
Large numbers of positional arguments to format strings make the code
harder to follow, and using the test numbers serves to increase the
number of arguments to such format strings, so the code is generally
changed to use %(name)s where all the arguments come from attributes
of the test object and so vars(self) is sufficient to provide all
those names for the format string. Cases where some arguments aren't
attributes of self still use positional format arguments.
Tested for x86_64, and with build-many-glibcs.py.
* conform/conformtest.py (ElementTest.run): Use unique identifiers
in tests. Use names for format arguments.
(ConstantTest.run): Likewise.
(SymbolTest.run): Likewise.
(TypeTest.run): Likewise.
(TagTest.run): Likewise.
(FunctionTest.run): Likewise.
(VariableTest.run): Likewise.
(MacroFunctionTest.run): Likewise.
(MacroStrTest.run): Likewise.
(HeaderTests.__init__): Set self.num_tests.
(HeaderTests.handle_test_line): Set test.num. Increment
self.num_tests.
diff --git a/ChangeLog b/ChangeLog
index bb2824c..d340866 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2018-11-20 Joseph Myers <joseph@codesourcery.com>
+
+ * conform/conformtest.py (ElementTest.run): Use unique identifiers
+ in tests. Use names for format arguments.
+ (ConstantTest.run): Likewise.
+ (SymbolTest.run): Likewise.
+ (TypeTest.run): Likewise.
+ (TagTest.run): Likewise.
+ (FunctionTest.run): Likewise.
+ (VariableTest.run): Likewise.
+ (MacroFunctionTest.run): Likewise.
+ (MacroStrTest.run): Likewise.
+ (HeaderTests.__init__): Set self.num_tests.
+ (HeaderTests.handle_test_line): Set test.num. Increment
+ self.num_tests.
+
2018-11-19 Samuel Thibault <samuel.thibault@ens-lyon.org>
* sysdeps/mach/hurd/f_setlk.c: Include <unistd.h>.
diff --git a/conform/conformtest.py b/conform/conformtest.py
index 46fdfde..792163e 100644
--- a/conform/conformtest.py
+++ b/conform/conformtest.py
@@ -41,24 +41,24 @@ class ElementTest(object):
def run(self, header_tests):
"""Run an ElementTest."""
- text = ('%s a;\n'
- '%s b;\n'
- 'extern void xyzzy (__typeof__ (&b.%s), __typeof__ (&a.%s), '
- 'unsigned);\n'
- 'void foobarbaz (void) {\n'
- 'xyzzy (&a.%s, &b.%s, sizeof (a.%s));\n'
+ text = ('%(type_name)s a_%(num)d;\n'
+ '%(type_name)s b_%(num)d;\n'
+ 'extern void xyzzy_%(num)d '
+ '(__typeof__ (&b_%(num)d.%(member_name)s), '
+ '__typeof__ (&a_%(num)d.%(member_name)s), unsigned);\n'
+ 'void foobarbaz_%(num)d (void) {\n'
+ 'xyzzy_%(num)d (&a_%(num)d.%(member_name)s, '
+ '&b_%(num)d.%(member_name)s, '
+ 'sizeof (a_%(num)d.%(member_name)s));\n'
'}\n'
- % (self.type_name, self.type_name,
- self.member_name, self.member_name,
- self.member_name, self.member_name, self.member_name))
+ % vars(self))
header_tests.compile_test('Availability of member %s'
% self.member_name,
text)
- text = ('%s a;\n'
- 'extern %s b%s;\n'
- 'extern __typeof__ (a.%s) b;\n'
- % (self.type_name, self.member_type, self.rest,
- self.member_name))
+ text = ('%(type_name)s a2_%(num)d;\n'
+ 'extern %(member_type)s b2_%(num)d%(rest)s;\n'
+ 'extern __typeof__ (a2_%(num)d.%(member_name)s) b2_%(num)d;\n'
+ % vars(self))
header_tests.compile_test('Type of member %s' % self.member_name,
text)
@@ -85,69 +85,73 @@ class ConstantTest(object):
def run(self, header_tests):
"""Run a ConstantTest."""
if 'macro' in self.symbol_type:
- text = ('#ifndef %s\n'
- '# error "Macro %s not defined"\n'
+ text = ('#ifndef %(symbol)s\n'
+ '# error "Macro %(symbol)s not defined"\n'
'#endif\n'
- % (self.symbol, self.symbol))
+ % vars(self))
header_tests.compile_test('Availability of macro %s'
% self.symbol,
text)
if 'constant' in self.symbol_type:
- text = ('__typeof__ (%s) a = %s;\n'
- % (self.symbol, self.symbol))
+ text = ('__typeof__ (%(symbol)s) a_%(num)d = %(symbol)s;\n'
+ % vars(self))
header_tests.compile_test('Availability of constant %s'
% self.symbol,
text)
if self.symbol_type == 'macro-int-constant':
sym_bits_def_neg = ''.join(
'# if %s & (1LL << %d)\n'
- '# define conformtest_bit_%d 0LL\n'
+ '# define conformtest_%d_bit_%d 0LL\n'
'# else\n'
- '# define conformtest_bit_%d (1LL << %d)\n'
+ '# define conformtest_%d_bit_%d (1LL << %d)\n'
'# endif\n'
- % (self.symbol, i, i, i, i) for i in range(63))
- sym_bits_or_neg = '|'.join('conformtest_bit_%d' % i
+ % (self.symbol, i, self.num, i, self.num, i, i)
+ for i in range(63))
+ sym_bits_or_neg = '|'.join('conformtest_%d_bit_%d' % (self.num, i)
for i in range(63))
sym_bits_def_pos = ''.join(
'# if %s & (1ULL << %d)\n'
- '# define conformtest_bit_%d (1ULL << %d)\n'
+ '# define conformtest_%d_bit_%d (1ULL << %d)\n'
'# else\n'
- '# define conformtest_bit_%d 0ULL\n'
+ '# define conformtest_%d_bit_%d 0ULL\n'
'# endif\n'
- % (self.symbol, i, i, i, i) for i in range(64))
- sym_bits_or_pos = '|'.join('conformtest_bit_%d' % i
+ % (self.symbol, i, self.num, i, i, self.num, i)
+ for i in range(64))
+ sym_bits_or_pos = '|'.join('conformtest_%d_bit_%d' % (self.num, i)
for i in range(64))
text = ('#if %s < 0\n'
- '# define conformtest_negative 1\n'
+ '# define conformtest_%d_negative 1\n'
'%s'
- '# define conformtest_value ~(%s)\n'
+ '# define conformtest_%d_value ~(%s)\n'
'#else\n'
- '# define conformtest_negative 0\n'
+ '# define conformtest_%d_negative 0\n'
'%s'
- '# define conformtest_value (%s)\n'
+ '# define conformtest_%d_value (%s)\n'
'#endif\n'
- '_Static_assert (((%s < 0) == conformtest_negative) '
- '&& (%s == conformtest_value), '
+ '_Static_assert (((%s < 0) == conformtest_%d_negative) '
+ '&& (%s == conformtest_%d_value), '
'"value match inside and outside #if");\n'
- % (self.symbol, sym_bits_def_neg, sym_bits_or_neg,
- sym_bits_def_pos, sym_bits_or_pos,
- self.symbol, self.symbol))
+ % (self.symbol, self.num, sym_bits_def_neg, self.num,
+ sym_bits_or_neg, self.num, sym_bits_def_pos, self.num,
+ sym_bits_or_pos, self.symbol, self.num, self.symbol,
+ self.num))
header_tests.compile_test('#if usability of symbol %s'
% self.symbol,
text)
if self.c_type is not None:
if self.c_type.startswith('promoted:'):
c_type = self.c_type[len('promoted:'):]
- text = ('__typeof__ ((%s) 0 + (%s) 0) a;\n'
- % (c_type, c_type))
+ text = ('__typeof__ ((%s) 0 + (%s) 0) a2_%d;\n'
+ % (c_type, c_type, self.num))
else:
- text = '__typeof__ ((%s) 0) a;\n' % self.c_type
- text += 'extern __typeof__ (%s) a;\n' % self.symbol
+ text = '__typeof__ ((%s) 0) a2_%d;\n' % (self.c_type, self.num)
+ text += 'extern __typeof__ (%s) a2_%d;\n' % (self.symbol, self.num)
header_tests.compile_test('Type of symbol %s' % self.symbol,
text)
if self.op is not None:
- text = ('_Static_assert (%s %s %s, "value constraint");\n'
- % (self.symbol, self.op, self.value))
+ text = ('_Static_assert (%(symbol)s %(op)s %(value)s, '
+ '"value constraint");\n'
+ % vars(self))
header_tests.compile_test('Value of symbol %s' % self.symbol,
text)
@@ -163,16 +167,16 @@ class SymbolTest(object):
def run(self, header_tests):
"""Run a SymbolTest."""
- text = ('void foobarbaz (void) {\n'
- '__typeof__ (%s) a = %s;\n'
+ text = ('void foobarbaz_%(num)d (void) {\n'
+ '__typeof__ (%(symbol)s) a_%(num)d = %(symbol)s;\n'
'}\n'
- % (self.symbol, self.symbol))
+ % vars(self))
header_tests.compile_test('Availability of symbol %s'
% self.symbol,
text)
if self.value is not None:
- text = ('int main (void) { return %s != %s; }\n'
- % (self.symbol, self.value))
+ text = ('int main (void) { return %(symbol)s != %(symbol)s; }\n'
+ % vars(self))
header_tests.execute_test('Value of symbol %s' % self.symbol,
text)
@@ -195,8 +199,8 @@ class TypeTest(object):
def run(self, header_tests):
"""Run a TypeTest."""
- text = ('%s %sa;\n'
- % (self.type_name, '*' if self.maybe_opaque else ''))
+ text = ('%s %sa_%d;\n'
+ % (self.type_name, '*' if self.maybe_opaque else '', self.num))
header_tests.compile_test('Availability of type %s' % self.type_name,
text)
@@ -218,9 +222,9 @@ class TagTest(object):
"""Run a TagTest."""
# If the tag is not declared, these function prototypes have
# incompatible types.
- text = ('void foo (%s *);\n'
- 'void foo (%s *);\n'
- % (self.type_name, self.type_name))
+ text = ('void foo_%(num)d (%(type_name)s *);\n'
+ 'void foo_%(num)d (%(type_name)s *);\n'
+ % vars(self))
header_tests.compile_test('Availability of tag %s' % self.type_name,
text)
@@ -243,14 +247,15 @@ class FunctionTest(object):
def run(self, header_tests):
"""Run a FunctionTest."""
- text = ('%s (*foobarbaz) %s = %s;\n'
- % (self.return_type, self.args, self.function_name))
+ text = ('%(return_type)s (*foobarbaz_%(num)d) %(args)s '
+ '= %(function_name)s;\n'
+ % vars(self))
header_tests.compile_test('Availability of function %s'
% self.function_name,
text)
- text = ('extern %s (*foobarbaz) %s;\n'
- 'extern __typeof__ (&%s) foobarbaz;\n'
- % (self.return_type, self.args, self.function_name))
+ text = ('extern %(return_type)s (*foobarbaz2_%(num)d) %(args)s;\n'
+ 'extern __typeof__ (&%(function_name)s) foobarbaz2_%(num)d;\n'
+ % vars(self))
header_tests.compile_test('Type of function %s' % self.function_name,
text)
@@ -267,14 +272,14 @@ class VariableTest(object):
def run(self, header_tests):
"""Run a VariableTest."""
- text = ('typedef %s xyzzy%s;\n'
- 'xyzzy *foobarbaz = &%s;\n'
- % (self.var_type, self.rest, self.var_name))
+ text = ('typedef %(var_type)s xyzzy_%(num)d%(rest)s;\n'
+ 'xyzzy_%(num)d *foobarbaz_%(num)d = &%(var_name)s;\n'
+ % vars(self))
header_tests.compile_test('Availability of variable %s'
% self.var_name,
text)
- text = ('extern %s %s%s;\n'
- % (self.var_type, self.var_name, self.rest))
+ text = ('extern %(var_type)s %(var_name)s%(rest)s;\n'
+ % vars(self))
header_tests.compile_test('Type of variable %s' % self.var_name,
text)
@@ -291,20 +296,19 @@ class MacroFunctionTest(object):
def run(self, header_tests):
"""Run a MacroFunctionTest."""
- text = ('#ifndef %s\n'
- '%s (*foobarbaz) %s = %s;\n'
+ text = ('#ifndef %(function_name)s\n'
+ '%(return_type)s (*foobarbaz_%(num)d) %(args)s '
+ '= %(function_name)s;\n'
'#endif\n'
- % (self.function_name, self.return_type, self.args,
- self.function_name))
+ % vars(self))
header_tests.compile_test('Availability of macro %s'
% self.function_name,
text)
- text = ('#ifndef %s\n'
- 'extern %s (*foobarbaz) %s;\n'
- 'extern __typeof__ (&%s) foobarbaz;\n'
+ text = ('#ifndef %(function_name)s\n'
+ 'extern %(return_type)s (*foobarbaz2_%(num)d) %(args)s;\n'
+ 'extern __typeof__ (&%(function_name)s) foobarbaz2_%(num)d;\n'
'#endif\n'
- % (self.function_name, self.return_type, self.args,
- self.function_name))
+ % vars(self))
header_tests.compile_test('Type of macro %s' % self.function_name,
text)
@@ -320,16 +324,17 @@ class MacroStrTest(object):
def run(self, header_tests):
"""Run a MacroStrTest."""
- text = ('#ifndef %s\n'
- '# error "Macro %s not defined"\n'
+ text = ('#ifndef %(macro_name)s\n'
+ '# error "Macro %(macro_name)s not defined"\n'
'#endif\n'
- % (self.macro_name, self.macro_name))
+ % vars(self))
header_tests.compile_test('Availability of macro %s' % self.macro_name,
text)
# We can't include <string.h> here.
text = ('extern int (strcmp)(const char *, const char *);\n'
- 'int main (void) { return (strcmp) (%s, %s) != 0; }\n'
- % (self.macro_name, self.value))
+ 'int main (void) { return (strcmp) (%(macro_name)s, '
+ '%(value)s) != 0; }\n'
+ % vars(self))
header_tests.execute_test('Value of macro %s' % self.macro_name,
text)
@@ -355,6 +360,7 @@ class HeaderTests(object):
self.allow = set()
self.allow_fnmatch = set()
self.headers_handled = set()
+ self.num_tests = 0
self.total = 0
self.skipped = 0
self.errors = 0
@@ -458,6 +464,8 @@ class HeaderTests(object):
test = test_classes[tokens[0]](*tokens)
test.xfail = xfail
test.optional = optional
+ test.num = self.num_tests
+ self.num_tests += 1
self.add_allow(test.allow_name, False)
if not allow:
self.tests.append(test)
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 16 +++++
conform/conformtest.py | 160 +++++++++++++++++++++++++-----------------------
2 files changed, 100 insertions(+), 76 deletions(-)
hooks/post-receive
--
GNU C Library master sources