This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils 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, ldemul_choose_target, AIX -b64 support

Alan Modra wrote:

>
> Space before open parenthesis.

Space's added...

>
> > + static char *
> > + choose_target (argc, argv)
> > +      int argc;
> > +      char **argv;
> > + {
> > +   int i, j, jmax;
> > +   static char *from_outside;
> > +   static char *from_inside;
> > +   static char *argv_to_target[][2] =
> > +     {
> > +       NULL,   "aixcoff-rs6000",
> > + #ifdef _AIX43
> > +       "-b32", "aixcoff-rs6000",
> > +       "-b64", "aixcoff64-rs6000",
> > + #endif
> > +     };
> > +
> > +   jmax = 1;
> > + #ifdef _AIX43
> > +   jmax = 3;
> > + #endif
>
> Why the #ifdefs here?  Think cross-tools.

Mmmm.  No the ifdef's aren't needed.
deleting...

Tom



--
Tom Rix
GCC Engineer
trix@redhat.com
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]