*** empty log message ***
[binutils-gdb.git] / binutils / nlmconv.c
index 2eaaa9f54735bd0dba898699ce8ce95cd47e7f5a..6709461d328ace7f89e4da73d39fe0229a9362a7 100644 (file)
@@ -1,21 +1,22 @@
 /* nlmconv.c -- NLM conversion program
-   Copyright (C) 1993, 94, 95, 96, 97, 1998 Free Software Foundation, Inc.
+   Copyright 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002
+   Free Software Foundation, Inc.
 
-This file is part of GNU Binutils.
+   This file is part of GNU Binutils.
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 /* Written by Ian Lance Taylor <ian@cygnus.com>.
 
@@ -34,14 +35,14 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 #include "bfd.h"
 #include "libiberty.h"
 #include "bucomm.h"
+#include "safe-ctype.h"
 
-#include <ansidecl.h>
+#include "ansidecl.h"
 #include <time.h>
-#include <ctype.h>
 #include <sys/stat.h>
 #include <sys/file.h>
 #include <assert.h>
-#include <getopt.h>
+#include "getopt.h"
 
 /* Internal BFD NLM header.  */
 #include "libnlm.h"
@@ -55,7 +56,7 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 /* If strerror is just a macro, we want to use the one from libiberty
    since it will handle undefined values.  */
 #undef strerror
-extern char *strerror ();
+extern char *strerror PARAMS ((int));
 
 #ifndef localtime
 extern struct tm *localtime ();
@@ -121,38 +122,40 @@ static struct option long_options[] =
 
 /* Local routines.  */
 
-static void show_help PARAMS ((void));
-static void show_usage PARAMS ((FILE *, int));
-static const char *select_output_format PARAMS ((enum bfd_architecture,
-                                                unsigned long, boolean));
-static void setup_sections PARAMS ((bfd *, asection *, PTR));
-static void copy_sections PARAMS ((bfd *, asection *, PTR));
-static void mangle_relocs PARAMS ((bfd *, asection *, arelent ***,
-                                  long *, char *,
-                                  bfd_size_type));
-static void default_mangle_relocs PARAMS ((bfd *, asection *, arelent ***,
-                                          long *, char *,
-                                          bfd_size_type));
-static char *link_inputs PARAMS ((struct string_list *, char *));
+int main PARAMS ((int, char **));
+
+static void show_usage
+  PARAMS ((FILE *, int));
+static const char *select_output_format
+  PARAMS ((enum bfd_architecture, unsigned long, bfd_boolean));
+static void setup_sections
+  PARAMS ((bfd *, asection *, PTR));
+static void copy_sections
+  PARAMS ((bfd *, asection *, PTR));
+static void mangle_relocs
+  PARAMS ((bfd *, asection *, arelent ***, long *, char *, bfd_size_type));
+static void default_mangle_relocs
+  PARAMS ((bfd *, asection *, arelent ***, long *, char *, bfd_size_type));
+static char *link_inputs
+  PARAMS ((struct string_list *, char *));
 
 #ifdef NLMCONV_I386
-static void i386_mangle_relocs PARAMS ((bfd *, asection *, arelent ***,
-                                       long *, char *,
-                                       bfd_size_type));
+static void i386_mangle_relocs
+  PARAMS ((bfd *, asection *, arelent ***, long *, char *, bfd_size_type));
 #endif
 
 #ifdef NLMCONV_ALPHA
-static void alpha_mangle_relocs PARAMS ((bfd *, asection *, arelent ***,
-                                        long *, char *,
-                                        bfd_size_type));
+static void alpha_mangle_relocs
+  PARAMS ((bfd *, asection *, arelent ***, long *, char *, bfd_size_type));
 #endif
 
 #ifdef NLMCONV_POWERPC
-static void powerpc_build_stubs PARAMS ((bfd *, bfd *, asymbol ***, long *));
-static void powerpc_resolve_stubs PARAMS ((bfd *, bfd *));
-static void powerpc_mangle_relocs PARAMS ((bfd *, asection *, arelent ***,
-                                          long *, char *,
-                                          bfd_size_type));
+static void powerpc_build_stubs
+  PARAMS ((bfd *, bfd *, asymbol ***, long *));
+static void powerpc_resolve_stubs
+  PARAMS ((bfd *, bfd *));
+static void powerpc_mangle_relocs
+  PARAMS ((bfd *, asection *, arelent ***, long *, char *, bfd_size_type));
 #endif
 \f
 /* The main routine.  */
@@ -184,7 +187,7 @@ main (argc, argv)
   asymbol *endsym;
   long i;
   char inlead, outlead;
-  boolean gotstart, gotexit, gotcheck;
+  bfd_boolean gotstart, gotexit, gotcheck;
   struct stat st;
   FILE *custom_data = NULL;
   FILE *help_data = NULL;
@@ -210,7 +213,12 @@ main (argc, argv)
   char *modname;
   char **matching;
 
+#if defined (HAVE_SETLOCALE) && defined (HAVE_LC_MESSAGES)
   setlocale (LC_MESSAGES, "");
+#endif
+#if defined (HAVE_SETLOCALE)
+  setlocale (LC_CTYPE, "");
+#endif
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
@@ -220,7 +228,7 @@ main (argc, argv)
   bfd_init ();
   set_default_bfd_target ();
 
-  while ((opt = getopt_long (argc, argv, "dhI:l:O:T:V", long_options,
+  while ((opt = getopt_long (argc, argv, "dHhI:l:O:T:Vv", long_options,
                             (int *) NULL))
         != EOF)
     {
@@ -229,9 +237,10 @@ main (argc, argv)
        case 'd':
          debug = 1;
          break;
+       case 'H':
        case 'h':
-         show_help ();
-         /*NOTREACHED*/
+         show_usage (stdout, 0);
+         break;
        case 'I':
          input_format = optarg;
          break;
@@ -244,14 +253,15 @@ main (argc, argv)
        case 'T':
          header_file = optarg;
          break;
+       case 'v':
        case 'V':
          print_version ("nlmconv");
-         /*NOTREACHED*/
+         break;
        case 0:
          break;
        default:
          show_usage (stderr, 1);
-         /*NOTREACHED*/
+         break;
        }
     }
 
@@ -269,10 +279,7 @@ main (argc, argv)
            show_usage (stderr, 1);
          if (strcmp (input_file, output_file) == 0)
            {
-             fprintf (stderr,
-                      _("%s: input and output files must be different\n"),
-                      program_name);
-             exit (1);
+             fatal (_("input and output files must be different"));
            }
        }
     }
@@ -290,18 +297,18 @@ main (argc, argv)
   memset ((PTR) &extended_hdr_struct, 0, sizeof extended_hdr_struct);
   check_procedure = NULL;
   custom_file = NULL;
-  debug_info = false;
+  debug_info = FALSE;
   exit_procedure = "_Stop";
   export_symbols = NULL;
   map_file = NULL;
-  full_map = false;
+  full_map = FALSE;
   help_file = NULL;
   import_symbols = NULL;
   message_file = NULL;
   modules = NULL;
   sharelib_file = NULL;
   start_procedure = "_Prelude";
-  verbose = false;
+  verbose = FALSE;
   rpc_file = NULL;
 
   parse_errors = 0;
@@ -319,10 +326,7 @@ main (argc, argv)
     {
       if (input_file != NULL)
        {
-         fprintf (stderr,
-                  _("%s: input file named both on command line and with INPUT\n"),
-                  program_name);
-         exit (1);
+         fatal (_("input file named both on command line and with INPUT"));
        }
       if (input_files->next == NULL)
        input_file = input_files->string;
@@ -331,7 +335,7 @@ main (argc, argv)
     }
   else if (input_file == NULL)
     {
-      fprintf (stderr, _("%s: no input file\n"), program_name);
+      non_fatal (_("no input file"));
       show_usage (stderr, 1);
     }
 
@@ -361,8 +365,7 @@ main (argc, argv)
      Otherwise use the file named in the OUTPUT statement.  */
   if (output_file == NULL)
     {
-      fprintf (stderr, _("%s: no name for output file\n"),
-              program_name);
+      non_fatal (_("no name for output file"));
       show_usage (stderr, 1);
     }
 
@@ -374,10 +377,9 @@ main (argc, argv)
 
   assert (bfd_get_flavour (outbfd) == bfd_target_nlm_flavour);
 
-  if (bfd_arch_get_compatible (inbfd, outbfd) == NULL)
-    fprintf (stderr,
-            _("%s: warning:input and output formats are not compatible\n"),
-            program_name);
+  /* XXX: Should we accept the unknown bfd format here ?  */
+  if (bfd_arch_get_compatible (inbfd, outbfd, TRUE) == NULL)
+    non_fatal (_("warning: input and output formats are not compatible"));
 
   /* Move the values read from the command file into outbfd.  */
   *nlm_fixed_header (outbfd) = fixed_hdr_struct;
@@ -459,9 +461,9 @@ main (argc, argv)
   /* Adjust symbol information.  */
   inlead = bfd_get_symbol_leading_char (inbfd);
   outlead = bfd_get_symbol_leading_char (outbfd);
-  gotstart = false;
-  gotexit = false;
-  gotcheck = false;
+  gotstart = FALSE;
+  gotexit = FALSE;
+  gotcheck = FALSE;
   newsymalloc = 10;
   newsyms = (asymbol **) xmalloc (newsymalloc * sizeof (asymbol *));
   newsymcount = 0;
@@ -559,7 +561,7 @@ main (argc, argv)
              sym->section = got_sec->output_section;
            }
 #endif
-       }
+       }
 
       /* If this is a global symbol, check the export list.  */
       if ((sym->flags & (BSF_EXPORT | BSF_GLOBAL)) != 0)
@@ -632,11 +634,10 @@ main (argc, argv)
                }
            }
          if (l == NULL)
-           fprintf (stderr,
-                    _("%s: warning: symbol %s imported but not in import list\n"),
-                    program_name, bfd_asymbol_name (sym));
+           non_fatal (_("warning: symbol %s imported but not in import list"),
+                      bfd_asymbol_name (sym));
        }
-       
+
       /* See if it's one of the special named symbols.  */
       if ((sym->flags & BSF_DEBUGGING) == 0)
        {
@@ -654,7 +655,7 @@ main (argc, argv)
                val += bfd_section_size (outbfd, text_sec);
              if (! bfd_set_start_address (outbfd, val))
                bfd_fatal (_("set start address"));
-             gotstart = true;
+             gotstart = TRUE;
            }
          if (strcmp (bfd_asymbol_name (sym), exit_procedure) == 0)
            {
@@ -663,7 +664,7 @@ main (argc, argv)
                  && text_sec != (asection *) NULL)
                val += bfd_section_size (outbfd, text_sec);
              nlm_fixed_header (outbfd)->exitProcedureOffset = val;
-             gotexit = true;
+             gotexit = TRUE;
            }
          if (check_procedure != NULL
              && strcmp (bfd_asymbol_name (sym), check_procedure) == 0)
@@ -673,7 +674,7 @@ main (argc, argv)
                  && text_sec != (asection *) NULL)
                val += bfd_section_size (outbfd, text_sec);
              nlm_fixed_header (outbfd)->checkUnloadProcedureOffset = val;
-             gotcheck = true;
+             gotcheck = TRUE;
            }
        }
     }
@@ -700,17 +701,13 @@ main (argc, argv)
     }
 
   bfd_set_symtab (outbfd, outsyms, symcount + newsymcount);
-    
+
   if (! gotstart)
-    fprintf (stderr, _("%s: warning: START procedure %s not defined\n"),
-            program_name, start_procedure);
+    non_fatal (_("warning: START procedure %s not defined"), start_procedure);
   if (! gotexit)
-    fprintf (stderr, _("%s: warning: EXIT procedure %s not defined\n"),
-            program_name, exit_procedure);
-  if (check_procedure != NULL
-      && ! gotcheck)
-    fprintf (stderr, _("%s: warning: CHECK procedure %s not defined\n"),
-            program_name, check_procedure);
+    non_fatal (_("warning: EXIT procedure %s not defined"), exit_procedure);
+  if (check_procedure != NULL && ! gotcheck)
+    non_fatal (_("warning: CHECK procedure %s not defined"), check_procedure);
 
   /* Add additional sections required for the header information.  */
   if (custom_file != NULL)
@@ -846,9 +843,8 @@ main (argc, argv)
              if (sharedhdr.uninitializedDataSize > 0)
                {
                  /* There is no place to record this information.  */
-                 fprintf (stderr,
-                          _("%s:%s: warning: shared libraries can not have uninitialized data\n"),
-                          program_name, sharelib_file);
+                 non_fatal (_("%s: warning: shared libraries can not have uninitialized data"),
+                            sharelib_file);
                }
              shared_offset = st.st_size;
              if (shared_offset > (size_t) sharedhdr.codeImageOffset)
@@ -876,8 +872,7 @@ main (argc, argv)
 
   /* Check whether a version was given.  */
   if (strncmp (version_hdr->stamp, "VeRsIoN#", 8) != 0)
-    fprintf (stderr, _("%s: warning: No version number given\n"),
-            program_name);
+    non_fatal (_("warning: No version number given"));
 
   /* At least for now, always create an extended header, because that
      is what NLMLINK does.  */
@@ -917,8 +912,7 @@ main (argc, argv)
 
       data = xmalloc (custom_size);
       if (fread (data, 1, custom_size, custom_data) != custom_size)
-       fprintf (stderr, _("%s:%s: read: %s\n"), program_name, custom_file,
-                strerror (errno));
+       non_fatal (_("%s: read: %s"), custom_file, strerror (errno));
       else
        {
          if (! bfd_set_section_contents (outbfd, custom_section, data,
@@ -940,17 +934,14 @@ main (argc, argv)
       nlm_fixed_header (outbfd)->debugInfoOffset = (file_ptr) -1;
     }
   if (map_file != NULL)
-    fprintf (stderr,
-            _("%s: warning: MAP and FULLMAP are not supported; try ld -M\n"),
-            program_name);
+    non_fatal (_("warning: MAP and FULLMAP are not supported; try ld -M"));
   if (help_file != NULL)
     {
       PTR data;
 
       data = xmalloc (help_size);
       if (fread (data, 1, help_size, help_data) != help_size)
-       fprintf (stderr, _("%s:%s: read: %s\n"), program_name, help_file,
-                strerror (errno));
+       non_fatal (_("%s: read: %s"), help_file, strerror (errno));
       else
        {
          if (! bfd_set_section_contents (outbfd, help_section, data,
@@ -968,8 +959,7 @@ main (argc, argv)
 
       data = xmalloc (message_size);
       if (fread (data, 1, message_size, message_data) != message_size)
-       fprintf (stderr, _("%s:%s: read: %s\n"), program_name, message_file,
-                strerror (errno));
+       non_fatal (_("%s: read: %s"), message_file, strerror (errno));
       else
        {
          if (! bfd_set_section_contents (outbfd, message_section, data,
@@ -1018,8 +1008,7 @@ main (argc, argv)
 
       data = xmalloc (rpc_size);
       if (fread (data, 1, rpc_size, rpc_data) != rpc_size)
-       fprintf (stderr, _("%s:%s: read: %s\n"), program_name, rpc_file,
-                strerror (errno));
+       non_fatal (_("%s: read: %s"), rpc_file, strerror (errno));
       else
        {
          if (! bfd_set_section_contents (outbfd, rpc_section, data,
@@ -1038,8 +1027,7 @@ main (argc, argv)
       data = xmalloc (shared_size);
       if (fseek (shared_data, shared_offset, SEEK_SET) != 0
          || fread (data, 1, shared_size, shared_data) != shared_size)
-       fprintf (stderr, _("%s:%s: read: %s\n"), program_name, sharelib_file,
-                strerror (errno));
+       non_fatal (_("%s: read: %s"), sharelib_file, strerror (errno));
       else
        {
          if (! bfd_set_section_contents (outbfd, shared_section, data,
@@ -1091,8 +1079,7 @@ main (argc, argv)
   for (modname = nlm_fixed_header (outbfd)->moduleName;
        *modname != '\0';
        modname++)
-    if (islower ((unsigned char) *modname))
-      *modname = toupper (*modname);
+    *modname = TOUPPER (*modname);
 
   strncpy (nlm_variable_header (outbfd)->oldThreadName, " LONG",
           NLM_OLD_THREAD_NAME_LENGTH);
@@ -1111,15 +1098,6 @@ main (argc, argv)
   return 0;
 }
 \f
-/* Display a help message and exit.  */
-
-static void
-show_help ()
-{
-  printf (_("%s: Convert an object file into a NetWare Loadable Module\n"),
-         program_name);
-  show_usage (stdout, 0);
-}
 
 /* Show a usage message and exit.  */
 
@@ -1128,15 +1106,19 @@ show_usage (file, status)
      FILE *file;
      int status;
 {
-  fprintf (file, _("\
-Usage: %s [-dhV] [-I bfdname] [-O bfdname] [-T header-file] [-l linker]\n\
-       [--input-target=bfdname] [--output-target=bfdname]\n\
-       [--header-file=file] [--linker=linker] [--debug]\n\
-       [--help] [--version]\n\
-       [in-file [out-file]]\n"),
-          program_name);
+  fprintf (file, _("Usage: %s [option(s)] [in-file [out-file]]\n"), program_name);
+  fprintf (file, _(" Convert an object file into a NetWare Loadable Module\n"));
+  fprintf (file, _(" The options are:\n\
+  -I --input-target=<bfdname>   Set the input binary file format\n\
+  -O --output-target=<bfdname>  Set the output binary file format\n\
+  -T --header-file=<file>       Read <file> for NLM header information\n\
+  -l --linker=<linker>          Use <linker> for any linking\n\
+  -d --debug                    Display on stderr the linker command line\n\
+  -h --help                     Display this information\n\
+  -v --version                  Display the program's version\n\
+"));
   if (status == 0)
-    fprintf (file, _("Report bugs to bug-gnu-utils@gnu.org\n"));
+    fprintf (file, _("Report bugs to %s\n"), REPORT_BUGS_TO);
   exit (status);
 }
 \f
@@ -1147,7 +1129,7 @@ static const char *
 select_output_format (arch, mach, bigendian)
      enum bfd_architecture arch;
      unsigned long mach;
-     boolean bigendian;
+     bfd_boolean bigendian ATTRIBUTE_UNUSED;
 {
   switch (arch)
     {
@@ -1168,11 +1150,8 @@ select_output_format (arch, mach, bigendian)
       return "nlm32-powerpc";
 #endif
     default:
-      fprintf (stderr, _("%s: support not compiled in for %s\n"),
-              program_name, bfd_printable_arch_mach (arch, mach));
-      exit (1);
-      /* Avoid warning.  */
-      return NULL;
+      fatal (_("support not compiled in for %s"),
+            bfd_printable_arch_mach (arch, mach));
     }
   /*NOTREACHED*/
 }
@@ -1183,7 +1162,7 @@ select_output_format (arch, mach, bigendian)
 
 static void
 setup_sections (inbfd, insec, data_ptr)
-     bfd *inbfd;
+     bfd *inbfd ATTRIBUTE_UNUSED;
      asection *insec;
      PTR data_ptr;
 {
@@ -1276,7 +1255,7 @@ copy_sections (inbfd, insec, data_ptr)
 
   /* FIXME: Why are these necessary?  */
   insec->_cooked_size = insec->_raw_size;
-  insec->reloc_done = true;
+  insec->reloc_done = TRUE;
 
   if ((bfd_get_section_flags (inbfd, insec) & SEC_HAS_CONTENTS) == 0)
     contents = NULL;
@@ -1403,16 +1382,15 @@ mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
 /* By default all we need to do for relocs is change the address by
    the output_offset.  */
 
-/*ARGSUSED*/
 static void
 default_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
                       contents_size)
-     bfd *outbfd;
+     bfd *outbfd ATTRIBUTE_UNUSED;
      asection *insec;
      arelent ***relocs_ptr;
      long *reloc_count_ptr;
-     char *contents;
-     bfd_size_type contents_size;
+     char *contents ATTRIBUTE_UNUSED;
+     bfd_size_type contents_size ATTRIBUTE_UNUSED;
 {
   if (insec->output_offset != 0)
     {
@@ -1440,15 +1418,15 @@ static reloc_howto_type nlm_i386_pcrel_howto =
         0,                     /* rightshift */
         2,                     /* size (0 = byte, 1 = short, 2 = long) */
         32,                    /* bitsize */
-        true,                  /* pc_relative */
+        TRUE,                  /* pc_relative */
         0,                     /* bitpos */
         complain_overflow_signed, /* complain_on_overflow */
         0,                     /* special_function */
         "DISP32",              /* name */
-        true,                  /* partial_inplace */
+        TRUE,                  /* partial_inplace */
         0xffffffff,            /* src_mask */
         0xffffffff,            /* dst_mask */
-        true);                 /* pcrel_offset */
+        TRUE);                 /* pcrel_offset */
 
 static void
 i386_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
@@ -1598,17 +1576,16 @@ static reloc_howto_type nlm32_alpha_nw_howto =
         0,                     /* rightshift */
         0,                     /* size (0 = byte, 1 = short, 2 = long) */
         0,                     /* bitsize */
-        false,                 /* pc_relative */
+        FALSE,                 /* pc_relative */
         0,                     /* bitpos */
         complain_overflow_dont, /* complain_on_overflow */
         0,                     /* special_function */
         "NW_RELOC",            /* name */
-        false,                 /* partial_inplace */
+        FALSE,                 /* partial_inplace */
         0,                     /* src_mask */
         0,                     /* dst_mask */
-        false);                /* pcrel_offset */
+        FALSE);                /* pcrel_offset */
 
-/*ARGSUSED*/
 static void
 alpha_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
                     contents_size)
@@ -1616,8 +1593,8 @@ alpha_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
      asection *insec;
      register arelent ***relocs_ptr;
      long *reloc_count_ptr;
-     char *contents;
-     bfd_size_type contents_size;
+     char *contents ATTRIBUTE_UNUSED;
+     bfd_size_type contents_size ATTRIBUTE_UNUSED;
 {
   long old_reloc_count;
   arelent **old_relocs;
@@ -1749,7 +1726,7 @@ static bfd_size_type powerpc_initial_got_size;
 static void
 powerpc_build_stubs (inbfd, outbfd, symbols_ptr, symcount_ptr)
      bfd *inbfd;
-     bfd *outbfd;
+     bfd *outbfd ATTRIBUTE_UNUSED;
      asymbol ***symbols_ptr;
      long *symcount_ptr;
 {
@@ -1833,7 +1810,7 @@ powerpc_build_stubs (inbfd, outbfd, symbols_ptr, symcount_ptr)
 
       item->next = powerpc_stubs;
       powerpc_stubs = item;
-      
+
       ++stubcount;
     }
 
@@ -1916,7 +1893,7 @@ powerpc_resolve_stubs (inbfd, outbfd)
       reloc->address = l->toc_index + got_sec->output_offset;
       reloc->addend = 0;
       reloc->howto = bfd_reloc_type_lookup (inbfd, BFD_RELOC_32);
-                                     
+
       *r++ = reloc;
     }
 
@@ -1929,7 +1906,6 @@ powerpc_resolve_stubs (inbfd, outbfd)
    r2, will be set to the correct TOC value, so there is no need for
    any further reloc.  */
 
-/*ARGSUSED*/
 static void
 powerpc_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
                       contents_size)
@@ -1938,7 +1914,7 @@ powerpc_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
      register arelent ***relocs_ptr;
      long *reloc_count_ptr;
      char *contents;
-     bfd_size_type contents_size;
+     bfd_size_type contents_size ATTRIBUTE_UNUSED;
 {
   reloc_howto_type *toc_howto;
   long reloc_count;
@@ -1998,8 +1974,8 @@ powerpc_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
             between two sections both of which were placed in the
             same output section.  This should not happen.  */
          if (bfd_get_section (sym) != insec->output_section)
-           fprintf (stderr, _("%s: unresolved PC relative reloc against %s\n"),
-                    program_name, bfd_asymbol_name (sym));
+           non_fatal (_("unresolved PC relative reloc against %s"),
+                      bfd_asymbol_name (sym));
          else
            {
              bfd_vma val;
@@ -2062,9 +2038,8 @@ powerpc_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
                        & rel->howto->dst_mask));
              if ((bfd_signed_vma) val < - 0x8000
                  || (bfd_signed_vma) val >= 0x8000)
-               fprintf (stderr,
-                        _("%s: overflow when adjusting relocation against %s\n"),
-                        program_name, bfd_asymbol_name (sym));
+               non_fatal (_("overflow when adjusting relocation against %s"),
+                          bfd_asymbol_name (sym));
              bfd_put_16 (outbfd, val, (bfd_byte *) contents + rel->address);
              break;
 
@@ -2110,9 +2085,6 @@ powerpc_mangle_relocs (outbfd, insec, relocs_ptr, reloc_count_ptr, contents,
 #define LD_NAME "ld"
 #endif
 
-/* Temporary file name base.  */
-static char *temp_filename;
-
 /* The user has specified several input files.  Invoke the linker to
    link them all together, and convert and delete the resulting output
    file.  */
@@ -2161,10 +2133,7 @@ link_inputs (inputs, ld)
   if (ld == NULL)
     ld = (char *) LD_NAME;
 
-  temp_filename = choose_temp_base ();
-
-  unlink_on_exit = xmalloc (strlen (temp_filename) + 3);
-  sprintf (unlink_on_exit, "%s.O", temp_filename);
+  unlink_on_exit = make_temp_file (".O");
 
   argv[0] = ld;
   argv[1] = (char *) "-Ur";
@@ -2201,7 +2170,7 @@ link_inputs (inputs, ld)
 
   if (status != 0)
     {
-      fprintf (stderr, _("%s: Execution of %s failed\n"), program_name, ld);
+      non_fatal (_("Execution of %s failed"), ld);
       unlink (unlink_on_exit);
       exit (1);
     }