diff options
author | Joseph Myers <joseph@codesourcery.com> | 2017-06-27 17:52:42 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2017-06-27 17:52:42 +0000 |
commit | ac782f9e9ab0a39a3054e4c97653fafa8ea47a62 (patch) | |
tree | 0afe5ff050d761b017bef5fecf09fe5975f9b6c1 /ChangeLog | |
parent | e4043b84c49e1cf9bcf1e8320233343ecc34f8eb (diff) | |
download | glibc-ac782f9e9ab0a39a3054e4c97653fafa8ea47a62.zip glibc-ac782f9e9ab0a39a3054e4c97653fafa8ea47a62.tar.gz glibc-ac782f9e9ab0a39a3054e4c97653fafa8ea47a62.tar.bz2 |
Fix elf/loadtest.c build with GCC 8.
Building the testsuite with current GCC mainline fails with:
loadtest.c: In function 'main':
loadtest.c:76:3: error: macro expands to multiple statements [-Werror=multistatement-macros]
for (map = MAPS; map != NULL; map = map->l_next) \
^
loadtest.c:165:2: note: in expansion of macro 'OUT'
OUT;
^~~
loadtest.c:164:7: note: some parts of macro expansion are not guarded by this 'if' clause
if (debug)
^~
This seems like a genuine bug, although fairly harmless; it means the
fflush call in the OUT macro is unconditional instead of being inside
the conditional as presumably intended. This patch makes this macro
use do { } while (0) to avoid the problem.
Tested for x86_64 (testsuite), and with build-many-glibcs.py for
aarch64-linux-gnu with GCC mainline.
* elf/loadtest.c (OUT): Define using do { } while (0).
Diffstat (limited to 'ChangeLog')
-rw-r--r-- | ChangeLog | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -1,5 +1,7 @@ 2017-06-27 Joseph Myers <joseph@codesourcery.com> + * elf/loadtest.c (OUT): Define using do { } while (0). + * time/strftime_l.c (DO_NUMBER): Define using do { } while (0). (DO_NUMBER_SPACEPAD): Likewise. |