Bug 3807 - binutils-all/objcopy.exp test-suite failure for mipsisa32r2el-elf
Summary: binutils-all/objcopy.exp test-suite failure for mipsisa32r2el-elf
Status: RESOLVED FIXED
Alias: None
Product: binutils
Classification: Unclassified
Component: binutils (show other bugs)
Version: 2.18
: P2 normal
Target Milestone: ---
Assignee: unassigned
URL: http://sourceware.org/ml/binutils/201...
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-27 07:05 UTC by Hans-Peter Nilsson
Modified: 2022-07-22 06:14 UTC (History)
2 users (show)

See Also:
Host: i686-pc-linux-gnu
Target: mipsisa32r2el-elf
Build:
Last reconfirmed:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Hans-Peter Nilsson 2006-12-27 07:05:11 UTC 
With HEAD binutils checked out as of Wed, 27 Dec 2006 06:05:09 UTC, I get this
failure for "make check-binutils" for --target=mips-elf:

Running
/mnt/test1/hp/axis/binutils/src/binutils/testsuite/binutils-all/objcopy.exp ...
Version /mnt/test1/hp/axis/binutils/mips/binutils/objcopy 20061227
FAIL: --localize-hidden test 1

with the .log file containing:
/mnt/test1/hp/axis/binutils/mips/binutils/objdump  --syms tmpdir/copy.o >
tmpdir/dump.out
extra regexps in
/mnt/test1/hp/axis/binutils/src/binutils/testsuite/binutils-all/localize-hidden-1.d
starting with "^0+2200 l .*\\
*ABS\* 0+ \.hidden Ghidden$"
EOF from tmpdir/dump.out
FAIL: --localize-hidden test 1
Comment 1 John David Anglin 2009-03-01 19:23:17 UTC 
With the test change proposed here
<http://sourceware.org/ml/binutils/2009-03/msg00006.html>,
I see a similar fail on hppa64-hp-hpux11.11 and hppa-unknown-linux-gnu.

tmpdir/copy.o:     file format elf32-hppa-linux

SYMBOL TABLE:
00000000 l    d  .text	00000000 .text
00000000 l    d  .data	00000000 .data
00000000 l    d  .bss	00000000 .bss
00002200 l       *ABS*	00000000 .hidden Ghidden
00002300 l       *ABS*	00000000 .internal Ginternal
00002100 g       *ABS*	00000000 Gdefault
00002400 g       *ABS*	00000000 .protected Gprotected

extra regexps in /home/dave/gnu/binutils/src/binutils/testsuite/binutils-all/loc
alize-hidden-1.d starting with "^0+1200 l .*\*ABS\*     0+ \.hidden Lhidden$"
EOF from tmpdir/dump.outFAIL: --localize-hidden test 1
Comment 2 Alan Modra 2009-03-03 00:05:44 UTC 
That'll be tc-hppa.h:tc_frob_symbol throwing away local symbols.
Comment 3 Hans-Peter Nilsson 2012-03-01 04:07:43 UTC 
Apaprently fixed for MIPS as of at least CVS HEAD of 2012-03-01 (post-2.22), unknown whether fixed in any release.
Comment 4 Hans-Peter Nilsson 2012-03-01 04:09:36 UTC 
(In reply to comment #3)
> Apaprently fixed for MIPS as of at least CVS HEAD of 2012-03-01 (post-2.22),
> unknown whether fixed in any release.

Oops, sorry, it's just xfailed.  Reopening.
Comment 5 Sourceware Commits 2012-03-13 00:41:25 UTC 
CVSROOT:	/cvs/src
Module name:	src
Changes by:	hp@sourceware.org	2012-03-13 00:41:22

Modified files:
	binutils/testsuite: ChangeLog 
	binutils/testsuite/binutils-all: objcopy.exp 

Log message:
	PR binutils/3807
	* binutils-all/objcopy.exp (localize-hidden-1): Correct xfailed
	mips-targets.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/testsuite/ChangeLog.diff?cvsroot=src&r1=1.254&r2=1.255
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/testsuite/binutils-all/objcopy.exp.diff?cvsroot=src&r1=1.76&r2=1.77
Comment 6 Hans-Peter Nilsson 2012-03-13 00:45:09 UTC 
And now with more correct xfails.  Not closing it, though; IMHO the underlying issue is fixable, see the URL.
Comment 7 Alan Modra 2022-07-22 06:14:18 UTC 
Closing.  Yes, irix compatibility means mips object files violate the ELF standard.  This isn't the place to agitate for a change in the default ABI for mips-elf and similar targets.