From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-15822-listarch-gdb-patches=sourceware.cygnus.com@sources.redhat.com>
Received: (qmail 17478 invoked by alias); 23 May 2002 16:05:05 -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 17457 invoked from network); 23 May 2002 16:05:03 -0000
Received: from unknown (HELO localhost.redhat.com) (216.138.202.10)
  by sources.redhat.com with SMTP; 23 May 2002 16:05:03 -0000
Received: from cygnus.com (localhost [127.0.0.1])
	by localhost.redhat.com (Postfix) with ESMTP
	id BEBB93CC5; Thu, 23 May 2002 12:05:16 -0400 (EDT)
Message-ID: <3CED133C.9060704@cygnus.com>
Date: Thu, 23 May 2002 09:07:00 -0000
From: Andrew Cagney <ac131313@cygnus.com>
User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0rc2) Gecko/20020518
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Joern Rennecke <joern.rennecke@superh.com>
Cc: gdb-patches@sources.redhat.com, ezannoni@redhat.com,
	bje@redhat.com
Subject: Re: [Fwd: Re: Enum constants for sh1-sh4, sh{3}-dsp register numbers
 in  thesimulator]
References: <3CED0B5C.7281676D@superh.com>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-SW-Source: 2002-05/txt/msg00870.txt.bz2

> 
> Do you mean to add  SIM_SH_FR1_REGNUM .. SIM_SH_FR15_REGNUM and the various
> individual banked regnums?  I could do that.
> 
> OTOH, if you mean to remove the seed values like 24 for SIM_SH_DSR_REGNUM and
> 40 for SIM_SH_MOD_REGNUM, making a flat enum, that would break backwards
> compatibility,
> as we have some gaps, and overlaps between floating point and dsp registers.
> (This mirrors an overlap in the instruction set.  In fact the instructions to
> manipulate
>  FPSCR and DSR are the same, and the simulator makes use of this by treating
> them really
>  as the same.)

I mean flatten it.

If FPSCR and DSR are the same physical register on the SIM then have 
just one of them in the enum.  Separate to that define constants that 
map onto that main enum.

So:

	enum sh_...
	{
	};

	enum
	{
	  DUP_REG = SH_SIM_..._REG,
	};

enjoy,
Andrew