s390: gdbarch_tdep.have_* int -> bool
authorPhilipp Rudo <prudo@linux.vnet.ibm.com>
Tue, 23 Jan 2018 12:37:42 +0000 (13:37 +0100)
committerAndreas Arnez <arnez@linux.vnet.ibm.com>
Tue, 23 Jan 2018 12:37:42 +0000 (13:37 +0100)
Currently the gdbarch_tdep.have_* flags are a mix of int and bool.  Clean
this up by making them all bool.

gdb/ChangeLog:

* s390-linux-tdep.c (gdbarch_tdep) <have_linux_v1, have_linux_v2>
<have_tdb>: Change type to bool.
(s390_gdbarch_tdep_alloc): Adjust.
(s390_gdbarch_init): Adjust.

gdb/ChangeLog
gdb/s390-linux-tdep.c

index cc4816b7a38fc28ffc7e27b8ce69e8befec20b97..52394ab6ecda600384d94e5adcf4fbbea1892a49 100644 (file)
@@ -1,3 +1,10 @@
+2018-01-23  Philipp Rudo  <prudo@linux.vnet.ibm.com>
+
+       * s390-linux-tdep.c (gdbarch_tdep) <have_linux_v1, have_linux_v2>
+       <have_tdb>: Change type to bool.
+       (s390_gdbarch_tdep_alloc): Adjust.
+       (s390_gdbarch_init): Adjust.
+
 2018-01-23  Philipp Rudo  <prudo@linux.vnet.ibm.com>
 
        * s390-linux-tdep (s390_abi_kind) <ABI_NONE>: New default field.
index e107af8b07dcdc110cabcced61ee6146db3b8295..7d41dfeb9b96fa206a56a354bf4c335d8273c72c 100644 (file)
@@ -113,9 +113,9 @@ struct gdbarch_tdep
   int v0_full_regnum;
 
   bool have_upper;
-  int have_linux_v1;
-  int have_linux_v2;
-  int have_tdb;
+  bool have_linux_v1;
+  bool have_linux_v2;
+  bool have_tdb;
   bool have_vx;
   bool have_gs;
 };
@@ -7824,9 +7824,9 @@ s390_gdbarch_tdep_alloc ()
   tdep->cc_regnum = -1;
 
   tdep->have_upper = false;
-  tdep->have_linux_v1 = 0;
-  tdep->have_linux_v2 = 0;
-  tdep->have_tdb = 0;
+  tdep->have_linux_v1 = false;
+  tdep->have_linux_v2 = false;
+  tdep->have_tdb = false;
   tdep->have_vx = false;
   tdep->have_gs = false;
 
@@ -7993,13 +7993,13 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
 
          if (tdesc_numbered_register (feature, tdesc_data,
                                       S390_LAST_BREAK_REGNUM, "last_break"))
-           tdep->have_linux_v1 = 1;
+           tdep->have_linux_v1 = true;
 
          if (tdesc_numbered_register (feature, tdesc_data,
                                       S390_SYSTEM_CALL_REGNUM, "system_call"))
-           tdep->have_linux_v2 = 1;
+           tdep->have_linux_v2 = true;
 
-         if (tdep->have_linux_v2 tdep->have_linux_v1)
+         if (tdep->have_linux_v2 && !tdep->have_linux_v1)
            valid_p = 0;
        }
 
@@ -8011,7 +8011,7 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
            valid_p &= tdesc_numbered_register (feature, tdesc_data,
                                                S390_TDB_DWORD0_REGNUM + i,
                                                tdb_regs[i]);
-         tdep->have_tdb = 1;
+         tdep->have_tdb = true;
        }
 
       /* Vector registers.  */