This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [patch] Support unordered .debug_info references to .debug_ranges
- From: Dave Korn <dave dot korn dot cygwin at googlemail dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- Cc: binutils at sourceware dot org, Sebastian Huber <sebastian dot huber at embedded-brains dot de>
- Date: Thu, 16 Jul 2009 03:39:53 +0100
- Subject: Re: [patch] Support unordered .debug_info references to .debug_ranges
- References: <20090713095453.GA21747@host0.dyn.jankratochvil.net>
Jan Kratochvil wrote:
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ binutils/testsuite/binutils-all/testranges.s 13 Jul 2009 08:49:22 -0000
> + .section .debug_ranges,"",@progbits
> + .section .debug_info,"",@progbits
> + .section .debug_abbrev,"",@progbits
These forms of the .section directive are ELF-specific and cause the test to
fail on all COFF targets. (Also, the test relies on readelf.) The attached
patch moves the line that invokes the "testranges" testcase up a couple of
lines to where it's under the control of an existing "if [is_elf_format]" clause.
binutils/testsuite/ChangeLog:
* binutils-all/objcopy.exp: Only run testranges if is_elf_format.
Tested on i686-pc-cygwin; Jan, would you mind verifying it still runs (and
passes) the test on your linux target?
cheers,
DaveK
Index: binutils/testsuite/binutils-all/objcopy.exp
===================================================================
RCS file: /cvs/src/src/binutils/testsuite/binutils-all/objcopy.exp,v
retrieving revision 1.58
diff -p -u -r1.58 objcopy.exp
--- binutils/testsuite/binutils-all/objcopy.exp 14 Jul 2009 16:57:18 -0000 1.58
+++ binutils/testsuite/binutils-all/objcopy.exp 16 Jul 2009 02:22:18 -0000
@@ -872,6 +872,6 @@ if [is_elf_format] {
}
run_dump_test "localize-hidden-1"
+ run_dump_test "testranges"
}
run_dump_test "localize-hidden-2"
-run_dump_test "testranges"