Skip to content

R
riscv-gcc-1
Commit 628ee3c2 by Ian Lance Taylor
Options

libbacktrace: always pass -g when compiling test code 

This approach required adding a few casts to ztest.c, as it is now
compiled with -Wall.

Fixes PR libbacktrace/90636
parent 8b117ad0
Showing with 52 additions and 17 deletions
libbacktrace/ChangeLog
2020-02-03 Ian Lance Taylor <iant@golang.org>
* Makefile.am (libbacktrace_TEST_CFLAGS): Define.
(test_elf32_CFLAGS): Use $(libbacktrace_test_CFLAGS).
(test_elf_64_CFLAGS, test_xcoff_32_CFLAGS): Likewise.
(test_xcoff_64_CFLAGS, test_pecoff_CFLAGS): Likewise.
(test_unknown_CFLAGS, unittest_CFLAGS): Likewise.
(unittest_alloc_CFLAGS, allocfail_CFLAGS): Likewise.
(b2test_CFLAGS, b3test_CFLAGS, btest_CFLAGS): Likewise.
(btest_lto_CFLAGS, btest_alloc_CFLAGS, stest_CFLAGS): Likewise.
(stest_alloc_CFLAGS): Likewise.
* Makefile.in: Regenerate.
* ztest.c (error_callback_compress): Mark vdata unused.
(test_large): Add casts to avoid warnings.
2020-01-01 Jakub Jelinek <jakub@redhat.com> 2020-01-01 Jakub Jelinek <jakub@redhat.com>
Update copyright years. Update copyright years.
......
libbacktrace/Makefile.am
...@@ -93,6 +93,9 @@ TESTS = ...@@ -93,6 +93,9 @@ TESTS =
# Add test to this variable, if you want it to be build and run. # Add test to this variable, if you want it to be build and run.
BUILDTESTS = BUILDTESTS =
# Flags to use when compiling test programs.
libbacktrace_TEST_CFLAGS = $(EXTRA_FLAGS) $(WARN_FLAGS) -g
if NATIVE if NATIVE
check_LTLIBRARIES = libbacktrace_alloc.la check_LTLIBRARIES = libbacktrace_alloc.la
...@@ -149,41 +152,49 @@ xcoff_%.c: xcoff.c ...@@ -149,41 +152,49 @@ xcoff_%.c: xcoff.c
mv $@.tmp $@ mv $@.tmp $@
test_elf_32_SOURCES = test_format.c testlib.c test_elf_32_SOURCES = test_format.c testlib.c
test_elf_32_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_elf_32_LDADD = libbacktrace_noformat.la elf_32.lo test_elf_32_LDADD = libbacktrace_noformat.la elf_32.lo
BUILDTESTS += test_elf_32 BUILDTESTS += test_elf_32
test_elf_64_SOURCES = test_format.c testlib.c test_elf_64_SOURCES = test_format.c testlib.c
test_elf_64_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_elf_64_LDADD = libbacktrace_noformat.la elf_64.lo test_elf_64_LDADD = libbacktrace_noformat.la elf_64.lo
BUILDTESTS += test_elf_64 BUILDTESTS += test_elf_64
test_xcoff_32_SOURCES = test_format.c testlib.c test_xcoff_32_SOURCES = test_format.c testlib.c
test_xcoff_32_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_xcoff_32_LDADD = libbacktrace_noformat.la xcoff_32.lo test_xcoff_32_LDADD = libbacktrace_noformat.la xcoff_32.lo
BUILDTESTS += test_xcoff_32 BUILDTESTS += test_xcoff_32
test_xcoff_64_SOURCES = test_format.c testlib.c test_xcoff_64_SOURCES = test_format.c testlib.c
test_xcoff_64_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_xcoff_64_LDADD = libbacktrace_noformat.la xcoff_64.lo test_xcoff_64_LDADD = libbacktrace_noformat.la xcoff_64.lo
BUILDTESTS += test_xcoff_64 BUILDTESTS += test_xcoff_64
test_pecoff_SOURCES = test_format.c testlib.c test_pecoff_SOURCES = test_format.c testlib.c
test_pecoff_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_pecoff_LDADD = libbacktrace_noformat.la pecoff.lo test_pecoff_LDADD = libbacktrace_noformat.la pecoff.lo
BUILDTESTS += test_pecoff BUILDTESTS += test_pecoff
test_unknown_SOURCES = test_format.c testlib.c test_unknown_SOURCES = test_format.c testlib.c
test_unknown_CFLAGS = $(libbacktrace_TEST_CFLAGS)
test_unknown_LDADD = libbacktrace_noformat.la unknown.lo test_unknown_LDADD = libbacktrace_noformat.la unknown.lo
BUILDTESTS += test_unknown BUILDTESTS += test_unknown
unittest_SOURCES = unittest.c testlib.c unittest_SOURCES = unittest.c testlib.c
unittest_CFLAGS = $(libbacktrace_TEST_CFLAGS)
unittest_LDADD = libbacktrace.la unittest_LDADD = libbacktrace.la
BUILDTESTS += unittest BUILDTESTS += unittest
unittest_alloc_SOURCES = $(unittest_SOURCES) unittest_alloc_SOURCES = $(unittest_SOURCES)
unittest_alloc_CFLAGS = $(libbacktrace_TEST_CFLAGS)
unittest_alloc_LDADD = libbacktrace_alloc.la unittest_alloc_LDADD = libbacktrace_alloc.la
BUILDTESTS += unittest_alloc BUILDTESTS += unittest_alloc
...@@ -200,6 +211,7 @@ libbacktrace_instrumented_alloc_la_DEPENDENCIES = \ ...@@ -200,6 +211,7 @@ libbacktrace_instrumented_alloc_la_DEPENDENCIES = \
instrumented_alloc.lo: alloc.c instrumented_alloc.lo: alloc.c
allocfail_SOURCES = allocfail.c testlib.c allocfail_SOURCES = allocfail.c testlib.c
allocfail_CFLAGS = $(libbacktrace_TEST_CFLAGS)
allocfail_LDADD = libbacktrace_instrumented_alloc.la allocfail_LDADD = libbacktrace_instrumented_alloc.la
check_PROGRAMS += allocfail check_PROGRAMS += allocfail
...@@ -212,7 +224,7 @@ if HAVE_ELF ...@@ -212,7 +224,7 @@ if HAVE_ELF
if HAVE_OBJCOPY_DEBUGLINK if HAVE_OBJCOPY_DEBUGLINK
b2test_SOURCES = $(btest_SOURCES) b2test_SOURCES = $(btest_SOURCES)
b2test_CFLAGS = $(btest_CFLAGS) b2test_CFLAGS = $(libbacktrace_TEST_CFLAGS)
b2test_LDFLAGS = -Wl,--build-id b2test_LDFLAGS = -Wl,--build-id
b2test_LDADD = libbacktrace_elf_for_test.la b2test_LDADD = libbacktrace_elf_for_test.la
...@@ -222,7 +234,7 @@ TESTS += b2test_buildid ...@@ -222,7 +234,7 @@ TESTS += b2test_buildid
if HAVE_DWZ if HAVE_DWZ
b3test_SOURCES = $(btest_SOURCES) b3test_SOURCES = $(btest_SOURCES)
b3test_CFLAGS = $(btest_CFLAGS) b3test_CFLAGS = $(libbacktrace_TEST_CFLAGS)
b3test_LDFLAGS = -Wl,--build-id b3test_LDFLAGS = -Wl,--build-id
b3test_LDADD = libbacktrace_elf_for_test.la b3test_LDADD = libbacktrace_elf_for_test.la
...@@ -235,7 +247,7 @@ endif HAVE_OBJCOPY_DEBUGLINK ...@@ -235,7 +247,7 @@ endif HAVE_OBJCOPY_DEBUGLINK
endif HAVE_ELF endif HAVE_ELF
btest_SOURCES = btest.c testlib.c btest_SOURCES = btest.c testlib.c
btest_CFLAGS = $(AM_CFLAGS) -g -O btest_CFLAGS = $(libbacktrace_TEST_CFLAGS) -O
btest_LDADD = libbacktrace.la btest_LDADD = libbacktrace.la
BUILDTESTS += btest BUILDTESTS += btest
...@@ -243,7 +255,7 @@ BUILDTESTS += btest ...@@ -243,7 +255,7 @@ BUILDTESTS += btest
if HAVE_ELF if HAVE_ELF
btest_lto_SOURCES = btest.c testlib.c btest_lto_SOURCES = btest.c testlib.c
btest_lto_CFLAGS = $(AM_CFLAGS) -g -O -flto btest_lto_CFLAGS = $(libbacktrace_TEST_CFLAGS) -O -flto
btest_lto_LDADD = libbacktrace.la btest_lto_LDADD = libbacktrace.la
BUILDTESTS += btest_lto BUILDTESTS += btest_lto
...@@ -251,7 +263,7 @@ BUILDTESTS += btest_lto ...@@ -251,7 +263,7 @@ BUILDTESTS += btest_lto
endif HAVE_ELF endif HAVE_ELF
btest_alloc_SOURCES = $(btest_SOURCES) btest_alloc_SOURCES = $(btest_SOURCES)
btest_alloc_CFLAGS = $(btest_CFLAGS) btest_alloc_CFLAGS = $(libbacktrace_TEST_CFLAGS)
btest_alloc_LDADD = libbacktrace_alloc.la btest_alloc_LDADD = libbacktrace_alloc.la
BUILDTESTS += btest_alloc BUILDTESTS += btest_alloc
...@@ -277,11 +289,13 @@ endif HAVE_OBJCOPY_DEBUGLINK ...@@ -277,11 +289,13 @@ endif HAVE_OBJCOPY_DEBUGLINK
endif HAVE_DWZ endif HAVE_DWZ
stest_SOURCES = stest.c stest_SOURCES = stest.c
stest_CFLAGS = $(libbacktrace_TEST_CFLAGS)
stest_LDADD = libbacktrace.la stest_LDADD = libbacktrace.la
BUILDTESTS += stest BUILDTESTS += stest
stest_alloc_SOURCES = $(stest_SOURCES) stest_alloc_SOURCES = $(stest_SOURCES)
stest_alloc_CFLAGS = $(libbacktrace_TEST_CFLAGS)
stest_alloc_LDADD = libbacktrace_alloc.la stest_alloc_LDADD = libbacktrace_alloc.la
BUILDTESTS += stest_alloc BUILDTESTS += stest_alloc
...@@ -289,7 +303,7 @@ BUILDTESTS += stest_alloc ...@@ -289,7 +303,7 @@ BUILDTESTS += stest_alloc
if HAVE_ELF if HAVE_ELF
ztest_SOURCES = ztest.c testlib.c ztest_SOURCES = ztest.c testlib.c
ztest_CFLAGS = -DSRCDIR=\"$(srcdir)\" ztest_CFLAGS = $(libbacktrace_TEST_CFLAGS) -DSRCDIR=\"$(srcdir)\"
ztest_LDADD = libbacktrace.la ztest_LDADD = libbacktrace.la
ztest_alloc_LDADD = libbacktrace_alloc.la ztest_alloc_LDADD = libbacktrace_alloc.la
...@@ -310,11 +324,13 @@ BUILDTESTS += ztest_alloc ...@@ -310,11 +324,13 @@ BUILDTESTS += ztest_alloc
endif HAVE_ELF endif HAVE_ELF
edtest_SOURCES = edtest.c edtest2_build.c testlib.c edtest_SOURCES = edtest.c edtest2_build.c testlib.c
edtest_CFLAGS = $(libbacktrace_TEST_CFLAGS)
edtest_LDADD = libbacktrace.la edtest_LDADD = libbacktrace.la
BUILDTESTS += edtest BUILDTESTS += edtest
edtest_alloc_SOURCES = $(edtest_SOURCES) edtest_alloc_SOURCES = $(edtest_SOURCES)
edtest_alloc_CFLAGS = $(libbacktrace_TEST_CFLAGS)
edtest_alloc_LDADD = libbacktrace_alloc.la edtest_alloc_LDADD = libbacktrace_alloc.la
BUILDTESTS += edtest_alloc BUILDTESTS += edtest_alloc
...@@ -330,7 +346,7 @@ if HAVE_PTHREAD ...@@ -330,7 +346,7 @@ if HAVE_PTHREAD
BUILDTESTS += ttest BUILDTESTS += ttest
ttest_SOURCES = ttest.c testlib.c ttest_SOURCES = ttest.c testlib.c
ttest_CFLAGS = $(AM_CFLAGS) -pthread ttest_CFLAGS = $(libbacktrace_TEST_CFLAGS) -pthread
ttest_LDADD = libbacktrace.la ttest_LDADD = libbacktrace.la
BUILDTESTS += ttest_alloc BUILDTESTS += ttest_alloc
...@@ -360,12 +376,12 @@ endif HAVE_OBJCOPY_DEBUGLINK ...@@ -360,12 +376,12 @@ endif HAVE_OBJCOPY_DEBUGLINK
if HAVE_COMPRESSED_DEBUG if HAVE_COMPRESSED_DEBUG
ctestg_SOURCES = btest.c testlib.c ctestg_SOURCES = btest.c testlib.c
ctestg_CFLAGS = $(AM_CFLAGS) -g ctestg_CFLAGS = $(libbacktrace_TEST_CFLAGS)
ctestg_LDFLAGS = -Wl,--compress-debug-sections=zlib-gnu ctestg_LDFLAGS = -Wl,--compress-debug-sections=zlib-gnu
ctestg_LDADD = libbacktrace.la ctestg_LDADD = libbacktrace.la
ctesta_SOURCES = btest.c testlib.c ctesta_SOURCES = btest.c testlib.c
ctesta_CFLAGS = $(AM_CFLAGS) -g ctesta_CFLAGS = $(libbacktrace_TEST_CFLAGS)
ctesta_LDFLAGS = -Wl,--compress-debug-sections=zlib-gabi ctesta_LDFLAGS = -Wl,--compress-debug-sections=zlib-gabi
ctesta_LDADD = libbacktrace.la ctesta_LDADD = libbacktrace.la
...@@ -388,7 +404,7 @@ endif ...@@ -388,7 +404,7 @@ endif
if HAVE_DWARF5 if HAVE_DWARF5
dwarf5_SOURCES = btest.c testlib.c dwarf5_SOURCES = btest.c testlib.c
dwarf5_CFLAGS = $(AM_CFLAGS) -gdwarf-5 dwarf5_CFLAGS = $(libbacktrace_TEST_CFLAGS) -gdwarf-5
dwarf5_LDADD = libbacktrace.la dwarf5_LDADD = libbacktrace.la
BUILDTESTS += dwarf5 BUILDTESTS += dwarf5
......
libbacktrace/ztest.c
...@@ -89,7 +89,8 @@ struct zlib_test ...@@ -89,7 +89,8 @@ struct zlib_test
/* Error callback. */ /* Error callback. */
static void static void
error_callback_compress (void *vdata, const char *msg, int errnum) error_callback_compress (void *vdata ATTRIBUTE_UNUSED, const char *msg,
int errnum)
{ {
fprintf (stderr, "%s", msg); fprintf (stderr, "%s", msg);
if (errnum > 0) if (errnum > 0)
...@@ -360,7 +361,7 @@ test_large (struct backtrace_state *state) ...@@ -360,7 +361,7 @@ test_large (struct backtrace_state *state)
fclose (e); fclose (e);
if (got > 0) if (got > 0)
{ {
orig_buf = rbuf; orig_buf = (unsigned char *) rbuf;
orig_bufsize = got; orig_bufsize = got;
break; break;
} }
...@@ -383,7 +384,7 @@ test_large (struct backtrace_state *state) ...@@ -383,7 +384,7 @@ test_large (struct backtrace_state *state)
} }
compress_sizearg = compressed_bufsize - 12; compress_sizearg = compressed_bufsize - 12;
r = compress (compressed_buf + 12, &compress_sizearg, r = compress ((unsigned char *) compressed_buf + 12, &compress_sizearg,
orig_buf, orig_bufsize); orig_buf, orig_bufsize);
if (r != Z_OK) if (r != Z_OK)
{ {
...@@ -406,7 +407,8 @@ test_large (struct backtrace_state *state) ...@@ -406,7 +407,8 @@ test_large (struct backtrace_state *state)
} }
uncompressed_bufsize = orig_bufsize; uncompressed_bufsize = orig_bufsize;
if (!backtrace_uncompress_zdebug (state, compressed_buf, compressed_bufsize, if (!backtrace_uncompress_zdebug (state, (unsigned char *) compressed_buf,
compressed_bufsize,
error_callback_compress, NULL, error_callback_compress, NULL,
&uncompressed_buf, &uncompressed_bufsize)) &uncompressed_buf, &uncompressed_bufsize))
{ {
...@@ -443,7 +445,8 @@ test_large (struct backtrace_state *state) ...@@ -443,7 +445,8 @@ test_large (struct backtrace_state *state)
return; return;
} }
if (!backtrace_uncompress_zdebug (state, compressed_buf, if (!backtrace_uncompress_zdebug (state,
(unsigned char *) compressed_buf,
compressed_bufsize, compressed_bufsize,
error_callback_compress, NULL, error_callback_compress, NULL,
&uncompressed_buf, &uncompressed_buf,
...@@ -472,8 +475,9 @@ test_large (struct backtrace_state *state) ...@@ -472,8 +475,9 @@ test_large (struct backtrace_state *state)
} }
uncompress_sizearg = uncompressed_bufsize; uncompress_sizearg = uncompressed_bufsize;
r = uncompress (uncompressed_buf, &uncompress_sizearg, r = uncompress ((unsigned char *) uncompressed_buf, &uncompress_sizearg,
compressed_buf + 12, compressed_bufsize - 12); (unsigned char *) compressed_buf + 12,
compressed_bufsize - 12);
if (clock_gettime (cid, &ts2) < 0) if (clock_gettime (cid, &ts2) < 0)
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment