]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix --help -Q output
authorMartin Liska <mliska@suse.cz>
Mon, 29 Nov 2021 13:46:47 +0000 (14:46 +0100)
committerMartin Liska <mliska@suse.cz>
Tue, 7 Dec 2021 13:37:02 +0000 (14:37 +0100)
PR middle-end/103438

gcc/ChangeLog:

* config/s390/s390.c (s390_valid_target_attribute_inner_p):
Use new enum CLVC_INTEGER.
* opt-functions.awk: Use new CLVC_INTEGER.
* opts-common.c (set_option): Likewise.
(option_enabled): Return -1,0,1 for CLVC_INTEGER.
(get_option_state): Use new CLVC_INTEGER.
(control_warning_option): Likewise.
* opts.h (enum cl_var_type): Likewise.

gcc/config/s390/s390.c
gcc/opt-functions.awk
gcc/opts-common.c
gcc/opts.h

index 510e7f58a3b8e07b86bb723339d7a5df49fa4caf..3a22f7833a94e9e3cfdf5834efa00a4758be97b8 100644 (file)
@@ -15926,7 +15926,7 @@ s390_valid_target_attribute_inner_p (tree args,
          new_opts_set->x_target_flags |= mask;
        }
 
-      else if (cl_options[opt].var_type == CLVC_BOOLEAN)
+      else if (cl_options[opt].var_type == CLVC_INTEGER)
        {
          int value;
 
index 9bc856040669e901ea04d8efe1a76b64ae74725c..ffe4eb920276a3dc390b5e3817e1276cda86e9f1 100644 (file)
@@ -303,7 +303,7 @@ function var_set(flags)
                return "0, CLVC_STRING, 0"
        if (flag_set_p("ByteSize", flags))
                return "0, CLVC_SIZE, 0"
-       return "0, CLVC_BOOLEAN, 0"
+       return "0, CLVC_INTEGER, 0"
 }
 
 # Given that an option called NAME has flags FLAGS, return an initializer
index 9d1914ff2ffeb089eb3d3062e78a65cff13d68a7..ef2130e318fb16355f4e3f47be1c8cab795fa110 100644 (file)
@@ -1458,7 +1458,7 @@ set_option (struct gcc_options *opts, struct gcc_options *opts_set,
 
   switch (option->var_type)
     {
-    case CLVC_BOOLEAN:
+    case CLVC_INTEGER:
        if (option->cl_host_wide_int)
          {
            *(HOST_WIDE_INT *) flag_var = value;
@@ -1586,7 +1586,8 @@ option_flag_var (int opt_index, struct gcc_options *opts)
 }
 
 /* Return 1 if option OPT_IDX is enabled in OPTS, 0 if it is disabled,
-   or -1 if it isn't a simple on-off switch.  */
+   or -1 if it isn't a simple on-off switch
+   (or if the value is unknown, typically set later in target).  */
 
 int
 option_enabled (int opt_idx, unsigned lang_mask, void *opts)
@@ -1606,11 +1607,17 @@ option_enabled (int opt_idx, unsigned lang_mask, void *opts)
   if (flag_var)
     switch (option->var_type)
       {
-      case CLVC_BOOLEAN:
+      case CLVC_INTEGER:
        if (option->cl_host_wide_int)
-         return *(HOST_WIDE_INT *) flag_var != 0;
+         {
+           HOST_WIDE_INT v = *(HOST_WIDE_INT *) flag_var;
+           return v != 0 ? (v < 0 ? -1 : 1) : 0;
+         }
        else
-         return *(int *) flag_var != 0;
+         {
+           int v = *(int *) flag_var;
+           return v != 0 ? (v < 0 ? -1 : 1) : 0;
+         }
 
       case CLVC_EQUAL:
        if (option->cl_host_wide_int) 
@@ -1658,7 +1665,7 @@ get_option_state (struct gcc_options *opts, int option,
 
   switch (cl_options[option].var_type)
     {
-    case CLVC_BOOLEAN:
+    case CLVC_INTEGER:
     case CLVC_EQUAL:
     case CLVC_SIZE:
       state->data = flag_var;
@@ -1725,7 +1732,7 @@ control_warning_option (unsigned int opt_index, int kind, const char *arg,
       const struct cl_option *option = &cl_options[opt_index];
 
       /* -Werror=foo implies -Wfoo.  */
-      if (option->var_type == CLVC_BOOLEAN
+      if (option->var_type == CLVC_INTEGER
          || option->var_type == CLVC_ENUM
          || option->var_type == CLVC_SIZE)
        {
index f5bc9a3149c911d7dc56b6752c156598f91b9db0..4c2b77ec0f0061263421d6668fe3f788aa573ffb 100644 (file)
@@ -24,8 +24,8 @@ along with GCC; see the file COPYING3.  If not see
 
 /* Specifies how a switch's VAR_VALUE relates to its FLAG_VAR.  */
 enum cl_var_type {
-  /* The switch is enabled when FLAG_VAR is nonzero.  */
-  CLVC_BOOLEAN,
+  /* The switch is an integer value.  */
+  CLVC_INTEGER,
 
   /* The switch is enabled when FLAG_VAR == VAR_VALUE.  */
   CLVC_EQUAL,