From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <binutils-return-21258-listarch-binutils=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 22871 invoked by alias); 3 Oct 2002 00:00:40 -0000
Mailing-List: contact binutils-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:binutils-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/binutils/>
List-Post: <mailto:binutils@sources.redhat.com>
List-Help: <mailto:binutils-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: binutils-owner@sources.redhat.com
Received: (qmail 22755 invoked from network); 3 Oct 2002 00:00:33 -0000
Received: from unknown (HELO dair.pair.com) (209.68.1.49)
  by sources.redhat.com with SMTP; 3 Oct 2002 00:00:33 -0000
Received: (qmail 46086 invoked by uid 20157); 3 Oct 2002 00:00:32 -0000
Received: from localhost (sendmail-bs@127.0.0.1)
  by localhost with SMTP; 3 Oct 2002 00:00:32 -0000
Date: Wed, 02 Oct 2002 17:00:00 -0000
From: Hans-Peter Nilsson <hp@bitrange.com>
X-X-Sender: hp@dair.pair.com
To: Stephen Clarke <stephen.clarke@earthling.net>
cc: "H. J. Lu" <hjl@lucon.org>,  <binutils@sources.redhat.com>
Subject: Re: [Patch] sh64: Fix gas testsuite expected output
In-Reply-To: <002701c26a6a$88281100$6301a8c0@wren>
Message-ID: <Pine.BSF.4.44.0210021943060.34969-100000@dair.pair.com>
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
X-SW-Source: 2002-10/txt/msg00073.txt.bz2

On Wed, 2 Oct 2002, Stephen Clarke wrote:

While I have your attention, can you please do some official
check regarding the TLS ELF reloc numbers that Kojima-san
proposed?

> From: "H. J. Lu"
> Sent: Wednesday, October 02, 2002 4:08 PM
>
>
> > Is sh64-linux a supported Linux target? If yes, I can add it to my
> > binutils test target. I already test sh-linux, adding sh64-linux won't
> > be a problem.
>
> Not yet ... there's a patch at
> http://sources.redhat.com/ml/binutils/2002-09/msg00048.html
> waiting to be reviewed.

That patch is ok.

> This would need sh64 gas and ld testsuite fixes too, see
> http://sources.redhat.com/ml/binutils/2002-09/msg00049.html

Ok.

> http://sources.redhat.com/ml/binutils/2002-09/msg00050.html

But for one thing (leading to some other changes), the patch is
ok:

Index: ld/testsuite/ld-sh/sh.exp
===================================================================
RCS file: /cvs/src/src/ld/testsuite/ld-sh/sh.exp,v
retrieving revision 1.8
diff -u -c -3 -p -r1.8 sh.exp
*** ld/testsuite/ld-sh/sh.exp   8 Feb 2002 06:49:55 -0000
1.8
--- ld/testsuite/ld-sh/sh.exp   30 Aug 2002 00:51:57 -0000
*************** if ![istarget sh*-*-*] {
*** 25,31 ****
      return
  }

! if [istarget sh64-*-elf] {
      # relaxing not supported on sh64 yet.
      return
  }
--- 25,32 ----
      return
  }

! if { ([istarget sh64-*-elf]
!       || [istarget sh64*-*-linux*])} {
      # relaxing not supported on sh64 yet.
      return
  }

I'd rather you tweak the target specifiers on the tests:
#targets: sh-*-* shl*-*-* shb*-*-*
in each *.d should suffice.

>
> I couldn't miss this opportunity to nag the maintainers could I :-)

Where's Joern these days?  Doesn't he work on SH stuff?

brgds, H-P