OSDN Git Service

* config/m32c/m32c-protos.h (m32c_function_arg): Delete.
[pf3gnuchains/gcc-fork.git] / gcc / opts.c
index c9255e0..fbee8f6 100644 (file)
@@ -372,31 +372,16 @@ bool flag_warn_unused_result = false;
 const char **in_fnames;
 unsigned num_in_fnames;
 
-static int common_handle_option (size_t scode, const char *arg, int value,
-                                unsigned int lang_mask, int kind);
+static bool common_handle_option (const struct cl_decoded_option *decoded,
+                                 unsigned int lang_mask, int kind,
+                                 const struct cl_option_handlers *handlers);
 static void handle_param (const char *);
 static char *write_langs (unsigned int lang_mask);
-static void complain_wrong_lang (const char *, const struct cl_option *,
+static void complain_wrong_lang (const struct cl_decoded_option *,
                                 unsigned int lang_mask);
 static void set_debug_level (enum debug_info_type type, int extended,
                             const char *arg);
 
-/* If ARG is a non-negative integer made up solely of digits, return its
-   value, otherwise return -1.  */
-static int
-integral_argument (const char *arg)
-{
-  const char *p = arg;
-
-  while (*p && ISDIGIT (*p))
-    p++;
-
-  if (*p == '\0')
-    return atoi (arg);
-
-  return -1;
-}
-
 /* Return a malloced slash-separated list of languages in MASK.  */
 static char *
 write_langs (unsigned int mask)
@@ -425,30 +410,35 @@ write_langs (unsigned int mask)
   return result;
 }
 
-/* Complain that switch OPT_INDEX does not apply to this front end.  */
+/* Complain that switch DECODED does not apply to this front end (mask
+   LANG_MASK).  */
 static void
-complain_wrong_lang (const char *text, const struct cl_option *option,
+complain_wrong_lang (const struct cl_decoded_option *decoded,
                     unsigned int lang_mask)
 {
-  char *ok_langs, *bad_lang;
-
-  /* The LTO front end inherits all the options from the first front
-     end that was used.  However, not all the original front end
-     options make sense in LTO.
+  const struct cl_option *option = &cl_options[decoded->opt_index];
+  const char *text = decoded->orig_option_with_args_text;
+  char *ok_langs = NULL, *bad_lang = NULL;
+  unsigned int opt_flags = option->flags;
 
-     A real solution would be to filter this in collect2, but collect2
-     does not have access to all the option attributes to know what to
-     filter.  So, in lto1 we silently accept inherited flags and do
-     nothing about it.  */
-  if (lang_mask & CL_LTO)
+  if (!lang_hooks.complain_wrong_lang_p (option))
     return;
 
-  ok_langs = write_langs (option->flags);
-  bad_lang = write_langs (lang_mask);
-
-  /* Eventually this should become a hard error IMO.  */
-  warning (0, "command line option \"%s\" is valid for %s but not for %s",
-          text, ok_langs, bad_lang);
+  opt_flags &= ((1U << cl_lang_count) - 1) | CL_DRIVER;
+  if (opt_flags != CL_DRIVER)
+    ok_langs = write_langs (opt_flags);
+  if (lang_mask != CL_DRIVER)
+    bad_lang = write_langs (lang_mask);
+
+  if (opt_flags == CL_DRIVER)
+    error ("command line option %qs is valid for the driver but not for %s",
+          text, bad_lang);
+  else if (lang_mask == CL_DRIVER)
+    gcc_unreachable ();
+  else
+    /* Eventually this should become a hard error IMO.  */
+    warning (0, "command line option %qs is valid for %s but not for %s",
+            text, ok_langs, bad_lang);
 
   free (ok_langs);
   free (bad_lang);
@@ -484,184 +474,64 @@ void print_ignored_options (void)
   input_location = saved_loc;
 }
 
+/* Handle an unknown option DECODED, returning true if an error should be
+   given.  */
 
-/* Handle option OPT_INDEX, and argument ARG, for the language
-   indicated by LANG_MASK.  VALUE is true, unless no- form of an -f or
-   -W option was given.  KIND is the diagnostic_t if this is a
-   diagnostics option, DK_UNSPECIFIED otherwise.  Returns false if the
-   switch was invalid.  */
-bool
-handle_option (int opt_index, int value, const char *arg,
-               unsigned int lang_mask, int kind)
+static bool
+unknown_option_callback (const struct cl_decoded_option *decoded)
 {
-  const struct cl_option *option = &cl_options[opt_index];
+  const char *opt = decoded->arg;
 
-  if (option->flag_var)
-    set_option (opt_index, value, arg, kind);
-  
-  if (option->flags & lang_mask)
+  if (opt[1] == 'W' && opt[2] == 'n' && opt[3] == 'o' && opt[4] == '-'
+      && !(decoded->errors & CL_ERR_NEGATIVE))
     {
-      if (lang_hooks.handle_option (opt_index, arg, value, kind) == 0)
-       return false;
-#ifdef ENABLE_LTO
-      else
-       lto_register_user_option (opt_index, arg, value, lang_mask);
- #endif
+      /* We don't generate warnings for unknown -Wno-* options unless
+        we issue diagnostics.  */
+      postpone_unknown_option_warning (opt);
+      return false;
     }
+  else
+    return true;
+}
 
-  if (option->flags & CL_COMMON)
-    {
-      if (common_handle_option (opt_index, arg, value, lang_mask, kind) == 0)
-       return false;
-#ifdef ENABLE_LTO
-      else
-       lto_register_user_option (opt_index, arg, value, CL_COMMON);
-#endif
-    }
+/* Note that an option DECODED has been successfully handled with a
+   handler for mask MASK.  */
 
-  if (option->flags & CL_TARGET)
-    {
-      if (!targetm.handle_option (opt_index, arg, value))
-       return false;
+static void
+post_handling_callback (const struct cl_decoded_option *decoded ATTRIBUTE_UNUSED,
+                       unsigned int mask ATTRIBUTE_UNUSED)
+{
 #ifdef ENABLE_LTO
-      else
-       lto_register_user_option (opt_index, arg, value, CL_TARGET);
+  lto_register_user_option (decoded->opt_index, decoded->arg,
+                           decoded->value, mask);
 #endif
-    }
-  return true;
 }
 
-/* Handle the switch beginning at ARGV for the language indicated by
-   LANG_MASK.  Returns the number of switches consumed.  */
-static unsigned int
-read_cmdline_option (const char **argv, unsigned int lang_mask)
-{
-  size_t opt_index;
-  const char *opt, *arg = 0;
-  char *dup = 0;
-  int value = 1;
-  unsigned int result = 0;
-  const struct cl_option *option;
-
-  opt = argv[0];
-
-  opt_index = find_opt (opt + 1, lang_mask | CL_COMMON | CL_TARGET);
-  if (opt_index == cl_options_count
-      && (opt[1] == 'W' || opt[1] == 'f' || opt[1] == 'm')
-      && opt[2] == 'n' && opt[3] == 'o' && opt[4] == '-')
-    {
-      /* Drop the "no-" from negative switches.  */
-      size_t len = strlen (opt) - 3;
-
-      dup = XNEWVEC (char, len + 1);
-      dup[0] = '-';
-      dup[1] = opt[1];
-      memcpy (dup + 2, opt + 5, len - 2 + 1);
-      opt = dup;
-      value = 0;
-      opt_index = find_opt (opt + 1, lang_mask | CL_COMMON | CL_TARGET);
-      if (opt_index == cl_options_count && opt[1] == 'W')
-       {
-         /* We don't generate warnings for unknown -Wno-* options
-             unless we issue diagnostics.  */
-         postpone_unknown_option_warning (argv[0]);
-         result = 1;
-         goto done;
-       }
-    }
-
-  if (opt_index == cl_options_count)
-    goto done;
-
-  option = &cl_options[opt_index];
-
-  /* Reject negative form of switches that don't take negatives as
-     unrecognized.  */
-  if (!value && (option->flags & CL_REJECT_NEGATIVE))
-    goto done;
-
-  /* We've recognized this switch.  */
-  result = 1;
-
-  /* Check to see if the option is disabled for this configuration.  */
-  if (option->flags & CL_DISABLED)
-    {
-      error ("command line option %qs"
-            " is not supported by this configuration", opt);
-      goto done;
-    }
-
-  /* Sort out any argument the switch takes.  */
-  if (option->flags & CL_JOINED)
-    {
-      /* Have arg point to the original switch.  This is because
-        some code, such as disable_builtin_function, expects its
-        argument to be persistent until the program exits.  */
-      arg = argv[0] + cl_options[opt_index].opt_len + 1;
-      if (!value)
-       arg += strlen ("no-");
-
-      if (*arg == '\0' && !(option->flags & CL_MISSING_OK))
-       {
-         if (option->flags & CL_SEPARATE)
-           {
-             arg = argv[1];
-             result = 2;
-           }
-         else
-           /* Missing argument.  */
-           arg = NULL;
-       }
-    }
-  else if (option->flags & CL_SEPARATE)
-    {
-      arg = argv[1];
-      result = 2;
-    }
+/* Handle a front-end option; arguments and return value as for
+   handle_option.  */
 
-  /* Now we've swallowed any potential argument, complain if this
-     is a switch for a different front end.  */
-  if (!(option->flags & (lang_mask | CL_COMMON | CL_TARGET)))
-    {
-      complain_wrong_lang (argv[0], option, lang_mask);
-      goto done;
-    }
-  else if ((option->flags & CL_TARGET)
-          && (option->flags & CL_LANG_ALL)
-          && !(option->flags & lang_mask))
-    {
-      /* Complain for target flag language mismatches if any languages
-        are specified.  */
-      complain_wrong_lang (argv[0], option, lang_mask);
-      goto done;
-    }
-
-  if (arg == NULL && (option->flags & (CL_JOINED | CL_SEPARATE)))
-    {
-      if (!lang_hooks.missing_argument (opt, opt_index))
-       error ("missing argument to \"%s\"", opt);
-      goto done;
-    }
-
-  /* If the switch takes an integer, convert it.  */
-  if (arg && (option->flags & CL_UINTEGER))
-    {
-      value = integral_argument (arg);
-      if (value == -1)
-       {
-         error ("argument to \"%s\" should be a non-negative integer",
-                option->opt_text);
-         goto done;
-       }
-    }
+static bool
+lang_handle_option (const struct cl_decoded_option *decoded,
+                   unsigned int lang_mask ATTRIBUTE_UNUSED, int kind,
+                   const struct cl_option_handlers *handlers)
+{
+  gcc_assert (decoded->canonical_option_num_elements <= 2);
+  return lang_hooks.handle_option (decoded->opt_index, decoded->arg,
+                                  decoded->value, kind, handlers);
+}
 
-  if (!handle_option (opt_index, value, arg, lang_mask, DK_UNSPECIFIED))
-    result = 0;
+/* Handle a back-end option; arguments and return value as for
+   handle_option.  */
 
- done:
-  if (dup)
-    free (dup);
-  return result;
+static bool
+target_handle_option (const struct cl_decoded_option *decoded,
+                     unsigned int lang_mask ATTRIBUTE_UNUSED, int kind,
+                     const struct cl_option_handlers *handlers ATTRIBUTE_UNUSED)
+{
+  gcc_assert (decoded->canonical_option_num_elements <= 2);
+  gcc_assert (kind == DK_UNSPECIFIED);
+  return targetm.handle_option (decoded->opt_index, decoded->arg,
+                               decoded->value);
 }
 
 /* Handle FILENAME from the command line.  */
@@ -723,14 +593,10 @@ flag_instrument_functions_exclude_p (tree fndecl)
       char *s;
 
       name = lang_hooks.decl_printable_name (fndecl, 0);
-      for (i = 0;
-          VEC_iterate (char_p, flag_instrument_functions_exclude_functions,
-                       i, s);
-          ++i)
-       {
-         if (strstr (name, s) != NULL)
-           return true;
-       }
+      FOR_EACH_VEC_ELT (char_p, flag_instrument_functions_exclude_functions,
+                       i, s)
+       if (strstr (name, s) != NULL)
+         return true;
     }
 
   if (VEC_length (char_p, flag_instrument_functions_exclude_files) > 0)
@@ -740,65 +606,58 @@ flag_instrument_functions_exclude_p (tree fndecl)
       char *s;
 
       name = DECL_SOURCE_FILE (fndecl);
-      for (i = 0;
-          VEC_iterate (char_p, flag_instrument_functions_exclude_files, i, s);
-          ++i)
-       {
-         if (strstr (name, s) != NULL)
-           return true;
-       }
+      FOR_EACH_VEC_ELT (char_p, flag_instrument_functions_exclude_files, i, s)
+       if (strstr (name, s) != NULL)
+         return true;
     }
 
   return false;
 }
 
 
-/* Decode and handle the vector of command line options.  LANG_MASK
+/* Handle the vector of command line options.  LANG_MASK
    contains has a single bit set representing the current
-   language.  */
+   language.  HANDLERS describes what functions to call for the options.  */
 static void
-read_cmdline_options (unsigned int argc, const char **argv, unsigned int lang_mask)
+read_cmdline_options (struct cl_decoded_option *decoded_options,
+                     unsigned int decoded_options_count,
+                     unsigned int lang_mask,
+                     const struct cl_option_handlers *handlers)
 {
-  unsigned int n, i;
+  unsigned int i;
 
-  for (i = 1; i < argc; i += n)
+  for (i = 1; i < decoded_options_count; i++)
     {
-      const char *opt = argv[i];
-
-      /* Interpret "-" or a non-switch as a file name.  */
-      if (opt[0] != '-' || opt[1] == '\0')
+      if (decoded_options[i].opt_index == OPT_SPECIAL_input_file)
        {
          if (main_input_filename == NULL)
            {
-             main_input_filename = opt;
+             main_input_filename = decoded_options[i].arg;
              main_input_baselength
                = base_of_path (main_input_filename, &main_input_basename);
            }
-         add_input_filename (opt);
-         n = 1;
+         add_input_filename (decoded_options[i].arg);
          continue;
        }
 
-      n = read_cmdline_option (argv + i, lang_mask);
-
-      if (!n)
-       {
-         n = 1;
-         error ("unrecognized command line option \"%s\"", opt);
-       }
+      read_cmdline_option (decoded_options + i, lang_mask, handlers);
     }
 }
 
 /* Parse command line options and set default flag values.  Do minimal
-   options processing.  */
+   options processing.  The decoded options are placed in *DECODED_OPTIONS
+   and *DECODED_OPTIONS_COUNT.  */
 void
-decode_options (unsigned int argc, const char **argv)
+decode_options (unsigned int argc, const char **argv,
+               struct cl_decoded_option **decoded_options,
+               unsigned int *decoded_options_count)
 {
   static bool first_time_p = true;
   static int initial_min_crossjump_insns;
   static int initial_max_fields_for_field_sensitive;
   static int initial_loop_invariant_max_bbs_in_loop;
   static unsigned int initial_lang_mask;
+  struct cl_option_handlers handlers;
 
   unsigned int i, lang_mask;
   int opt1;
@@ -810,7 +669,7 @@ decode_options (unsigned int argc, const char **argv)
   if (first_time_p)
     {
       /* Perform language-specific options initialization.  */
-      initial_lang_mask = lang_mask = lang_hooks.init_options (argc, argv);
+      initial_lang_mask = lang_mask = lang_hooks.option_lang_mask ();
 
       lang_hooks.initialize_diagnostics (global_dc);
 
@@ -825,40 +684,45 @@ decode_options (unsigned int argc, const char **argv)
   else
     lang_mask = initial_lang_mask;
 
+  decode_cmdline_options_to_array (argc, argv,
+                                  lang_mask | CL_COMMON | CL_TARGET,
+                                  decoded_options, decoded_options_count);
+  if (first_time_p)
+    /* Perform language-specific options initialization.  */
+    lang_hooks.init_options (*decoded_options_count, *decoded_options);
+
+  handlers.unknown_option_callback = unknown_option_callback;
+  handlers.wrong_lang_callback = complain_wrong_lang;
+  handlers.post_handling_callback = post_handling_callback;
+  handlers.num_handlers = 3;
+  handlers.handlers[0].handler = lang_handle_option;
+  handlers.handlers[0].mask = lang_mask;
+  handlers.handlers[1].handler = common_handle_option;
+  handlers.handlers[1].mask = CL_COMMON;
+  handlers.handlers[2].handler = target_handle_option;
+  handlers.handlers[2].mask = CL_TARGET;
+
   /* Scan to see what optimization level has been specified.  That will
      determine the default value of many flags.  */
-  for (i = 1; i < argc; i++)
+  for (i = 1; i < *decoded_options_count; i++)
     {
-      if (!strcmp (argv[i], "-O"))
+      struct cl_decoded_option *opt = &(*decoded_options)[i];
+      switch (opt->opt_index)
        {
-         optimize = 1;
-         optimize_size = 0;
-         ofast = 0;
-       }
-      else if (argv[i][0] == '-' && argv[i][1] == 'O')
-       {
-         /* Handle -Os, -O2, -O3, -O69, ...  */
-         const char *p = &argv[i][2];
-
-         if ((p[0] == 's') && (p[1] == 0))
+       case OPT_O:
+         if (*opt->arg == '\0')
            {
-             optimize_size = 1;
-
-             /* Optimizing for size forces optimize to be 2.  */
-             optimize = 2;
-             ofast = 0;
-           }
-         else if (strcmp (p, "fast") == 0)
-           {
-             /* -Ofast only adds flags to -O3.  */
+             optimize = 1;
              optimize_size = 0;
-             optimize = 3;
-             ofast = 1;
+             ofast = 0;
            }
          else
            {
-             const int optimize_val = read_integral_parameter (p, p - 2, -1);
-             if (optimize_val != -1)
+             const int optimize_val = integral_argument (opt->arg);
+             if (optimize_val == -1)
+               error ("argument to %qs should be a non-negative integer",
+                      "-O");
+             else
                {
                  optimize = optimize_val;
                  if ((unsigned int) optimize > 255)
@@ -867,6 +731,26 @@ decode_options (unsigned int argc, const char **argv)
                  ofast = 0;
                }
            }
+         break;
+
+       case OPT_Os:
+         optimize_size = 1;
+
+         /* Optimizing for size forces optimize to be 2.  */
+         optimize = 2;
+         ofast = 0;
+         break;
+
+       case OPT_Ofast:
+         /* -Ofast only adds flags to -O3.  */
+         optimize_size = 0;
+         optimize = 3;
+         ofast = 1;
+         break;
+
+       default:
+         /* Ignore other options in this prescan.  */
+         break;
        }
     }
 
@@ -895,6 +779,7 @@ decode_options (unsigned int argc, const char **argv)
   flag_merge_constants = opt1;
   flag_split_wide_types = opt1;
   flag_tree_ccp = opt1;
+  flag_tree_bit_ccp = opt1;
   flag_tree_dce = opt1;
   flag_tree_dom = opt1;
   flag_tree_dse = opt1;
@@ -910,6 +795,7 @@ decode_options (unsigned int argc, const char **argv)
   opt2 = (optimize >= 2);
   flag_inline_small_functions = opt2;
   flag_indirect_inlining = opt2;
+  flag_partial_inlining = opt2;
   flag_thread_jumps = opt2;
   flag_crossjumping = opt2;
   flag_optimize_sibling_calls = opt2;
@@ -946,6 +832,7 @@ decode_options (unsigned int argc, const char **argv)
 
   /* -O3 optimizations.  */
   opt3 = (optimize >= 3);
+  flag_tree_loop_distribute_patterns = opt3;
   flag_predictive_commoning = opt3;
   flag_inline_functions = opt3;
   flag_unswitch_loops = opt3;
@@ -991,7 +878,7 @@ decode_options (unsigned int argc, const char **argv)
     }
 
   /* Enable -Werror=coverage-mismatch by default */
-  enable_warning_as_error("coverage-mismatch", 1, lang_mask);
+  enable_warning_as_error ("coverage-mismatch", 1, lang_mask, &handlers);
 
   if (first_time_p)
     {
@@ -1019,7 +906,8 @@ decode_options (unsigned int argc, const char **argv)
   OPTIMIZATION_OPTIONS (optimize, optimize_size);
 #endif
 
-  read_cmdline_options (argc, argv, lang_mask);
+  read_cmdline_options (*decoded_options, *decoded_options_count, lang_mask,
+                       &handlers);
 
   if (dump_base_name && ! IS_ABSOLUTE_PATH (dump_base_name))
     {
@@ -1062,6 +950,11 @@ decode_options (unsigned int argc, const char **argv)
               "is disabled.");
       flag_toplevel_reorder = 0;
     }
+
+  /* -Wmissing-noreturn is alias for -Wsuggest-attribute=noreturn.  */
+  if (warn_missing_noreturn)
+    warn_suggest_attribute_noreturn = true;
+    
   /* Unless the user has asked for section anchors, we disable toplevel
      reordering at -O0 to disable transformations that might be surprising
      to end users and to get -fno-toplevel-reorder tested.  */
@@ -1304,6 +1197,12 @@ print_filtered_help (unsigned int include_flags,
       if ((option->flags & exclude_flags) != 0)
        continue;
 
+      /* The driver currently prints its own help text.  */
+      if ((option->flags & CL_DRIVER) != 0
+         && (option->flags & (((1U << cl_lang_count) - 1)
+                              | CL_COMMON | CL_TARGET)) == 0)
+       continue;
+
       found = true;
       /* Skip switches that have already been printed.  */
       if (printed[i])
@@ -1444,6 +1343,7 @@ print_specific_help (unsigned int include_flags,
       switch (flag & include_flags)
        {
        case 0:
+       case CL_DRIVER:
          break;
 
        case CL_TARGET:
@@ -1506,15 +1406,21 @@ print_specific_help (unsigned int include_flags,
 /* Handle target- and language-independent options.  Return zero to
    generate an "unknown option" message.  Only options that need
    extra handling need to be listed here; if you simply want
-   VALUE assigned to a variable, it happens automatically.  */
+   DECODED->value assigned to a variable, it happens automatically.  */
 
-static int
-common_handle_option (size_t scode, const char *arg, int value,
-                     unsigned int lang_mask, int kind ATTRIBUTE_UNUSED)
+static bool
+common_handle_option (const struct cl_decoded_option *decoded,
+                     unsigned int lang_mask, int kind ATTRIBUTE_UNUSED,
+                     const struct cl_option_handlers *handlers)
 {
+  size_t scode = decoded->opt_index;
+  const char *arg = decoded->arg;
+  int value = decoded->value;
   static bool verbose = false;
   enum opt_code code = (enum opt_code) scode;
 
+  gcc_assert (decoded->canonical_option_num_elements <= 2);
+
   switch (code)
     {
     case OPT__param:
@@ -1541,7 +1447,8 @@ common_handle_option (size_t scode, const char *arg, int value,
        print_specific_help (0, undoc_mask, all_langs_mask);
        /* Then display any remaining, non-language options.  */
        for (i = CL_MIN_OPTION_CLASS; i <= CL_MAX_OPTION_CLASS; i <<= 1)
-         print_specific_help (i, undoc_mask, 0);
+         if (i != CL_DRIVER)
+           print_specific_help (i, undoc_mask, 0);
        exit_after_options = true;
        break;
       }
@@ -1552,8 +1459,8 @@ common_handle_option (size_t scode, const char *arg, int value,
       exit_after_options = true;
 
       /* Allow the target a chance to give the user some additional information.  */
-      if (targetm.target_help)
-       targetm.target_help ();
+      if (targetm.help)
+       targetm.help ();
       break;
 
     case OPT_fhelp_:
@@ -1680,11 +1587,6 @@ common_handle_option (size_t scode, const char *arg, int value,
       exit_after_options = true;
       break;
 
-    case OPT_G:
-      g_switch_value = value;
-      g_switch_set = true;
-      break;
-
     case OPT_O:
     case OPT_Os:
     case OPT_Ofast:
@@ -1692,7 +1594,7 @@ common_handle_option (size_t scode, const char *arg, int value,
       break;
 
     case OPT_Werror_:
-      enable_warning_as_error (arg, value, lang_mask);
+      enable_warning_as_error (arg, value, lang_mask, handlers);
       break;
 
     case OPT_Wlarger_than_:
@@ -1818,7 +1720,7 @@ common_handle_option (size_t scode, const char *arg, int value,
        diagnostic_prefixing_rule (global_dc)
          = DIAGNOSTICS_SHOW_PREFIX_EVERY_LINE;
       else
-       return 0;
+       return false;
       break;
 
     case OPT_fdiagnostics_show_option:
@@ -1827,7 +1729,7 @@ common_handle_option (size_t scode, const char *arg, int value,
 
     case OPT_fdump_:
       if (!dump_switch_p (arg))
-       return 0;
+       return false;
       break;
 
     case OPT_fexcess_precision_:
@@ -1991,7 +1893,7 @@ common_handle_option (size_t scode, const char *arg, int value,
     case OPT_frandom_seed:
       /* The real switch is -fno-random-seed.  */
       if (value)
-       return 0;
+       return false;
       set_random_seed (NULL);
       break;
 
@@ -2009,7 +1911,7 @@ common_handle_option (size_t scode, const char *arg, int value,
       fix_sched_param ("verbose", arg);
       break;
 #else
-      return 0;
+      return false;
 #endif
 
     case OPT_fsched_stalled_insns_:
@@ -2056,7 +1958,7 @@ common_handle_option (size_t scode, const char *arg, int value,
     case OPT_fstack_limit:
       /* The real switch is -fno-stack-limit.  */
       if (value)
-       return 0;
+       return false;
       stack_limit_rtx = NULL_RTX;
       break;
 
@@ -2182,11 +2084,12 @@ common_handle_option (size_t scode, const char *arg, int value,
       break;
 
     case OPT_pedantic_errors:
-      global_dc->pedantic_errors = flag_pedantic_errors = pedantic = 1;
+      flag_pedantic_errors = pedantic = 1;
+      global_dc->pedantic_errors = 1;
       break;
 
     case OPT_fwhopr:
-      flag_whopr = value;
+      flag_whopr = arg;
       break;
 
     case OPT_w:
@@ -2222,7 +2125,7 @@ common_handle_option (size_t scode, const char *arg, int value,
       break;
     }
 
-  return 1;
+  return true;
 }
 
 /* Handle --param NAME=VALUE.  */
@@ -2437,48 +2340,6 @@ get_option_state (int option, struct cl_option_state *state)
   return true;
 }
 
-/* Set *OPTION according to VALUE and ARG.  */
-
-void
-set_option (int opt_index, int value, const char *arg, int kind)
-{
-  const struct cl_option *option = &cl_options[opt_index];
-
-  if (!option->flag_var)
-    return;
-
-  switch (option->var_type)
-    {
-    case CLVC_BOOLEAN:
-       *(int *) option->flag_var = value;
-       break;
-
-    case CLVC_EQUAL:
-       *(int *) option->flag_var = (value
-                                    ? option->var_value
-                                    : !option->var_value);
-       break;
-
-    case CLVC_BIT_CLEAR:
-    case CLVC_BIT_SET:
-       if ((value != 0) == (option->var_type == CLVC_BIT_SET))
-         *(int *) option->flag_var |= option->var_value;
-       else
-         *(int *) option->flag_var &= ~option->var_value;
-       if (option->flag_var == &target_flags)
-         target_flags_explicit |= option->var_value;
-       break;
-
-    case CLVC_STRING:
-       *(const char **) option->flag_var = arg;
-       break;
-    }
-
-  if ((diagnostic_t)kind != DK_UNSPECIFIED)
-    diagnostic_classify_diagnostic (global_dc, opt_index, (diagnostic_t)kind);
-}
-
-
 /* Callback function, called when -Werror= enables a warning.  */
 
 static void (*warning_as_error_callback) (int) = NULL;
@@ -2496,7 +2357,8 @@ register_warning_as_error_callback (void (*callback) (int))
    also by legacy Werror-implicit-function-declaration.  */
 
 void
-enable_warning_as_error (const char *arg, int value, unsigned int lang_mask)
+enable_warning_as_error (const char *arg, int value, unsigned int lang_mask,
+                        const struct cl_option_handlers *handlers)
 {
   char *new_option;
   int option_index;
@@ -2505,7 +2367,7 @@ enable_warning_as_error (const char *arg, int value, unsigned int lang_mask)
   new_option[0] = 'W';
   strcpy (new_option + 1, arg);
   option_index = find_opt (new_option, lang_mask);
-  if (option_index == N_OPTS)
+  if (option_index == OPT_SPECIAL_unknown)
     {
       error ("-Werror=%s: No option -%s", arg, new_option);
     }
@@ -2513,14 +2375,16 @@ enable_warning_as_error (const char *arg, int value, unsigned int lang_mask)
     {
       const diagnostic_t kind = value ? DK_ERROR : DK_WARNING;
 
-      diagnostic_classify_diagnostic (global_dc, option_index, kind);
+      diagnostic_classify_diagnostic (global_dc, option_index, kind,
+                                     UNKNOWN_LOCATION);
       if (kind == DK_ERROR)
        {
          const struct cl_option * const option = cl_options + option_index;
 
          /* -Werror=foo implies -Wfoo.  */
          if (option->var_type == CLVC_BOOLEAN)
-           handle_option (option_index, value, arg, lang_mask, (int)kind);
+           handle_generated_option (option_index, NULL, value, lang_mask,
+                                    (int)kind, handlers);
 
          if (warning_as_error_callback)
            warning_as_error_callback (option_index);