[RFC/patch 5.1] change NUM_REALREGS to NUM_REGS

[RFC/patch 5.1] change NUM_REALREGS to NUM_REGS

[Andrew Volkov]

[-- Attachment #1: Type: text/plain, Size: 185 bytes --]

The attached patch replace NUM_REALREGS macro to NUM_REGS, 
and NUM_REGS to NUM_REGS + NUM_PSEUDO_REGS.

I need this for H8300 maltiarching.

Any suggestions/comments ?

Andrey Volkov


[-- Attachment #2: remote-e7000.diff --]
[-- Type: application/octet-stream, Size: 885 bytes --]

--- remote-e7000.c.old	Mon Jul 16 00:34:13 2001
+++ remote-e7000.c	Thu Dec 27 19:16:06 2001
@@ -922,7 +922,7 @@
   fetch_regs_from_dump (gch, wanted);
 
   /* And supply the extra ones the simulator uses */
-  for (regno = NUM_REALREGS; regno < NUM_REGS; regno++)
+  for (regno = NUM_REGS; regno < NUM_REGS + NUM_PSEUDO_REGS; regno++)
     {
       int buf = 0;
 
@@ -946,7 +946,7 @@
 {
   int regno;
 
-  for (regno = 0; regno < NUM_REALREGS; regno++)
+  for (regno = 0; regno < NUM_REGS; regno++)
     e7000_store_register (regno);
 
   registers_changed ();
@@ -2059,7 +2059,7 @@
   fetch_regs_from_dump (gch, wanted_nopc);
 
   /* And supply the extra ones the simulator uses */
-  for (regno = NUM_REALREGS; regno < NUM_REGS; regno++)
+  for (regno = NUM_REGS; regno < NUM_REGS + NUM_PSEUDO_REGS; regno++)
     {
       int buf = 0;
       supply_register (regno, (char *) &buf);

Re: [RFC/patch 5.1] change NUM_REALREGS to NUM_REGS

[Elena Zannoni]

Andrew Volkov writes:
 > The attached patch replace NUM_REALREGS macro to NUM_REGS, 
 > and NUM_REGS to NUM_REGS + NUM_PSEUDO_REGS.
 > 
 > I need this for H8300 maltiarching.
 > 
 > Any suggestions/comments ?
 > 
 > Andrey Volkov
 > 

Andrey,
You should also eliminate the definitions of the NUM_REALREGS macro.

One definition of that macro is in config/h8300/tm-h8300.h (defined to
10). Hmm, h8300-tdep.c defines NUM_REGS to 11, instead. Oh, and look(!)
config/tm-h8300.h defines NUM_REGS to 13.

You also compile remote-e7000.c when the target triplet is one of
sh-*-hms, sh-*-coff*, sh-*-elf*, sh-*-linux*. The definition of
NUM_REALREGS then comes from config/sh/tm-sh.h (in this case is 59).

Pretty messy. Maybe Kazu has some more insights on the h8300
definitions of NUM_REGS.

Elena

RE: [RFC/patch 5.1] change NUM_REALREGS to NUM_REGS

[Andrew Volkov]

Elena,

> You should also eliminate the definitions of the NUM_REALREGS macro.
> 

Certainly. I already remove NUM_REALREGS in h8300 part, but since
multiarching 
in progress, I send this changes with first version of multiarching h8300. 

> One definition of that macro is in config/h8300/tm-h8300.h (defined to
> 10). Hmm, h8300-tdep.c defines NUM_REGS to 11, instead. Oh, 
> and look(!)
> config/tm-h8300.h defines NUM_REGS to 13.

Yea, and h8300-simulator using self regnums :).

> 
> You also compile remote-e7000.c when the target triplet is one of
> sh-*-hms, sh-*-coff*, sh-*-elf*, sh-*-linux*. The definition of
> NUM_REALREGS then comes from config/sh/tm-sh.h (in this case is 59).
> 

Therefore I send copy to you personally, as maintaner of SH. Could you
remove 
this obsolete definition from tm-sh.h?


Anrey Volkov