From bb3b5316332875b436b4a88d1ec6968eba46a870 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Mon, 28 May 2018 16:30:23 +0930 Subject: [PATCH] PR23235, make check uses system installed ld This patch doesn't stop the build-id and debuglink tests using the installed ld, it just prevents a compiler failure from resulting in a test fail. We could move the tests to the ld testsuite but it doesn't seem all that important. PR 23235 * testsuite/binutils-all/compress.exp (test_gnu_debuglink): Return unsupported rather than fail on compile errors. * testsuite/binutils-all/objdump.exp (test_build_id_debuglink): Likewise. Save and restore CFLAGS_FOR_TARGET. --- binutils/ChangeLog | 8 ++++++++ binutils/testsuite/binutils-all/compress.exp | 2 +- binutils/testsuite/binutils-all/objdump.exp | 11 +++++++++-- 3 files changed, 18 insertions(+), 3 deletions(-) diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 2943b1d31b4..185dffdc3c5 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,11 @@ +2018-05-28 Alan Modra + + PR 23235 + * testsuite/binutils-all/compress.exp (test_gnu_debuglink): Return + unsupported rather than fail on compile errors. + * testsuite/binutils-all/objdump.exp (test_build_id_debuglink): + Likewise. Save and restore CFLAGS_FOR_TARGET. + 2018-05-18 Kito Cheng * readelf.c (get_machine_flags): Handle EF_RISCV_RVE. diff --git a/binutils/testsuite/binutils-all/compress.exp b/binutils/testsuite/binutils-all/compress.exp index 12a43b025c5..edbe9dd5ca6 100644 --- a/binutils/testsuite/binutils-all/compress.exp +++ b/binutils/testsuite/binutils-all/compress.exp @@ -692,7 +692,7 @@ proc test_gnu_debuglink {} { } if { [target_compile $srcdir/$subdir/testprog.c tmpdir/testprog exectuable debug] != "" } { - fail "$test (build)" + unsupported "$test (build)" return } set got [remote_exec host "$OBJDUMP -S tmpdir/testprog" "" "/dev/null" "tmpdir/testprog.dump"] diff --git a/binutils/testsuite/binutils-all/objdump.exp b/binutils/testsuite/binutils-all/objdump.exp index b9232e92294..d9d07138aa0 100644 --- a/binutils/testsuite/binutils-all/objdump.exp +++ b/binutils/testsuite/binutils-all/objdump.exp @@ -363,14 +363,21 @@ proc test_build_id_debuglink {} { } # Use a fixed build-id. + if { [info exists CFLAGS_FOR_TARGET] } { + set save_CFLAGS_FOR_TARGET $CFLAGS_FOR_TARGET + } set CFLAGS_FOR_TARGET "-g -Wl,--build-id=0x12345678abcdef01" if { [target_compile $srcdir/$subdir/testprog.c tmpdir/testprog exectuable debug] != "" } { - fail "$test (build)" + unsupported "$test (build)" return } - # FIXME: Do we need to restore CFLAGS_FOR_TARGET to its old value ? + if { [info exists save_CFLAGS_FOR_TARGET] } { + set CFLAGS_FOR_TARGET $save_CFLAGS_FOR_TARGET + } else { + unset CFLAGS_FOR_TARGET + } if { [binutils_run $STRIP "--strip-debug --remove-section=.comment tmpdir/testprog -o tmpdir/testprog.strip"] != "" } { fail "$test (strip debug info)" -- 2.30.2