Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* ppc-po-tdep.c?
@ 2003-09-12 19:52 Andrew Cagney
  2003-09-12 20:43 ` ppc-po-tdep.c? Kevin Buettner
  0 siblings, 1 reply; 6+ messages in thread
From: Andrew Cagney @ 2003-09-12 19:52 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: gdb-patches

Kevin,

What do you think of a second ABI file (ppc-po-tdep.c?) along the lines 
of ppc-sysv-tdep.c that would contain the "AIX" / "Apple" / "PowerOpen" 
  / "SYSVr4 PPC64" ABIs (they are all a close fit).

Andrew


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

* Re: ppc-po-tdep.c?
  2003-09-12 19:52 ppc-po-tdep.c? Andrew Cagney
@ 2003-09-12 20:43 ` Kevin Buettner
  2003-09-12 21:58   ` ppc-po-tdep.c? Andrew Cagney
  2003-09-13 17:03   ` ppc-po-tdep.c? Andrew Cagney
  0 siblings, 2 replies; 6+ messages in thread
From: Kevin Buettner @ 2003-09-12 20:43 UTC (permalink / raw)
  To: Andrew Cagney, Kevin Buettner; +Cc: gdb-patches

On Sep 12,  3:52pm, Andrew Cagney wrote:

> What do you think of a second ABI file (ppc-po-tdep.c?) along the lines 
> of ppc-sysv-tdep.c that would contain the "AIX" / "Apple" / "PowerOpen" 
>   / "SYSVr4 PPC64" ABIs (they are all a close fit).

I think this is a good idea.  The name you've suggested is okay with
me also.

As I suggested to Jimi a few days ago, I'd like to eventually see the
generic PPC code moved to its own file.  The remainder of
rs6000-tdep.c would then be AIX specific and this file could be
renamed to ppc-aix-tdep.c.

In "outline" form, this is what I'd eventually like to see:

    Generic PPC ISA:
	ppc-tdep.c	- replaces rs6000-tdep.c for generic bits
    
    ABI specific:
	ppc-sysv-tdep.c
	ppc-po-tdep.c	- PowerOpen ABI specific bits; some of rs6000-tdep.c
                          is moved here.

    OS specific:
	ppc-aix-tdep.c
	ppc-linux-tdep.c
	ppcnbsd-tdep.c	- I prefer ppc-nbsd-tdep.c though

    Native support:
	ppc-aix-nat.c	- renamed from rs6000-nat.c
	ppc-linux-nat.c
	ppcnbsd-nat.c	- I prefer ppc-nbsd-nat.c though

Kevin


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

* Re: ppc-po-tdep.c?
  2003-09-12 20:43 ` ppc-po-tdep.c? Kevin Buettner
@ 2003-09-12 21:58   ` Andrew Cagney
  2003-09-13 17:03   ` ppc-po-tdep.c? Andrew Cagney
  1 sibling, 0 replies; 6+ messages in thread
From: Andrew Cagney @ 2003-09-12 21:58 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: gdb-patches


> 	ppcnbsd-nat.c	- I prefer ppc-nbsd-nat.c though

The BSDs consistently use <cpu><x>bsd-....  Given that there will likely 
ba a x86-64 -> amd64 rename, now might be a good time to consider a 
renaming the bsd files to be <cpu>-<x>bsd-...

Andrew



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

* Re: ppc-po-tdep.c?
  2003-09-12 20:43 ` ppc-po-tdep.c? Kevin Buettner
  2003-09-12 21:58   ` ppc-po-tdep.c? Andrew Cagney
@ 2003-09-13 17:03   ` Andrew Cagney
  2003-09-15 17:05     ` ppc-po-tdep.c? Kevin Buettner
  1 sibling, 1 reply; 6+ messages in thread
From: Andrew Cagney @ 2003-09-13 17:03 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: gdb-patches

> On Sep 12,  3:52pm, Andrew Cagney wrote:
> 
> 
>> What do you think of a second ABI file (ppc-po-tdep.c?) along the lines 
>> of ppc-sysv-tdep.c that would contain the "AIX" / "Apple" / "PowerOpen" 
>>   / "SYSVr4 PPC64" ABIs (they are all a close fit).
> 
> 
> I think this is a good idea.  The name you've suggested is okay with
> me also.
> 
> As I suggested to Jimi a few days ago, I'd like to eventually see the
> generic PPC code moved to its own file.  The remainder of
> rs6000-tdep.c would then be AIX specific and this file could be
> renamed to ppc-aix-tdep.c.

BTW, I'm really tempted by the names:

	ppc-aix-abi.[ch]
	ppc-svr4-abi.[ch]

as that really reflect what's going on.  (ppc-sysv, isn't right - it is 
only for release 4 of system 5.

Andrew



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

* Re: ppc-po-tdep.c?
  2003-09-13 17:03   ` ppc-po-tdep.c? Andrew Cagney
@ 2003-09-15 17:05     ` Kevin Buettner
  2003-09-15 21:35       ` ppc-po-tdep.c? Jim Blandy
  0 siblings, 1 reply; 6+ messages in thread
From: Kevin Buettner @ 2003-09-15 17:05 UTC (permalink / raw)
  To: Andrew Cagney; +Cc: gdb-patches

On Sep 13,  1:02pm, Andrew Cagney wrote:

> BTW, I'm really tempted by the names:
> 
> 	ppc-aix-abi.[ch]
> 	ppc-svr4-abi.[ch]
> 
> as that really reflect what's going on.  (ppc-sysv, isn't right - it is 
> only for release 4 of system 5.

It may be possible to persuade me to go with "ppc-aix-abi.c".  As I
understand it, the Power Open ABI was based on the ABI used by AIX,
but I've never actually seen a separate AIX ABI document.  (To be
honest though, I haven't looked that hard.)  If you have a pointer to
a separate AIX ABI document, I'd like to look at it.

I don't agree regarding the name "ppc-svr4-abi.c".

The title of the ABI document is:

    System V Application Binary Interface
    PowerPC Processor Supplement

In the introduction under the heading "The PowerPC Processor and the
System V ABI", the document says:

    The System V Application Binary Interface, or System V ABI,
    defines a system interface for compiled application programs.  Its
    purpose is to establish a standard binary interface for
    application programs on systems that implement the interfaces
    defined in the System V Interface Definition, Issue 3.  This
    includes systems that have implemented UNIX System V Release 4.

Kevin


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

* Re: ppc-po-tdep.c?
  2003-09-15 17:05     ` ppc-po-tdep.c? Kevin Buettner
@ 2003-09-15 21:35       ` Jim Blandy
  0 siblings, 0 replies; 6+ messages in thread
From: Jim Blandy @ 2003-09-15 21:35 UTC (permalink / raw)
  To: Kevin Buettner; +Cc: Andrew Cagney, gdb-patches


Kevin Buettner <kevinb@redhat.com> writes:
> It may be possible to persuade me to go with "ppc-aix-abi.c".  As I
> understand it, the Power Open ABI was based on the ABI used by AIX,
> but I've never actually seen a separate AIX ABI document.  (To be
> honest though, I haven't looked that hard.)  If you have a pointer to
> a separate AIX ABI document, I'd like to look at it.

I looked very hard recently, and didn't find one either.  That's
hardly an authoritative answer, but I've usually been able to find
such things in the past.


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

end of thread, other threads:[~2003-09-15 21:35 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-09-12 19:52 ppc-po-tdep.c? Andrew Cagney
2003-09-12 20:43 ` ppc-po-tdep.c? Kevin Buettner
2003-09-12 21:58   ` ppc-po-tdep.c? Andrew Cagney
2003-09-13 17:03   ` ppc-po-tdep.c? Andrew Cagney
2003-09-15 17:05     ` ppc-po-tdep.c? Kevin Buettner
2003-09-15 21:35       ` ppc-po-tdep.c? Jim Blandy

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