This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Committed: fix "[Patch] Fix nm --size-sort on non-elf targets"
- From: Hans-Peter Nilsson <hp at bitrange dot com>
- To: Tristan Gingold <gingold at adacore dot com>
- Cc: binutils at sourceware dot org
- Date: Mon, 19 Aug 2013 21:32:18 -0400 (EDT)
- Subject: Committed: fix "[Patch] Fix nm --size-sort on non-elf targets"
- References: <F4CED898-FFDB-40F2-A970-255896BAE90D at adacore dot com> <20130810093420 dot GB3294 at bubble dot grove dot modra dot org> <84192DFA-3FDC-4485-8C7D-57D486847E57 at adacore dot com>
On Mon, 19 Aug 2013, Tristan Gingold wrote:
> On Aug 10, 2013, at 11:34 AM, Alan Modra <amodra@gmail.com> wrote:
> > On Fri, Jul 19, 2013 at 09:50:00AM +0200, Tristan Gingold wrote:
> >> binutils/
> >> 2013-07-19 Tristan Gingold <gingold@adacore.com>
> >>
> >> * nm.c (print_size_symbols): Directly get symbol size.
> >>
> >> binutils/testsuite/
> >> 2013-07-19 Tristan Gingold <gingold@adacore.com>
> >>
> >> * binutils-all/nm.exp: Add a test for nm --size-sort
> >> * binutils-all/nm-elf-1.s: New file.
> >> * binutils-all/nm-1.s: New file.
> >
> > This is OK.
>
> Thanks, now committed.
Didn't work on mmix-knuth-mmixware. I assume by "Fix nm
--size-sort on non-elf targets" you actually mean "Fix nm
--size-sort on non-elf targets where symbols have a size
attribute", else please shout. I committed the following.
Conceptually it's wrong to xfail the test for targets where it's
(by design) not supposed to work; it should instead be skipped.
Though, adding skip-machinery would just clutter up the code for
no actual benefit, at least when those targets that should be
skipped is an insignificant number of one, IMHO. To the point,
nm --size-sort should actually be made to work for those ECOFF
targets, right?
* binutils-all/nm.exp: Xfail test for nm --size-sort for
mmix-knuth-mmixware.
Index: nm.exp
===================================================================
RCS file: /cvs/src/src/binutils/testsuite/binutils-all/nm.exp,v
retrieving revision 1.7
diff -p -u -C 7 -r1.7 nm.exp
cvs diff: conflicting specifications of output style
*** nm.exp 19 Aug 2013 15:08:52 -0000 1.7
--- nm.exp 20 Aug 2013 01:16:57 -0000
*************** if [is_remote host] {
*** 175,188 ****
--- 175,192 ----
# This test does not work correctly on ECOFF targets, because ECOFF
# stores most symbols twice, which messes up the nm output.
setup_xfail "alpha*-*-osf*" "alpha*-*-netware*"
setup_xfail "mips*-*-ultrix*" "mips*-*-ecoff*" "mips*-*-irix4*"
setup_xfail "mips*-*-riscos*" "mips*-*-sysv3*" "mips*-sony-bsd*"
+ # It also doesn't work with the MMO object format, because a MMO
+ # symbol has no size information.
+ setup_xfail "mmix-knuth-mmixware"
+
set got [binutils_run $NM "$NMFLAGS --size-sort $tempfile"]
set want "0*4 T text_symbol3.*0*8 T text_symbol2.*0*c T text_symbol1"
if [regexp $want $got] then {
pass "nm --size-sort"
} else {
brgds, H-P