Consolidate run_dump_test

This merges the three versions of run_dump_test.  Improved warning
handling versus the old gas version shows up a number of tests that
need their disassembly updating:
arm-linuxeabi  +FAIL: ARM v1 instructions
arm-linuxeabi  +FAIL: Accepted v8-a with ARMv8.1 AdvSIMD.
arm-linuxeabi  +FAIL: bl local instructions for v4t.
arm-linuxeabi  +FAIL: UDF
and some that now fail due to detecting assembly warnings:
h8300-elf  +FAIL: binutils-all/strip-13
h8300-elf  +FAIL: binutils-all/strip-14
h8300-elf  +FAIL: binutils-all/strip-15

I've generally kept the union of all run_dump_test features, except
that the ld target aliases "cfi" and "shared" have disappeared, as has
the binutils substitution of $srcdir as $scrdir/$subdir.

binutils/
	* testsuite/binutils-all/add-symbol.d: Add "section_subst: no".
	* testsuite/binutils-all/elfedit.exp: Don't set tempfile or copyfile.
	* testsuite/binutils-all/symbols-1.d,
	* testsuite/binutils-all/symbols-2.d,
	* testsuite/binutils-all/symbols-3.d,
	* testsuite/binutils-all/symbols-4.d: Quote '*' and '!'.
	* testsuite/binutils-all/add-empty-section.d,
	* testsuite/binutils-all/add-section.d,
	* testsuite/binutils-all/elfedit.exp,
	* testsuite/binutils-all/note-1.d,
	* testsuite/binutils-all/pr23633.d: Add $subdir after $srcdir.
	* testsuite/config/default.exp (AS, ASFLAGS): Define.
	* testsuite/config/hppa.sed: Handle all common symbols.
	* testsuite/lib/binutils-common.exp (run_dump_test): New proc,
	merged from three other versions.
	(slurp_options, file_contents, set_file_contents): Likewise.
	(big_or_little_endian, get_standard_section_names): Likewise.
	* testsuite/lib/utils-lib.exp (run_dump_test): Delete.
	(slurp_options, proc file_contents): Delete.
	(get_standard_section_names): Delete.
gas/
	* testsuite/config/default.exp: Make tmpdir.
	* testsuite/lib/gas-defs.exp (run_dump_test): Delete.
	(get_standard_section_names, slurp_options): Delete.
ld/
	* testsuite/ld-d10v/reloc-007.d,
	* testsuite/ld-d10v/reloc-008.d,
	* testsuite/ld-d10v/reloc-015.d,
	* testsuite/ld-d10v/reloc-016.d: Remove "error:".
	* testsuite/ld-elf/eh-frame-hdr.d,
	* testsuite/ld-elf/eh5.d,
	* testsuite/ld-gc/personality.d: Replace "cfi" and "shared" in
	target list with appropriate proc.
	* testsuite/ld-elf/frame.exp: Use check_shared_lib_support rather
	than "istarget shared".
	* testsuite/lib/ld-lib.exp (proc big_or_little_endian): Delete.
	(run_dump_test, slurp_options, file_contents): Delete.
	(set_file_contents, istarget): Delete.

2 files changed, 890 insertions, 561 deletions

diff --git a/binutils/testsuite/lib/binutils-common.exp b/binutils/testsuite/lib/binutils-common.exp
index bac1407..4aa5c7c 100644
--- a/binutils/testsuite/lib/binutils-common.exp
+++ b/binutils/testsuite/lib/binutils-common.exp
@@ -444,3 +444,893 @@ if ![string length [info proc prune_warnings]] {
 	return $text
     }
 }
+
+# run_dump_test FILE (optional:) EXTRA_OPTIONS
+#
+# Assemble a .s file, then run some utility on it and check the output.
+#
+# There should be an assembly language file named FILE.s in the test
+# suite directory, and a pattern file called FILE.d.  run_dump_test
+# will assemble FILE.s, optionally run objcopy on the object file,
+# optionally run ld, optionally run another objcopy, optionally run
+# another tool under test specified by PROG, then run a dump tool like
+# addr2line, nm, objdump, readelf or size on the object file to produce
+# textual output, and then analyze that with regexps.
+# The FILE.d file specifies what program to run, and what to expect in
+# its output.
+#
+# The FILE.d file begins with zero or more option lines, which specify
+# flags to pass to the assembler, the program to run to dump the
+# assembler's output, and the options it wants.  The option lines have
+# the syntax:
+#
+#         # OPTION: VALUE
+#
+# OPTION is the name of some option, like "name" or "objdump", and
+# VALUE is OPTION's value.  The valid options are described below.
+# Whitespace is ignored everywhere, except within VALUE.  The option
+# list ends with the first line that doesn't match the above syntax.
+# However, a line within the options that begins with a #, but doesn't
+# have a recognizable option name followed by a colon, is considered a
+# comment and entirely ignored.
+#
+# The optional EXTRA_OPTIONS argument to `run_dump_test' is a list of
+# two-element lists.  The first element of each is an option name, and
+# the second additional arguments to be added on to the end of the
+# option list as given in FILE.d.  (If omitted, no additional options
+# are added.)
+#
+# The interesting options are:
+#
+#   name: TEST-NAME
+#	The name of this test, passed to DejaGNU's `pass' and `fail'
+#	commands.  If omitted, this defaults to FILE, the root of the
+#	.s and .d files' names.
+#
+#   as: FLAGS
+#	When assembling, pass FLAGS to the assembler.
+#	If assembling several files, you can pass different assembler
+#	options in the "source" directives.  See below.
+#
+#   ld: FLAGS
+#	Link assembled files using FLAGS, in the order of the "source"
+#	directives, when using multiple files.
+#
+#   ld_after_inputfiles: FLAGS
+#	Similar to "ld", but put FLAGS after all input files.
+#
+#   objcopy_objects: FLAGS
+#	Run objcopy with the specified flags after assembling any source
+#	that has the special marker RUN_OBJCOPY in the source specific
+#	flags.
+#
+#   objcopy_linked_file: FLAGS
+#	Run objcopy on the linked file with the specified flags.
+#	This lets you transform the linked file using objcopy, before the
+#	result is analyzed by an analyzer program specified below.
+#
+#   PROG: PROGRAM-NAME
+#	The name of a program under test, to run to modify or analyze the
+#	.o file produced by the assembler.  Recognised names are: ar,
+#	elfedit, nm, objcopy, ranlib, strings, and strip.
+#
+#   DUMPPROG: PROGRAM-NAME
+#       The name of the program to run to analyze the file produced
+#       by the assembler or the linker.  This can be omitted;
+#       run_dump_test will guess which program to run from which of
+#       the flags options below is present.
+#
+#   addr2line: FLAGS
+#   nm: FLAGS
+#   objdump: FLAGS
+#   readelf: FLAGS
+#   size: FLAGS
+#	Use the specified program to analyze the output file, and pass it
+#	FLAGS, in addition to the output name.  Note that they are run
+#	with LC_ALL=C in the environment to give consistent sorting of
+#	symbols.  If no FLAGS are needed then you can use:
+#	    DUMPPROG: [nm objdump readelf addr2line]
+#	instead, or just pass a flag that happens to be the default.
+#	If objdump is the dump tool and we're not dumping binary, nor
+#	have run ld, then the standard section names (.text, .data and
+#	.bss) are replaced by target ones if any (eg. rx-elf uses "P"
+#	instead of .text).  The substition is done for both the
+#	objdump options (eg: "-j .text" is replaced by "-j P") and the
+#	reference file.
+#
+#   source: SOURCE [FLAGS]
+#	Assemble the file SOURCE.s using the flags in the "as" directive
+#	and the (optional) FLAGS.  If omitted, the source defaults to
+#	FILE.s.
+#	This is useful if several .d files want to share a .s file.
+#	More than one "source" directive can be given, which is useful
+#	when testing linking.
+#
+#   dump: DUMP
+#	Match against DUMP.d.  If omitted, this defaults to FILE.d.  This
+#	is useful if several .d files differ by options only.  Options are
+#	always read from FILE.d.
+#
+#   target: GLOB|PROC ...
+#	Run this test only on a specified list of targets.  More precisely,
+#	in the space-separated list each glob is passed to "istarget" and
+#	each proc is called as a TCL procedure.  List items are interpreted
+#	such that procs are denoted by surrounding square brackets, and any
+#	other items are consired globs.  If the call evaluates true for any
+#	of them, the test will be run, otherwise it will be marked
+#	unsupported.
+#
+#   notarget: GLOB|PROC ...
+#	Do not run this test on a specified list of targets.  Again, each
+#	glob in the space-separated list is passed to "istarget" and each
+#	proc is called as a TCL procedure, and the test is run if it
+#	evaluates *false* for *all* of them.  Otherwise it will be marked
+#	unsupported.
+#
+#   alltargets: GLOB|PROC ...
+#	Run this test on a specified list of targets.  Again, each
+#	glob in the space-separated list is passed to "istarget" and each
+#	proc is called as a TCL procedure, and the test is run if it
+#	evaluates *true* for *all* of them.  Otherwise it will be marked
+#	unsupported.
+#
+#   skip: GLOB|PROC ...
+#   anyskip: GLOB|PROC ...
+#   noskip: GLOB|PROC ...
+#	These are exactly the same as "notarget", "alltargets" and
+#	"target" respectively, except that they do nothing at all if the
+#	check fails.  They should only be used in groups, to construct a
+#	single test which is run on all targets but with variant options
+#	or expected output on some targets.  (For example, see
+#	gas/arm/inst.d and gas/arm/wince_inst.d.)
+#
+#   xfail: GLOB|PROC ...
+#	Run this test and it is is expected to fail on a specified list
+#	of targets.
+#
+#   error: REGEX
+#	An error with message matching REGEX must be emitted for the test
+#	to pass.  The DUMPPROG, addr2line, nm, objdump, readelf and size
+#	options have no meaning and need not supplied if this is present.
+#	Multiple "error" directives append to the expected error message.
+#
+#   error_output: FILE
+#	Means the same as 'error', except the regular expression lines
+#	are contains in FILE.
+#
+#   warning: REGEX
+#	Expect a warning matching REGEX.  It is an error to issue
+#	both "error" and "warning".  Multiple "warning" directives
+#	append to the expected warning message.
+#
+#   warning_output: FILE
+#	Means the same as 'warning', except the regular expression
+#	lines are contains in FILE.
+#
+#   map: FILE
+#	Adding this option will cause the linker to generate a linker
+#	map file, using the -Map=MAPFILE command line option.  If
+#	there is no -Map=MAPFILE in the 'ld: FLAGS' then one will be
+#	added to the linker command line.  The contents of the
+#	generated MAPFILE are then compared against the regexp lines
+#	in FILE using `regexp_diff' (see below for details).
+#
+#   section_subst: no
+#	Means that the section substitution for objdump is disabled.
+#
+# Each option may occur at most once unless otherwise mentioned.
+#
+# After the option lines come regexp lines.  run_dump_test calls
+# regexp_diff to compare the output of the dumping tool against the
+# regexps in FILE.d.
+#
+proc run_dump_test { name {extra_options {}} } {
+    global ADDR2LINE ADDR2LINEFLAGS AS ASFLAGS ELFEDIT ELFEDITFLAGS LD LDFLAGS
+    global NM NMFLAGS OBJCOPY OBJCOPYFLAGS OBJDUMP OBJDUMPFLAGS
+    global READELF READELFFLAGS STRIP STRIPFLAGS
+    global copyfile env ld_elf_shared_opt runtests srcdir subdir verbose
+
+    if [string match "*/*" $name] {
+	set file $name
+	set name [file tail $name]
+    } else {
+	set file "$srcdir/$subdir/$name"
+    }
+
+    if ![runtest_file_p $runtests $name] then {
+	return
+    }
+
+    set opt_array [slurp_options "${file}.d"]
+    if { $opt_array == -1 } {
+	perror "error reading options from $file.d"
+	unresolved $subdir/$name
+	return
+    }
+    set dumpfile tmpdir/dump.out
+    set run_ld 0
+    set run_objcopy 0
+    set objfile_names {}
+    set opts(PROG) {}
+    set opts(DUMPPROG) {}
+    set opts(addr2line) {}
+    set opts(alltargets) {}
+    set opts(anyskip) {}
+    set opts(ar) {}
+    set opts(as) {}
+    set opts(dump) {}
+    set opts(elfedit) {}
+    set opts(error) {}
+    set opts(error_output) {}
+    set opts(ld) {}
+    set opts(ld_after_inputfiles) {}
+    set opts(map) {}
+    set opts(name) {}
+    set opts(nm) {}
+    set opts(noskip) {}
+    set opts(notarget) {}
+    set opts(objcopy) {}
+    set opts(objcopy_linked_file) {}
+    set opts(objcopy_objects) {}
+    set opts(objdump) {}
+    set opts(ranlib) {}
+    set opts(readelf) {}
+    set opts(section_subst) {}
+    set opts(size) {}
+    set opts(strings) {}
+    set opts(strip) {}
+    set opts(skip) {}
+    set opts(source) {}
+    set opts(strip) {}
+    set opts(target) {}
+    set opts(warning) {}
+    set opts(warning_output) {}
+    set opts(xfail) {}
+
+    set in_extra 0
+    foreach i [concat $opt_array {{} {}} $extra_options] {
+	set opt_name [lindex $i 0]
+	set opt_val [lindex $i 1]
+	if { $opt_name == "" } {
+	    set in_extra 1
+	    continue
+	}
+	if ![info exists opts($opt_name)] {
+	    perror "unknown option $opt_name in file $file.d"
+	    unresolved $subdir/$name
+	    return
+	}
+
+	# Allow more substitutions, including tcl functions, for as and ld.
+	# Not done in general because extra quoting is needed for glob
+	# args used for example in binutils-all/remove-relocs-04.d.
+	if { $opt_name == "as" || $opt_name == "ld" } {
+	    set opt_val [subst $opt_val]
+	} else {
+	    # Just substitute $srcdir and $subdir
+	    regsub -all {\$srcdir} "$opt_val" "$srcdir" opt_val
+	    regsub -all {\$subdir} "$opt_val" "$subdir" opt_val
+	}
+
+	switch -- $opt_name {
+	    xfail {}
+	    target {}
+	    alltargets {}
+	    notarget {}
+	    skip {}
+	    anyskip {}
+	    noskip {}
+	    warning {}
+	    error {}
+	    source {
+		# Move any source-specific as-flags to a separate list to
+		# simplify processing.
+		if { [llength $opt_val] > 1 } {
+		    lappend asflags [lrange $opt_val 1 end]
+		    set opt_val [lindex $opt_val 0]
+		} else {
+		    lappend asflags {}
+		}
+
+		# Create the object file name based on nothing but the source
+		# file name.
+		set new_objfile \
+		    [concat tmpdir/[file rootname [file tail [lindex $opt_val 0]]].o]
+		# But, sometimes, we have the exact same source filename in
+		# different directories (foo/src.s bar/src.s) which would lead
+		# us to try and create two src.o files.  We detect this
+		# conflict here, and instead create src.o and src1.o.
+		set j 0
+		while { [lsearch $objfile_names $new_objfile] != -1 } {
+		    incr j
+		    set new_objfile \
+			[concat tmpdir/[file rootname [file tail  [lindex $opt_val 0]]]${j}.o]
+		}
+		lappend objfile_names $new_objfile
+	    }
+	    default {
+		if { !$in_extra && [string length $opts($opt_name)] } {
+		    perror "option $opt_name multiply set in $file.d"
+		    unresolved $subdir/$name
+		    return
+		}
+
+		# A single "#ld:" with no options should do the right thing.
+		if { $opt_name == "ld" } {
+		    set run_ld 1
+		}
+		# Likewise objcopy_linked_file.
+		if { $opt_name == "objcopy_linked_file" } {
+		    set run_objcopy 1
+		}
+	    }
+	}
+
+	# Append differently whether it's a message (without space) or
+	# an option or list (with space).
+	switch -- $opt_name {
+	    warning -
+	    error {
+		append opts($opt_name) $opt_val
+	    }
+	    default {
+		set opts($opt_name) [concat $opts($opt_name) $opt_val]
+	    }
+	}
+    }
+
+    foreach opt { as ld } {
+	regsub {\[big_or_little_endian\]} $opts($opt) \
+	    [big_or_little_endian] opts($opt)
+    }
+
+    if { $opts(name) == "" } {
+	set testname "$subdir/$name"
+    } else {
+	set testname $opts(name)
+    }
+
+    set err_warn 0
+    foreach opt { warning error warning_output error_output } {
+	if { $opts($opt) != "" } {
+	    if { $err_warn } {
+		perror "$testname: bad mix of warning and error test directives"
+		unresolved $testname
+		return
+	    }
+	    set err_warn 1
+	}
+    }
+
+    # Decide early whether we should run the test for this target.
+    if { [llength $opts(noskip)] > 0 } {
+	set targmatch 0
+	foreach targ $opts(noskip) {
+	    if [match_target $targ] {
+		set targmatch 1
+		break
+	    }
+	}
+	if { $targmatch == 0 } {
+	    return
+	}
+    }
+    foreach targ $opts(anyskip) {
+	if ![match_target $targ] {
+	    return
+	}
+    }
+    foreach targ $opts(skip) {
+	if [match_target $targ] {
+	    return
+	}
+    }
+    if { [llength $opts(target)] > 0 } {
+	set targmatch 0
+	foreach targ $opts(target) {
+	    if [match_target $targ] {
+		set targmatch 1
+		break
+	    }
+	}
+	if { $targmatch == 0 } {
+	    unsupported $testname
+	    return
+	}
+    }
+    foreach targ $opts(alltargets) {
+	if ![match_target $targ] {
+	    unsupported $testname
+	    return
+	}
+    }
+    foreach targ $opts(notarget) {
+	if [match_target $targ] {
+	    unsupported $testname
+	    return
+	}
+    }
+
+    set dumpprogram ""
+    # It's meaningless to require an output-testing method when we
+    # expect an error.
+    if { $opts(error) == "" && $opts(error_output) == "" } {
+	if { $opts(DUMPPROG) != "" } {
+	    switch -- $opts(DUMPPROG) {
+		addr2line	{ set dumpprogram addr2line }
+		nm		{ set dumpprogram nm }
+		objdump		{ set dumpprogram objdump }
+		readelf		{ set dumpprogram readelf }
+		size		{ set dumpprogram size }
+		default		{
+		    perror "unrecognized DUMPPROG option $opts(DUMPPROG) in $file.d"
+		    unresolved $testname
+		    return
+		}
+	    }
+	} else {
+	    # Guess which program to run, by seeing which option was specified.
+	    foreach p {addr2line nm objdump readelf size} {
+		if {$opts($p) != ""} {
+		    if {$dumpprogram != ""} {
+			perror "ambiguous dump program in $file.d"
+			unresolved $testname
+			return
+		    } else {
+			set dumpprogram $p
+		    }
+		}
+	    }
+	}
+	if { $dumpprogram == "" && $opts(map) == "" && !$err_warn } {
+	    perror "dump program unspecified in $file.d"
+	    unresolved $testname
+	    return
+	}
+    }
+
+    if { $opts(source) == "" } {
+	set sourcefiles [list ${file}.s]
+	set asflags [list ""]
+	set objfile_names [list tmpdir/[file tail ${file}].o]
+    } else {
+	set sourcefiles {}
+	foreach sf $opts(source) {
+	    if { [string match "/*" $sf] } {
+		lappend sourcefiles "$sf"
+	    } else {
+		lappend sourcefiles "$srcdir/$subdir/$sf"
+	    }
+	}
+    }
+
+    if { $opts(dump) == "" } {
+	set dfile ${file}.d
+    } else {
+	set dfile $srcdir/$subdir/$opts(dump)
+    }
+
+    # Time to setup xfailures.
+    foreach targ $opts(xfail) {
+	setup_xfail $targ
+    }
+
+    # Assemble each file.
+    set objfiles {}
+    for { set i 0 } { $i < [llength $sourcefiles] } { incr i } {
+	set sourcefile [lindex $sourcefiles $i]
+	set sourceasflags [lindex $asflags $i]
+	set run_objcopy_objects 0
+
+	if { [string match "*RUN_OBJCOPY*" $sourceasflags] } {
+	    set run_objcopy_objects 1
+	}
+	regsub "RUN_OBJCOPY" $sourceasflags "" sourceasflags
+
+	set objfile [lindex $objfile_names $i]
+	catch "exec rm -f $objfile" exec_output
+	lappend objfiles $objfile
+
+	if { $opts(as) == "binary" } {
+	    while {[file type $sourcefile] eq "link"} {
+		set newfile [file readlink $sourcefile]
+		if {[string index $newfile 0] ne "/"} {
+		    set newfile [file dirname $sourcefile]/$newfile
+		}
+		set sourcefile $newfile
+	    }
+	    set newfile [remote_download host $sourcefile $objfile]
+	    set cmdret 0
+	    if { $newfile == "" } {
+		set cmdret 1
+	    }
+	} else {
+	    if { [istarget "hppa*-*-*"] && ![istarget "*-*-linux*" ] } {
+		set cmd "sed -e 's/^\[	 \]*\.comm \\(\[^,\]*\\),\\(.*\\)/\\1 .comm \\2/' < $sourcefile > tmpdir/asm.s"
+		send_log "$cmd\n"
+		set cmdret [remote_exec host [concat sh -c [list "$cmd"]]]
+		set cmdret [lindex $cmdret 0]
+		if { $cmdret != 0 } {
+		    perror "sed failure"
+		    unresolved $testname
+		    return
+		}
+		set sourcefile tmpdir/asm.s
+	    }
+	    set cmd "$AS $ASFLAGS $opts(as) $sourceasflags -o $objfile $sourcefile"
+
+	    send_log "$cmd\n"
+	    set cmdret [remote_exec host [concat sh -c [list "$cmd 2>&1"]] "" "/dev/null" "dump.tmp"]
+	    remote_upload host "dump.tmp"
+	    set comp_output [prune_warnings [file_contents "dump.tmp"]]
+	    remote_file host delete "dump.tmp"
+	    remote_file build delete "dump.tmp"
+	    set cmdret [lindex $cmdret 0]
+	}
+	if { $cmdret == 0 && $run_objcopy_objects } {
+	    set cmd "$OBJCOPY $opts(objcopy_objects) $objfile"
+
+	    send_log "$cmd\n"
+	    set cmdret [remote_exec host [concat sh -c [list "$cmd 2>&1"]] \
+			"" "/dev/null" "dump.tmp"]
+	    remote_upload host "dump.tmp"
+	    append comp_output [prune_warnings [file_contents "dump.tmp"]]
+	    remote_file host delete "dump.tmp"
+	    remote_file build delete "dump.tmp"
+	    set cmdret [lindex $cmdret 0]
+	}
+    }
+
+    # Perhaps link the file(s).
+    if { $cmdret == 0 && $run_ld } {
+	set objfile "tmpdir/dump"
+	catch "exec rm -f $objfile" exec_output
+
+	set ld_extra_opt ""
+	if { [is_elf_format] && [check_shared_lib_support] } {
+	    set ld_extra_opt "$ld_elf_shared_opt"
+	}
+
+	# Add -L$srcdir/$subdir so that the linker command can use
+	# linker scripts in the source directory.
+	set cmd "$LD $ld_extra_opt $LDFLAGS -L$srcdir/$subdir \
+		   $opts(ld) -o $objfile $objfiles $opts(ld_after_inputfiles)"
+
+	# If needed then check for, or add a -Map option.
+	set mapfile ""
+	if { $opts(map) != "" } then {
+	    if { [regexp -- "-Map=(\[^ \]+)" $cmd all mapfile] } then {
+		# Found existing mapfile option
+		verbose -log "Existing mapfile '$mapfile' found"
+	    } else {
+		# No mapfile option.
+		set mapfile "tmpdir/dump.map"
+		verbose -log "Adding mapfile '$mapfile'"
+		set cmd "$cmd -Map=$mapfile"
+	    }
+	}
+
+	send_log "$cmd\n"
+	set cmdret [remote_exec host [concat sh -c [list "$cmd 2>&1"]] "" "/dev/null" "dump.tmp"]
+	remote_upload host "dump.tmp"
+	append comp_output [file_contents "dump.tmp"]
+	remote_file host delete "dump.tmp"
+	remote_file build delete "dump.tmp"
+	set cmdret [lindex $cmdret 0]
+
+	if { $cmdret == 0 && $run_objcopy } {
+	    set infile $objfile
+	    set objfile "tmpdir/dump1"
+	    remote_file host delete $objfile
+
+	    # Note that we don't use OBJCOPYFLAGS here; any flags must be
+	    # explicitly specified.
+	    set cmd "$OBJCOPY $opts(objcopy_linked_file) $infile $objfile"
+
+	    send_log "$cmd\n"
+	    set cmdret [remote_exec host [concat sh -c [list "$cmd 2>&1"]] "" "/dev/null" "dump.tmp"]
+	    remote_upload host "dump.tmp"
+	    append comp_output [file_contents "dump.tmp"]
+	    remote_file host delete "dump.tmp"
+	    remote_file build delete "dump.tmp"
+	    set cmdret [lindex $cmdret 0]
+	}
+    } else {
+	set objfile [lindex $objfiles 0]
+    }
+
+    if { $cmdret == 0 && $opts(PROG) != "" } {
+	set destopt ${copyfile}.o
+	switch -- $opts(PROG) {
+	    ar		{ set program ar }
+	    elfedit	{
+		set program elfedit
+		set destopt ""
+	    }
+	    nm		{ set program nm }
+	    objcopy	{ set program objcopy }
+	    ranlib	{ set program ranlib }
+	    strings	{ set program strings }
+	    strip	{
+		set program strip
+		set destopt "-o $destopt"
+	    }
+	    default	{
+		perror "unrecognized PROG option $opts(PROG) in $file.d"
+		unresolved $testname
+		return
+	    }
+	}
+
+	set progopts1 $opts($program)
+	eval set progopts \$[string toupper $program]FLAGS
+	eval set binary \$[string toupper $program]
+
+	if { ![is_remote host] && [which $binary] == 0 } {
+	    untested $testname
+	    return
+	}
+
+	verbose "running $binary $progopts $progopts1" 3
+	set cmd "$binary $progopts $progopts1 $objfile $destopt"
+
+	# Ensure consistent sorting of symbols
+	if {[info exists env(LC_ALL)]} {
+	    set old_lc_all $env(LC_ALL)
+	}
+	set env(LC_ALL) "C"
+	send_log "$cmd\n"
+	set cmdret [remote_exec host [concat sh -c [list "$cmd 2>dump.tmp"]] "" "/dev/null"]
+	set cmdret [lindex $cmdret 0]
+	remote_upload host "dump.tmp"
+	append comp_output [prune_warnings [file_contents "dump.tmp"]]
+	remote_file host delete "dump.tmp"
+	remote_file build delete "dump.tmp"
+	if {[info exists old_lc_all]} {
+	    set env(LC_ALL) $old_lc_all
+	} else {
+	    unset env(LC_ALL)
+	}
+	if { $destopt != "" } {
+	    set objfile ${copyfile}.o
+	}
+    }
+
+    set want_out(source) ""
+    set want_out(terminal) 0
+    if { $err_warn } {
+	if { $opts(error) != "" || $opts(error_output) != "" } {
+	    set want_out(terminal) 1
+	}
+
+	if { $opts(error) != "" || $opts(warning) != "" } {
+	    set want_out(source) "regex"
+	    if { $opts(error) != "" } {
+		set want_out(regex) $opts(error)
+	    } else {
+		set want_out(regex) $opts(warning)
+	    }
+	} else {
+	    set want_out(source) "file"
+	    if { $opts(error_output) != "" } {
+		set want_out(file) $opts(error_output)
+	    } else {
+		set want_out(file) $opts(warning_output)
+	    }
+	}
+    }
+
+    regsub "\n$" $comp_output "" comp_output
+    if { $cmdret != 0 || $comp_output != "" || $want_out(source) != "" } {
+	set exitstat "succeeded"
+	if { $cmdret != 0 } { set exitstat "failed" }
+
+	if { $want_out(source) == "regex" } {
+	    verbose -log "$exitstat with: <$comp_output>, expected: <$want_out(regex)>"
+	} elseif { $want_out(source) == "file" } {
+	    verbose -log "$exitstat with: <$comp_output>, expected in file $want_out(file)"
+	    set_file_contents "tmpdir/ld.messages" "$comp_output"
+	} else {
+	    verbose -log "$exitstat with: <$comp_output>, no expected output"
+	}
+
+	if { (($want_out(source) == "") == ($comp_output == "")) \
+	     && (($cmdret == 0) == ($want_out(terminal) == 0)) \
+	     && ((($want_out(source) == "regex") \
+		  && [regexp -- $want_out(regex) $comp_output]) \
+		 || (($want_out(source) == "file") \
+		     && (![regexp_diff "tmpdir/ld.messages" "$srcdir/$subdir/$want_out(file)"]))) } {
+	    # We have the expected output.
+	    if { $want_out(terminal) || $dumpprogram == "" } {
+		pass $testname
+		return
+	    }
+	} else {
+	    fail $testname
+	    return
+	}
+    }
+
+    # We must not have expected failure if we get here.
+    if { $opts(error) != "" } {
+	fail $testname
+	return
+    }
+
+    if { $opts(map) != "" } then {
+	# Check the map file matches.
+	set map_pattern_file $srcdir/$subdir/$opts(map)
+	verbose -log "Compare '$mapfile' against '$map_pattern_file'"
+	if { [regexp_diff $mapfile $map_pattern_file] } then {
+	    fail "$testname (map file check)"
+	} else {
+	    pass "$testname (map file check)"
+	}
+
+	if { $dumpprogram == "" } then {
+	    return
+	}
+    }
+
+    set progopts1 $opts($dumpprogram)
+    eval set progopts \$[string toupper $dumpprogram]FLAGS
+    eval set binary \$[string toupper $dumpprogram]
+
+    if { ![is_remote host] && [which $binary] == 0 } {
+	untested $testname
+	return
+    }
+
+    # For objdump of gas output, automatically translate standard section names
+    set sect_names ""
+    if { !$run_ld && $dumpprogram == "objdump" \
+	     && $opts(section_subst) != "no" \
+	     && ![string match "*-b binary*" $progopts1] } {
+	set sect_names [get_standard_section_names]
+	if { $sect_names != ""} {
+	    regsub -- "\\.text" $progopts1 "[lindex $sect_names 0]" progopts1
+	    regsub -- "\\.data" $progopts1 "[lindex $sect_names 1]" progopts1
+	    regsub -- "\\.bss"  $progopts1 "[lindex $sect_names 2]" progopts1
+	}
+    }
+
+    if { $progopts1 == "" } { set $progopts1 "-r" }
+    verbose "running $binary $progopts $progopts1" 3
+
+    set cmd "$binary $progopts $progopts1 $objfile > $dumpfile"
+
+    # Ensure consistent sorting of symbols
+    if {[info exists env(LC_ALL)]} {
+	set old_lc_all $env(LC_ALL)
+    }
+    set env(LC_ALL) "C"
+    send_log "$cmd\n"
+    set cmdret [remote_exec host [concat sh -c [list "$cmd 2>dump.tmp"]] "" "/dev/null"]
+    set cmdret [lindex $cmdret 0]
+    remote_upload host "dump.tmp"
+    set comp_output [prune_warnings [file_contents "dump.tmp"]]
+    remote_file host delete "dump.tmp"
+    remote_file build delete "dump.tmp"
+    if {[info exists old_lc_all]} {
+	set env(LC_ALL) $old_lc_all
+    } else {
+	unset env(LC_ALL)
+    }
+    if { $cmdret != 0 || $comp_output != "" } {
+	send_log "exited abnormally with $cmdret, output:$comp_output\n"
+	fail $testname
+	return
+    }
+
+    if { $verbose > 2 } then { verbose "output is [file_contents $dumpfile]" 3 }
+
+    # Create the substition list for objdump output.
+    set regexp_subst ""
+    if { $sect_names != "" } {
+	set regexp_subst [list "\\\\?\\.text" [lindex $sect_names 0] \
+			      "\\\\?\\.data" [lindex $sect_names 1] \
+			      "\\\\?\\.bss" [lindex $sect_names 2] ]
+    }
+
+    if { [regexp_diff $dumpfile "${dfile}" $regexp_subst] } then {
+	fail $testname
+	if { $verbose == 2 } then { verbose "output is [file_contents $dumpfile]" 2 }
+	return
+    }
+
+    pass $testname
+}
+
+proc slurp_options { file } {
+    # If options_regsub(foo) is set to {a b}, then the contents of a
+    # "#foo:" line will have regsub -all applied to replace a with b.
+    global options_regsub
+
+    if [catch { set f [open $file r] } x] {
+	#perror "couldn't open `$file': $x"
+	perror "$x"
+	return -1
+    }
+    set opt_array {}
+    # whitespace expression
+    set ws  {[ 	]*}
+    set nws {[^ 	]*}
+    # whitespace is ignored anywhere except within the options list;
+    # option names are alphanumeric plus underscore.
+    set pat "^#${ws}(\[a-zA-Z0-9_\]*)$ws:${ws}(.*)$ws\$"
+    while { [gets $f line] != -1 } {
+	set line [string trim $line]
+	# Whitespace here is space-tab.
+	if [regexp $pat $line xxx opt_name opt_val] {
+	    # match!
+	    if [info exists options_regsub($opt_name)] {
+		set subst $options_regsub($opt_name)
+		regsub -all -- [lindex $subst 0] $opt_val [lindex $subst 1] \
+		    opt_val
+	    }
+	    lappend opt_array [list $opt_name $opt_val]
+	} elseif {![regexp "^#" $line ]} {
+	    break
+	}
+    }
+    close $f
+    return $opt_array
+}
+
+proc file_contents { filename } {
+    set file [open $filename r]
+    set contents [read $file]
+    close $file
+    return $contents
+}
+
+proc set_file_contents { filename contents } {
+    set file [open $filename w]
+    puts $file "$contents"
+    close $file
+}
+
+# Look for big-endian or little-endian switches in the multlib
+# options and translate these into a -EB or -EL switch.  Note
+# we cannot rely upon proc process_multilib_options to do this
+# for us because for some targets the compiler does not support
+# -EB/-EL but it does support -mbig-endian/-mlittle-endian, and
+# the site.exp file will include the switch "-mbig-endian"
+# (rather than "big-endian") which is not detected by proc
+# process_multilib_options.
+#
+proc big_or_little_endian {} {
+
+    if [board_info [target_info name] exists multilib_flags] {
+	set tmp_flags " [board_info [target_info name] multilib_flags]"
+
+	foreach x $tmp_flags {
+	    case $x in {
+		{*big*endian eb EB -eb -EB -mb -meb} {
+		    set flags " -EB"
+		    return $flags
+		}
+		{*little*endian el EL -el -EL -ml -mel} {
+		    set flags " -EL"
+		    return $flags
+		}
+	    }
+	}
+    }
+
+    set flags ""
+    return $flags
+}
+
+# Internal procedure: return the names of the standard sections
+#
+proc get_standard_section_names {} {
+    if [istarget "rx-*-*"] {
+	return { "P" "D_1" "B_1" }
+    }
+    if [istarget "alpha*-*-*vms*"] {
+	# Double quote: for TCL and for sh.
+	return { "\\\$CODE\\\$" "\\\$DATA\\\$" "\\\$BSS\\\$" }
+    }
+    return
+}
diff --git a/binutils/testsuite/lib/utils-lib.exp b/binutils/testsuite/lib/utils-lib.exp
index 5f5ce03..45c1f03 100644
--- a/binutils/testsuite/lib/utils-lib.exp
+++ b/binutils/testsuite/lib/utils-lib.exp
@@ -149,568 +149,7 @@ proc exe_ext {} {
     }
 }
 
-# Copied and modified from gas.
-
-# run_dump_test FILE (optional:) EXTRA_OPTIONS
-#
-# Assemble a .s file, then run some utility on it and check the output.
-#
-# There should be an assembly language file named FILE.s in the test
-# suite directory, and a pattern file called FILE.d.  `run_dump_test'
-# will assemble FILE.s, run some tool like `objdump', `objcopy', or
-# `nm' on the .o file to produce textual output, and then analyze that
-# with regexps.  The FILE.d file specifies what program to run, and
-# what to expect in its output.
-#
-# The FILE.d file begins with zero or more option lines, which specify
-# flags to pass to the assembler, the program to run to dump the
-# assembler's output, and the options it wants.  The option lines have
-# the syntax:
-#
-#         # OPTION: VALUE
-#
-# OPTION is the name of some option, like "name" or "objdump", and
-# VALUE is OPTION's value.  The valid options are described below.
-# Whitespace is ignored everywhere, except within VALUE.  The option
-# list ends with the first line that doesn't match the above syntax.
-# However, a line within the options that begins with a #, but doesn't
-# have a recognizable option name followed by a colon, is considered a
-# comment and entirely ignored.
-#
-# The optional EXTRA_OPTIONS argument to `run_dump_test' is a list of
-# two-element lists.  The first element of each is an option name, and
-# the second additional arguments to be added on to the end of the
-# option list as given in FILE.d.  (If omitted, no additional options
-# are added.)
-#
-# The interesting options are:
-#
-#   name: TEST-NAME
-#	The name of this test, passed to DejaGNU's `pass' and `fail'
-#       commands.  If omitted, this defaults to FILE, the root of the
-#       .s and .d files' names.
-#
-#   as: FLAGS
-#	When assembling FILE.s, pass FLAGS to the assembler.
-#
-#   PROG: PROGRAM-NAME
-#       The name of the program to run to modify or analyze the .o file
-#       produced by the assembler.  This option is required.  Recognised
-#       names are: ar, elfedit, nm, objcopy, ranlib, strings, and strip.
-#
-#   DUMPPROG: PROGRAM-NAME
-#       The name of the program to run to analyze the .o file after it has
-#       has been modified by PROG.  This can be omitted; run_dump_test will
-#       guess which program to run by seeing if any of the flags options
-#       for the recognised dump programs are set.  Recognised names are:
-#       addr2line, nm, objdump, readelf and size.
-#
-#   nm: FLAGS
-#   objcopy: FLAGS
-#   objdump: FLAGS
-#   readelf: FLAGS
-#   size: FLAGS
-#	Use the specified program to analyze the .o file, and pass it
-#	FLAGS, in addition to the .o file name.  Note that they are run
-#	with LC_ALL=C in the environment to give consistent sorting
-#	of symbols.
-#
-#   source: SOURCE
-#	Assemble the file SOURCE.s.  If omitted, this defaults to FILE.s.
-#       This is useful if several .d files want to share a .s file.
-#
-#   dump: DUMP
-#	Match against DUMP.d.  If omitted, this defaults to FILE.d.  This
-#	is useful if several .d files differ by options only.  Options are
-#	always read from FILE.d.
-#
-#   target: GLOB|PROC ...
-#	Run this test only on a specified list of targets.  More precisely,
-#	in the space-separated list each glob is passed to "istarget" and
-#	each proc is called as a TCL procedure.  List items are interpreted
-#	such that procs are denoted by surrounding square brackets, and any
-#	other items are considered globs.  If the call evaluates true for
-#	any of them, the test will be run, otherwise it will be marked
-#	unsupported.
-#
-#   notarget: GLOB|PROC ...
-#	Do not run this test on a specified list of targets.  Again, each
-#	glob in the space-separated list is passed to "istarget" and each
-#	proc is called as a TCL procedure, and the test is run if it
-#	evaluates *false* for *all* of them.  Otherwise it will be marked
-#	unsupported.
-#
-#   skip: GLOB|PROC ...
-#   noskip: GLOB|PROC ...
-#       These are exactly the same as "notarget" and "target",
-#       respectively, except that they do nothing at all if the check
-#       fails.  They should only be used in groups, to construct a single
-#       test which is run on all targets but with variant options or
-#       expected output on some targets.  (For example, see
-#       gas/arm/inst.d and gas/arm/wince_inst.d.)
-#
-#   error: REGEX
-#	An error with message matching REGEX must be emitted for the test
-#	to pass.  The DUMPPROG, addr2line, nm, objdump, readelf and size
-#	options have no meaning and need not supplied if this is present.
-#	Multiple "error" directives append to the expected error message.
-#
-#   error_output: FILE
-#	Means the same as 'error', except the regular expression lines
-#	are contains in FILE.
-#
-#   warning: REGEX
-#	Expect a warning matching REGEX.  It is an error to issue both
-#	"error" and "warning".  Multiple "warning" directives append to
-#	the expected linker warning message.
-#
-#   warning_output: FILE
-#	Means the same as 'warning', except the regular expression
-#	lines are contains in FILE.
-#
-# Each option may occur at most once.
-#
-# After the option lines come regexp lines.  `run_dump_test' calls
-# `regexp_diff' to compare the output of the dumping tool against the
-# regexps in FILE.d.  `regexp_diff' is defined in binutils-common.exp;
-# see further comments there.
-
-proc run_dump_test { name {extra_options {}} } {
-    global subdir srcdir
-    global OBJDUMP NM OBJCOPY READELF STRIP
-    global OBJDUMPFLAGS NMFLAGS OBJCOPYFLAGS READELFFLAGS STRIPFLAGS
-    global ELFEDIT ELFEDITFLAGS
-    global binutils_run_status
-    global host_triplet
-    global env
-    global copyfile
-    global tempfile
-
-    if [string match "*/*" $name] {
-	set file $name
-	set name [file tail $name]
-    } else {
-	set file "$srcdir/$subdir/$name"
-    }
-    set opt_array [slurp_options "${file}.d"]
-    if { $opt_array == -1 } {
-	perror "error reading options from $file.d"
-	unresolved $subdir/$name
-	return
-    }
-    set opts(addr2line) {}
-    set opts(ar) {}
-    set opts(as) {}
-    set opts(elfedit) {}
-    set opts(name) {}
-    set opts(nm) {}
-    set opts(objcopy) {}
-    set opts(objdump) {}
-    set opts(ranlib) {}
-    set opts(readelf) {}
-    set opts(size) {}
-    set opts(strings) {}
-    set opts(strip) {}
-    set opts(PROG) {}
-    set opts(DUMPPROG) {}
-    set opts(source) {}
-    set opts(dump) {}
-    set opts(error) {}
-    set opts(warning) {}
-    set opts(error_output) {}
-    set opts(warning_output) {}
-    set opts(target) {}
-    set opts(notarget) {}
-    set opts(skip) {}
-    set opts(noskip) {}
-
-    foreach i $opt_array {
-	set opt_name [lindex $i 0]
-	set opt_val [lindex $i 1]
-	if ![info exists opts($opt_name)] {
-	    perror "unknown option $opt_name in file $file.d"
-	    unresolved $subdir/$name
-	    return
-	}
-
-	# Permit the option to use $srcdir to refer to the source
-	# directory.
-	regsub -all "\\\$srcdir" "$opt_val" "$srcdir/$subdir" opt_val
-
-	switch -- $opt_name {
-	    warning {}
-	    error {}
-	    default {
-		if [string length $opts($opt_name)] {
-		    perror "option $opt_name multiply set in $file.d"
-		    unresolved $subdir/$name
-		    return
-		}
-	    }
-	}
-	append opts($opt_name) $opt_val
-    }
-
-    foreach i $extra_options {
-	set opt_name [lindex $i 0]
-	set opt_val [lindex $i 1]
-	if ![info exists opts($opt_name)] {
-	    perror "unknown option $opt_name given in extra_opts"
-	    unresolved $subdir/$name
-	    return
-	}
-
-	# Permit the option to use $srcdir to refer to the source
-	# directory.
-	regsub -all "\\\$srcdir" "$opt_val" "$srcdir/$subdir" opt_val
-
-	# add extra option to end of existing option, adding space
-	# if necessary.
-	if { ![regexp "warning|error" $opt_name]
-	     && [string length $opts($opt_name)] } {
-	    append opts($opt_name) " "
-	}
-	append opts($opt_name) $opt_val
-    }
-
-    if { $opts(name) == "" } {
-	set testname "$subdir/$name"
-    } else {
-	set testname $opts(name)
-    }
-    verbose "Testing $testname"
-
-    if {$opts(PROG) == ""} {
-	perror "PROG isn't set in $file.d"
-	unresolved $testname
-	return
-    }
-
-    set destopt ""
-    switch -- $opts(PROG) {
-	ar	{ set program ar }
-	elfedit { set program elfedit }
-	nm      { set program nm }
-	objcopy	{ set program objcopy }
-	ranlib	{ set program ranlib }
-	strings	{ set program strings }
-	strip	{
-	    set program strip
-	    set destopt "-o"
-	}
-	default	{
-	    perror "unrecognized program option $opts(PROG) in $file.d"
-	    unresolved $testname
-	    return }
-    }
-
-    set dumpprogram ""
-    # It's meaningless to require an output-testing method when we
-    # expect an error.
-    if { $opts(error) == "" && $opts(error_output) == "" } {
-	if { $opts(DUMPPROG) != "" } {
-	    switch -- $opts(DUMPPROG) {
-		addr2line	{ set dumpprogram addr2line }
-		nm		{ set dumpprogram nm }
-		objdump		{ set dumpprogram objdump }
-		readelf		{ set dumpprogram readelf }
-		size		{ set dumpprogram size }
-		default		{
-		    perror "unrecognized dump program option $opts(DUMPPROG)\
-			    in $file.d"
-		    unresolved $testname
-		    return
-		}
-	    }
-	} else {
-	    # Guess which program to run, by seeing which option was specified.
-	    foreach p {addr2line nm objdump readelf size} {
-		if {$opts($p) != ""} {
-		    if {$dumpprogram != ""} {
-			perror "more than one possible dump program specified\
-				in $file.d"
-			unresolved $testname
-			return
-		    } else {
-			set dumpprogram $p
-		    }
-		}
-	    }
-	}
-    }
-
-    # Handle skipping the test on specified targets.
-    # You can have both skip/noskip and target/notarget, but you can't
-    # have both skip and noskip, or target and notarget, in the same file.
-    if { $opts(skip) != "" } then {
-	if { $opts(noskip) != "" } then {
-	    perror "$testname: mixing skip and noskip directives is invalid"
-	    unresolved $testname
-	    return
-	}
-	foreach glob $opts(skip) {
-	    if {[match_target $glob]} { return }
-	}
-    }
-    if { $opts(noskip) != "" } then {
-	set skip 1
-	foreach glob $opts(noskip) {
-	    if {[match_target $glob]} {
-		set skip 0
-		break
-	    }
-	}
-	if {$skip} { return }
-    }
-    if { $opts(target) != "" } then {
-	set skip 1
-	foreach glob $opts(target) {
-	    if {[match_target $glob]} {
-		set skip 0
-		break
-	    }
-	}
-	if {$skip} {
-	    unsupported $testname
-	    return
-	}
-    }
-    if { $opts(notarget) != "" } then {
-	foreach glob $opts(notarget) {
-	    if {[match_target $glob]} {
-		unsupported $testname
-		return
-	    }
-	}
-    }
-
-    if { $opts(source) == "" } {
-	set srcfile ${file}.s
-    } else {
-	set srcfile $srcdir/$subdir/$opts(source)
-    }
-
-    if { $opts(dump) == "" } {
-	set dumpfile ${file}.d
-    } else {
-	set dumpfile $srcdir/$subdir/$opts(dump)
-    }
-
-    if { $opts(as) == "binary" } {
-	while {[file type $srcfile] eq "link"} {
-	    set newfile [file readlink $srcfile]
-	    if {[string index $newfile 0] ne "/"} {
-		set newfile [file dirname $srcfile]/$newfile
-	    }
-	    set srcfile $newfile
-	}
-	# Make sure we copy the file if we are doing remote host testing.
-	remote_download host ${srcfile} $tempfile
-    } else {
-	set exec_output [binutils_assemble_flags ${srcfile} $tempfile $opts(as)]
-	if [string match "" $exec_output] then {
-	    send_log "$exec_output\n"
-	    verbose "$exec_output"
-	    fail $testname
-	   return
-	}
-    }
-
-    if { (($opts(warning) != "") && ($opts(error) != "")) \
-	 || (($opts(warning) != "") && ($opts(error_output) != "")) \
-	 || (($opts(warning) != "") && ($opts(warning_output) != "")) \
-	 || (($opts(error) != "") && ($opts(warning_output) != "")) \
-	 || (($opts(error) != "") && ($opts(error_output) != "")) \
-	 || (($opts(warning_output) != "") && ($opts(error_output) != "")) } {
-	perror "bad mix of warning, error, warning_output, and error_output\
-		test-directives"
-	unresolved $testname
-	return
-    }
-
-    set check_prog(source) ""
-    set check_prog(terminal) 0
-    if { $opts(error) != "" \
-	 || $opts(warning) != "" \
-	 || $opts(error_output) != "" \
-	 || $opts(warning_output) != "" } {
-
-	if { $opts(error) != "" || $opts(error_output) != "" } {
-	    set check_prog(terminal) 1
-	} else {
-	    set check_prog(terminal) 0
-	}
-
-	if { $opts(error) != "" || $opts(warning) != "" } {
-	    set check_prog(source) "regex"
-	    if { $opts(error) != "" } {
-		set check_prog(regex) $opts(error)
-	    } else {
-		set check_prog(regex) $opts(warning)
-	    }
-	} else {
-	    set check_prog(source) "file"
-	    if { $opts(error_output) != "" } {
-		set check_prog(file) $opts(error_output)
-	    } else {
-		set check_prog(file) $opts(warning_output)
-	    }
-	}
-    }
-
-    set progopts1 $opts($program)
-    eval set progopts \$[string toupper $program]FLAGS
-    eval set binary \$[string toupper $program]
-
-    set exec_output [binutils_run $binary "$progopts $progopts1 $tempfile $destopt ${copyfile}.o"]
-    set cmdret 0
-    if [info exists binutils_run_status] {
-	set cmdret $binutils_run_status
-    }
-
-    regsub "\n$" $exec_output "" exec_output
-    if { $cmdret != 0 || $exec_output != "" || $check_prog(source) != "" } {
-	set exitstat "succeeded"
-	if { $cmdret != 0 } {
-	    set exitstat "failed"
-	}
-
-	if { $check_prog(source) == "regex" } {
-	    verbose -log "$exitstat with: <$exec_output>,\
-			  expected: <$check_prog(regex)>"
-	} elseif { $check_prog(source) == "file" } {
-	    verbose -log "$exitstat with: <$exec_output>,\
-			  expected in file $check_prog(file)"
-	    set_file_contents "tmpdir/prog.messages" "$exec_output"
-	} else {
-	    verbose -log "$exitstat with: <$exec_output>, no expected output"
-	}
-	send_log -- "$exec_output\n"
-	verbose "$exec_output"
-
-	if { (($check_prog(source) == "") == ($exec_output == "")) \
-	     && (($cmdret == 0) == ($check_prog(terminal) == 0)) \
-	     && ((($check_prog(source) == "regex") \
-		  && ($check_prog(regex) == "") == ($exec_output == "") \
-		  && [regexp -- $check_prog(regex) $exec_output]) \
-		 || (($check_prog(source) == "file") \
-		     && (![regexp_diff "tmpdir/prog.messages" \
-			   "$srcdir/$subdir/$check_prog(file)"]))) } {
-	    # We have the expected output from prog.
-	    if { $check_prog(terminal) || $program == "" } {
-		pass $testname
-		return
-	    }
-	} else {
-	    fail $testname
-	    return
-	}
-    }
-
-    set progopts1 $opts($dumpprogram)
-    eval set progopts \$[string toupper $dumpprogram]FLAGS
-    eval set binary \$[string toupper $dumpprogram]
-
-    if { ![is_remote host] && [which $binary] == 0 } {
-	untested $testname
-	return
-    }
-
-    # For objdump, automatically translate standard section names to the targets one,
-    # if they are different.
-    set sect_names [get_standard_section_names]
-    if { $sect_names != "" && $dumpprogram == "objdump"} {
-        regsub -- "-j \\.text" $progopts1 "-j [lindex $sect_names 0]" progopts1
-        regsub -- "-j \\.data" $progopts1 "-j [lindex $sect_names 1]" progopts1
-        regsub -- "-j \\.bss"  $progopts1 "-j [lindex $sect_names 2]" progopts1
-    }
-
-    verbose "running $binary $progopts $progopts1" 3
-
-    set cmd "$binary $progopts $progopts1 ${copyfile}.o"
-
-    # Ensure consistent sorting of symbols
-    if {[info exists env(LC_ALL)]} {
-	set old_lc_all $env(LC_ALL)
-    }
-    set env(LC_ALL) "C"
-    send_log "$cmd\n"
-    set comp_output [remote_exec host $cmd "" "/dev/null" "tmpdir/dump.out"]
-    if {[info exists old_lc_all]} {
-	set env(LC_ALL) $old_lc_all
-    } else {
-	unset env(LC_ALL)
-    }
-    if { [lindex $comp_output 0] != 0 } then {
-	send_log "$comp_output\n"
-	fail $testname
-	return
-    }
-    set comp_output [prune_warnings [lindex $comp_output 1]]
-    if ![string match "" $comp_output] then {
-	send_log "$comp_output\n"
-	fail $testname
-	return
-    }
-
-    verbose_eval {[file_contents "tmpdir/dump.out"]} 3
-    if { [regexp_diff "tmpdir/dump.out" "${dumpfile}"] } then {
-	fail $testname
-	verbose "output is [file_contents "tmpdir/dump.out"]" 2
-	return
-    }
-
-    pass $testname
-}
-
-proc slurp_options { file } {
-    if [catch { set f [open $file r] } x] {
-	#perror "couldn't open `$file': $x"
-	perror "$x"
-	return -1
-    }
-    set opt_array {}
-    # whitespace expression
-    set ws  {[ 	]*}
-    set nws {[^ 	]*}
-    # whitespace is ignored anywhere except within the options list;
-    # option names are alphabetic plus dash
-    set pat "^#${ws}(\[a-zA-Z-\]*)$ws:${ws}(.*)$ws\$"
-    while { [gets $f line] != -1 } {
-	set line [string trim $line]
-	# Whitespace here is space-tab.
-	if [regexp $pat $line xxx opt_name opt_val] {
-	    # match!
-	    lappend opt_array [list $opt_name $opt_val]
-	} elseif {![regexp "^#" $line ]} {
-	    break
-	}
-    }
-    close $f
-    return $opt_array
-}
-
-proc file_contents { filename } {
-    set file [open $filename r]
-    set contents [read $file]
-    close $file
-    return $contents
-}
-
 proc verbose_eval { expr { level 1 } } {
     global verbose
     if $verbose>$level then { eval verbose "$expr" $level }
 }
-
-# Internal procedure: return the names of the standard sections
-#
-proc get_standard_section_names {} {
-    if [istarget "rx-*-*"] {
-        return { "P" "D_1" "B_1" }
-    }
-    if [istarget "alpha*-*-*vms*"] {
-        # Double quote: for TCL and for sh.
-        return { "\\\$CODE\\\$" "\\\$DATA\\\$" "\\\$BSS\\\$" }
-    }
-    return
-}