This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Fix power7 ppc64 binaries loading


On Wed, 4 Mar 2009, Alan Modra wrote:

> On Tue, Mar 03, 2009 at 11:30:27PM +0000, Joseph S. Myers wrote:
> > On Wed, 4 Mar 2009, Alan Modra wrote:
> > 
> > > Google for E500 ABI.  It should only be enabled when you are
> > > assembling for e500.  I've committed the following to mainline
> > > and 2.19 branch.
> > > 
> > >        * config/tc-ppc.c (md_assemble): APUinfo only for e500.
> > 
> > > -  if (ppc_cpu & (PPC_OPCODE_SPE
> > > -   	       | PPC_OPCODE_ISEL | PPC_OPCODE_EFS
> > > -	       | PPC_OPCODE_BRLOCK | PPC_OPCODE_PMR | PPC_OPCODE_CACHELCK
> > > -	       | PPC_OPCODE_RFMCI))
> > > +  if ((ppc_cpu & PPC_OPCODE_E500MC) != 0)
> > 
> > Most e500 are not e500mc (indeed, e500mc is mostly like classic PowerPC at 
> > the userspace level, not like e500).
> 
> The effect of the new test is to generate .PPC.EMB.apuinfo when
> gas is given any of -me500, -me500x2 or -me500mc options.  Is that
> not correct?  I freely admit to not being 100% certain.

That seems correct; it just seems odd for PPC_OPCODE_E500MC to apply also 
to the other e500 processors.

-- 
Joseph S. Myers
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]