This is the mail archive of the gdb-patches@sources.redhat.com 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: [RFA] rs6000-tdep.c: figure out e500 machine from bfd.


On Aug 20,  6:36pm, Elena Zannoni wrote:

> +  /* For e500 executables, the apuinfo section is of help here.  */
> +  if (info.abfd)
> +    {
> +      sect = bfd_get_section_by_name (info.abfd, ".PPC.EMB.apuinfo");
> +      if (sect)

What is the apuinfo section?  If you know what it's about (I sure
don't), consider expanding the comment somewhat.

Is there any chance that this section could appear in non-e500
executables?  If so, make a note of this gotcha in the comment too.

Otherwise, fine.  Go ahead and commit it after expanding the comment
as necessary.

Kevin


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