Your gas changes caused weakref failures for cris-elf, supposedly others too

[Jan Beulich]

>>> Alan Modra <amodra@bigpond.net.au> 04.11.05 02:57:47 >>>
>On Mon, Oct 31, 2005 at 10:15:49AM +0100, Jan Beulich wrote:
>> >While you two discuss where and how to fix this, I've entered
>[snip]
>> 
>> Thanks. And yes, I suppose the failure exists everywhere. And no,
>> neither of the two of use could really see the problem while
testing
>> since the submissions and check-ins were too close to one another
in
>> time.
>
>gas/read.c
>
>revision 1.111
>date: 2005/10/27 07:40:07;  author: jbeulich;  state: Exp;  lines: +1
-1
>
>revision 1.110
>date: 2005/10/24 17:51:41;  author: aoliva;  state: Exp;  lines: +119
-0
>
>62 hours is hardly "too close".  There seems to be no progress on
>resolving these annoying testsuite failures on all targets, even
though
>the fix is fairly simple.  I will revert your last patch if something
is
>not done soon.

This you just look at the commit time, not the submission time. If I
hadn't checked in that patch, another (x86) test would have failed as
was realized and reported
(http://sourceware.org/ml/binutils/2005-10/msg00393.html), since the
OK for a dependent x86 patch was given before the OK for the general
patch (and I'm sorry for having forgotten about the dependency). Also,
from my perspective, the weakref patch was already broken when it got
checked in, the broken-ness was just hidden (it silently assumed symbols
can be redefined without making sure that really worked, much like other
parts of the assembler before the adjustment
[http://sourceware.org/ml/binutils/2005-09/msg00377.html]). Finally,
Alexandre promised to address the problems (attached, unfortunately not
copied to the list), which is why I didn't spend any time on addressing
these (not to mention that I didn't really have time to do so).
Meanwhile I suggest the following (probably I could even consider this
obvious and don't wait for approval...):

gas/testsuite/
2005-11-04  Jan Beulich  <jbeulich@novell.com>

	* gas/all/gas.exp: xfail weakref dump tests for all targets.

---
/home/jbeulich/src/binutils/mainline/2005-11-04/gas/testsuite/gas/all/gas.exp	2005-11-04
08:29:48.000000000 +0100
+++ 2005-11-04/gas/testsuite/gas/all/gas.exp	2005-11-04
08:49:50.000000000 +0100
@@ -256,15 +256,15 @@ if { ![istarget "i960-*-*"] } {
 case $target_triplet in {
     { z80-*-* } { }
     default {
-	setup_xfail cris-*-* mmix-*-*
+	setup_xfail *-*-*
 	run_dump_test weakref1
-	setup_xfail cris-*-* mmix-*-*
+	setup_xfail *-*-*
 	run_dump_test weakref1g
-	setup_xfail cris-*-* mmix-*-*
+	setup_xfail *-*-*
 	run_dump_test weakref1l
-	setup_xfail cris-*-* mmix-*-*
+	setup_xfail *-*-*
 	run_dump_test weakref1u
-	setup_xfail cris-*-* mmix-*-*
+	setup_xfail *-*-*
 	run_dump_test weakref1w
     }
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: binutils-mainline-weakref-xfail.patch
Type: application/octet-stream
Size: 841 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20051104/aa35570d/attachment.obj>
-------------- next part --------------
An embedded message was scrubbed...
From: Alexandre Oliva <aoliva@redhat.com>
Subject: Re: Your gas changes caused weakref failures for cris-elf, supposedly others too
Date: Sat, 29 Oct 2005 16:27:24 -0200
Size: 3416
URL: <https://sourceware.org/pipermail/binutils/attachments/20051104/aa35570d/attachment.eml>