Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message
@ 2002-07-31 13:29 Kevin Buettner
  2002-07-31 16:22 ` Michael Snyder
  0 siblings, 1 reply; 5+ messages in thread
From: Kevin Buettner @ 2002-07-31 13:29 UTC (permalink / raw)
  To: gdb-patches

I've just committed the patch below...

	* mips-tdep.c (_initialize_mips_tdep): Add "n64" to "set mips abi"
	help message.

Index: mips-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/mips-tdep.c,v
retrieving revision 1.84
diff -u -p -r1.84 mips-tdep.c
--- mips-tdep.c	31 Jul 2002 20:08:20 -0000	1.84
+++ mips-tdep.c	31 Jul 2002 20:18:57 -0000
@@ -5126,6 +5126,7 @@ This option can be set to one of:\n\
      "  o32\n"
      "  o64\n"
      "  n32\n"
+     "  n64\n"
      "  eabi32\n"
      "  eabi64",
      &setmipscmdlist);


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message
  2002-07-31 13:29 [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message Kevin Buettner
@ 2002-07-31 16:22 ` Michael Snyder
  2002-07-31 16:39   ` Kevin Buettner
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Snyder @ 2002-07-31 16:22 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: gdb-patches, echristo

Kevin Buettner wrote:
> 
> I've just committed the patch below...
> 
>         * mips-tdep.c (_initialize_mips_tdep): Add "n64" to "set mips abi"
>         help message.
> 
> Index: mips-tdep.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/mips-tdep.c,v
> retrieving revision 1.84
> diff -u -p -r1.84 mips-tdep.c
> --- mips-tdep.c 31 Jul 2002 20:08:20 -0000      1.84
> +++ mips-tdep.c 31 Jul 2002 20:18:57 -0000
> @@ -5126,6 +5126,7 @@ This option can be set to one of:\n\
>       "  o32\n"
>       "  o64\n"
>       "  n32\n"
> +     "  n64\n"
>       "  eabi32\n"
>       "  eabi64",
>       &setmipscmdlist);

Hey Kevin, Eric, 

Isn't there also an "MEABI32" and "MEABI64"?
Something developed at mips?
Perhaps we should add placeholders for those?


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message
  2002-07-31 16:22 ` Michael Snyder
@ 2002-07-31 16:39   ` Kevin Buettner
  2002-07-31 18:30     ` Eric Christopher
  0 siblings, 1 reply; 5+ messages in thread
From: Kevin Buettner @ 2002-07-31 16:39 UTC (permalink / raw)
  To: Michael Snyder, Kevin Buettner; +Cc: gdb-patches, echristo

On Jul 31,  3:57pm, Michael Snyder wrote:

> Isn't there also an "MEABI32" and "MEABI64"?

I've seen these before somewhere.  (bfd maybe).

> Something developed at mips?

I don't know anything about them.

> Perhaps we should add placeholders for those?

Sounds right to me so long as they're really different from the ones
we already have.

Kevin


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message
  2002-07-31 16:39   ` Kevin Buettner
@ 2002-07-31 18:30     ` Eric Christopher
  2002-07-31 18:40       ` Kevin Buettner
  0 siblings, 1 reply; 5+ messages in thread
From: Eric Christopher @ 2002-07-31 18:30 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: Michael Snyder, gdb-patches

On Wed, 2002-07-31 at 16:32, Kevin Buettner wrote:
> On Jul 31,  3:57pm, Michael Snyder wrote:
> 
> > Isn't there also an "MEABI32" and "MEABI64"?
> 
> I've seen these before somewhere.  (bfd maybe).
> 

They are in gcc and stuff.

> > Something developed at mips?
> 
> I don't know anything about them.
> 

It was actually a multivendor (Jim Wilson was on the committee)
development. It was never completed.

> > Perhaps we should add placeholders for those?
> 
> Sounds right to me so long as they're really different from the ones
> we already have.
> 

I wouldn't worry about it. I put it in gcc because I was told it was
"almost complete" - almost 2 years ago. I pinged mips a few times and
haven't heard anything. I'll probably remove support for it in the next
few months if a final, useful, ABI doc doesn't materialize.

-eric

-- 
I don't want a pony, I want a rocket
powered jetpack!


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message
  2002-07-31 18:30     ` Eric Christopher
@ 2002-07-31 18:40       ` Kevin Buettner
  0 siblings, 0 replies; 5+ messages in thread
From: Kevin Buettner @ 2002-07-31 18:40 UTC (permalink / raw)
  To: Eric Christopher; +Cc: Michael Snyder, gdb-patches

On Jul 31,  6:14pm, Eric Christopher wrote:

> > > Perhaps we should add placeholders for those?
> > 
> > Sounds right to me so long as they're really different from the ones
> > we already have.
> > 
> 
> I wouldn't worry about it. I put it in gcc because I was told it was
> "almost complete" - almost 2 years ago. I pinged mips a few times and
> haven't heard anything. I'll probably remove support for it in the next
> few months if a final, useful, ABI doc doesn't materialize.

Okay, we'll hold off on adding them then.  Please let us know if you
do ever see a final, useful ABI doc...

Thanks,

Kevin


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2002-08-01  1:33 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-07-31 13:29 [PATCH] mips-tdep.c: Add "n64" to "set mips abi" help message Kevin Buettner
2002-07-31 16:22 ` Michael Snyder
2002-07-31 16:39   ` Kevin Buettner
2002-07-31 18:30     ` Eric Christopher
2002-07-31 18:40       ` Kevin Buettner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox