From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-15453-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 10559 invoked by alias); 13 May 2002 22:13:34 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 10541 invoked from network); 13 May 2002 22:13:31 -0000
Received: from unknown (HELO cygnus.com) (205.180.83.203)
  by sources.redhat.com with SMTP; 13 May 2002 22:13:31 -0000
Received: from localhost.redhat.com (remus.sfbay.redhat.com [172.16.27.252])
	by runyon.cygnus.com (8.8.7-cygnus/8.8.7) with ESMTP id PAA19676;
	Mon, 13 May 2002 15:13:27 -0700 (PDT)
Received: by localhost.redhat.com (Postfix, from userid 469)
	id DF0A110AAD; Mon, 13 May 2002 17:10:27 -0400 (EDT)
From: Elena Zannoni <ezannoni@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Message-ID: <15584.11203.728429.774659@localhost.redhat.com>
Date: Mon, 13 May 2002 15:13:00 -0000
To: Nick Clifton <nickc@cambridge.redhat.com>
Cc: thorpej@wasabisystems.com, binutils@sources.redhat.com,
        gdb-patches@sources.redhat.com
Subject: Re: [PATCH/RFA] Include sh64 support for shle-*-netbsdelf*
In-Reply-To: <m3lmaokl2u.fsf@north-pole.nickc.cambridge.redhat.com>
References: <20020511115603.W3435@dr-evil.shagadelic.org>
	<m3n0v4o2lh.fsf@north-pole.nickc.cambridge.redhat.com>
	<20020513082324.R3435@dr-evil.shagadelic.org>
	<m3lmaokl2u.fsf@north-pole.nickc.cambridge.redhat.com>
X-SW-Source: 2002-05/txt/msg00501.txt.bz2

Nick Clifton writes:
 > Hi Jason,
 > 
 > >  > > +  shle-*-netbsdelf*)
 > >  > > +    targ_defvec=bfd_elf32_shlnbsd_vec
 > >  > > +    targ_selvecs="bfd_elf32_shnbsd_vec shcoff_vec shlcoff_vec"
 > >  > > +#ifdef BFD64
 > >  > > +    targ_selvecs="${targ_selvecs} bfd_elf32_sh64_vec bfd_elf32_sh64l_vec bfd_elf64_sh64_vec bfd_elf64_sh64l_vec"
 > >  > > +#endif
 > >  > > +    ;;
 > >  > >    sh*le-*-netbsdelf*)
 > >  > >      targ_defvec=bfd_elf32_shlnbsd_vec
 > >  > >      targ_selvecs="bfd_elf32_shnbsd_vec shcoff_vec shlcoff_vec"
 > >  > 
 > >  > Doesn't the "sh*le-*-netbsdelf*" target (immediately below the lines
 > >  > you have added) already cover this situation ?
 > > 
 > > It appears as if the intent in other configurations was to include sh64
 > > support only if the target was "sh", not "sh3" or "sh4", etc.  Look down
 > > one and two more, and you see the sh-*-netbsdelf* (added by hp when the
 > > sh64 code went in) and the sh*-*-netbsdelf* (added by me when I checked
 > > in NetBSD SH support) configs.
 > 
 > Oh right, I understand now.  In which case please consider this part
 > of the patch approved too.
 > 

Wouldn't the same change be required to build sh*le-*-netbdself* ?
The tdep gdb file is going to be built for all the sh targets. And that file
requires the sh64 disassembly functions.
Even the sh coff targets will require the sh64 stuff. :-(

Elena


 > Cheers
 >         Nick