OSDN Git Service

2002-12-12 Jeff Johnston <jjohnstn@redhat.com>
[pf3gnuchains/pf3gnuchains4x.git] / ld / lexsup.c
index d544df5..8ce8f18 100644 (file)
@@ -1,24 +1,24 @@
 /* Parse options for the GNU linker.
    Copyright 1991, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000,
-   2001
+   2001, 2002
    Free Software Foundation, Inc.
 
-This file is part of GLD, the Gnu Linker.
+   This file is part of GLD, the Gnu Linker.
 
-GLD 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, or (at your option)
-any later version.
+   GLD 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, or (at your option)
+   any later version.
 
-GLD 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.
+   GLD 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 GLD; see the file COPYING.  If not, write to the Free
-Software Foundation, 59 Temple Place - Suite 330, Boston, MA
-02111-1307, USA.  */
+   You should have received a copy of the GNU General Public License
+   along with GLD; see the file COPYING.  If not, write to the Free
+   Software Foundation, 59 Temple Place - Suite 330, Boston, MA
+   02111-1307, USA.  */
 
 #include "bfd.h"
 #include "sysdep.h"
@@ -33,7 +33,7 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 #include "ldmisc.h"
 #include "ldexp.h"
 #include "ldlang.h"
-#include "ldgram.h"
+#include <ldgram.h>
 #include "ldlex.h"
 #include "ldfile.h"
 #include "ldver.h"
@@ -53,13 +53,9 @@ Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 #define        S_ISDIR(m) (((m) & S_IFMT) == S_IFDIR)
 #endif
 
-/* Omit args to avoid the possibility of clashing with a system header
-   that might disagree about consts.  */
-unsigned long strtoul ();
-
 static int is_num PARAMS ((const char *, int, int, int));
-static void set_default_dirlist PARAMS ((char *dirlist_ptr));
-static void set_section_start PARAMS ((char *sect, char *valstr));
+static void set_default_dirlist PARAMS ((char *));
+static void set_section_start PARAMS ((char *, char *));
 static void help PARAMS ((void));
 
 /* Non-zero if we are processing a --defsym from the command line.  */
@@ -130,8 +126,13 @@ int parsing_defsym = 0;
 #define OPTION_UNIQUE                  (OPTION_SECTION_START + 1)
 #define OPTION_TARGET_HELP              (OPTION_UNIQUE + 1)
 #define OPTION_ALLOW_SHLIB_UNDEFINED   (OPTION_TARGET_HELP + 1)
-#define OPTION_DISCARD_NONE            (OPTION_ALLOW_SHLIB_UNDEFINED + 1)
+#define OPTION_ALLOW_MULTIPLE_DEFINITION (OPTION_ALLOW_SHLIB_UNDEFINED + 1)
+#define OPTION_NO_UNDEFINED_VERSION    (OPTION_ALLOW_MULTIPLE_DEFINITION + 1)
+#define OPTION_DISCARD_NONE            (OPTION_NO_UNDEFINED_VERSION + 1)
 #define OPTION_SPARE_DYNAMIC_TAGS      (OPTION_DISCARD_NONE + 1)
+#define OPTION_NO_DEFINE_COMMON                (OPTION_SPARE_DYNAMIC_TAGS + 1)
+#define OPTION_NOSTDLIB                        (OPTION_NO_DEFINE_COMMON + 1)
+#define OPTION_NO_OMAGIC               (OPTION_NOSTDLIB + 1)
 
 /* The long options.  This structure is used for both the option
    parsing and the help text.  */
@@ -203,7 +204,7 @@ static const struct ld_option ld_options[] =
   { {"soname", required_argument, NULL, OPTION_SONAME},
       'h', N_("FILENAME"), N_("Set internal name of shared library"), ONE_DASH },
   { {"dynamic-linker", required_argument, NULL, OPTION_DYNAMIC_LINKER},
-      'I', N_("PROGRAM"), N_("Set the dynamic linker to use"), TWO_DASHES },
+      'I', N_("PROGRAM"), N_("Set PROGRAM as the dynamic linker to use"), TWO_DASHES },
   { {"library", required_argument, NULL, 'l'},
       'l', N_("LIBNAME"), N_("Search for library LIBNAME"), TWO_DASHES },
   { {"library-path", required_argument, NULL, 'L'},
@@ -217,6 +218,8 @@ static const struct ld_option ld_options[] =
   { {"omagic", no_argument, NULL, 'N'},
       'N', NULL, N_("Do not page align data, do not make text readonly"),
       EXACTLY_TWO_DASHES },
+  { {"no-omagic", no_argument, NULL, OPTION_NO_OMAGIC},
+      '\0', NULL, N_("Page align data, make text readonly"), EXACTLY_TWO_DASHES },
   { {"output", required_argument, NULL, 'o'},
       'o', N_("FILE"), N_("Set output file name"), EXACTLY_TWO_DASHES },
   { {NULL, required_argument, NULL, '\0'},
@@ -311,6 +314,8 @@ static const struct ld_option ld_options[] =
      '\0', N_("SYMBOL"), N_("Call SYMBOL at load-time"), ONE_DASH },
   { {"Map", required_argument, NULL, OPTION_MAP},
       '\0', N_("FILE"), N_("Write a map file"), ONE_DASH },
+  { {"no-define-common", no_argument, NULL, OPTION_NO_DEFINE_COMMON},
+      '\0', NULL, N_("Do not define Common storage"), TWO_DASHES },
   { {"no-demangle", no_argument, NULL, OPTION_NO_DEMANGLE },
       '\0', NULL, N_("Do not demangle symbol names"), TWO_DASHES },
   { {"no-keep-memory", no_argument, NULL, OPTION_NO_KEEP_MEMORY},
@@ -319,6 +324,10 @@ static const struct ld_option ld_options[] =
      '\0', NULL, N_("Allow no undefined symbols"), TWO_DASHES },
   { {"allow-shlib-undefined", no_argument, NULL, OPTION_ALLOW_SHLIB_UNDEFINED},
      '\0', NULL, N_("Allow undefined symbols in shared objects"), TWO_DASHES },
+  { {"allow-multiple-definition", no_argument, NULL, OPTION_ALLOW_MULTIPLE_DEFINITION},
+     '\0', NULL, N_("Allow multiple definitions"), TWO_DASHES },
+  { {"no-undefined-version", no_argument, NULL, OPTION_NO_UNDEFINED_VERSION},
+     '\0', NULL, N_("Disallow undefined version"), TWO_DASHES },
   { {"no-warn-mismatch", no_argument, NULL, OPTION_NO_WARN_MISMATCH},
       '\0', NULL, N_("Don't warn about mismatched input files"), TWO_DASHES},
   { {"no-whole-archive", no_argument, NULL, OPTION_NO_WHOLE_ARCHIVE},
@@ -327,6 +336,8 @@ static const struct ld_option ld_options[] =
       '\0', NULL, N_("Create an output file even if errors occur"), TWO_DASHES },
   { {"noinhibit_exec", no_argument, NULL, OPTION_NOINHIBIT_EXEC},
       '\0', NULL, NULL, NO_HELP },
+  { {"nostdlib", no_argument, NULL, OPTION_NOSTDLIB},
+      '\0', NULL, N_("Only use library directories specified on\n\t\t\t\tthe command line"), ONE_DASH },
   { {"oformat", required_argument, NULL, OPTION_OFORMAT},
       '\0', N_("TARGET"), N_("Specify target of output file"), EXACTLY_TWO_DASHES },
   { {"qmagic", no_argument, NULL, OPTION_IGNORE},
@@ -536,7 +547,7 @@ parse_args (argc, argv)
   last_optind = -1;
   while (1)
     {
-      int longind, errind;
+      int longind;
       int optc;
 
       /* Using last_optind lets us avoid calling ldemul_parse_args
@@ -546,21 +557,18 @@ parse_args (argc, argv)
         -nx, in which the -n is parsed as a single option, and we
         loop around to pick up the -x.  */
       if (optind != last_optind)
-       {
-         if (ldemul_parse_args (argc, argv))
-           continue;
-         last_optind = optind;
-       }
+       if (ldemul_parse_args (argc, argv))
+         continue;
 
       /* getopt_long_only is like getopt_long, but '-' as well as '--'
         can indicate a long option.  */
       opterr = 0;
-      errind = optind;
+      last_optind = optind;
       optc = getopt_long_only (argc, argv, shortopts, longopts, &longind);
       if (optc == '?')
        {
-         --optind;
-         optc = getopt_long (argc, argv, shortopts, really_longopts, &longind);
+         optind = last_optind;
+         optc = getopt_long (argc, argv, "-", really_longopts, &longind);
        }
 
       if (optc == -1)
@@ -569,13 +577,10 @@ parse_args (argc, argv)
       switch (optc)
        {
        case '?':
-         fprintf (stderr, _("%s: unrecognized option '%s'\n"),
-                  program_name, argv[errind]);
+         einfo (_("%P: unrecognized option '%s'\n"), argv[last_optind]);
        default:
-         fprintf (stderr,
-                  _("%s: use the --help option for usage information\n"),
-                  program_name);
-         xexit (1);
+         einfo (_("%P%F: use the --help option for usage information\n"));
+
        case 1:                 /* File name.  */
          lang_add_input_file (optarg, lang_input_file_is_file_enum,
                               (char *) NULL);
@@ -588,10 +593,10 @@ parse_args (argc, argv)
              ``use only shared libraries'' but, then, we don't
              currently support shared libraries on HP/UX anyhow.  */
          if (strcmp (optarg, "archive") == 0)
-           config.dynamic_link = false;
+           config.dynamic_link = FALSE;
          else if (strcmp (optarg, "shared") == 0
                   || strcmp (optarg, "default") == 0)
-           config.dynamic_link = true;
+           config.dynamic_link = TRUE;
          else
            einfo (_("%P%F: unrecognized -a option `%s'\n"), optarg);
          break;
@@ -620,17 +625,17 @@ parse_args (argc, argv)
          yyparse ();
          break;
        case OPTION_CALL_SHARED:
-         config.dynamic_link = true;
+         config.dynamic_link = TRUE;
          break;
        case OPTION_NON_SHARED:
-         config.dynamic_link = false;
+         config.dynamic_link = FALSE;
          break;
        case OPTION_CREF:
-         command_line.cref = true;
-         link_info.notice_all = true;
+         command_line.cref = TRUE;
+         link_info.notice_all = TRUE;
          break;
        case 'd':
-         command_line.force_common_definition = true;
+         command_line.force_common_definition = TRUE;
          break;
        case OPTION_DEFSYM:
          lex_string = optarg;
@@ -642,7 +647,7 @@ parse_args (argc, argv)
          lex_string = NULL;
          break;
        case OPTION_DEMANGLE:
-         demangling = true;
+         demangling = TRUE;
          if (optarg != NULL)
            {
              enum demangling_styles style;
@@ -666,14 +671,14 @@ parse_args (argc, argv)
          command_line.endian = ENDIAN_LITTLE;
          break;
        case OPTION_EMBEDDED_RELOCS:
-         command_line.embedded_relocs = true;
+         command_line.embedded_relocs = TRUE;
          break;
        case OPTION_EXPORT_DYNAMIC:
        case 'E': /* HP/UX compatibility.  */
-         link_info.export_dynamic = true;
+         link_info.export_dynamic = TRUE;
          break;
        case 'e':
-         lang_add_entry (optarg, true);
+         lang_add_entry (optarg, TRUE);
          break;
        case 'f':
          if (command_line.auxiliary_filters == NULL)
@@ -702,7 +707,7 @@ parse_args (argc, argv)
          command_line.filter_shlib = optarg;
          break;
        case OPTION_FORCE_EXE_SUFFIX:
-         command_line.force_exe_suffix = true;
+         command_line.force_exe_suffix = TRUE;
          break;
        case 'G':
          {
@@ -716,14 +721,14 @@ parse_args (argc, argv)
          /* Ignore.  */
          break;
        case OPTION_GC_SECTIONS:
-         command_line.gc_sections = true;
+         command_line.gc_sections = TRUE;
          break;
        case OPTION_HELP:
          help ();
          xexit (0);
          break;
        case 'L':
-         ldfile_add_library_path (optarg, true);
+         ldfile_add_library_path (optarg, TRUE);
          break;
        case 'l':
          lang_add_input_file (optarg, lang_input_file_is_l_enum,
@@ -739,37 +744,55 @@ parse_args (argc, argv)
          config.map_filename = optarg;
          break;
        case 'N':
-         config.text_read_only = false;
-         config.magic_demand_paged = false;
-         config.dynamic_link = false;
+         config.text_read_only = FALSE;
+         config.magic_demand_paged = FALSE;
+         config.dynamic_link = FALSE;
+         break;
+       case OPTION_NO_OMAGIC:
+         config.text_read_only = TRUE;
+         config.magic_demand_paged = TRUE;
+         /* NB/ Does not set dynamic_link to TRUE.
+            Use --call-shared or -Bdynamic for this.  */
          break;
        case 'n':
-         config.magic_demand_paged = false;
-         config.dynamic_link = false;
+         config.magic_demand_paged = FALSE;
+         config.dynamic_link = FALSE;
+         break;
+       case OPTION_NO_DEFINE_COMMON:
+         command_line.inhibit_common_definition = TRUE;
          break;
        case OPTION_NO_DEMANGLE:
-         demangling = false;
+         demangling = FALSE;
          break;
        case OPTION_NO_GC_SECTIONS:
-         command_line.gc_sections = false;
+         command_line.gc_sections = FALSE;
          break;
        case OPTION_NO_KEEP_MEMORY:
-         link_info.keep_memory = false;
+         link_info.keep_memory = FALSE;
          break;
        case OPTION_NO_UNDEFINED:
-         link_info.no_undefined = true;
+         link_info.no_undefined = TRUE;
          break;
        case OPTION_ALLOW_SHLIB_UNDEFINED:
-         link_info.allow_shlib_undefined = true;
+         link_info.allow_shlib_undefined = TRUE;
+         break;
+       case OPTION_ALLOW_MULTIPLE_DEFINITION:
+         link_info.allow_multiple_definition = TRUE;
+         break;
+       case OPTION_NO_UNDEFINED_VERSION:
+         link_info.allow_undefined_version = FALSE;
          break;
        case OPTION_NO_WARN_MISMATCH:
-         command_line.warn_mismatch = false;
+         command_line.warn_mismatch = FALSE;
          break;
        case OPTION_NOINHIBIT_EXEC:
-         force_make_executable = true;
+         force_make_executable = TRUE;
+         break;
+       case OPTION_NOSTDLIB:
+         config.only_cmd_line_lib_dirs = TRUE;
          break;
        case OPTION_NO_WHOLE_ARCHIVE:
-         whole_archive = false;
+         whole_archive = FALSE;
          break;
        case 'O':
          /* FIXME "-O<non-digits> <value>" used to set the address of
@@ -779,7 +802,7 @@ parse_args (argc, argv)
             getopt can't handle two args to an option without kludges.  */
 
          /* Enable optimizations of output files.  */
-         link_info.optimize = strtoul (optarg, NULL, 0) ? true : false;
+         link_info.optimize = strtoul (optarg, NULL, 0) ? TRUE : FALSE;
          break;
        case 'o':
          lang_add_output (optarg, 0);
@@ -788,15 +811,26 @@ parse_args (argc, argv)
          lang_add_output_format (optarg, (char *) NULL, (char *) NULL, 0);
          break;
        case 'q':
-         link_info.emitrelocations = true;
+         link_info.emitrelocations = TRUE;
          break;
        case 'i':
        case 'r':
-         link_info.relocateable = true;
-         config.build_constructors = false;
-         config.magic_demand_paged = false;
-         config.text_read_only = false;
-         config.dynamic_link = false;
+         if (optind == last_optind)
+           /* This can happen if the user put "-rpath,a" on the command
+              line.  (Or something similar.  The comma is important).
+              Getopt becomes confused and thinks that this is a -r option
+              but it cannot parse the text after the -r so it refuses to
+              increment the optind counter.  Detect this case and issue
+              an error message here.  We cannot just make this a warning,
+              increment optind, and continue because getopt is too confused
+              and will seg-fault the next time around.  */
+           einfo(_("%P%F: bad -rpath option\n"));
+
+         link_info.relocateable = TRUE;
+         config.build_constructors = FALSE;
+         config.magic_demand_paged = FALSE;
+         config.text_read_only = FALSE;
+         config.dynamic_link = FALSE;
          break;
        case 'R':
          /* The GNU linker traditionally uses -R to mean to include
@@ -833,6 +867,7 @@ parse_args (argc, argv)
              do
                {
                  size_t idx = 0;
+
                  while (optarg[idx] != '\0' && optarg[idx] == cp[idx])
                    ++idx;
                  if (optarg[idx] == '\0'
@@ -872,7 +907,7 @@ parse_args (argc, argv)
            }
          break;
        case OPTION_RELAX:
-         command_line.relax = true;
+         command_line.relax = TRUE;
          break;
        case OPTION_RETAIN_SYMBOLS_FILE:
          add_keepsyms_file (optarg);
@@ -885,7 +920,7 @@ parse_args (argc, argv)
          break;
        case OPTION_SHARED:
          if (config.has_shared)
-           link_info.shared = true;
+           link_info.shared = TRUE;
          else
            einfo (_("%P%F: -shared not supported\n"));
          break;
@@ -894,16 +929,16 @@ parse_args (argc, argv)
          command_line.soname = optarg;
          break;
        case OPTION_SORT_COMMON:
-         config.sort_common = true;
+         config.sort_common = TRUE;
          break;
        case OPTION_STATS:
-         config.stats = true;
+         config.stats = TRUE;
          break;
        case OPTION_SYMBOLIC:
-         link_info.symbolic = true;
+         link_info.symbolic = TRUE;
          break;
        case 't':
-         trace_files = true;
+         trace_files = TRUE;
          break;
        case 'T':
          ldfile_open_command_file (optarg);
@@ -919,23 +954,13 @@ parse_args (argc, argv)
            /* Check for <something>=<somthing>...  */
            optarg2 = strchr (optarg, '=');
            if (optarg2 == NULL)
-             {
-               fprintf (stderr,
-                        _("%s: Invalid argument to option \"--section-start\"\n"),
-                        program_name);
-               xexit (1);
-             }
+             einfo (_("%P%F: invalid argument to option \"--section-start\"\n"));
 
            optarg2++;
 
            /* So far so good.  Are all the args present?  */
            if ((*optarg == '\0') || (*optarg2 == '\0'))
-             {
-               fprintf (stderr,
-                        _("%s: Missing argument(s) to option \"--section-start\"\n"),
-                        program_name);
-               xexit (1);
-             }
+             einfo (_("%P%F: missing argument(s) to option \"--section-start\"\n"));
 
            /* We must copy the section name as set_section_start
               doesn't do it for us.  */
@@ -950,8 +975,8 @@ parse_args (argc, argv)
          break;
        case OPTION_TARGET_HELP:
          /* Mention any target specific options.  */
-          ldemul_list_emulation_options (stdout);
-          exit (0);
+         ldemul_list_emulation_options (stdout);
+         exit (0);
        case OPTION_TBSS:
          set_section_start (".bss", optarg);
          break;
@@ -962,17 +987,17 @@ parse_args (argc, argv)
          set_section_start (".text", optarg);
          break;
        case OPTION_TRADITIONAL_FORMAT:
-         link_info.traditional_format = true;
+         link_info.traditional_format = TRUE;
          break;
        case OPTION_TASK_LINK:
-         link_info.task_link = true;
+         link_info.task_link = TRUE;
          /* Fall through - do an implied -r option.  */
        case OPTION_UR:
-         link_info.relocateable = true;
-         config.build_constructors = true;
-         config.magic_demand_paged = false;
-         config.text_read_only = false;
-         config.dynamic_link = false;
+         link_info.relocateable = TRUE;
+         config.build_constructors = TRUE;
+         config.magic_demand_paged = FALSE;
+         config.text_read_only = FALSE;
+         config.dynamic_link = FALSE;
          break;
        case 'u':
          ldlang_add_undef (optarg);
@@ -981,38 +1006,23 @@ parse_args (argc, argv)
          if (optarg != NULL)
            lang_add_unique (optarg);
          else
-           config.unique_orphan_sections = true;
+           config.unique_orphan_sections = TRUE;
          break;
        case OPTION_VERBOSE:
          ldversion (1);
-         version_printed = true;
-         trace_file_tries = true;
+         version_printed = TRUE;
+         trace_file_tries = TRUE;
          break;
        case 'v':
          ldversion (0);
-         version_printed = true;
+         version_printed = TRUE;
          break;
        case 'V':
          ldversion (1);
-         version_printed = true;
+         version_printed = TRUE;
          break;
        case OPTION_VERSION:
-         /* This output is intended to follow the GNU standards document.  */
-         printf ("GNU ld %s\n", ld_program_version);
-         printf (_("Copyright 2001 Free Software Foundation, Inc.\n"));
-         printf (_("\
-This program is free software; you may redistribute it under the terms of\n\
-the GNU General Public License.  This program has absolutely no warranty.\n"));
-         {
-           ld_emulation_xfer_type **ptr = ld_emulations;
-
-           printf (_("  Supported emulations:\n"));
-           while (*ptr)
-             {
-               printf ("   %s\n", (*ptr)->emulation_name);
-               ptr++;
-             }
-         }
+         ldversion (2);
          xexit (0);
          break;
        case OPTION_VERSION_SCRIPT:
@@ -1020,7 +1030,7 @@ the GNU General Public License.  This program has absolutely no warranty.\n"));
              version information.  Read it, but don't assume that
              we've seen a linker script.  */
          {
-           FILE * hold_script_handle;
+           FILE *hold_script_handle;
 
            hold_script_handle = saved_script_handle;
            ldfile_open_command_file (optarg);
@@ -1036,25 +1046,25 @@ the GNU General Public License.  This program has absolutely no warranty.\n"));
          command_line.version_exports_section = optarg;
          break;
        case OPTION_WARN_COMMON:
-         config.warn_common = true;
+         config.warn_common = TRUE;
          break;
        case OPTION_WARN_CONSTRUCTORS:
-         config.warn_constructors = true;
+         config.warn_constructors = TRUE;
          break;
        case OPTION_WARN_FATAL:
-         config.fatal_warnings = true;
+         config.fatal_warnings = TRUE;
          break;
        case OPTION_WARN_MULTIPLE_GP:
-         config.warn_multiple_gp = true;
+         config.warn_multiple_gp = TRUE;
          break;
        case OPTION_WARN_ONCE:
-         config.warn_once = true;
+         config.warn_once = TRUE;
          break;
        case OPTION_WARN_SECTION_ALIGN:
-         config.warn_section_align = true;
+         config.warn_section_align = TRUE;
          break;
        case OPTION_WHOLE_ARCHIVE:
-         whole_archive = true;
+         whole_archive = TRUE;
          break;
        case OPTION_WRAP:
          add_wrap (optarg);
@@ -1092,50 +1102,40 @@ the GNU General Public License.  This program has absolutely no warranty.\n"));
            config.split_by_file = 1;
          break;
        case OPTION_CHECK_SECTIONS:
-         command_line.check_section_addresses = true;
+         command_line.check_section_addresses = TRUE;
          break;
        case OPTION_NO_CHECK_SECTIONS:
-         command_line.check_section_addresses = false;
+         command_line.check_section_addresses = FALSE;
          break;
        case '(':
          if (ingroup)
-           {
-             fprintf (stderr,
-                      _("%s: may not nest groups (--help for usage)\n"),
-                      program_name);
-             xexit (1);
-           }
+           einfo (_("%P%F: may not nest groups (--help for usage)\n"));
+
          lang_enter_group ();
          ingroup = 1;
          break;
        case ')':
          if (! ingroup)
-           {
-             fprintf (stderr,
-                      _("%s: group ended before it began (--help for usage)\n"),
-                      program_name);
-             xexit (1);
-           }
+           einfo (_("%P%F: group ended before it began (--help for usage)\n"));
+
          lang_leave_group ();
          ingroup = 0;
          break;
        case OPTION_MPC860C0:
-         link_info.mpc860c0 = 20;      /* default value (in bytes) */
+         /* Default value (in bytes).  */
+         link_info.mpc860c0 = 20;
          if (optarg)
            {
              unsigned words;
 
              words = is_num (optarg, 1, 10, 0);
              if (words == 0)
-               {
-                 fprintf (stderr,
-                          _("%s: Invalid argument to option \"mpc860c0\"\n"),
-                          program_name);
-                 xexit (1);
-               }
-             link_info.mpc860c0 = words * 4;   /* convert words to bytes */
+               einfo (_("%P%F: invalid argument to option \"mpc860c0\"\n"));
+
+             /* Convert words to bytes.  */
+             link_info.mpc860c0 = words * 4;
            }
-         command_line.relax = true;
+         command_line.relax = TRUE;
          break;
 
        case OPTION_INIT:
@@ -1171,7 +1171,7 @@ set_default_dirlist (dirlist_ptr)
       if (p != NULL)
        *p = '\0';
       if (*dirlist_ptr != '\0')
-       ldfile_add_library_path (dirlist_ptr, true);
+       ldfile_add_library_path (dirlist_ptr, TRUE);
       if (p == NULL)
        break;
       dirlist_ptr = p + 1;
@@ -1204,13 +1204,13 @@ help ()
     {
       if (ld_options[i].doc != NULL)
        {
-         boolean comma;
+         bfd_boolean comma;
          int len;
          unsigned j;
 
          printf ("  ");
 
-         comma = false;
+         comma = FALSE;
          len = 2;
 
          j = i;
@@ -1231,7 +1231,7 @@ help ()
                      printf ("%s", _(ld_options[j].arg));
                      len += strlen (_(ld_options[j].arg));
                    }
-                 comma = true;
+                 comma = TRUE;
                }
              ++j;
            }
@@ -1246,7 +1246,7 @@ help ()
                  int two_dashes =
                    (ld_options[j].control == TWO_DASHES
                     || ld_options[j].control == EXACTLY_TWO_DASHES);
-                 
+
                  printf ("%s-%s%s",
                          comma ? ", " : "",
                          two_dashes ? "-" : "",
@@ -1260,7 +1260,7 @@ help ()
                      printf (" %s", _(ld_options[j].arg));
                      len += 1 + strlen (_(ld_options[j].arg));
                    }
-                 comma = true;
+                 comma = TRUE;
                }
              ++j;
            }